Какой сделать главный класс и/или какая правильная структура?

@stupidstudent · Регистрация: 25.10.2011

Студворк — интернет-сервис помощи студентам

Написал авторизацию и регистрацию без классов. Все работает как положено.
Переделываю через классы:

Какой сделать главный класс и/или какая правильная структура?
Например, я создам класс fns(методы проверки почты, создания хэша, определения ip и подключение к базе данных), а потом создам унаследованные дочерние подклассы - auth и reg.

Туда будут передаваться соответствующие аргументы получаемые из форм авторизации или регистрации.

Такой подход верен? Или предложите свой, пожалуйста.

@stupidstudent · 12.08.2012, 23:07 **[ТС]**

Сообщение от caballero

недостаточно для чего?
Не можешь в двух простых классах разобратся а претендуешь на имплементацию хеширования лучшего чем писаные спецами до которых ты вряд ли дорастешь.

вы не так поняли, а тут сразу наезжаете. Дорасту не дорасту, не вам решать.
а про использование CRYPT_BLOWFISH(и еще других методов шифрования), соли и т.д. не объединяют в одну функцию?
Писать и разрабатывать свой алгоритм шифрования без возможности обратной дешифрации, я не собирался и не собираюсь.
Кстати, md5 и sha1 в легкую ломаются с использованием радужных таблиц.
О небезопасности md5 и sha1
http://ru2.php.net/manual/ru/f... s.fasthash

Добавлено через 7 минут
еще вопрос, будет ли целесообразно выделить все запросы к бд в отдельный класс, а там уже наследовать авториз и регист?

@Василий Макогон · 12.08.2012, 23:47

ВООБЩЕ ТО ЧТО-ТО СОВЕТОВАТЬ ТУТ БЕСПОЛЕЗНОЮ. ЕАДО КАК БЭ ПРИМЕРЫ КОДА И ОТ НЕГО ПЛЯСАТЬ

d1gi · 12.08.2012, 23:51

stupidstudent, подумайте над предложением создать классик UsersProvider - т.е. поставщик юзеров, а в нём уже будет реализованы методы проверки пары логин/пароль, он же сам будет юзать БД если ему надо, и наследовать его ненадо, лучше просто юзать объект этого класса, например как сервис.

@caballero · 13.08.2012, 00:15

Кстати, md5 и sha1 в легкую ломаются с использованием радужных таблиц.

Во первых еще надо написать что то такое ценное чтобы ктото потратил туеву кучу ресурса для взлома (вы всяком случае я не слышал о массовом взломе линуховых паролей).
во вторых чтобы взломать пароль, в данном случае, хакеру надо добратся до БД где они лежат, а если он туда добрался то пароли юзеров ему уже не нужны.

еще вопрос, будет ли целесообразно выделить все запросы к бд в отдельный класс, а там уже наследовать авториз и регист?

Разумеется нет. Это разные сущности.
Тут подходит агрегация а не наследование. Тем более что сами по себе классы авторизации и регистрации бессмыслены поскольку не имеют смысла объекты для которых они создаются.

@Darevill · 17.08.2012, 13:52

Задачу можно разбить на 3 сущности:
Работа с базой

PHP
1
2
3
4
5
6
class db
{
   function connect() {}
   function query() {}
   ...
}

Вспомогательные функции

PHP
1
2
3
4
5
6
class tools
{
   function hash() {}
   function check_email() {}
   ...
}

Обработчик пользователей, который использует для работы объекты первых двух классов

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
class user
{
   var $db;
   var $tools;
 
   function __construct()
   {
      $this->db = new db();
      $this->db->connect();
      $this->tools = new tools();
   }
   function auth() {}
   function reg() {}
   ...
}

можно наследовать, а можно пользоваться $this->tools->hash() это уже не суть. Главное - использовать классы для тематического разделения методов.
Авторизация и регистрация - это по сути методы по работе с таблицей пользователей, потому должны находиться в одном классе как и другие методы по этой-же теме (update, delete, userinfo, и т.п.)
Если же нужно назначать пользователю права, то систему управления правами можно вынести в отдельный класс, который будет заниматься назначением/удалением прав, созданием групп пользователей и т.п.

d1gi · 17.08.2012, 13:58

Только ненадо юзать устаревшее ключевое слово var, сейчас надо явно указывать, это же относится и к методам... а то новички начитаются и начнут дрова ломать

@caballero · 17.08.2012, 14:06

Главное - использовать классы для тематического разделения методов.

в таком случае методы должны быть объявлены как static поскольку сохдание объектов таких типов бессмысленно

@Василий Макогон · 17.08.2012, 14:12

основная загвоздка - в таинственном методе hash()
ТС упорно не говорит что он с ним делает
если бы мы знали, за что отвечает hash() и кто его использует, мы могли бы что-то сделать

подозреваю, что тут нужен класс для работы с этих хэшом, КОТОРЫЙ И НУЖНО ВПРЫСКИВАТЬ ВО ВСЕ ОСТАЛЬНОЕ
таким образом мы абстрагируемся от способа шифрования пароля и можем в случае надобности подменить способ шифровки

@caballero · 17.08.2012, 14:25

ТС упорно не говорит что он с ним делает

какая разница - это делать в любом случае глупо

подозреваю, что тут нужен класс для работы с этих хэшом,

а зачем там целый класс?

@Darevill · 17.08.2012, 14:48

