Правила не из топ правил из интернета.
Я, например, ВСЕГДА в switch в default делаю стоп и логирование, на случай чего-то непредвиденного.
Code: Select all
switch ($this->view_acc) {
case 1:
require '../des/acc_all.php';
break;
case 2:
require '../des/acc_frend.php';
break;
case 3:
require '../des/acc_private.php';
break;
default:
$e->logerr_exit('error $view_acc in ' . __METHOD__);
break; //лишний брек в дефолте? та и х с ним
}
###############################################
switch ($this->action) {
case '+':
$this->new_crypto_balans = $this->one_user_current_balans + $this->second_digit;
break;
case '-':
$this->new_crypto_balans = $this->one_user_current_balans - $this->second_digit;
break;
case '*':
$this->new_crypto_balans = $this->one_user_current_balans * $this->second_digit;
break;
case '/': //если вторая цифра ноль, помненять знак
if ($this->second_digit == 0) {
$this->new_crypto_balans = -$this->one_user_current_balans;
break;
}
$this->new_crypto_balans = $this->one_user_current_balans / $this->second_digit;
break;
default: //всегда должен быть дефолтное опове
$e = new Err();
$e->logerr_exit("unk action in cryptoCu");
break;
}
#####################################################
switch ($wsh) {
//друзі
//мои друзья и друзья польщов - если 5 то енопки држбы= . убрать лишие классы!
case 1:
$vld->is_vars_set($pg, $sort);
$myfr = new MyAndOneUserFrends($_SESSION['uid']);
$vld->is_numpg_correct_pg_sort($myfr);
$user->view($myfr);
break;
...
...
...
default:
$ip = new Ip();
$e = new Err();
$str = "unk wsh in show.php";
// $e->logerr($str);
$e->ins_err($ip->ip, $str);
// $rd->rd2_404();
exit;
break;
}
Page::eop();
Какие правила вы применяете, используете?
ДОписываю: у меня лишние break в default: потому что я использую автодополнния/сокращения при кнопкотыканьи. для того шобы быстро создатть блок switch case я набираю а клавиатуре swca жму на Enter и оно у меня автоматом выводмится с несколькими case :
В ИДЕ давно так сделал и шаблоны автодополнения не пересматривал/не исправлял.
Дописываю: ого, а что если $this->one_user_current_balans уже будет минусовое. Нужно протестировать. Возможны проблемы. Как хорошо что я именно этот кусок решил показать.