[ Поиск ] - [ Пользователи ] - [ Календарь ]
Полная Версия: Простая регистрация
Страницы: 1, 2, 3
Valick
Цитата
на коленке писал и на хтмл забил

Дело даже не в отделении логики от представления и уж никак не в хтмл.
Говнокод у тебя именно на стороне РНР.
Ну и зачем писать что-то "на коленке", что бы потом просить найти ошибки и прокомментировать код, ты сознательно нас заставляешь ковыряться в какашках?
Если не получается писать грамотно "на коленке", то стоит задуматься о других местах.
В общем "у меня говно потому что на коленке" - это ни разу не отмаз.

_____________
Стимулятор ~yoomoney - 41001303250491
Миша
Это я про хтмл, что бы на него не смотрели, в остальном делаю выводы.

_____________
Принимаю заказы, писать в ЛС
Миша
Цитата (OleKh @ 19.02.2015 - 12:47)
Цитата (Медведь @ 19.02.2015 - 10:41)
а сейчас пхп


ну, так как правильно уложить) переменную $notification в $dubl?

$notification = "<font color='red'>Пользователь с таким логином УСПЕШНО СОЗДАН.</font><br><br>";

echo $dubl;

В переменной $dubl в нужное место вставить $notification

_____________
Принимаю заказы, писать в ЛС
OleKh
Цитата (Медведь @ 19.02.2015 - 10:52)
про хтмл, что бы на него не смотрели

если бы комментарий был по поводу незакрытых тегов, другое дело, внимание обратил просто на этот участок кода, вероятно что планируется подключение какого-то супер современного шаблонизатора, т.к. задача не настолько проста чтобы пренебрегать
johniek_comp
комментарии в коде то что надо. я бы сам не понял что там происходит)))

_____________
user posted image
OleKh
да уж) там только Медведь смог бы меня понять)))
stump
Цитата (Медведь @ 19.02.2015 - 11:22)
Цитата (stump @ 18.02.2015 - 23:38)

Медведь

CREATE DATABASE $db IF NOT EXISTS

Кстати говорите книги читай, а этот фрагмент с @ именно из книги Котерова, 479 страница.

Котеров может классный мужик и рассказывает правильно. Только как правило подразумевается что БД уже существует и надо только селекты и инсерты делать для новичков.

Если БД не создана ранее то надо делать инсталятор который создаст БД, таблицы и пр. и позволит нормально логиниться и создаваться.

Создавать постоянно БД - почти вершина хардкода!

_____________
Трус не играет в хокей
johniek_comp
если у ТС какие-то проблемы с БД, то почитай как создать файл миграции, и что там писать

_____________
user posted image
volter9
Написал простую ООП версию той же регистрации:
https://github.com/Volter9/Medved/tree/master/oop

Кому интересно могут сравнить smile.gif (и попинать biggrin.gif )

_____________
Мой блог
Быстрый ответ:

 Графические смайлики |  Показывать подпись
Здесь расположена полная версия этой страницы.
Invision Power Board © 2001-2024 Invision Power Services, Inc.