Сообщение от Василий Макогон

основная загвоздка - в таинственном методе hash() ...

Вообще-то без разницы как будут хешироваться данные, вопрос то был не об этом

Если он хочет какие-то свои алгоритмы реализовывать, может и понадобится класс, а может обойдётся одной функцией.

Добавлено через 7 минут

Сообщение от d1gi

Только ненадо юзать устаревшее ключевое слово var, сейчас надо явно указывать, это же относится и к методам... а то новички начитаются и начнут дрова ломать

Я же привёл схему, а не рабочие классы, потому не указывал какими эти переменные и методы будут в работе. Не зная всей задачи и загнав что-то в приват, я могу опять-же запутать новичков

@Василий Макогон · 17.08.2012, 15:00

Сообщение от caballero

а зачем там целый класс?

затем, что скорее всего нужно будет как минимум 2 метода - проверка какого-нибудь значения из куки и выдача нового хэша. это - бизнес логика и если в хэш будет ещё и соль шифроваться, то без класса никак не обойтись

@caballero · 17.08.2012, 15:06

затем, что скорее всего нужно будет как минимум 2 метода - проверка какого-нибудь значения из куки и выдача нового хэша. это - бизнес логика и если в хэш будет ещё и соль шифроваться, то без класса никак не обойтись

Не вижу проблем почему это не может быть в одной функции.

@stupidstudent · 17.08.2012, 21:13 **[ТС]**

Сообщение от Darevill

Работа с базой

PHP
1
2
3
4
5
6
class db
{
* *function connect() {}
* *function query() {}
* *...
}

насчет query(), функция должна обработать запрос и вернуть значение или в самой функции должен быть запрос?

@Darevill · 17.08.2012, 22:33

Сообщение от stupidstudent

насчет query(), функция должна обработать запрос и вернуть значение или в самой функции должен быть запрос?

Функция принимает запрос в качестве аргумента, выполняет запрос через mysql_query() и возвращает результат ввиде массива данных, если это был SELECT запрос.

@stupidstudent · 17.08.2012, 23:02 **[ТС]**

Сообщение от Darevill

Функция принимает запрос в качестве аргумента, выполняет запрос через mysql_query() и возвращает результат ввиде массива данных, если это был SELECT запрос.

Да, так и делал. Спасибо.

Новые блоги и статьи Все статьи Все блоги /
Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .	Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .
интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .	Сам себя обучал rest api anaschu 02.07.2026 Педагогический лайфхак: Почему чистый REST API для ученика намного круче, чем готовые библиотеки Когда мы отказались от капризного JAR-файла AnyLogic и переписали код на стандартный HttpClient,. . .	rest api anylogic - выполнение модели на своём русском сайте anaschu 02.07.2026 Как подружиться с AnyLogic Cloud API, победить провайдеров и развернуться Java-бэкенд в Docker на бесплатном хостинге: Двухдневный лог борьбы Всем привет! Хочу поделиться свежим (и довольно. . .

@stupidstudent 18 / 15 / 0 Регистрация: 25.10.2011 Сообщений: 241

	Какой сделать главный класс и/или какая правильная структура? 11.08.2012, 18:31. Показов 4481. Ответов 34 Метки нет (Все метки) Написал авторизацию и регистрацию без классов. Все работает как положено. Переделываю через классы: Какой сделать главный класс и/или какая правильная структура? Например, я создам класс fns(методы проверки почты, создания хэша, определения ip и подключение к базе данных), а потом создам унаследованные дочерние подклассы - auth и reg. Туда будут передаваться соответствующие аргументы получаемые из форм авторизации или регистрации. Такой подход верен? Или предложите свой, пожалуйста. 0

@Василий Макогон 270 / 226 / 11 Регистрация: 20.04.2012 Сообщений: 817
	12.08.2012, 23:47
	ВООБЩЕ ТО ЧТО-ТО СОВЕТОВАТЬ ТУТ БЕСПОЛЕЗНОЮ. ЕАДО КАК БЭ ПРИМЕРЫ КОДА И ОТ НЕГО ПЛЯСАТЬ 1

d1gi 28 / 50 / 2 Регистрация: 10.01.2012 Сообщений: 347
	12.08.2012, 23:51
	stupidstudent, подумайте над предложением создать классик UsersProvider - т.е. поставщик юзеров, а в нём уже будет реализованы методы проверки пары логин/пароль, он же сам будет юзать БД если ему надо, и наследовать его ненадо, лучше просто юзать объект этого класса, например как сервис. 1

d1gi 28 / 50 / 2 Регистрация: 10.01.2012 Сообщений: 347
	17.08.2012, 13:58
	Только ненадо юзать устаревшее ключевое слово var, сейчас надо явно указывать, это же относится и к методам... а то новички начитаются и начнут дрова ломать 2

@Василий Макогон 270 / 226 / 11 Регистрация: 20.04.2012 Сообщений: 817
	17.08.2012, 14:12
	основная загвоздка - в таинственном методе hash() ТС упорно не говорит что он с ним делает если бы мы знали, за что отвечает hash() и кто его использует, мы могли бы что-то сделать подозреваю, что тут нужен класс для работы с этих хэшом, КОТОРЫЙ И НУЖНО ВПРЫСКИВАТЬ ВО ВСЕ ОСТАЛЬНОЕ таким образом мы абстрагируемся от способа шифрования пароля и можем в случае надобности подменить способ шифровки 0