Практически биллинг :)

Статус
В этой теме нельзя размещать новые ответы.

Nova

Постоялец
Регистрация
31 Мар 2007
Сообщения
63
Реакции
14
Вот навоял в кратце может кто укажет мне на мои ошибки.



Ну или как лучше с мерчантом такие операции проводить.

P>S
Я уверен что я сделал очень много лишних действий плз укажите на ошибки да и как вообще можно упростить данный вариант.
 
Я так понял никто мне не поможет =(
 
Глубоко смотреть не стал.
1. Где разделение html от логики?
2. if($_SESSION['admin']==1) - это что такое?
3. Про стиль кода вообще молчу. :)
 
Глубоко смотреть не стал.
1. Где разделение html от логики?
2. if($_SESSION['admin']==1) - это что такое?
3. Про стиль кода вообще молчу. :)

1. Ну хз где )
2. Делал статистику по платежам но потом убрал за особой ненадобностью (Виимо забыл убрать)
3. Ну блин я исправлюсь.....

Есть ещё замечания ?
Интересует работа с мерчантом, точнее замечания по работе.... :thenks:
 
Чтобы обсуждать работу с мерчантом - нужно смотреть подробно... Имхо, даже не глядя в тот код, можно запросто сказать что так негладко...

Хотя, некоторые говорят, что хороший код -рабочий код :)
 
Хотя, некоторые говорят, что хороший код -рабочий код :)
Угу код гавно зато рабочий (я старался) :)

Чтобы обсуждать работу с мерчантом - нужно смотреть подробно... Имхо, даже не глядя в тот код, можно запросто сказать что так негладко...
Если будет время глянь плз :ah:
 
Установил. Зайти в панель не смог, вылетают ошибки. Открыл файл login.php старт сесии после вывода куска кода в браузер, такого делать (насколько я знаю нельзя), в базе храниш пароли, а не хеши, безопасность кульгает. Ну и т.д. Детально расматривать не стал.
 
сильно хромает безопасность.
плюс структура конечно немного непродуманная.
в остальном - работает и ладно :)
в основном вопросы к стилю... особо сильно не капал.
всетаки это скорее не скрипт или заготовка, а так, поделка чтобы ознакомится с технологиями :) потому что структура сильно осложняет изменения и расширения (тот же html в перемешку с кодом)....
 
Статус
В этой теме нельзя размещать новые ответы.
Назад
Сверху