Пуля — дура. Пых — молодец.

Добро пожаловать на Пыху!

Логин:
Пароль:
 

Нет прописки? Зарегистрируйся!

Новости

Пыха переехала на новый сервер, ура!

Краснодарское время: 26 Май, 2012, 03:14:20

Страниц: 1 [2] 3
Печать
Автор Тема: Правильно ли я пишу ?  (Прочитано 980 раз)
0 Пользователей и 2 Гостей смотрят эту тему.
phpdude    ↓ 
07 Март, 2010, 12:43:29
НЕ ХУЕТА! ХУЕТА!

я - ЭМО
Группа: в ухо

Карма: 345
Сообщений: д-о-х-у-я!
Сила слова: 1.66

md5, эээээ :====э
Записан

забанен. могу забанить других, пишите в личку
BEER. Helping ugly people have sex since 1862.
Givi    ↓ 
07 Март, 2010, 02:36:29 , спустя 1 час 53 минуты
НЕ ХУЕТА! ХУЕТА!

Группа: Адекваты

Карма: 42
Сообщений: 2305
Сила слова: 1.82

Troy, поехали срать:
1. <h3 style="display:inline;">{title}</h3> - это что за хрен у тебя? нафиг заголовку стиль инлайн ? не знаешь как сделать чтоб у заголовка не было всяких отступов? Но то так, херня по сути :)
2. Нафиг в папке с темплейтами шаблоны админки? Они должны быть где-то в системных разделах, так как независимо от основного шаблона эти темы не меняются. А в твоем случае придется тупо копировать папку админ каждый раз, когда создаешь свою тему.
3. Про названия классов тебе уже сказали. То же и относительно названия файлов: делай их меленьким регистром.
4. Зачем в папке конфигов столько файлов? Может удобнее сделать максимум 2: юзерская часть и административная часть, а то и вовсе все в один объединить? Где-то провтыкал с настройками, и епи себе моск где что пропустил. Да и подключение одного файла - это быстрее недели подключать их десяток.
5. Не совсем понимаю (не особо разбирался, ибо лень + утро) нафиг аж три файла users.php: в модулях, в классах, в экшенах? Может достаточно их в одно место запихнуть (к примеру, в папке модулей создать папку "юзерс" и туда пихать все, что касается модуля юзверей) ?
6. Пагинатор в папке библиотек. А на кой он там? Он уже или модуль (что мне более всего импонирует) или же в папке классес (у тебя ведь там все вспомогательные классы). А так как в нем вшит уже шаблон отображения (ХТМЛ-код), то тем более в ядро его пихать не стоит (файлы ядра ведь неззя изменять), потому как твой ХТМЛ-код не есть во всех случаях удобным. Но это ИМХО
 
Ну и не совсем понятно нафиг нужна папка "экшенс" с имеющимися внутри файлами, если по сути это должны быть либо модульные составляющие (то есть где-то в папке с модулями должны быть) либо "ядрённые" (ака системные). А сейчас они непришейкобылехвост, и не совсем понятно что с ними делать. Но это опять таки ИМХО.
Записан

Все, что говорят другие - неправда! До тех пор, пока ты сам в это не поверишь.
Если человек дурак, то... чур это не я!
Troy    ↓ 
07 Март, 2010, 05:48:59 , спустя 3 часа 12 минут 30 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи

Карма: 45
Сообщений: 2393
Сила слова: 1.88

Поехали отмазыватся)
1. с верстальщиком спорить ? нее
2. Это да. сам вот думаю убрать
3. Уже говорили
4. конфиги отвечают на совсем разные настройки, некоторые из них не всегда используются, сваливать все в 1 файл тоже не айс
5. Их по моей логике должно быть 2-3, 1. Это сам экшен, там регистрация , авторизация и тп. 2. Это вспомогательный класс для 1 3. Это сам модуль, он загружается на каждой странице, служит для разделения прав и тп.
6. Пагинатор считаю отдельой библиотекой , папка classes это папка для моделей
7. "Экшенс" - это контроллеры
Записан

Givi    ↓ 
07 Март, 2010, 07:46:18 , спустя 1 час 57 минут 19 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Адекваты

Карма: 42
Сообщений: 2305
Сила слова: 1.82

