Искусство работы с foreach, или Парадигма for-case

[norritt]

Хотя этот пост я обещал очень давно, руки дошли только сейчас. До того писал велосипед в виде парсера кода, так как бывший примитивный ломался на этом участке.

Ну да не суть. Сегодня у нас довольно большой кусок кода, в котором имеется довольно обширный набор индуизмов, антипаттернов, и просто идиотских решений. Итак, добро пожаловать

Comments

html

[ext_366780]

генерируемый html жутко нехорош

Re: html

[norritt]

Не спорю, хотя хотелось бы выслушать более развёрнутое мнение на эту тему.

Так как не было времени на переделку всей вёрстки, да и верстальщик из меня так себе (опыта нет), я остановился на варианте «переделать код, оставить html as-is». Рано или поздно вернёмся к этой части и переверстаем.

Re: html

[ext_366780]

ну в данном случае надо все стили вынести в файл, поскольку идет повторение оформления для каждой строки.
надо поработать над валидностью, раз img закрыт с помощью /> я понимаю что доктайп xhtml, получается все атрибуты надо брать в скобки.
ну и еще $result здесь по моему можно не собирать а выводить сразу ) не суммируя

Re: html

[norritt]

Ну, по порядку :)

1. в отдельный файл css энтот орёл, по всей видимости, принципиально не выносил. У меня это стоит в очереди, надо заняться, но рук нет. Вообще - в планах;
2. валидность да. Где могу и вижу - закрываю как />, но не всегда;
3. $result да, выводить теоретически можно сразу, но так было в оригинале, и я не рискнул от этого отходить. Чисто чтобы не поломалось ещё где-нить;
4. труъ-программеры не мешают html с кодом, так как это моветон. Но так как переделывать всё - не те трудозатраты, которые я мог себе позволить (особенно помножив на время, которое я потратил, раскуривая парадигму for-case).

За комменты спасибо, постараюсь ещё что-нить нарыть, если интересно :)