Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.79/47: Рейтинг темы: голосов - 47, средняя оценка - 4.79
0 / 0 / 0
Регистрация: 19.11.2012
Сообщений: 114

Оценка кода

17.10.2019, 16:14. Показов 10300. Ответов 21
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Добрый день! Есть задание в ВУЗе:
Заработная плата специалиста составляет А руб., курс доллара—В руб. Предполагается ежемесячное увеличение заработной платы в рублях на (р+k) % (по отношению к заработной плате предыдущего месяца, k—номер текущего месяца), и прогнозируется изменение курса доллара на q %. Определить, через сколько месяцев заработная плата специалиста в долларовом эквиваленте увеличится на 50 %. Рассматривать срок не более одного года.
Я сделал следующий код:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
#include <iostream>
#include <conio.h>
 
int main()
{
    setlocale(LC_ALL, "");
    int k; float A, B, p, q;
    printf("Введите в программу значения, подтверждая их нажатием Enter (запятая в числах ','), в следующем порядке:\n");
    printf("1.Заработная плата(руб)\n2.Курс доллара\n3.Значение p\n4.Текущий месяц(k)\n5.Прогнозируемое изменение курса(проценты)(q)\n");
    scanf_s("%f%f%f%d%f", &A, &B, &p, &k, &q);
    printf("Входные данные:\n");
    printf("1.Заработная плата(руб): %.2f\n2.Курс доллара: %.2f\n3.Значение p: %.2f\n4.Текущий месяц(k): %d\n5.Прогнозируемое изменение курса(проценты) (q): %.2f\n", A, B, p, k, q);
    float WageInDollarsStart, WageInDollarsChange, DollarRate;
    int Month = k, MonthResult = NULL;
    WageInDollarsStart = A/B;
    DollarRate = (B / 100) * q + B;
    for (int i = 0; i < 12; i++)
    {
        if (Month < 12) { Month = k++; }
        else { Month = 1; }
        A = (A / 100) * (Month + p) + A;
        WageInDollarsChange = A /DollarRate;
        MonthResult = i + 1;
        if (WageInDollarsChange / WageInDollarsStart >= 2){break;}
    }
    printf("Начальная зарплата в долларах: %.3f\n", WageInDollarsStart);
    printf("Зарплата в рублях:%.2f\nЗарплата в долларах:%.2f\n", A, WageInDollarsChange);
    printf("Увеличилась на %.3f\n",WageInDollarsChange / WageInDollarsStart);
    printf("Курс доллара %.3f\n", DollarRate);
    printf("За %d месяцев\n", MonthResult);
    _getch();
}
Подскажите пожалуйста с профессиональной точки зрения, что я неправильно сделал и вы бы сделали по другому?

И вопрос. VisualStudio ругается на _getch() :Предупреждение C6031 Возвращаемое значение пропущено: "_getch". Что это значит и как убрать это предупреждение?
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
17.10.2019, 16:14
Ответы с готовыми решениями:

Советы по ускорению работы кода + оценка самого кода
Вчера вечером сел написать 3 консольные программки для работы с шаблонами размножения текста: 1.Выборка групп синонимов(создание словаря)...

оценка кода
Дайте оценку моему коду (https://onlinegdb.com/r1i-i8Cqm), укажите на ошибки

Оценка стиля кода
Народ, есть предложение. Оцените мой код. Не в смысле функционала - в смысле стиля написания. Значится, вот тут есть некий код: bool...

21
43 / 39 / 5
Регистрация: 16.09.2019
Сообщений: 285
17.10.2019, 16:20
Цитата Сообщение от BuJIJI Посмотреть сообщение
int MonthResult = NULL;
Так не принято, вроде....=0;
0
0 / 0 / 0
Регистрация: 19.11.2012
Сообщений: 114
17.10.2019, 16:24  [ТС]
Окей, спасибо. Не принято среди программистов C++? Просто последняя время занимался скриптами PowerShell там такое обычная практика. Да и разницы между NULL и 0 как я понимаю нет.
0
Life Builder
 Аватар для sk007
531 / 496 / 374
Регистрация: 12.01.2011
Сообщений: 1,755
17.10.2019, 16:25
Цитата Сообщение от BuJIJI Посмотреть сообщение
И вопрос. VisualStudio ругается на _getch() :Предупреждение C6031 Возвращаемое значение пропущено: "_getch". Что это значит и как убрать это предупреждение?
1) Это функция считывает символ из потока ввода, поэтому выдает предупреждение.
2) Просто убрать это старье, вместе с инклюдом(или использовать другой вариант, например: system("pause") - если вы пользуетесь ОС Windows)
1
0 / 0 / 0
Регистрация: 19.11.2012
Сообщений: 114
17.10.2019, 16:29  [ТС]
Как я понял, _getch() возвращает этот символ и VisualStudio мне просто говорит, что я нигде, то что она возвращает не использую?
Да, system(pause) даже поудобнее будет.
0
43 / 39 / 5
Регистрация: 16.09.2019
Сообщений: 285
17.10.2019, 16:30
Цитата Сообщение от BuJIJI Посмотреть сообщение
C++
1
2
if (Month < 12) { Month = k++; }
* * * * else { Month = 1; }
Странное место...наличие лишней переменной сбивает с толку и по моему там будет считать не верно
0
0 / 0 / 0
Регистрация: 19.11.2012
Сообщений: 114
17.10.2019, 16:48  [ТС]
Да. Убрал переменную month. Вот что получилось:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
#include <iostream>
#include <conio.h>
 
