Правила форума Гаранты форума
Размещение рекламы AMX-X компилятор

Здравствуйте, гость Вход | Регистрация

Наши новости:

14-дек
Добавлена поддержка utf8mb4
24-апр
Telegram группа
10-апр
Обновление PHP
11-апр
Раздача читов

Прошу оценки

, Прошу оценки
MbProsto
сообщение 9.5.2016, 20:03
Сообщение #1
Стаж: 10 лет

Сообщений: 176
Благодарностей: 16
Полезность: < 0

Не так давно начал изучать язык php
И так сказать хочется увидеть вашей критике по поводу моего кода.

Установка

1. Залить дамп бд на phpmyadmin
2. Прописать данные бд в конфиге config.php

Дабы не утруждались регистрироваться:
логин и пароль от пользователя - user
логин и пароль от админа - admin


PS Шаблон брал инета, тк не обладаю сильными знаниями html/css

Тех кто гуру php и если их не затруднит прошу подсказать где и в чём я допустил ошибки

Отредактировал: MbProsto, - 9.5.2016, 20:03
Прикрепленные файлы:
Прикрепленный файл  testshop.zip ( 6,71 мегабайт ) Кол-во скачиваний: 48
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   Цитировать сообщение
Статус пользователя Profan
сообщение 9.5.2016, 20:23
Сообщение #2


Стаж: 10 лет
Город: Сочи

Сообщений: 1317
Благодарностей: 403
Полезность: 698

MbProsto, Было бы быстрее и проще тебе самому залить на хост и поделится ссылкой, не думаю что кто-то будет этим заниматься.
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
Поблагодарили 1 раз
   + Цитировать сообщение
Статус пользователя BaHeK
сообщение 9.5.2016, 20:43
Сообщение #3


Стаж: 17 лет

Сообщений: 571
Благодарностей: 505
Полезность: 651

Что увидел за 5 минут проверки
1) mysql считается устаревшим с версии 5.5 и удален в 5.6+, используй mysqli или PDO.
2) $login = trim(mysql_real_escape_string($_POST['user'])); - нет проверки на isset($_POST['user']), если переменной $_POST['user'] нет, возникнет NOTICE предупреждение, при разработке всегда включай все ошибка: error_reporting(E_ALL);
3) та же строка выдаст warning предупреждение, если $_POST['user'] - массив
4) зачем?
Код
function clear_admin($var){
    $var = mysql_real_escape_string($var);
    return $var;
}

Остальное проверять не стал, может кто-нибудь еще что-то найдет


Чуть-чуть нарушаю
http://img-host.su/aoN1.png
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
MbProsto
сообщение 9.5.2016, 20:51
Сообщение #4
Стаж: 10 лет

Сообщений: 176
Благодарностей: 16
Полезность: < 0

Цитата(Profan @ 9.5.2016, 22:23) *
MbProsto, Было бы быстрее и проще тебе самому залить на хост и поделится ссылкой, не думаю что кто-то будет этим заниматься.

Залить это понятно, код же не увидите

BaHeK,
Первый двиг написанный лично мной, так что в принципе не удивлён некоторым ошибкам.
Насчёт mysql_ уже начал изучать PDO



Отредактировал: MbProsto, - 9.5.2016, 20:53
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
Статус пользователя onotole
сообщение 10.5.2016, 8:52
Сообщение #5


Иконка группы

Стаж: 13 лет

Сообщений: 1572
Благодарностей: 1119
Полезность: 1550

Меценат Меценат
Кроме замечаний Ванька придраться особо не к чему) Единственное, раз уж начал делать в MVC, делай в OOP =)
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
Поблагодарили 1 раз
   + Цитировать сообщение
MbProsto
сообщение 10.5.2016, 11:21
Сообщение #6
Стаж: 10 лет

Сообщений: 176
Благодарностей: 16
Полезность: < 0

Цитата(onotole @ 10.5.2016, 10:52) *
Кроме замечаний Ванька придраться особо не к чему) Единственное, раз уж начал делать в MVC, делай в OOP =)

Хорошо спасибо большое
Перейти в начало страницы         Просмотр профиля    Отправить личное сообщение
   + Цитировать сообщение
« Предыдущая тема · Скрипты · Следующая тема »
 
  Ответить в данную темуНачать новую тему
 
0 пользователей и 1 гостей читают эту тему:
Режим отображения: Стандартный · Переключить на: Линейный · Переключить на: Древовидный
Сообщить другу · Версия для печати

Powered By Invision Power Board  © 2005-2026 
Counter Strike Support Community