Код php(Пых)

Ребят, кто хорошо шарит в php, прошу посмотреть на код, что тут не так?
Этот код, по какой-то причине не выполняется, не могу понять почему.
Очень большая просьба, не писать "ГАмнАкод!" и т.п. Как умею, так и пишу

PHP:

<?php
    define("DATALIFEENGINE", true);
    include("/engine/classes/mysqli.class.php");
    include("/engine/data/dbconfig.php");
    #Получаем всех не подтвержденных игроков.
    $results = mysql_query("SELECT * FROM `dle_users` WHERE `confirm` = 'FALSE'");
   
    #Получаем время игрока на сервере.
    while($rows = mysql_fetch_array($results)) {
    $byurls = $rows['byurl'];
    $rowsname = $rows['byurl'];
    if($byurls != ''){ 
    $resulttime = mysql_query("SELECT * FROM `playTime` WHERE `username` = '".$rows['name']."'");   
    $row1 = mysql_fetch_array($resulttime);
    $playedTime = $row1['playtime'];
   
    #Проверка и выдача денег.
    if($playedTime > 480 || $playedTime < 1439){
            mysql_query("UPDATE `dle_users` SET `confirm` = 'TRUE' WHERE `name` = '".$rowsname."'");
            mysql_query("UPDATE `dle_users` SET `money` = `money`+10 WHERE `name` = '".$rowsname."'");
            mysql_query("UPDATE `dle_users` SET `returnmoney` = 'returnmoney'+10 WHERE `name` = '".$rowsname."'");
            echo "1";
            //SaveLog("'".$rows['byurl']."'", "<i class='uk-icon-user-plus'></i> Игрок <b>".$rows['byurl']."</b> прогласил игрока <b>".$rows['name']."</b>");
        } elseif($playedTime > 1440 || $playedTime < 2879) {
            mysql_query("UPDATE `dle_users` SET `confirm` = 'TRUE' WHERE `name` = '".$rowsname."'");
            mysql_query("UPDATE `dle_users` SET `money` = `money`+20 WHERE `name` = '".$rowsname."'");
            mysql_query("UPDATE `dle_users` SET `returnmoney` = 'returnmoney'+20 WHERE `name` = '".$rowsname."'");
            echo "2";
             //SaveLog("'".$rows['byurl']."'", "<i class='uk-icon-user-plus'></i> Игрок <b>".$rows['byurl']."</b> отыграл 24 часа, его пригласил <b>".$rows['name']."</b> и он получил '".$money1440."'");
        } elseif($playedTime > 2880) {
            mysql_query("UPDATE `dle_users` SET `confirm` = 'TRUE' WHERE `name` = '".$rowsname."'");
            mysql_query("UPDATE `dle_users` SET `money` = `money`+30 WHERE `name` = '".$rowsname."'");
            mysql_query("UPDATE `dle_users` SET `returnmoney` = 'returnmoney'+30 WHERE `name` = '".$rowsname."'");
            echo "3";
            //SaveLog("'".$rows['byurl']."'", "<i class='uk-icon-user-plus'></i> Игрок <b>".$rows['byurl']."</b> отыграл 48 часов, его пригласил <b>".$rows['name']."</b> и он получил '".$money2880."'");
        }
    } else {
          mysql_query("UPDATE `dle_users` SET `confirm` = 'TRUE'  WHERE `name` = '".$rowsname."'");
    }
    echo "Cron success";
    }
    ?>


 

Комментов много! Уберёшь их и заробит. ИНФА СОТКА

 

Разве не mysql.class.php?

 

> mysql_*
> 2016
Вскройся

 

Сделай вывод ошибок да и се

 

Дерек подсказал уже.
5-й пых использует mysqli_, вместо mysql_ .

 

Сейчас сюда прибежит @Negezor и заорётзакричит про то, что не используются сложные запросы, да и вообще используются циклы.

 

Код использует библиотеку базы DLE, так че не используются его функции. Что-то мне подсказывает проблема в этом. И в одном запросе UPDATE можно задавать сразу несколько изменений колонок
UPDATE `dle_users` SET `confirm` = 'TRUE', `money` = `money`+10 , `returnmoney` = 'returnmoney'+10 WHERE `name` = '".$rowsname."'

 

А как выводить все строки из бд?
mysqli_fetch_array по одной же строке отдает.

 

Ну, он бы сказал, что для каждой строки выполняется ещё один запрос(чтобы снова получить данные), когда можно было бы те "конечные" данные получить как результат выполнения сложного запроса.

 

Так же можно объединить запрос выборки из DLE и выборку времени. LEFT OUTER JOIN в помощь, и вместо сотни и тысяч запросов, будет всего один

 

Леха ну я не такой( АААААААААААААА ГОВНОКОД!!!! НЕ ИСПОЛЬЗУЮТСЯ СЛОЖНЫЕ ЗАПРОСЫ!!!
А по делу я бы переделал это так: http://pastebin.com/HQrJ7wfK
И судя по ошибка что ты показывал, там точно как модуль у тебя CRON.

Спойлер

PHP:

<?php

    $notConfirmed = $db->super_row('
        SELECT
            u.name AS login,
            u.byurl,
            pt.playtime
        FROM
            dle_users u
        INNER JOIN
            playTime pt
        ON
            pt.username=u.name
        WHERE
            confirm="FALSE"
    ');

    $updateSql = '
        UPDATE
            dle_users
        SET
            confirm="TRUE",
            money=money+%d,
            returnmoney=returnmoney+%d
        WHERE
            name="%s"
    ';

    foreach ($notConfirmed as $user) {
        if (empty($user['byurl'])) {
            $db->query('UPDATE dle_users SET confirm="TRUE" WHERE name="'.$user['login'].'"');

            continue;
        }

        $time = $user['playtime'];

        if ($time < 480) {
            continue;
        } else if ($time < 1439) {
            $sum = 10;
        } else if ($time < 2879) {
            $sum = 20;
        } else if ($time > 2880) {
            $sum = 30;
        }

        $db->query(sprintf($updateSql,$sum,$sum,$user['login']));
    }




 

А я?

 

Лучше же поменять функции на mysqli, чем подключать более старый класс, разве нет?

 

Я предположил, что в mysqli ошибка.

 

Вариант с PDO, возможно там не как модуль конектится: http://pastebin.com/zX0dRaGk

Спойлер

PHP:

<?php

    $pdo = new PDO('mysql:host=YOU_HOST;dbname=YOU_DB','YOU_USER','YOU_PASSWORD');

    $stmt = $pdo->prepare('
        SELECT
            u.name AS login,
            u.byurl,
            pt.playtime
        FROM
            dle_users u
        INNER JOIN
            playTime pt
        ON
            pt.username=u.name
        WHERE
            confirm=?
    ');
    $stmt->execute(['FALSE']);

    $notConfirmed = $stmt->fetchAll();

    $updateSql = '
        UPDATE
            dle_users
        SET
            confirm=:confirm,
            money=money+:money,
            returnmoney=:return
        WHERE
            name=:login
    ';

    foreach ($notConfirmed as $user) {
        if (empty($user['byurl'])) {
            $stmt = $pdo->prepare('UPDATE dle_users SET confirm=:confirm WHERE name=:login');
            $stmt->execute([
                'confirm' => 'TRUE',
                'login' => $user['login']
            ]);

            continue;
        }

        $time = $user['playtime'];

        if ($time < 480) {
            continue;
        } else if ($time < 1439) {
            $sum = 10;
        } else if ($time < 2879) {
            $sum = 20;
        } else if ($time > 2880) {
            $sum = 30;
        }

        $pdo->prepare($updateSql);
        $stmt->execute([
            'confirm' => 'TRUE',
            'money' => $sum,
            'return' => $sum,
            'login' => $user['login']
        ]);
    }


 

Ну вы и костыльщики, есть же куча плагинов, которые сами начисляют баланс по времени.