- 1
$txt['aeva_gallery'] = isset($txt['aeva_gallery']) ? $txt['aeva_gallery'] : 'Media';
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+149
$txt['aeva_gallery'] = isset($txt['aeva_gallery']) ? $txt['aeva_gallery'] : 'Media';
Кусочек языкового файла модуля медиагалереи
Oonitaz 27.03.2011 15:38 # −5
wvxvw 27.03.2011 17:09 # +5
Зачем 2 раза одно и то же делать?
Еще большее говно в том, что обращения делаются по каким-то строковым ключам неизвестно откуда полученым, но это, похоже норма для PHP - работать с динамическими массивами вместо объектов с задаными свойствами.
Oonitaz 29.03.2011 17:09 # 0
так правильнее
wvxvw 29.03.2011 17:32 # +1
Кроме того, если уж на то пошло и мы хотим игнорировать предупреждения, и нам не важно, что по этому индексу массива могло быть что-то, что может быть истолковано как false, тогда есть такой вариант, опять же, более приемлимый в виду того, что многие языки с похожей структурой его тоже поддерживают:
В отличие от последнего, который существует только в последних версиях языка, и, заметте, ни чем принципиально не лучше других предложеных.
Int 27.03.2011 17:17 # +2
Lure Of Chaos 27.03.2011 17:50 # +1
Oonitaz 28.03.2011 06:06 # +1
Так делать можно только в 5.3
prodigy 27.03.2011 19:14 # −1
Автор присваивает значение новой переменной(причем работать она будет как константа - ведь языковой файл, значения не будут меняться) путем проверки ее существования))
1. Нечитабельность кода
2. Путаница для переводчика(см. пункт №1)
3. Избыточность, ведь проще некуда было бы сделать вот так:
$txt['aeva_gallery'] = 'Галерея';
Все! Проще некуда, зачем на пустом месте что-то усложнять? В этом я и увидел говно.
Lure Of Chaos 27.03.2011 21:26 # +2
$txt['aeva_gallery'] = 'Галерея';
перепишет значение в любом случае, а сабж - только если такого индекса нет
в общем, надо делать так: http://govnokod.ru/6100#comment79186
bugmenot 27.03.2011 22:54 # +1
striker 28.03.2011 03:20 # +1
Lure Of Chaos 28.03.2011 15:21 # 0