Веб OnePage Free-Kassa

Госпади, там тааак мнооого дорабатывать.. Конечно сделаю, завтра днем будет ссылка для скачивания.[DOUBLEPOST=1482789448,1482789411][/DOUBLEPOST]

Да, я оставил ссылку на свой ВК для таких вопросов.

 

Прошу проявить уважение к человеку, и не обращаться к нему на "Слышь".

 

Обновлено. Теперь выдача не через ркон, а через чуть измененный плагин от @alexandrage.

 

Могу провести анализ кода на говнокод.

 

Проведи, но еще посоветуй, как его убрать.

 

Отличная работа! Заберу себе

 

Могу сразу сказать: html в коде, вызов js из кода php (вообще такого говна давно не видел), определение функций внутри операторов, использование mysqli, многократное дублирование кода + я вообще молчу про логические ошибки. Чтобы его исправить, нужно с нуля все по-нормальному переписать.

 

О, лучше бы на свой скрипт посмотрел)
Про HTML и JS в PHP знаю.
Ну а MySQLi-то чем тебе не угодил?

 

А что тебе не нравится в моем скрипте?

 

Зачем делать скрипт на RCON, если будет привязан PermissionsEX?

 

Мой скрипт не был привязан к плагинан. Когда разговор дошел до безопасности, я перестал его разрабатывать.

 

Полностью поддерживаю @Cames

Самая первая и самая грубейшая ошибка при написание кода при работе с базой, это соединятся с базой без нужной на то причины. Это просто прям какой то DDoS соединениями с базой.
Решение: Соединятся с базой когда это действительно необходимо. При этом ты подключаешься это в главном файле даже не задумываясь что там даже одного запроса нету.

Вторая ошибка связанная уже непосредственно с вёрсткой, зачем ты подключаешь весь bootstrap включая JS составляющую, если используешь от туда меньше 1% и то это выглядит уродливо
Не использование css по назначению что там творится в главном файле...
Решение: Написание собственных стилей (хотя это уже кто использует должен), убрать всё что не связанно.

Третья ошибка, зачем ты подключаешь отдельный файл в котором только AJAX который зависит от jQuery и на этом вся его функциональность заканчивается библиотеки.
Решение: Переписать на нативном JS, да можно даже использовать Fetch API.

Четвёртая ошибка, связанна с самим запросом MySQL в базу.
> `id` int(50) NOT NULL AUTO_INCREMENT,
Ты не шутишь? INT 50?! Максимальная длина INT равняется 11, да и в жизни у тебя не будет там более 2,147,483,647 записей.
> `nickname` text NOT NULL,
TEXT серьёзно? Ты думаешь что никнейм будет длинной 65,535? Да тут даже VARCHAR 30 будет достаточно, даже меньше.
> `date` int(50) NOT NULL,
Снова TIMESTAMP, для него есть специальная дата хранения, а вообще этого необходимо избегать и использовать DATETIME, хотя если там только секунды не UNIX время.
> `paid` int(1) NOT NULL,
Для этого существует тип TINYINT
Остальные ошибки просто уже абсурдно приводить, не хочу лишний раз указывать на одно и то же.
Решение: Переделать строение таблицы и дать уникальные поля если они присутствуют, это ускорит выборку.

А теперь самое сочное, последние ошибки (но не все)
1. Никогда не доверяй пользователю!
2. Тебя мама в детстве не училась что не нужно жадничать? SELECT *, перечисляй только те поля что действительно нужны!
3. Как уже и говорил @Cames не используй MySQLi расширение, юзай PDO с препарированием запросов.
4. Какая топовая генерация SQL запроса...

Мой вариант без изменений другого кода ну и использование конечно же PDO.
Код: pastebin.com/zVNnEayw

5. Костыли и копипаст кода, тебе стоит познакомится с такой вещью как рефакторинг!
6. Использование двойных кавычек вместо одинарных, хоть это и экономия на спичка но всё же, строки легче читаются, а во вторых PHP не будет искать в них переменных!
7. Не нужно плодить лесенки! А ещё пиши else if, а не elseif это фича тебя погубит!
8. Доверяй только $_SERVER['REMOTE_ADDR']!!!
Всё что начинается с HTTP_ это уже заголовки в которых можно послать что угодно.
(Как то давно один крупный сайт упал из за этого)

9. Снова возвращаюсь к тому что не нужно делать глубокую вложенность!
Как же я ненавижу использовать горизонтальный скролл ради этого!
А сворачивать строки не очень удобно.

10. Я устал, если я продолжу перечислять ошибки то я просто скину весь код)

 

Чуваку даже не лень было это все писать! Как стараемся для ТСа, а он никак не может научиться писать хороший код.

 

Спасибо, что указали на ошибки) Буду стараться в следующий раз.

 

Спасибо, было интересно.
#лукас

 

Connect Error (1045) Access denied for user ' ошибка такая после скобок идёт бд

 

Полностью с вами согласен

 

у меня пишет типо не может подключится к бд не верный пароль а в конфиге верный пароль всё правельно в ведено что делать ?

 

Лично поставил что бы удостовериться, но как бы у вас что то с конфигами

 

да видемо да но я всё правельно в вожу и не фига