Troy, если 7 пункт - это контроллеры, то что же тогда будет модулем? неужели модули не имеют контроллером? Или, если имеют, то мне что, нужно закинуть файлы с модулем и в папку "модули" и в папку "контроллеры" ? не совсем корректно, как по мне. Хоть, это малость ИМХО. Потому это я не спорю, просто пытаюсь понять зачем.
 
А относительно конфига: ну загрузишь ты лишний 1 кб информации системного конфига (заметь, не какого-то там, а именно системного, то есть очень важного и необходимого в том или ином месте). Зато когда это все сразу понадобится. то оно вот уже готово к использованию. Ты ведь не сопротивляешься, когда загружаешь класс с n-количеством методов, из которых в данный момент по сути нужны максимум лишь парочка. И таких классов у тебя много. Вот и с конфигами сделай нечто вида класса (а можно и даже сам класс), в котором будут разного рода "методы" (ну или же реальные методы) и вызывай те из них, которые в данный момент нужны. Остальные пусть повисят в памяти. они много не съедят, зато всегда под рукой.
 
Пагинатор - это все же не библиотека, а банальный набор из пары математических функций, и у тебя ещё и при этом всем он разбавлен ХТМЛ-кодом. Все же склонен к модулю, но не либе.
 
По юзерам: почему модуль, который делит юзверей на группы и т.д. не может также заниматься аторизацией и регистрацией этих самых юзверей? По сути ведь обе этих части программы должны работать с одними и теми же таблицами и данными. И если для одной из частей изменить структуру данных, то и вторую тоже нужно менять. Получается, что модуль - это уже не отдельная, в какой-то мере независимая частица, а выходит что это неотъемлемая часть общей системы (хреново бубет если этот модуль отключить).
Записан

Все, что говорят другие - неправда! До тех пор, пока ты сам в это не поверишь.
Если человек дурак, то... чур это не я!
Troy    ↓ 
07 Март, 2010, 08:19:55 , спустя 33 минуты 37 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи

Карма: 45
Сообщений: 2393
Сила слова: 1.88

Под контроллер я понимаю класс , который грубо говоря отвечает за контент страницы , под модулем я понимаю кусок кода, который постоянно подгружается в систему, а библиотека это кусок кода (класс) выполняющий определенные задачи , но нужный не постоянно , а иногда , поэтому по моему мнению пагинатор это все же библиотека (надеюсь нас рассудят)
 
Конфиги это уже личное предпочтение, я люблю именно делить
 
Сайт может работать и без юзеров , так как доступ к контенту предоставить можно и без регистрации
Спустя 18 секунд добавил
з.ы.  1070 круто
Записан

Givi    ↓ 
08 Март, 2010, 02:26:00 , спустя 6 часов 6 минут 5 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Адекваты

Карма: 42
Сообщений: 2305
Сила слова: 1.82


Под контроллер я понимаю класс , который грубо говоря отвечает за контент страницы , под модулем я понимаю кусок кода, который постоянно подгружается в систему, а библиотека это кусок кода (класс) выполняющий определенные задачи , но нужный не постоянно , а иногда , поэтому по моему мнению пагинатор это все же библиотека (надеюсь нас рассудят)
Херня получается: берем модуль "новости" - он нужен только на страницах новостей, тогда нафига нам его всегда подгружать? То же и с пагинатором. А контроллер - это часть модуля, но неотъемлемая часть. И модуль тоже занимается контентом страницы, если он для этого предназначен.
 
Конфиги это уже личное предпочтение, я люблю именно делить
С этим спорить не буду, так как каждому свое.
 
Сайт может работать и без юзеров , так как доступ к контенту предоставить можно и без регистрации
Точно также и без новостей, если это станица галереи, и без сисег, если это страница стихотворений А.С. Пушкина.
Короче, тут не стану спорить. У тебя и так есть чего исправлять.
 
п.с. А в целом молодец, по-моему очень даже отлично стараешься.
Записан

Все, что говорят другие - неправда! До тех пор, пока ты сам в это не поверишь.
Если человек дурак, то... чур это не я!
krasun    ↓ 
08 Март, 2010, 03:39:09 , спустя 1 час 13 минут 9 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи
Группа: Джедаи

Карма: 41
Сообщений: 1379
Сила слова: 2.97