int main()
{
    setlocale(LC_ALL, "");
    int k; float A, B, p, q;
    printf("Введите в программу значения, подтверждая их нажатием Enter (запятая в числах ','), в следующем порядке:\n");
    printf("1.Заработная плата(руб)\n2.Курс доллара\n3.Значение p\n4.Текущий месяц(k)\n5.Прогнозируемое изменение курса(проценты)(q)\n");
    scanf_s("%f%f%f%d%f", &A, &B, &p, &k, &q);
    printf("Входные данные:\n");
    printf("1.Заработная плата(руб): %.2f\n2.Курс доллара: %.2f\n3.Значение p: %.2f\n4.Текущий месяц(k): %d\n5.Прогнозируемое изменение курса(проценты) (q): %.2f\n", A, B, p, k, q);
    float WageInDollarsStart, WageInDollarsChange, DollarRate;
    int MonthResult = NULL;
    WageInDollarsStart = A/B;
    DollarRate = (B / 100) * q + B;
    for (int i = 0; i < 12; i++)
    {
        if (k < 12) { k++; }
        else { k = 1; }
        A = (A / 100) * (k + p) + A;
        WageInDollarsChange = A /DollarRate;
        MonthResult = i + 1;
        if (WageInDollarsChange / WageInDollarsStart >= 2){break;}
    }
    printf("Начальная зарплата в долларах: %.3f\n", WageInDollarsStart);
    printf("Зарплата в рублях:%.2f\nЗарплата в долларах:%.2f\n", A, WageInDollarsChange);
    printf("Увеличилась на %.3f\n",WageInDollarsChange / WageInDollarsStart);
    printf("Курс доллара %.3f\n", DollarRate);
    printf("За %d месяцев\n", MonthResult);
    system("pause");
}
0
2 / 2 / 0
Регистрация: 08.10.2019
Сообщений: 11
17.10.2019, 17:45
Цитата Сообщение от BuJIJI Посмотреть сообщение
C++
1
if (WageInDollarsChange / WageInDollarsStart >= 2){break;}
Здесь не нужны фигурные скобки, иначе break не будет работать.
0
Неэпический
 Аватар для Croessmah
18146 / 10730 / 2066
Регистрация: 27.09.2012
Сообщений: 27,029
Записей в блоге: 1
17.10.2019, 17:47
Alexei55555, ???
BuJIJI, = NULL всё-таки поменяйте на = 0
Переменные в одну строку тоже не очень объявлять, особенно, если они разных типов.
0
2 / 2 / 0
Регистрация: 08.10.2019
Сообщений: 11
17.10.2019, 18:13
Цитата Сообщение от Croessmah Посмотреть сообщение
Alexei55555, ???
А, ну работать может и будет, но фигурные скобки там все равно не нужны.
Я просто не знаю, как break работает в циклах. Я раньше думал, что он только из одного блока выходит.
0
43 / 39 / 5
Регистрация: 16.09.2019
Сообщений: 285
17.10.2019, 21:18
Цитата Сообщение от Alexei55555 Посмотреть сообщение
из одного блока
цикла
0
490 / 286 / 129
Регистрация: 30.10.2018
Сообщений: 1,309
17.10.2019, 21:27
Цитата Сообщение от BuJIJI Посмотреть сообщение
что я неправильно сделал и вы бы сделали по другому?
забыл бы о старом printf/scanf, все таки с потоками лучше работать. С++ же.
0
с++
1282 / 523 / 225
Регистрация: 15.07.2015
Сообщений: 2,562
17.10.2019, 21:27
Цитата Сообщение от BuJIJI Посмотреть сообщение
что я неправильно сделал и вы бы сделали по другому?
cout для кого сделан или это СИ, кстати можно использовать cin.get(); место _getch();
0
0 / 0 / 0
Регистрация: 19.11.2012
Сообщений: 114
17.10.2019, 22:18  [ТС]
До начала обучения в ВУЗе я делал так как вы пишите: использовал cout,cin; переменные в одну строку не объявлял. Но изучив материалы, которые дал ВУЗ, я увидел там эти "фишки" и подумал, что я делал всё неправильно.
0
7438 / 5030 / 2892
Регистрация: 18.12.2017
Сообщений: 15,692
17.10.2019, 22:35
BuJIJI, на каком языке должен быть написан код ? (Ваш код - на С, не С++)
цикл for c break работать конечно будет, но зачем применять лишний оператор, есть же while
0
 Аватар для Avaddon74
