C++ : Магические числа

Вспомним один из прошлых уроков:

// Перевод евро в рубли через доллары
int main() {
  int euros { 1000 };
  auto dollars { euros * 1.25 }; // 1250
  auto rubles { dollars * 60 }; // 75000

  std::cout << rubles << std::endl; // => 75000
}

С точки зрения профессиональной разработки, такой код «пахнет». Так описывают код, который не соответствует так называемым лучшим практикам (best practices). И причина здесь вот в чем: уже сейчас, глядя на числа 60 и 1.25, вы скорее всего задаетесь вопросом: «что это за числа?». А представьте, что будет через месяц! А как его поймет новый программист, не видевший код ранее? В нашем примере контекст восстанавливается благодаря грамотному именованию, но в реальной жизни код значительно сложнее, и поэтому догадаться до смысла чисел зачастую невозможно.

Этот «запах» называют магические числа (magic numbers). Числа, происхождение которых невозможно понять без глубокого знания происходящего внутри данного участка кода.

Выход из ситуации прост: достаточно создать переменные с правильными именами, как все встанет на свои места.

int main() {
  auto dollars_in_euro { 1.25 };
  int rubles_in_dollar { 60 };

  int euros { 1000 };
  auto dollars { euros * dollars_in_euro }; // 1250
  auto rubles  { dollars * rubles_in_dollar }; // 75000

  std::cout << rubles << std::endl; // => 75000
}

Обратите внимание на следующие детали:

  • Именование snake_case
  • Две новые переменные отделены от последующих вычислений пустой строчкой. Эти переменные имеют смысл и без вычислений, поэтому такое отделение уместно, оно повышает читаемость.
  • Получился хорошо именованный и структурированный код, но он длиннее прошлой версии. Так часто бывает, и это нормально, потому что код должен быть читабельным.

Задание

Вы столкнулись с таким кодом, который выводит на экран среднесуточную температуру в Фаренгейтах:

std::cout << "Average daily temperature: " << 588 / 24;

Как видите, это магические числа: непонятно, что такое 588 и что такое 24.

Избавьтесь от магических чисел, создав новые переменные, а затем выведите текст на экран.

Получится так:

Average daily temperature: 24

Названия переменных должны передавать смысл чисел, но должны при этом оставаться достаточно короткими и ёмкими для комфортного чтения.

Помните: код будет работать с любыми названиями, а наша система всегда проверяет только результат на экране, поэтому выполнение этого задания — под вашу ответственность.

В С++ при делении если оба оператора являются целыми числами, то результат будет равен целой доли частного. Незабудьте в конце вывода добавить перевод строки.

Как с вами связаться? 😶

Команда проекта находится в телеграм-сообществе по ссылке https://ttttt.me/HexletLearningBot. Там можно задать любой вопрос и повлиять на проект

Упражнение не проходит проверку — что делать? 😶

Если вы зашли в тупик, то самое время поговорить с нашим асситентом Тота во вкладке "Обсуждение". Как правильно задать вопрос:

В моей среде код работает, а здесь нет 🤨

Тесты устроены таким образом, что они проверяют решение разными способами и на разных данных. Часто решение работает с одними входными данными, но не работает с другими. Чтобы разобраться с этим моментом, изучите вкладку «Тесты» и внимательно посмотрите на вывод ошибок, в котором есть подсказки.

Мой код отличается от решения учителя 🤔

Это нормально 🙆, в программировании одну задачу можно выполнить множеством способов. Если ваш код прошел проверку, то он соответствует условиям задачи. В редких случаях бывает, что решение подогнано под тесты, но это видно сразу.

Прочитал урок — ничего не понятно 🙄

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

Нашли ошибку? Есть что добавить? Пулреквесты приветствуются

Привет! Я Тота и моя задача помочь в обучении. Чтобы активировать меня, нужно зарегистрироваться или залогиниться, если у вас уже есть аккаунт

Loading...

Ваше упражнение проверяется по этим тестам

1#include "test_helper.h"
2
3TEST_CASE("testing solution") {
4  std::string expected = "Average daily temperature: 24\n";
5  std::string actual = get_output(sizeof(expected));
6
7  CHECK(actual == expected);
8}
9

Решение учителя откроется через:

20:00
waiting_clock