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)} | ||