Цитата (glock18 @ 2.07.2009 - 10:34) |
ты случаем тот мануал то качал |
Цитата |
так вот я лично ошибок пока там не находил вовсе |
Цитата (glock18 @ 2.07.2009 - 10:42) |
Я уже столько лет (целых, боюсь, 2 года!) работаю с мануалом на php.net |
Цитата (twin @ 2.07.2009 - 10:50) |
Для тех, кто хочет действительно научиться. |
Цитата (twin @ 2.07.2009 - 10:50) |
Там есть некоторые неточности, но это такая мелочь, которая вообще ни на что не влияет. |
Цитата (twin @ 2.07.2009 - 12:56) |
Это разделитель |
PHP |
$myrow = mysql_fetch_array($result); |
Цитата |
1. $alp - что обозначает? Если эта переменная находится в зоне видимости (визуальной), то можно сообразить, что массив значений, и то приходится отвлекаться, что бы разобраться в этом. Читаем мануал: Возвращает массив с обработанным рядом результата Логично назвать переменную $row или $rows на худой конец. Тогда ясно, что это такое. 2. Применение mysql_fetch_array() тут неоправдано, потому что используется только ассоциативный массив. Бережливее и прозрачнее, а значит уместнее тут mysql_fetch_assoc() 3. Открывающая фигурная скобка спрятана так, что не сразу её разглядишь. 4. Использование printf вообще моветон. Очень путанная и непрозрачная конструкция. 5. При обращении к элементам ассоциативного массива обязательно нужно ставить апострофы, так как иначе это интерпретируется не как строка, а как константа. 6. Стиль не выдержан. Операторы сравнения не отделены пробелом, хотя выше это сделано. 7. Что такое Z? Константа? Если да, то она должна быть осмысленной, отображать суть. Если строка - то в апострофах. 8. оператор break стоит в одной строке с if, что путает. Потому что не сразу ясно, что он относится к циклу. 9. Закрывающая фигурная скобка спрятана не лучше открывающей. 10. Теперь ляп. Очень серьёзный. Текст из базы выводится в браузер без предварительной обработки. Это чревато кривостью отображения в лучшем случае, XSS в худшем. Второй цикл аналогично. Так что почти каждый знак этого кода - кривой. Про логику отдельная тема, я не буду затрагивать. Ровно как и разделение её с представлением. |
PHP |
if ($alp[alp]==Z) break;} |
Цитата |
2. смешение html и php. |