EK
ручное тестирование по времени занимает столько же как и юнит тесты, разница только в том что юнит можно хоть 10 раз на день крутить
нет, юнит тестами можно регресс искать
Size: a a a
2020 June 25
EK
или нарушения прошлых договоренностей
EK
чем больше проект тем больше договоренностей которые надо соблюдат
EK
и тем больше надо проверять вручную
EK
начиная с какого то этапа, автоматизация начинает выигрывать
другое дело что она далеко не всем и не всегда
другое дело что она далеко не всем и не всегда
EK
имхо естественно
MM
Что в php-шторме отвечает за уродование else if
MM
Вместо else if получил лестницы из else { if(..) ... else { if(..) }
ВУ
1) получил в результате чего?
2) else/elseif не стоит юзать вообще. заменяй на приватные методы с ранним return
2) else/elseif не стоит юзать вообще. заменяй на приватные методы с ранним return
MM
1) получил в результате чего?
2) else/elseif не стоит юзать вообще. заменяй на приватные методы с ранним return
2) else/elseif не стоит юзать вообще. заменяй на приватные методы с ранним return
1) автоматического рефакторинга
S
1) получил в результате чего?
2) else/elseif не стоит юзать вообще. заменяй на приватные методы с ранним return
2) else/elseif не стоит юзать вообще. заменяй на приватные методы с ранним return
новые веяния? if не в тренде?
MM
//+проверочки
if (preg_match("@Микроволновые печи@ui", $category)) {
if ($e = Advert::CheckIsSelected($microwave_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
if (preg_match("@Холодильники@ui", $category)) {
if ($e = Advert::CheckIsSelected($refrigerators_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
if (preg_match("@Микроволновые печи@ui", $category)) {
if ($e = Advert::CheckIsSelected($microwave_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
if (preg_match("@Холодильники@ui", $category)) {
if ($e = Advert::CheckIsSelected($refrigerators_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
MM
раньше всё это представляло красивый if .. elseif
S
//+проверочки
if (preg_match("@Микроволновые печи@ui", $category)) {
if ($e = Advert::CheckIsSelected($microwave_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
if (preg_match("@Холодильники@ui", $category)) {
if ($e = Advert::CheckIsSelected($refrigerators_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
if (preg_match("@Микроволновые печи@ui", $category)) {
if ($e = Advert::CheckIsSelected($microwave_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
if (preg_match("@Холодильники@ui", $category)) {
if ($e = Advert::CheckIsSelected($refrigerators_brand)) {
$errors[] = '<i>Марка</i> - обязательное поле';
}
} else {
тут избыточность кода, но рефакторинг может быть неочевиден
ВУ
1) автоматического рефакторинга
Editor > CodeStyle > PHP > вкладка Code Conversion > convert else if
ВУ
новые веяния? if не в тренде?
я хоть слово про if сказал?)
S
Editor > CodeStyle > PHP > вкладка Code Conversion > convert else if
этими инструментами пользоваться осторожно