571 / 353 / 133
Регистрация: 15.09.2017
Сообщений: 1,239
17.10.2019, 22:45
Цитата Сообщение от BuJIJI Посмотреть сообщение
До начала обучения в ВУЗе я делал так как вы пишите
С одной стороны хорошо, что вам это показывают, и чтоб вы знали и С и ++, но ведь об этом объяснить нужно, чего я подозреваю не сделали...
Цитата Сообщение от BuJIJI Посмотреть сообщение
int k; float A, B, p, q;
Тут вам уже объяснили, я просто уточню, в строчку можно, НО разные типы разнесите по разным строчкам иначе сливается в один тип
C++
1
2
int k;
float A, B, p, q;
Цитата Сообщение от BuJIJI Посмотреть сообщение
printf("1.Заработная плата(руб)\n2.Курс доллара\n
Если в выводе будет на разных строках, то лучше и в коде это так же оформить,
во-первых, удобнее читать и сразу представлять как будет в выводе
во-вторых, сразу заметите где пропустили перенос строки, а можно и raw-строки использовать:
C++
1
2
3
    string str = R"(test
test2
test3)";
И последнее, больше пробелов и переносов строк, отделяйте блоки друг от друга отдельной строкой,блок вывода, строка, блок ввода, строка, цикл, строка и т.д. Не нужно стараться уместить весь код таким способом в маленькую зону, понижается читаемость. У вас блок кода или функция по возможности должны вмещаться в экран, но не весь код
Конечно это моё мнение, а сколько людей, столько и мнений
0
0 / 0 / 0
Регистрация: 19.11.2012
Сообщений: 114
17.10.2019, 23:11  [ТС]
Почему код C работает с компилятором C++?

Цикл for с break потому как по условию необходимо рассмотреть 12 итераций, не более.
0
490 / 286 / 129
Регистрация: 30.10.2018
Сообщений: 1,309
18.10.2019, 19:22
Цитата Сообщение от BuJIJI Посмотреть сообщение
Почему код C работает с компилятором C++?
С++ полностью поддерживает все что есть в С без каких либо (почти) исключения. С - это база для С++, даже в названии можно это увидить.
0
Эксперт С++
 Аватар для hoggy
8973 / 4319 / 960
Регистрация: 15.11.2014
Сообщений: 9,760
19.10.2019, 02:00
Цитата Сообщение от kitsoRik Посмотреть сообщение
С++ полностью поддерживает все что есть в С без каких либо (почти) исключения.
как ты интересно сформулировал.

перевожу на нормальный русский:
с++ не полностью совместим с языком си,
есть противопоказания.
0
Вездепух
Эксперт CЭксперт С++
 Аватар для TheCalligrapher
12937 / 6804 / 1821
Регистрация: 18.10.2014
Сообщений: 17,217
19.10.2019, 02:49
Цитата Сообщение от kitsoRik Посмотреть сообщение
С++ полностью поддерживает все что есть в С без каких либо (почти) исключения.
Нет, С++ уже давно НЕ поддерживает "все что есть в С". И даже если вести речь о "классическом" С (не принимая во внимание специфику С99 и позже), "исключений" очень много.

Поэтому ни о каком "С++ полностью поддерживает все что есть в С" не может быть и речи.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
19.10.2019, 02:49
Помогаю со студенческими работами здесь

Оценка стиля написания кода
Здравствуйте! Я к вам пожаловал не с очередной проблемой. Код у меня рабочий. Меня интересует то, насколько стандартен мой стиль...

Оценка кода динамического двумерного массива
Доброго времени суток. Прошу оценить правильность выделения и удаления памяти, а также заполнения массива. Два примера. Вот первый. В...

Список учеников имеет следующую структуру: фамилия – класс - оценка по алгебре - оценка по физике - средний балл
Задание такое Список учеников имеет следующую структуру: фамилия – класс - оценка по алгебре - оценка по физике - средний балл. При...

Вынести основную часть кода кода в отдельную функцию
помогите пожалуйста переделать это на функцию... #include &lt;locale.h&gt; #include &lt;conio.h&gt; int main() { setlocale(0,&quot;&quot;); ...

Исправление кода. Функция по нахождение суммы между мин и макс показателями массива. Корректировка кода
Прошу помощи. Написал программку, проблема состоит в функции summ в ней происходит нахождение минимального и максимального элемента. Что то...


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

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
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, то после закрытия окошка. . .
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer
8Observer8 08.02.2026
Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .
SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств
8Observer8 07.02.2026
Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве
8Observer8 02.02.2026
Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru