есть скрипт авторизации который процедурно при вводе пароля и логина правельного пишет гуд.и есть тот же скрипт но я его переделал на ООП и при вводе правельного логина с паролем или не правельного без разницы пишет бэд!помогите пожалучта голова уже трещит.

процедурный исправный скрипт:



и вот ООПэшный в котором проблема(ошибку не выдает просто не правельнор фунциклирует) 

а где нерабочий скрипт-то?

извиняюсь че то я забородил вот он:
<?php
include "../conf/config.php";

Смысл оформлять всё это в класс?

хочу свой форум переделать на классы,да и интересно почему не получается

PHP FAQ: Ничего не работает! Что делать??? 

Какое преимущество даёт то, что ты всё завернул в класс?

По моему, надо бы вам сделать так:


и соответственно вызывать:



Должно помочь.
NightmareZ, Хотя бы такие, что библиотеку можно будет повторно использовать (только если подредактировать, т.к. сейчас её не только повторно, но и вообще использовать нельзя ;))

Это сообщение отредактировал(а) Fally - 26.7.2007, 16:18

Да, вариант Fally должен помочь и с точки зрения повторного использования кода он удобнее, поскольку не накладывает зависимости на источник получения login и password. А помочь он должен, поскольку обходится Ваша конкретная ошибка.



Вы сначала пишете данные в поля класса $this->login, $this->password, а потом пытаетесь использовать не определенные ранее локальные переменные $login и $password. Они всегда пустые. Вот и результат всегда "bad".

P.S. Ну и к проблемам безопасности стоит отнестить внимательнее. Не стоит прямо в mysql_query() вставлять неэкранированные и нефильтрованные значения, прямо из $_POST

И ещё на существование проверь   



Ой сори извиняюсь опять не заметил 


Тогда можно в md5 запяхнуть 
 

   ещё раз извиняюсь

Это сообщение отредактировал(а) wolfmessing - 27.7.2007, 09:33

разве этого не хватит для защиты ?

pr1deВозможно и хватит, утверждать не буду, т.к. сам всегда использую mysql_real_escape_string(), да и разработчиками она рекомендована. 
К тому же до сих пор не пойму 2 вещи в вашем коде:
1) зачем так обрабатывать пароль, не лучше ли его хранить в БД в виде MD5-хэша, и при сравнении его хэшировать? И SQL-injection по полю пароля вы точно не получите.
2) Поле с логином. Вопрос, а зачем вы допускаете потенциально опасные символы в логине. Мне кажется, что можно допускать для логина только символы: a-z, а-я, _, 0-9 . Всё, больше ничего не нужно. А проверить на корректность такой логин можно регулярным выражением.



Пишите проверь на существование, а проверяете пустая переменная или нет.. Определитесь с тем чего хотите и чего вы делаете ;)

Это сообщение отредактировал(а) Fally - 27.7.2007, 10:07

большое спасибо всем кто помогал!

ОК спосибо за замечание учту