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

@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

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

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

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

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

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