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