2018-04-12 Не Протестировал Код
Различия
Здесь показаны различия между двумя версиями данной страницы.
|
правила_разработки:как_не_надо_делать:2018-04-12_не_протестировал_код [12.04.2018 09:28] admin Approved(admin 2018/04/12 09:28) |
правила_разработки:как_не_надо_делать:2018-04-12_не_протестировал_код [20.05.2019 15:18] |
||
|---|---|---|---|
| Строка 1: | Строка 1: | ||
| - | **Нарушено правило**: Тестировать код после любых изменений.\\ | ||
| - | **Постановка задачи**: Использовать в функции определения выходного техподдержки функции из библиотеки date_lib.\\ | ||
| - | **Ошибка:** После первой ошибки добавил "return false", и думая, что это очень небольшие изменения не стал тестировать код. В коде появилась новая ошибка забыл поставить ";" после false\\ | ||
| - | **Как надо делать:** Тестировать код, даже после минимальных изменений. Даже если Вы просто добавили комментарий или изменили string.\\ | ||
| - | <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)} | ||
admin