Форум программистов, компьютерный форум CyberForum.ru

Упрощение кода программы - C++

Восстановить пароль Регистрация
 
Delvox
 Аватар для Delvox
7 / 7 / 1
Регистрация: 09.03.2013
Сообщений: 54
12.05.2013, 14:28     Упрощение кода программы #1
Здравствуйте уважаемые форумчане!
Решил начать изучать язык Си++, начал с простого также как и при изучении языка Pascal.
Дана следующая задача:

Задача:

Составить программу, нахождения площади пола прямоугольной комнаты по длине и ширине.
1. Постановка задачи
Дано: длина и ширина прямоугольной комнаты
Найти: площадь комнаты

2. Математическая модель
Введём переменные
dl – длина комнаты
sh – ширина комнаты
S – площадь комнаты
S=dl*sh

Программу составил исходя из тех знаний которые у меня имеются + кое-что нашел в интернете.
Вопрос в следующем: Логично ли составлен код программы?, т.е. как его можно упростить и прийти к минимальному виду кода? Заранее спасибо

Код компилируется без ошибок

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include <stdio.h> //Подключаемые библиотеки
#include <iostream>
using namespace std;
 
int main()
{
    setlocale(0, ""); //Для того чтобы в консоле тектс был русский 
 
    int dl, sh, s; //Обьявление переменных
 
    cout<<"Введите длину: "; //Вывод сообщения
      scanf("%d", &dl);      //Считывание числа с клавиатуры в переменную dl
 
    cout<<"Введите ширину: "; 
      scanf("%d", &sh);
 
    s = dl * sh; //Нахождение самой площади посредством умножение длины на ширину
    cout<< s; //Вывод полученного результата в консоль
 
cin.get(); // Для того чтобы консоль сразу после вывода результата не закрылась
cin.get();
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
12.05.2013, 14:28     Упрощение кода программы
Посмотрите здесь:

Упрощение кода C++
Упрощение кода C++
Комментрирование кода программы C++
C++ Упрощение кода
C++ Упрощение программы
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
HighPredator
 Аватар для HighPredator
5352 / 1735 / 320
Регистрация: 10.12.2010
Сообщений: 5,120
Записей в блоге: 3
12.05.2013, 15:47     Упрощение кода программы #2
Определитесь наконец с вводом-выводом. Используйте либо cin/cout или scanf/printf. Не нужно смешивать.
Delvox
 Аватар для Delvox
7 / 7 / 1
Регистрация: 09.03.2013
Сообщений: 54
13.05.2013, 03:06  [ТС]     Упрощение кода программы #3
Вот переписанный код:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include <iostream>
#include <conio.h>
using namespace std;
 
int main()
{
setlocale(0, ""); //Для того чтобы в консоле тектс был русский
 
int dl, sh, s; //Обьявление переменных
 
cout<<"Введите длину и ширину. \n"; //Вывод сообщения
cin>> dl >> sh; //Считывание чисел с клавиатуры в переменные dl и sh
 
s = dl * sh; //Нахождение самой площади посредством умножение длины на ширину
cout<< s; //Вывод полученного результата в консоль
 
 _getch(); // Для того чтобы консоль сразу после вывода результата не закрылась
}
Tulosba
:)
Эксперт С++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
13.05.2013, 08:15     Упрощение кода программы #4
Delvox, упрощать особо некуда, кроме как избавиться от переменной s и избыточных комментариев. Еще не помешает код отформатировать единообразно.
Eagle123
6 / 6 / 0
Регистрация: 03.10.2012
Сообщений: 51
13.05.2013, 08:31     Упрощение кода программы #5
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include <iostream>
#include <conio.h>
using namespace std;
 
int main()
{
setlocale(0, ""); //Для того чтобы в консоле тектс был русский
 
int dl, sh; //Обьявление переменных
 
cout<<"Введите длину и ширину. \n"; //Вывод сообщения
cin>> dl >> sh; //Считывание чисел с клавиатуры в переменные dl и sh
 
 
cout<< dl * sh; //Вывод  самой площади посредством умножение длины на ширину
 
 _getch(); // Для того чтобы консоль сразу после вывода результата не закрылась
}
ValeryS
Модератор
6377 / 4843 / 442
Регистрация: 14.02.2011
Сообщений: 16,057
13.05.2013, 09:00     Упрощение кода программы #6
Eagle123,
бесполезные комментарии
например здесь
Цитата Сообщение от Eagle123 Посмотреть сообщение
int dl, sh; //Обьявление переменных
это и так понятно
а что за переменные за что отвечают?
Цитата Сообщение от Eagle123 Посмотреть сообщение
cin>> dl >> sh; //Считывание чисел с клавиатуры в переменные dl и sh
то же самое
не лучше ли написать
вводим длину и ширину
или вообще ничего не писать
строчкой выше все объяснено
лучше писать самодокументруемый код например
C++
1
int Length, Width;
переменные своим именем уже рассказывают зачем они нужны

Добавлено через 4 минуты
Delvox,
к тебе тоже относится
и если пишешь на плюсах так и пиши по плюсовски объявляй переменные где они нужны
например
Цитата Сообщение от Delvox Посмотреть сообщение
s = dl * sh; //Нахождение самой площади посредством умножение длины на ширину
cout<< s; //Вывод полученного результата в консоль
C++
1
2
int s = dl * sh; // находим площадь прямоугольника
cout<< s; //можно не коментьть и так ясно
programina
 Аватар для programina
1912 / 597 / 37
Регистрация: 23.10.2011
Сообщений: 4,468
Записей в блоге: 2
13.05.2013, 09:50     Упрощение кода программы #7
C++
int s(int &a int b)
{
    return a*=b;
}
 
int main()
{
    int dl, sh;
 
    cout << "длина : " << endl;
    cin >> dl;
    cout << "ширина : " << endl;
    cin >> sh;
 
    cout << s(dl, sh) << endl;
 
    system("pause");
}
Tulosba
:)
Эксперт С++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
13.05.2013, 10:45     Упрощение кода программы #8
Цитата Сообщение от programina Посмотреть сообщение
C++
1
2
3
4
int s(int &a int b) 
{ 
   return a*=b; 
}
Совершенно не нужно в данном случае использовать ссылку, да и вообще изменять аргумент.
programina
 Аватар для programina
1912 / 597 / 37
Регистрация: 23.10.2011
Сообщений: 4,468
Записей в блоге: 2
13.05.2013, 10:52     Упрощение кода программы #9
Цитата Сообщение от Tulosba Посмотреть сообщение
Совершенно не нужно в данном случае использовать ссылку, да и вообще изменять аргумент.
в данном случае можно воспользоваться калькулятором
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
13.05.2013, 11:07     Упрощение кода программы
Еще ссылки по теме:

Упрощение программного кода C++
Упрощение и оптимизация кода (MS VCPP) C++
Упрощение кода C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Tulosba
:)
Эксперт С++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
13.05.2013, 11:07     Упрощение кода программы #10
Цитата Сообщение от programina Посмотреть сообщение
в данном случае можно воспользоваться калькулятором
Лучше вообще в уме Но я Вам говорил вполне серьёзно. Передача по ссылке и изменение аргумента здесь ни к чему.
Yandex
Объявления
13.05.2013, 11:07     Упрощение кода программы
Ответ Создать тему
Опции темы

Текущее время: 23:53. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru