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

@Delvox · Регистрация: 09.03.2013

Студворк — интернет-сервис помощи студентам

Здравствуйте уважаемые форумчане!
Решил начать изучать язык Си++, начал с простого также как и при изучении языка 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();
}

HighPredator · 12.05.2013, 15:47

Определитесь наконец с вводом-выводом. Используйте либо cin/cout или scanf/printf. Не нужно смешивать.

@Delvox · 13.05.2013, 03:06 **[ТС]**

Вот переписанный код:

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 · 13.05.2013, 08:15

Delvox, упрощать особо некуда, кроме как избавиться от переменной s и избыточных комментариев. Еще не помешает код отформатировать единообразно.

@Eagle123 · 13.05.2013, 08:31

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 · 13.05.2013, 09:00

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 · 13.05.2013, 09:50

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 · 13.05.2013, 10:45

Сообщение от programina

C++
1
2
3
4
int s(int &a int b) 
{ 
   return a*=b; 
}

Совершенно не нужно в данном случае использовать ссылку, да и вообще изменять аргумент.

@programina · 13.05.2013, 10:52

Сообщение от Tulosba

Совершенно не нужно в данном случае использовать ссылку, да и вообще изменять аргумент.

в данном случае можно воспользоваться калькулятором

Tulosba · 13.05.2013, 11:07

Сообщение от programina

в данном случае можно воспользоваться калькулятором

Лучше вообще в уме

Но я Вам говорил вполне серьёзно. Передача по ссылке и изменение аргумента здесь ни к чему.

Новые блоги и статьи Все статьи Все блоги /
модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .	Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ
Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .	Использование значений реквизитов справочника в документе, с определенными условиями и правами Maks 07.04.2026 1. Контроль срока действия договора Алгоритм из решения ниже реализован на примере нетипового документа "ЗаявкаНаРаботу", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если. . .	Доступность команды формы по условию Maks 07.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: сделать доступной кнопку (команда формы "ЗавершитьСписание") при. . .

HighPredator 6045 / 2160 / 753 Регистрация: 10.12.2010 Сообщений: 6,005 Записей в блоге: 3
	12.05.2013, 15:47
	Определитесь наконец с вводом-выводом. Используйте либо cin/cout или scanf/printf. Не нужно смешивать. 1

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	13.05.2013, 08:15
	Delvox, упрощать особо некуда, кроме как избавиться от переменной s и избыточных комментариев. Еще не помешает код отформатировать единообразно. 1

@programina 2062 / 619 / 41 Регистрация: 23.10.2011 Сообщений: 4,468 Записей в блоге: 2
	13.05.2013, 09:50
	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"); } 1