2018-04-12 Не Протестировал Код
Различия
Здесь показаны различия между двумя версиями данной страницы.
Следующая версия | Предыдущая версия | ||
правила_разработки:как_не_надо_делать:2018-04-12_не_протестировал_код [12.04.2018 07:48] николай_глазов создано |
правила_разработки:как_не_надо_делать:2018-04-12_не_протестировал_код [20.05.2019 15:18] (текущий) |
||
---|---|---|---|
Строка 1: | Строка 1: | ||
- | /*Не удаляйте эту строку и ниже!*/ | + | **Нарушено правило**: Тестировать код после любых изменений.\\ |
+ | **Постановка задачи**: Использовать в функции определения выходного техподдержки функции из библиотеки date_lib.\\ | ||
+ | **Ошибка:** После первой ошибки добавил "return false", и думая, что это очень небольшие изменения не стал тестировать код. В коде появилась новая ошибка забыл поставить ";" после false\\ | ||
+ | **Как надо делать:** Тестировать код, даже после минимальных изменений. Даже если Вы просто добавили комментарий или изменили string.\\ | ||
- | {(rater>id=1|name=Прочитал_правила_разработки:как_не_надо_делать:2018-04-12_тестировать_код|type=vote|trace=user|tracedetails=1)} | + | <color #ed1c24>**Неверное решение**</color>\\ |
+ | <code php> | ||
+ | /** | ||
+ | * Возвращает выходной ли у техподдержки | ||
+ | * @param string $product . reductor.sup или другие | ||
+ | * @return bool. phone_number | ||
+ | */ | ||
+ | function support_weekend() | ||
+ | { | ||
+ | if (isWeekendStrong() or date('H')< 10 or date('H') > 18 ) | ||
+ | return true; | ||
+ | return false | ||
+ | } | ||
+ | </code> | ||
+ | <color #22b14c>**Правильное решение:**</color>\\ | ||
+ | <code php> | ||
+ | /** | ||
+ | * Возвращает выходной ли у техподдержки | ||
+ | * @param string $product . reductor.sup или другие | ||
+ | * @return bool. phone_number | ||
+ | */ | ||
+ | function support_weekend() | ||
+ | { | ||
+ | if (isWeekendStrong() or date('H')< 10 or date('H') > 18 ) | ||
+ | return true; | ||
+ | return false; | ||
+ | } | ||
+ | </code> | ||
+ | /*Не удаляйте эту строку и ниже!*/ | ||
+ | ~~META: | ||
+ | user = sergey_osintsev | ||
+ | ~~ | ||
+ | ~~OWNERAPPROVE~~ | ||
+ | {(rater>id=1|name=Прочитал_правила_разработки:как_не_надо_делать:2018-04-12_не_протестировал_код|type=vote|trace=user|tracedetails=1)} |