SS
Вы только полюбуйтесь этим шедевром
Size: a a a
2021 August 16
IO
+++
AE
Ёбаный стыд. Это попытка в динамически подключаемые скрипты 91го года ?
SS
Это меня попросили добавить фичу в сайт, сделанный на скрипте PlayTube. Устанавливаешь скрипт, подключаешь к БД, ставишь логотип, и у тебя свой видео-хостинг.
Открываю я код этого PlayTube, а там такое.
Открываю я код этого PlayTube, а там такое.
SS
Еще переменная $pt (видна на скрине) - это глобальный объект, с неопределенным количеством свойств. Я прошёл регуляркой по проекту - насчитал около 170 свойств. И 10000 обращений.
AE
Так может он переопределяется в каждом файле? Просто назван одинаково.
AE
Или его requireом тащат ?
SS
Там один require файла с объявлением этой переменной, и она становится доступна везде, без всяких испортов
S
Какая то ошибочная карьерная лестница
S
Забагованная
S
И там ступеньку одну не нашло
SS
Я сперва подумал, что за хрень? Почему код выглядит так, как будто человек хорошо знает пхп и посвятил ему много времени, но в то же время настолько похож на спагетти? А потом понял - вот она, парадигма динамической типизации - всё есть свалка данных
AE
Если ты допускаешь эту свалку - да.
SS
Кстати, вы же вроде на php пишете? Не хотите на исходники посмотреть?
SS
Мне нужно мнение, стоит ли тратить время на этот код, или выкинуть к чертям и написать заново
AE
Я js пишу. С пхп не работал уже года 4 да и не знал его досконально никогда. Но я уже от сюда могу сказать что выкинуть нахрен, если оплатят время на переделывание, если нет - допили только то что просят.
AE
Но могу и глянуть, если Гит есть
SS
Не, гита нет
SS
Ну ладно тогда
AE
Ну можно и архивчиком конечно, но тогда только завтра получится, когда за компом буду