StalkerClasses
Новичок
Кто-то про IF-ы говорил
github.com
GitHub - piotrplenik/clean-code-php: :bathtub: Clean Code concepts adapted for PHP
:bathtub: Clean Code concepts adapted for PHP. Contribute to piotrplenik/clean-code-php development by creating an account on GitHub.
github.com
PHP:
<?php
/**
* Чистый код (рекомендации)
* 1) Понятен
* 2) Простая логика
* 3) Одна задача
* 4) Минимальные зависимости
* 5) Нет того, что можно улучшить
*/
// Название переменных должно иметь смысл
// Имена классов не должны содержать глагола
// Имена методов должны говорить что они делают и содержать в своем имени глагол
$dateStart = '----';
class Car;
function openDoor();
// В значении переменных вместо цифр используйте константы
$accessRead = 1; // ?? лучше class::accessRead = 1;)
// Вместо названия переменной $val - лучше назвать что это
foreach($locations as $location){ // вместо foreach($locations as $k => $v)
}
// Избегайте вложенных if, откажитесь от else
function () {
if ($a == $b) {
// if(){} // ! вложенные if
return false;
}
if ($dateStart == $dateEnd) {
return false;
}
return true; // ! Вместо else
}
// Избегайте условий с "!" отрицанием
if(!in_array($a,"val")){
}
// При сравнении значений используйте ===
if ($a === $b) { // вместо $a == $b
}
// Количество аргументов функции не должно превышать 7 - иначе она пытается делать слишком много
// Если функция имеет слишком большое тело, ее имеет смысл разбить на подфункции
// Функция должна решать 1 задачу и не иметь побочных эффектов (пишем в файл, но не авторизуем пользователя)
function openDoor(string $openComment = ''){
// Далее если тело функции идет большым имеет смыслы разбить его на отдельные функции
// А часть функций вынести в отдельные классы
}
