Jump to content

Leaderboard

Popular Content

Showing content with the highest reputation on 10/12/2011 in all areas

  1. Ой, много чего можно сказать по поводу представленного кода. Тут видно, что автор страдает заболеванием «ООП головного мозга» (без обид, ага?). Видно, что нет никакого понимания для чего нужны классы, объекты и как их правильно использовать. Я сам таким раньше страдал из-за незнания синтаксиса PHP. В первую очередь непонятно зачем было создавать класс для работы с MySQL — для этого уже есть mysqli (от использования которого я уже тоже отказался) и PDO. Само собой, если нужен какой-то дополнительный функционал, то можно всегда сделать class SQL extends PDO и уже там наворачивать. Вот этот кусок кода public $server= '*****', $user='******', $password='******', $database='****' тоже не выдежривает никакой критики — классы нужны для абстракций и интерфейсов и указывать параметры соединения с БД прямо в классе — нелепо. Может же возникнуть ситуация, когда надо в одном скрипте работать с двумя БД, а то и вообще с двумя серверами. Правильно делать так: function __construct($server, $user, $password, $database) { // Тут подключаемся к БД } Ну и, само собой, заводить данные при создании объекта. Короче, весь класс SQL надо переделать. Теперь класс для регистрации пользователей. Зачем он наследует класс SQL? Если вам надо делать запросы к БД, то я бы это решил так $DB=new SQL($server, $name, $password, $database); class Register { function __construct() { global $DB; $this->db=$DB; } } И теперь вы можете делать запросы типа так: $this->db->query('SELECT * FROM `teablename` LIMIT 0, 10'); Ну и несколько вопросов на дом: 1. Зачем в тексте ошибок есть элементы разметки? 2. Почему пароль должен содержать от 6 до 20 букв или цифр? Спецсимволы повышают стойкость пароля. Ну и, если ограничение «снизу» как-то ещё можно объяснить, то зачем стоит органичение в 20 символов? Если нужна разумность, то ставьте 255, например — столько символов в пароле маловероятно, а больше 20 — вполне. 3. Регулярка на проверку email, по-моему, не идеальна — адрес «-@-.qq» сматчится, но не будет являться хоть сколь-нибудь валидным адресом, а, например, «username@domainname.travel» не сматчится. 4. Зачем делать addslashes? Мало того, что у вас логин и так не может содержать ничего такого, что бы надо было экранировать, так ещё и вообще не понятно зачем это делать в наш атомный век, когда есть prepared statements. 5. Зачем нужен метод install? Если вы заранее знаете какую структуру будет иметь у вас табличка, то не будет ли правильным её создать один раз, а не хранить постоянно бесполезную информацию в коде? 6. Для чего у вас один метод проверяет, а второй добавляет? Не будет ли правильнее написать один метод для добавления, который бы заодно и проверял всё, что надо? А если что-то не так, что бросал бы исключение или бы возвращал код ошибки? Типа такого: try {$reg -> add($_POST['rlogin'] , $_POST['rpass'] , $_POST['rpassr'] , isset($_POST['email']);} catch (Exception $ex) { $error=$ex->getMessage(); } ну или как-то так, например if (!$reg -> add($_POST['rlogin'] , $_POST['rpass'] , $_POST['rpassr'] , isset($_POST['email'])) $error=$reg->error(); Вот краткий список того, на что следует обратить внимание в первую очередь. Ждём следующую версию.
    2 points
  2. Никакого бага нет. 1) открываем фаербаг и смотрим: <table class="releco" style="width: 960px;"> <caption>Solid state relay</caption> <colgroup> <col> <!-- 1st child --> <col> <col> <col> <col> <col> <col> </colgroup> 2) Открываем Оперу и смотрим: <table class="releco" style="width: 960px;"> <caption>Solid state relay</caption> <!-- 1st child --> <col/> <!-- 2nd child --> <col/> <col/> <col/> <col/> <col/> <col/> Варианты решения: 1) Явно добавить colgroup, к слову в Опере 12 он уже проставляется автоматически также как и ФФ 2) Использовать :nth-of-type
    2 points
  3. Приветствую. Создал новый проект, сначала создавал для себя, а затем решил открыть для всех http://likecode.ru — хостинг кода. Если вам нужно передать кому то код, но через skype/icq неудобно, по email много кликов, или показать на сайте, то Likecode отлично подойдет для этой задачи. А html код можно еще и запустить сразу на сайте и увидеть его в действии, например http://likecode.ru/code/4e81a9ed89682/preview/ (если изменять код, то обновления видны без перезагрузки страницы)
    1 point
  4. Ссылка не открывается. Ну да и фиг с ней. В общем предлагаю маркеры сделать абсолютными элементами, желательно на :before
    1 point
  5. heywait и nightgremlin, извиняюсь. Действительно про это упоминается. Текст перечитывал 2 или 3 раза, но т.к. был с просони - мозг не зацепился за эту фразу.
    1 point
  6. Все! Спасибо Вам всем большое! Получилось!!!... Пойду забухаю
    1 point
  7. об этом вскользь упомянул автор поста
    1 point
  8. Хватит, пожалуйста
    1 point
  9. Ну вы слоупок... И вообще, я не понимаю этой истерии вокруг Джобса. Да он известный человек в определенных кругах, да он талантливый бизнесмен, ну и что? Все это похоже на какую-то секту чесслово... Вот смотрел в пятницу в новостях как люди цветы несут (!) к американскому посольству, ужос, а штатах так вообще - айпады кто-то не жалеет, в кучку складывает... Вам вот такой вопрос: будете ли вы так же скорбеть когда умрет глава Самсунга скажем? Не менее инновационная компания, между прочим, за 15 лет (на моих глазах) из никому неизвестного корейского производителя, превратившаяся в мирового гиганта. Будете ли вы расстраиваться и писать "пичалька" когда умрет, скажем, Ларри Пейдж? Не подумайте, я не циник, просто для меня немного странно все это.
    1 point
  10. Привет ребят! Нужен очень совет! Как пишутся сайты, и т.д. все знаю. Разобрать чужой код могу, даже рипы делала, и меняла для себя. (уж извините ) А вот, просто по макету сверстать сама вообще не могу. Вроде мысль есть, представляю как все будет, и т.д. А сверстать не могу, никак... Читала учебники, но там только по тегам примерно проходятся, а как их связывать все вместе нету... Упс не в тот раздел Оо
    0 points
  11. nightgremlin, а где ты увидел, что человек писал, что это заказ? Может он для себя блог создал. VioletaMeynell, врядли кто-то будет ковыряться в твоём блоге. Если хочешь сама и бесплатно сделать свой блог красивым, то придётся изучать html и css.
    0 points
  12. Зачем брали заказ тогда? И причем тут оптимизация? Надо код смотреть, а перед тем как выкладывать сайт в сеть лучше проверить на кроссбраузерность, а ещё лучше делать это при создании блога. В Хроме тоже не очень отображается.
    0 points
  13. Добрый вечер! Я создала блог на blogspot.com ,сегодня купила домен.. Но столкнулась с проблемой , заходя на другие браузеры , такие как Safari (Mac) , Internet Explorer .. всё передвигается ,цвета другие , расширение тоже. В этом деле я новичок , поэтому не судите строго! Блог по заказу и нужно чтоб на всех браузерах отображался одинаково. Поэтому прошу у вас помощи , в оптимизации мало что понимаю . Подскажите что нужно делать , на мозиле всё отображается хорошо. Вот сам блог : buddyleeblog.blogspot.com
    -1 points
This leaderboard is set to Kiev/GMT+02:00
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy