Как протестировать наследство без боли и страха

[@ErickSkrauch]

Ваши примеры не отвечают действительности — они читаемые.

[@ErickSkrauch]

Должно быть хотя бы как-то так:

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

[@jced]

Вы знаете, я минут 10 потратил на то, чтобы понять, что с моим кодом не так и что плохого в том, что примеры читаемы… Сходил выпил чаю, и только прочитав Ваш комментарий раз 5-й, наконец понял, что это сарказм!
Видимо вечер пятницы сказывается :)

[@Stalker_RED]

Инструменты автоматического форматирования существуют очень давно, и даже встроены в большинство IDE.

[@ErickSkrauch]

Когда это кого останавливало? Как будто вы не работали с джуниорами.

[@Stalker_RED]

Я не о том. Если вдруг вам приходится читать плохо отформатированный код, что мешает отформатировать его перед тем как читать?

[@kester]

часто останавливает куча «мусорных» изменений и git blame после них

[@search]

Мой любимый паттерн программирования на случай столкновения с легаси кодом: Garbage Wrapper. Только что попробовал погуглить по этому запросу но ничего не нашел. Может он как-то по-другому называется? Но суть: оборачиваем код, который будет в последстивии рефактриться, своим классом и таким образом получаем API, которое в последствии тестируется и вызывается другими методами.

[@TheGodfather]

На моем проекте как раз похожая ситуация, но пока не могу собраться с силами, чтобы начать рефакторинг. Может, подкинете ссылок, где почитать про этот подход?

[@search]

Ищу уже четвёртый день и не могу никак найти. Мой товарищ однажды рассказал про этот паттерн и мне название понравилось. Он, наверное, сам его изобрёл.

[@jced]

Есть подозрение, что речь идет о паттерне «Адаптер», а Ваш товарищ, скорее, изобрел для него новое название, которые точнее описывает его применение по отношению к устаревшему коду. По крайней мере из Вашего описания мне так показалось.

[@search]

Скорее уж фасад.

[@jced]

Согласен, «фасад» более подходящий.

[@0xC0CAC01A]

После того как проект меняет несколько владельцев, получаем несколько слоёв garbage wrapper-ов, каждый из которых немного глючит. Поэтому только честный рефакторинг, но не сразу.

[@search]

Ну не знаю. Обычно, если программист догадался написать апиху к инди-коду, вместо того чтоб продолжать великое костыльное дело, то такую апиху волне можно повторно использововать и развивать. Ну и по-тихоньку рефакторить оставшуюся красоту.

[@alekssamos]

Я практически так и делаю бывает…

[@vanxant]

В примерах не хватает глобальных переменных!

global $USER, $DB;