Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/topic.php on line 737
Форумы портала PHP.SU :: Укажите на ошибки [2]

 PHP.SU

Программирование на PHP, MySQL и другие веб-технологии
PHP.SU Портал     На главную страницу форума Главная     Помощь Помощь     Поиск Поиск     Поиск Яндекс Поиск Яндекс     Вакансии  Пользователи Пользователи


 Страниц (4): « 1 [2] 3 4 »   

> Без описания
digi
Отправлено: 16 Мая, 2012 - 14:40:45
Post Id


Посетитель


Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012  


Помог: 4 раз(а)




etoYA, ну какбы это ошибки и на них указано Улыбка
 
 Top
Мелкий Супермодератор
Отправлено: 16 Мая, 2012 - 14:47:15
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


Помог: 618 раз(а)




etoYA пишет:
Действительно хитро заныкано). В library/debug.php

Это очень плохо.
Во-первых, ужасное место и название.
Во-вторых, хитро спрятано.
В-третьих - кастированная эмуляция магических кавычек, вместо защиты.
GET, видимо, всё же уязвим. По крайней мере потенциально.

И база в глобальной переменной $db - весьма печально при, казалось бы, ООП подходе.


-----
PostgreSQL DBA
 
 Top
etoYA
Отправлено: 16 Мая, 2012 - 15:05:26
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)




Мелкий пишет:
И база в глобальной переменной $db - весьма печально при, казалось бы, ООП подходе.


А вот этого не понял..
(Добавление)
Мелкий пишет:
GET, видимо, всё же уязвим. По крайней мере потенциально.

GET не использую
(Добавление)
Ну, а вообще нормально? Учу пхп чуть более пол года)
 
 Top
ams
Отправлено: 16 Мая, 2012 - 16:10:36
Post Id



Посетитель


Покинул форум
Сообщений всего: 351
Дата рег-ции: Нояб. 2010  


Помог: 13 раз(а)




У знакомого видил что то типо /application/controllers/AdminController.php, строки 46 - 65...

Так же делал он, но потом поля росли, добовлялость внутрь условие и ВЕЗДЕ одинаково все. Нельзя ли 1 массив с именами ключей масива, а его значение - ошибка. И циклом по нему бежать...
 
 Top
Мелкий Супермодератор
Отправлено: 16 Мая, 2012 - 16:10:54
Post Id



Активный участник


Покинул форум
Сообщений всего: 11926
Дата рег-ции: Июль 2009  
Откуда: Россия, Санкт-Петербург


Помог: 618 раз(а)




etoYA пишет:
GET не использую

Да ну? А роутер чем занят тогда?
CategoryController, Films::getCatByTName($arg[1]); - я практически уверен, что здесь инъекция пройдёт.

etoYA пишет:
А вот этого не понял..

Функция db, static $db - глобальная переменная. Вместо синглтона/фабрики
А не, она действительно существует только для этой функции.

Вообще:
уязвим
вызов методов статически без указания метода статическим
нет модификаторов видимости методов - хороший тон их писать всегда
CODE (SQL):
скопировать код в буфер обмена
  1. SELECT * FROM `categories` DESK ORDER BY `id`
я чего-то не знаю о синтаксисе order by? Оно работает?
Крайне криво смотрю на function page($arg = false){, тогда как потом идёт обращение к массиву. Однако, действительно приведение типов в NULL и нет нотайсов.
Почему роутер вообще занимается layout'ами?
Безусловное включение всей library, при том, ещё и дважды? (почему дважды - домашнее задание)
Есть сомнения в том, что нафиг в fatal error свалитесь, если вдруг конфига на месте не будет. Хотя проверка на существование файла есть.

В общем, на работу не возьму.


-----
PostgreSQL DBA
 
 Top
digi
Отправлено: 16 Мая, 2012 - 16:17:40
Post Id


Посетитель


Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012  


Помог: 4 раз(а)




etoYA, вы пробовали изучать фреймворки? теже Yii и Symfony2?
 
 Top
etoYA
Отправлено: 16 Мая, 2012 - 16:46:30
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)




нет).

Мелкий, спасибо, будет время перепишу все)
(Добавление)
Мелкий пишет:
CategoryController, Films::getCatByTName($arg[1]); - я практически уверен, что здесь инъекция пройдёт.


Не пройдет, если фильма не существует, то вылетает исключение с ошибкой
 
 Top
digi
Отправлено: 16 Мая, 2012 - 16:56:59
Post Id


Посетитель


Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012  


Помог: 4 раз(а)




etoYA, тогда натоятельно рекомендую вам изучить симфони2 и поиграться с ним Улыбка во первых увидите как должен выглядеть правильный код и больше не будете гадать как хорошо, а как плохо, в вторых сразу изучите все самые передовые технологии, библиотеки и паттерны проектирования и программирования, в третьих получите инструмент на котором можно создавать любые проекты Улыбка да и вообще массу наслаждения от самого процесса ;)

я скачивал книжку на русском вот отсюда http://symf.ru/, но там перевод не всего... остальное читаю в офф доке на енг.
 
 Top
etoYA
Отправлено: 16 Мая, 2012 - 17:00:16
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)




Я попробую изучить Yii, с чего начинать?
(Добавление)
И вообще нужно ли учить фреймворки для первого сайта?
 
 Top
DeepVarvar Супермодератор
Отправлено: 16 Мая, 2012 - 17:29:10
Post Id



Активный участник


Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008  
Откуда: Альфа Центавра


Помог: 353 раз(а)




etoYA пишет:
учить фреймворки для первого сайта?
Нет.
Но учитывая твой уровень знаний - уже пора посмотреть что там во фреймворках есть интересного. Закатив глазки
 
 Top
digi
Отправлено: 16 Мая, 2012 - 17:54:17
Post Id


Посетитель


Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012  


Помог: 4 раз(а)




etoYA Yii второй ветки пока еще даже в альфа тесте нету... так что лучше изучить сначала симфони2, а потом посмотреть йии 1, чтобы понять на сколько велика разница между ними и с радостной миной на лице продолжить использовать симфони 2! ;))
 
 Top
DeepVarvar Супермодератор
Отправлено: 16 Мая, 2012 - 17:57:56
Post Id



Активный участник


Покинул форум
Сообщений всего: 10377
Дата рег-ции: Дек. 2008  
Откуда: Альфа Центавра


Помог: 353 раз(а)




Щас маслицца в огонь подкину: Zend forewer!!! Радость
 
 Top
digi
Отправлено: 16 Мая, 2012 - 18:02:14
Post Id


Посетитель


Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012  


Помог: 4 раз(а)




DeepVarvar, флеймить можно долго... но ЗФ2 еще не релизнули... так что на него и документации пока нету и поменяться еще может что угодно... а ЗФ1 уже морально устаревший, да и не развивается уже т.к. все силы сейчас на вторую ветку перекинуты, а вот тотже симфони 2 сейчас впереди планеты всей стоит в плане РНР Улыбка

но в тоже время с симфони можно легко юзать либы от зента, что от первого, что он разрабатываемого второго т.к. у сф2 очень грамотный автолоадер и легко в него всё интегрируется Улыбка
 
 Top
etoYA
Отправлено: 16 Мая, 2012 - 18:32:34
Post Id



Участник


Покинул форум
Сообщений всего: 1859
Дата рег-ции: Июль 2011  
Откуда: Крым


Помог: 21 раз(а)




DeepVarvar пишет:
Щас маслицца в огонь подкину: Zend forewer!!!

Не, не)) Нужно сначала что-то простенькое)
 
 Top
digi
Отправлено: 16 Мая, 2012 - 19:07:04
Post Id


Посетитель


Покинул форум
Сообщений всего: 406
Дата рег-ции: Янв. 2012  


Помог: 4 раз(а)




Мелкий, сделал пулл реквест автору вышеуказанного бенчмарка https://github[dot]com/eryx/php-fram[dot][dot][dot]benchmark/pull/2

ждём обновления результатов Улыбка
(Добавление)
etoYA пишет:
Не, не)) Нужно сначала что-то простенькое)
, простенькое вы уже написали ;)) теперь пишите нормальное ;) [разумется про sf2 речь ;)]
 
 Top
Страниц (4): « 1 [2] 3 4 »
Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)
« Объектно-ориентированное программирование »


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



Powered by PHP  Powered By MySQL  Powered by Nginx  Valid CSS  RSS

 
Powered by ExBB FM 1.0 RC1. InvisionExBB