Comments 18
Ваши примеры не отвечают действительности — они читаемые.
Должно быть хотя бы как-то так:
function poschitatchtonibud($abc, $def){
// Здесь обязательно должно быть подключение к базе дынных, без него никак
$result1 = mysql_query( "SELECT * FROM table1");
if ($result1["field"] == $abc) $result2 = mysql_query("Select * FROM table2" );
if ($result2["field"] == $def)
$result3 = mysql_query("SELECT * fRom table3");
return isset($result3)?$result3:$result2;
}
UFO just landed and posted this here
Вы знаете, я минут 10 потратил на то, чтобы понять, что с моим кодом не так и что плохого в том, что примеры читаемы… Сходил выпил чаю, и только прочитав Ваш комментарий раз 5-й, наконец понял, что это сарказм!
Видимо вечер пятницы сказывается :)
Видимо вечер пятницы сказывается :)
Инструменты автоматического форматирования существуют очень давно, и даже встроены в большинство IDE.
Мой любимый паттерн программирования на случай столкновения с легаси кодом: Garbage Wrapper. Только что попробовал погуглить по этому запросу но ничего не нашел. Может он как-то по-другому называется? Но суть: оборачиваем код, который будет в последстивии рефактриться, своим классом и таким образом получаем API, которое в последствии тестируется и вызывается другими методами.
На моем проекте как раз похожая ситуация, но пока не могу собраться с силами, чтобы начать рефакторинг. Может, подкинете ссылок, где почитать про этот подход?
После того как проект меняет несколько владельцев, получаем несколько слоёв garbage wrapper-ов, каждый из которых немного глючит. Поэтому только честный рефакторинг, но не сразу.
Я практически так и делаю бывает…
В примерах не хватает глобальных переменных!
global $USER, $DB;
Sign up to leave a comment.
Как протестировать наследство без боли и страха