Показать сообщение отдельно
Старый 04.12.2011, 09:09   #38
tempura
Forum bot
 
Аватар для tempura
 
Регистрация: 01.02.2010
Адрес: пусто
Сообщений: 841
Сказал(а) спасибо: 286
Поблагодарили 418 раз(а) в 190 сообщениях
Записей в дневнике: 60
tempura отключил(а) отображение уровня репутации
По умолчанию

Извините за слайтли оффтопик, но словами сантехника пришедшего в обком: "тут всю систему менять надо".
1. Невозможно писать систему безграмотно. Не существует слова "modul". Чуток поднатаскаешься в "инглишском изыке", глаз резать начнет, опечатываться сам же будешь. А пакетная замена уже может и не прокатить, так как вполне возможно, что появятся всякие "modul_n" или там "n_modul".
2. Ссылки вида "mysite.ru/index.php?modul=settings" - полный отстой. Крайне рекомендуется в движок сразу закладывать обработку ссылок в формат "mysite.ru/settings/" - причем без использования реврайта, все в РНР делать.
3. Выше уже упоминали, что
Цитата:
если с базы доставать и заносить в массив с помощью explode()! разбивая ссылки и удаляя символы /r/n! Вроде как гуд!
это совсем не гуд, а вовсе даже и бед. И бред. На какого хрена делать (помянем старый мангос) аналог поля "data"? Зафига в одно поле вводить пицод ссылок, при чтении разбивать их там куда-то зачем-то, рисковать из-за одного неверного знака все грохнуть напрочь, и в итоге даже и самому перестать разбираться в формате записи в этом поле - просто позабыв через пару месяцев принцип его формирования? одна строка - одно значение, и не хрена городить. Надо массив сделать - всю таблицу сгрызть.

Короче, либо я чего-то не понимаю, либо я вообще ничего не понимаю.
__________________
Совершенно безопасен для людей, обладающих хотя бы некоторыми минимальными зачатками интеллекта, и способными строить причинно-следственные цепочки.
tempura вне форума   Ответить с цитированием
3 пользователя(ей) сказали cпасибо:
MaxXx2021 (04.12.2011), Кот ДаWINчи (04.12.2011)