Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012
Помог: 4 раз(а)
etoYA, ну какбы это ошибки и на них указано
Мелкий
Отправлено: 16 Мая, 2012 - 14:47:15
Активный участник
Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург
Помог: 618 раз(а)
etoYA пишет:
Действительно хитро заныкано). В library/debug.php
Это очень плохо.
Во-первых, ужасное место и название.
Во-вторых, хитро спрятано.
В-третьих - кастированная эмуляция магических кавычек, вместо защиты.
GET, видимо, всё же уязвим. По крайней мере потенциально.
И база в глобальной переменной $db - весьма печально при, казалось бы, ООП подходе.
----- PostgreSQL DBA
etoYA
Отправлено: 16 Мая, 2012 - 15:05:26
Участник
Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011 Откуда: Крым
Помог: 21 раз(а)
Мелкий пишет:
И база в глобальной переменной $db - весьма печально при, казалось бы, ООП подходе.
А вот этого не понял.. (Добавление)
Мелкий пишет:
GET, видимо, всё же уязвим. По крайней мере потенциально.
GET не использую (Добавление)
Ну, а вообще нормально? Учу пхп чуть более пол года)
ams
Отправлено: 16 Мая, 2012 - 16:10:36
Посетитель
Покинул форум
Сообщений всего: 351
Дата рег-ции: Нояб. 2010
Помог: 13 раз(а)
У знакомого видил что то типо /application/controllers/AdminController.php, строки 46 - 65...
Так же делал он, но потом поля росли, добовлялость внутрь условие и ВЕЗДЕ одинаково все. Нельзя ли 1 массив с именами ключей масива, а его значение - ошибка. И циклом по нему бежать...
Мелкий
Отправлено: 16 Мая, 2012 - 16:10:54
Активный участник
Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург
Помог: 618 раз(а)
etoYA пишет:
GET не использую
Да ну? А роутер чем занят тогда?
CategoryController, Films::getCatByTName($arg[1]); - я практически уверен, что здесь инъекция пройдёт.
etoYA пишет:
А вот этого не понял..
Функция db, static $db - глобальная переменная. Вместо синглтона/фабрики
А не, она действительно существует только для этой функции.
Вообще:
уязвим
вызов методов статически без указания метода статическим
нет модификаторов видимости методов - хороший тон их писать всегда
я чего-то не знаю о синтаксисе order by? Оно работает?
Крайне криво смотрю на function page($arg = false){, тогда как потом идёт обращение к массиву. Однако, действительно приведение типов в NULL и нет нотайсов.
Почему роутер вообще занимается layout'ами?
Безусловное включение всей library, при том, ещё и дважды? (почему дважды - домашнее задание)
Есть сомнения в том, что нафиг в fatal error свалитесь, если вдруг конфига на месте не будет. Хотя проверка на существование файла есть.
В общем, на работу не возьму.
----- PostgreSQL DBA
digi
Отправлено: 16 Мая, 2012 - 16:17:40
Посетитель
Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012
Помог: 4 раз(а)
etoYA, вы пробовали изучать фреймворки? теже Yii и Symfony2?
etoYA
Отправлено: 16 Мая, 2012 - 16:46:30
Участник
Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011 Откуда: Крым
Помог: 21 раз(а)
нет).
Мелкий, спасибо, будет время перепишу все) (Добавление)
Мелкий пишет:
CategoryController, Films::getCatByTName($arg[1]); - я практически уверен, что здесь инъекция пройдёт.
Не пройдет, если фильма не существует, то вылетает исключение с ошибкой
digi
Отправлено: 16 Мая, 2012 - 16:56:59
Посетитель
Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012
Помог: 4 раз(а)
etoYA, тогда натоятельно рекомендую вам изучить симфони2 и поиграться с ним во первых увидите как должен выглядеть правильный код и больше не будете гадать как хорошо, а как плохо, в вторых сразу изучите все самые передовые технологии, библиотеки и паттерны проектирования и программирования, в третьих получите инструмент на котором можно создавать любые проекты да и вообще массу наслаждения от самого процесса ;)
я скачивал книжку на русском вот отсюда http://symf.ru/, но там перевод не всего... остальное читаю в офф доке на енг.
etoYA
Отправлено: 16 Мая, 2012 - 17:00:16
Участник
Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011 Откуда: Крым
Помог: 21 раз(а)
Я попробую изучить Yii, с чего начинать? (Добавление)
И вообще нужно ли учить фреймворки для первого сайта?
DeepVarvar
Отправлено: 16 Мая, 2012 - 17:29:10
Активный участник
Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008 Откуда: Альфа Центавра
Помог: 353 раз(а)
etoYA пишет:
учить фреймворки для первого сайта?
Нет.
Но учитывая твой уровень знаний - уже пора посмотреть что там во фреймворках есть интересного.
Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012
Помог: 4 раз(а)
etoYA Yii второй ветки пока еще даже в альфа тесте нету... так что лучше изучить сначала симфони2, а потом посмотреть йии 1, чтобы понять на сколько велика разница между ними и с радостной миной на лице продолжить использовать симфони 2! ;))
DeepVarvar
Отправлено: 16 Мая, 2012 - 17:57:56
Активный участник
Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008 Откуда: Альфа Центавра
Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012
Помог: 4 раз(а)
DeepVarvar, флеймить можно долго... но ЗФ2 еще не релизнули... так что на него и документации пока нету и поменяться еще может что угодно... а ЗФ1 уже морально устаревший, да и не развивается уже т.к. все силы сейчас на вторую ветку перекинуты, а вот тотже симфони 2 сейчас впереди планеты всей стоит в плане РНР
но в тоже время с симфони можно легко юзать либы от зента, что от первого, что он разрабатываемого второго т.к. у сф2 очень грамотный автолоадер и легко в него всё интегрируется
etoYA
Отправлено: 16 Мая, 2012 - 18:32:34
Участник
Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011 Откуда: Крым
Помог: 21 раз(а)
DeepVarvar пишет:
Щас маслицца в огонь подкину: Zend forewer!!!
Не, не)) Нужно сначала что-то простенькое)
digi
Отправлено: 16 Мая, 2012 - 19:07:04
Посетитель
Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012
Все гости форума могут просматривать этот раздел. Только зарегистрированные пользователи могут создавать новые темы в этом разделе. Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.