Алексей Владимирович

Есть мнение, что это

else if ($result1['scored'] == $result2['scored']) {
            return null;

лишнее условие.

1 0

Kirill Mokevnin

Сработает и без этого, но код это не просто способ решить задачу, в первую очередь это способ выразить мысль (спецификацию) и на практике важнее соответствовать этой спецификации, чем микрооптимизациям. Вот тут разбирается другой случай, но с похожей идеей: https://github.com/Hexlet/code-reviews/blob/master/01.php.md

0

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

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

Нажимая кнопку «Зарегистрироваться», вы даёте своё согласие на обработку персональных данных в соответствии с «Политикой конфиденциальности» и соглашаетесь с «Условиями оказания услуг».

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

Тимур Савин 06 июня 2020 →

:( Warning: Test case class not matching filename is deprecated in /usr/src/app/tests/...

Иван Орешков 26 мая 2020 →

В подсказке сказано: arraymap сохраняет ключи. Чтобы их сбросить, используйте arrayvalues Наверное, имелось...

Михаил Чудинов 21 июля 2017 →

запихнул zip оба входных массива в один при помощи map и анонимной функции которая сравнивает результаты и ...

Дмитрий Макух 15 апреля 2017 →

Добрый день я так и не понял почему код не проходит тесты без arrayvalues() если я эту программу запускаю н...

Ruslan Demin 13 апреля 2017 →

Имеет ли место быть данное решение? ``` function bestAttempt($first, $second) { if (empty($first) || em...