1. я просто когда то привык так делать...теперь всегда так делаю
А Вам самим то приятно, разбирать такой код?)))
HTML'ом должен заниматься браузер, а не PHP)
GEN_18 пишет:
2. это опять же по старинке, другого способа вывода данных из массива я не знаю
Есть while(). Просто while. Разница в том, что do while, выполнится хотя бы один раз, даже если условие ложное. Не уверен, что в Вашем случае это надо.
GEN_18 пишет:
if (!empty($sertif)){
Это вообще бред. Если Вы хотите проверить, есть ли записи в таблице, используйте mysql_num_rows().
GEN_18 пишет:
3. вот это вот учту. но ))
А еще лучше, включить запись ошибок в логи. Ибо думаю посетителю незачем видеть ошибки на сайте.
GEN_18 пишет:
4. это опять же, по старинке
Никогда не останавливайтесь на одной функции. Найдите ее аналоги, сравните, и делайте выводы.
GEN_18 пишет:
5. я с такими функциями не сталкивался....
На этом же PHP.su, в разделе "Функции баз данных", есть описания. Почитайте. Или PHP.net.
Все что я пишу - это мое личное мнение. Я ни в коем случае не заставляю никого прислушиваться к нему. А то сейчас народ подумает "что за умник нашелся, учит тут всех")))))
1. Не надо HTML выводить через всякие echo, или держать в переменной, как у Вас.
2. Вот вы используете do while. Можете пояснить, почему? Или Попова насмотрелись?)))
3. После mysql_query, вставьте OR die(mysql_error()). Ибо если возникнет ошибка в запросе - Вы даже об этом не узнаете.
4. mysql_fetch_array - она возвращает несколько видов массивов. А зачем, если Вы используете только ассоциативный? Смотрите в сторону mysql_fetch_assoc.
5. Не используйте эти mysql_* функции, используйте MySQLi, PDO... Современно, удобно, выгодно.
Когда вижу капчу, то хочется плеваться и разбить клавиатуру об голову того человека, который эту капчу установил.
Ну тут еще смотря какая каптча. Можно сделать что-то типа "Решите пример: 1+3" и т.п.
Не думаю что посетителю будет сложно решить пример первого класса)))