Действительно запутанно, библиотека, модуль, контроллеры. Опиши сначала принципы работы, или допустим, полностью путь работы движка при запросе. А далее будет легче разобраться.
Записан
krasun    ↓ 
08 Март, 2010, 03:46:08 , спустя 6 минут 59 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи
Группа: Джедаи

Карма: 41
Сообщений: 1379
Сила слова: 2.97

Пагинатор в библиотеку + к нему адаптеры, + скрипты вида
Записан
Troy    ↓ 
08 Март, 2010, 07:36:49 , спустя 3 часа 50 минут 41 секунду
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи

Карма: 45
Сообщений: 2393
Сила слова: 1.88

Givi, новости это не модуль, это контроллер, в системе 4 вида файлов : 1. это контроллеры , они находятся в папки actions , отвечают за вывод контента 2. Это модели, лежат в папке classes , 3. Это модули, модули могут влиять на систему , либо выводить какой-нибудь блок и тп. 4. Библиотеки , это вспомагательные классы, такие как пагинатор, класс для фида новостей и тп.
Записан

Troy    ↓ 
08 Март, 2010, 07:48:04 , спустя 11 минут 15 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи

Карма: 45
Сообщений: 2393
Сила слова: 1.88

krasun, хорошо, начинаю:
к примеру у нас такой запрос site.ru/news/add/tynimce/ , сначало урл обрабатывается роутером и получается так
Router::$Action = 'news';
Router::$Event = 'add';
Router::$Params = array('tynimce');
 
Исходя из значения Action система подгружает дефолтный экшен, нужный экшен или экшен с ошибкой
 
Пример экшена
PHP

<?
class news extends Action {
   
    function _load () {
       
        $this->SetDefaultEvent('main');
        $this->AddEventPreg('(.*)', 'full');   
       
    }
   
    function main () {
    }
 
    function full () {
    }
 
}
 

сначало в функции _load мы регистрируем доступные урл и функции соответственно которые будут запускаться, тут более менее ясно
 
Теперь про модели, модели находятся в папке classes , каждый файл называется так же как и контроллер (экшен) и содержит внутри себя класс с названием %controller%_class
 
Теперь про модули , модули загружаются автоматически (точнее это можно настроить, но пока не об этом) ДО загрузки контроллера (экшена)
Записан

Givi    ↓ 
08 Март, 2010, 02:42:21 , спустя 6 часов 54 минуты 17 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Адекваты

Карма: 42
Сообщений: 2305
Сила слова: 1.82

Все, перестаю спорить. У нас совсем разное понятие о системе.
Записан

Все, что говорят другие - неправда! До тех пор, пока ты сам в это не поверишь.
Если человек дурак, то... чур это не я!
Troy    ↓ 
08 Март, 2010, 02:48:51 , спустя 6 минут 30 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Джедаи

Карма: 45
Сообщений: 2393
Сила слова: 1.88

Givi, а мне наоборот нравятся споры , можно взглянуть на вопрос с другой стороны, узнать чужое мнение , посмотреть на минусы своего мнения
Записан

Professor    ↓ 
08 Март, 2010, 03:02:52 , спустя 14 минут 1 секунду
НЕ ХУЕТА! ХУЕТА!

Группа: Адекваты

Карма: 30
Сообщений: 2074
Сила слова: 1.45

Спор - плохо
Дискуссия - хорошо
=)
Записан

Записки профессора : Кто тут, к примеру, в цари крайний? Никого?! Так я первый буду!
mario    ↓ 
08 Март, 2010, 04:34:26 , спустя 1 час 31 минуту 34 секунды
НЕ ХУЕТА! ХУЕТА!

wazzup
Группа: Джедаи

Карма: 67
Сообщений: 6162
Сила слова: 1.09


Спор - плохо
Дискуссия - хорошо
=)
в споре познаётся истина! :)
Записан

с ув. mario.
Контакты
Professor    ↓ 
08 Март, 2010, 05:40:03 , спустя 1 час 5 минут 37 секунд
НЕ ХУЕТА! ХУЕТА!

Группа: Адекваты

Карма: 30
Сообщений: 2074
Сила слова: 1.45

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

В дискуссии рождается истина =)
Записан

Записки профессора : Кто тут, к примеру, в цари крайний? Никого?! Так я первый буду!
Страниц: 1 [2] 3
Печать
 

Перейти в: