Веб Бонус в определённый период

[KronosHD]

Всем привет, нужен был скрипт, который давал бы бонус после 7 дней заходов на сайт. Да, я не очень -то умею писать, сделал, что умел.

Вобщем, чтоб было легко, там есть конфиг, чтоб настроить.

Что делает скрипт:
При авторизации или просто нажатии на кнопку(которая будет отправлять запрос на этот скрипт)
Юзеру добавляется в колонку day + 1
А в колонку last_bonus дату в формате timestamp.
И через 24 часа(это можно настроить) при нажатии добавляется + 1 в колонку day и так столько, сколько вы укажите в config.php.

Если юзер не отметился в течение 48 часов(тоже можно настроить в config.php), то его колонка day обнуляется и он должен отмечаться заного.

В config.php вы должны указать сколько дней нужно отмечаться и какой бонус за это.

И если он отметился столько, сколько вы указали в конфиге, то его счёт пополнится на столько, сколько указали в config.php

 

[Vladlen_198]

Да ты говнокодер, отвечаю.

Спойлер

Больше одинаковых SQL ведь мы не знаешь что такое запятая. Конфиг отдельная история. mysql_connect, уууу.

 

[KronosHD]

Да ты говнокодер, отвечаю.

Спойлер

Посмотреть вложение 37925

Больше одинаковых SQL ведь мы не знаешь что такое запятая. Конфиг отдельная история. mysql_connect, уууу.

Я нубик
Я же написал, делал как умел
Лучше бы подсказал как правильней.
Завтра переделаю, сделаю с немного меньшим говнокодом

 

[KronosHD]

Хоть кто - то оценит скрипт нормально?

 

[KronosHD]

Хорошая работа для новчика. Все начинали с документации и говнокода

Спасибо
Немного исправил, надеюсь не будут агриться
И убрал Mysql. Им скоро и пользоваться перестанут

Да ты говнокодер, отвечаю.

Спойлер

Посмотреть вложение 37925

Больше одинаковых SQL ведь мы не знаешь что такое запятая. Конфиг отдельная история. mysql_connect, уууу.

Щас лучше?

 

[GHOST-MEN]

А импорт? Кто-то должен это написать, ибо некоторым лень самим делать.

 

[Kirich]

А импорт? Кто-то должен это написать, ибо некоторым лень самим делать.

Эм, 2 строчки?

Спойлер: .

ALTER TABLE `users` ADD `days` varchar(255) NOT NULL DEFAULT '';
ALTER TABLE `users` ADD `last_bonus` varchar(255) NOT NULL DEFAULT '';

 

[GHOST-MEN]

Эм, 2 строчки?

Спойлер: .

ALTER TABLE `users` ADD `days` varchar(255) NOT NULL DEFAULT '';
ALTER TABLE `users` ADD `last_bonus` varchar(255) NOT NULL DEFAULT '';

Именно! Прост ваще чёт я ленивый котэ...

Всем привет, нужен был скрипт, который давал бы бонус после 7 дней заходов на сайт. Да, я не очень -то умею писать, сделал, что умел.

Вобщем, чтоб было легко, там есть конфиг, чтоб настроить.

Что делает скрипт:
При авторизации или просто нажатии на кнопку(которая будет отправлять запрос на этот скрипт)
Юзеру добавляется в колонку day + 1
А в колонку last_bonus дату в формате timestamp.
И через 24 часа(это можно настроить) при нажатии добавляется + 1 в колонку day и так столько, сколько вы укажите в config.php.

Если юзер не отметился в течение 48 часов(тоже можно настроить в config.php), то его колонка day обнуляется и он должен отмечаться заного.

В config.php вы должны указать сколько дней нужно отмечаться и какой бонус за это.

И если он отметился столько, сколько вы указали в конфиге, то его счёт пополнится на столько, сколько указали в config.php

А можно пример? Пример как сделать под авторизацию.
Я просто в этом немножко(пи*дёж ваще нуб) нуб.
Ну или любой пример.

 

[Vladlen_198]

Я нубик
Я же написал, делал как умел
Лучше бы подсказал как правильней.
Завтра переделаю, сделаю с немного меньшим говнокодом

Я написал это ещё когда тема была в написание описания.
То что ты использовал PDO ничего не изменилось.
Использую кодировку UTF-8.
Это не нужно, php в любом случае выведет ошибку.

try
{
    $connect = "mysql:host={$host};dbname={$base}";
    $db = new PDO($connect, $user, $pass); 
}
catch(PDOException $e)
{
    echo "Ошибка PDO: {$e->getMessage()}";
}

Ты использовал PDO но ты не использовал statment почему ?
А ты не думал что переменная $_GET['name'] может не существовать и её можно заменить ? Выдаст NOTICE.

$name = $_GET['name'];

Зачем столько переменных ? Дели на массивы.

Спойлер

Зачем вот такое делать ?

Спойлер

Просто лишняя перменная. И ещё не надо получат все элементы, указывай что нужно

$result = $db->prepare('SELECT last_bonus,day FROM '.$table.' WHERE name=:name LIMIT 1')->execute([
    'name' => $name
])->fetch(PDO::FETCH_ASSOC);

Вот как нужно sql update делать если несколько.

UPDATE '.$table.' SET '.$last_bonus.'=:last, '.$money.'=:money, '.$day.'=:day WHERE name=:login

$db->prepare($sql)->execute([
    'last' => $time,
    'bonus' => $bonus,
    'day' => 0,
    'login' => $name,
]);

Дам совет, если переменная не будет использоваться больше одного раза, не создавай её.

Иди на ху*й от сюда хуе*путоло говно кодер твой отец и не более мне лично нравится такой вид а не такой

OMG

 

[TneRED]

$name = $_GET['name']; //Ник юзера
Тут я понял что проверки и в помине нету

 

[Vladlen_198]

$name = $_GET['name']; //Ник юзера
Тут я понял что проверки и в помине нету

Да, желательно бы так, а лучше сразу иметь представления в виде класса.

$name = isset($_GET['name'])?$_GET['name']:null;

 

[GHOST-MEN]

Может всё-же кто-нибудь напишет как эту штуку можно использовать?!

 

[Kirich]

Да, желательно бы так, а лучше сразу иметь представления в виде класса.

$name = isset($_GET['name'])?$_GET['name']:null;

$member_id['name']

 

[GHOST-MEN]

$member_id['name']

Ахаха, а вот и блокировщик. Нах заблочил? Не спасёт же есть VPN

 

[Vladlen_198]

$member_id['name']

Там не указано что это dle

 

[Kirich]

Ахаха, а вот и блокировщик. Нах заблочил? Не спасёт же есть VPN

У тебя 3 аккаунта что-ли на рб? Потому что за*б*л копировать.

 

[GHOST-MEN]

У тебя 3 аккаунта что-ли на рб? Потому что за*б*л копировать.

Нет, больше. А сказать не мог?

 

[KronosHD]

Обновил, оцени и скажи что не так, пожалуйста))

 

[Vladlen_198]

Обновил, оцени и скажи что не так, пожалуйста))

SQL ты не переделал как я показал. С настройками уже лучше. Ключа name в $_GET может и не быть добавляй проверку в начале и уже если $name === null тогда выдавай исключение.

Спойлер

Используй одинарные кавычки, это во первых улучшит читаемость и скрипт будет работать быстрее (Ибо php не нужно искать спец символы/переменные и заменять. Точка и одинарные кавычки очень хороший вариант.). У тебя здесь не указан WHERE и для замены не забудь использовать execute

Спойлер

Общепринято в PDO писать $stmt - statment

Спойлер

Здесь и так строка не сложная использую одинарные кавычки или как вариант ниже так лучше будет.

Спойлер

Спойлер: Альтернативный вариант для примера из моей CMF

Я использую ООП но это можно сделать и в процедурном стиле.

isset здесь уже не нужен ибо переменная создана, нужно понимать разницу isset и empty.
isset - Проверяет что переменная/ключ массива созданы
empty - Проверяет пустое ли значение в этой переменной ('',null,false,0) ну ты понял.

Спойлер

Очень информативные переменные, описывай переменную правильно и не нужно дублировать с цифрой или префиксом. Не используй нижнее подчёркивание лучше так voteTime чем vote_time, это значительней ускорит разработку и будет более читабельно.

Спойлер

 

[KronosHD]

SQL ты не переделал как я показал. С настройками уже лучше. Ключа name в $_GET может и не быть добавляй проверку в начале и уже если $name === null тогда выдавай исключение.

Спойлер

Посмотреть вложение 37947

Используй одинарные кавычки, это во первых улучшит читаемость и скрипт будет работать быстрее (Ибо php не нужно искать спец символы/переменные и заменять. Точка и одинарные кавычки очень хороший вариант.). У тебя здесь не указан WHERE и для замены не забудь использовать execute

Спойлер

Посмотреть вложение 37946

Общепринято в PDO писать $stmt - statment

Спойлер

Посмотреть вложение 37948

Здесь и так строка не сложная использую одинарные кавычки или как вариант ниже так лучше будет.

Спойлер

Посмотреть вложение 37949

Спойлер: Альтернативный вариант для примера из моей CMF

Я использую ООП но это можно сделать и в процедурном стиле.
Посмотреть вложение 37950

isset здесь уже не нужен ибо переменная создана, нужно понимать разницу isset и empty.
isset - Проверяет что переменная/ключ массива созданы
empty - Проверяет пустое ли значение в этой переменной ('',null,false,0) ну ты понял.

Спойлер

Посмотреть вложение 37951

Очень информативные переменные, описывай переменную правильно и не нужно дублировать с цифрой или префиксом. Не используй нижнее подчёркивание лучше так voteTime чем vote_time, это значительней ускорит разработку и будет более читабельно.

Спойлер

Посмотреть вложение 37952

Вот что получилось