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

Оценка кода

17.10.2019, 16:14. Показов 10240. Ответов 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
18144 / 10728 / 2066
Регистрация: 27.09.2012
Сообщений: 27,026
Записей в блоге: 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
12928 / 6796 / 1819
Регистрация: 18.10.2014
Сообщений: 17,194
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
Ответ Создать тему
Новые блоги и статьи
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Ниже машинный перевод статьи The Thinkpad X220 Tablet is the best budget school laptop period . Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы,. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru