Вопрос пользователя Александр Михайлов в испытании «Сравнение версий»

Александр Михайлов

https://ru.hexlet.io/code_reviews/228615 Мне интересно насколько мое решение костыльное? есть ли что то концептуально неправильное в нем? про spaceship только узнал, прикольно решение учителя конечно вау, но неочевидно на мой взгляд

3 0

Roman Kozlov

Александр, добрый вечер!
Такой вариант тоже допустим.
Не обязательно было создавать константы, можно сразу возвращать нужное число (так код выглядит ещё понятнее).
Почитайте нашу статью про именование, это будет полезно.

0

Александр Михайлов

Константы сделал чтоб не было "магических чисел"

Вместо FIRST_MORE нужно было использовать что то типа FIRST_BIGGER

Спасибо, про именование читал, понравилось, но нужно будет еще перечитать

0

Roman Kozlov

Про магические числа вы верно подметили!
Отлично! :)

0

Есть вопрос или хотите участвовать в обсуждении?

Зарегистрируйтесь или войдите в свой аккаунт

Отправляя форму, вы соглашаетесь c «Политикой конфиденциальности» и «Условиями оказания услуг».

Похожие вопросы

Ilia D 13 октября 2020 →

Добрый день! подскажите - где почитать про решение?! прошелся по урокам ранее - не смог найти. спасибо $1 ...

Илья Идущий 11 мая 2020 →

Невнимательно прочитал задание, и не прошел валидацию со значениями '0.2' и '0.12'. Потом никак не мог поня...

Валентин Иванов 21 января 2019 →

Подскажите, где можно почитать как работает сравнение массивов (как в решении Учителя)? С ходу не получилос...

Luba Parfilova 17 июля 2018 →

Как же работает оператор сравнения > с массивами? http://php.net/manual/ru/language.operators.array.php

Константин Кулишов 04 октября 2016 →

Добрый день! Пытался решить данную задачу через функцию compareVersion (думал пойти самым простым путем), р...