1

Есть скрипт, который пингует все IP адреса в столбце "ip_address". Нужно чтобы после пинга записывались результат в столбец "status". Т.е. если компьютер находится онлайн, в столбец "status" писать значение 1, а если оффлайн - 0. Пример базы: введите сюда описание изображения

Вот мой скрипт:

<?php
$link = mysqli_connect("192.168.10.56", "test", "test", "test");

if (!$link) {
    echo "Error: Unable to connect to MySQL." . PHP_EOL;
    echo "Debugging errno: " . mysqli_connect_errno() . PHP_EOL;
    echo "Debugging error: " . mysqli_connect_error() . PHP_EOL;
    exit;
}

echo "Работает: Подключение с базой данных имеется!" . PHP_EOL;
echo "Информация о хосте: " . mysqli_get_host_info($link) . PHP_EOL;

foreach($devices as $value){
    exec("ping -c 1 " . $value->ip_address, $output, $result);
    if ($result == 0) {
        $insert_sql = mysqli_query($link, "UPDATE 'test' SET 'status'='1' WHERE ip_addres = '".$value->ip_address."'");
     }else{ 
        $insert_sql = mysqli_query($link, "UPDATE 'test' SET 'status'='0' WHERE ip_addres = '".$value->ip_address."'");
}
    }

mysqli_close($link);
?>

Меня удивляет то, что, когда я запускаю скрипт в самом сервере, он не показывает никаких ошибок, но и не пишет в столбец. Но когда запускаю его на виртуальном сервере, то вот что: введите сюда описание изображения

3
  • $devices - это список IP адресов? покажите его структуру, пожалуйста. Откуда вы его берете?
    – cyadvert
    23 окт 2015 в 5:57
  • мои боги. Php вообще учили? Читайте хотя бы мануал php.net/manual/en/mysqli.examples-basic.php
    – korytoff
    23 окт 2015 в 7:34
  • что хочешь этим сказать? Вроде мы делаем всё правильно... или нет?
    – TriSTaR
    23 окт 2015 в 9:08

1 ответ 1

5

Вообще-то правильно он все пишет.

Invalid argument supplied for foreach() - он ссылается на строку foreach($devices as $value). Судя по коду, переменная $devices нигде не создается, вот он ее и не видит.

Предположим, что $devices это массив с IP адресами.
В таком случае Ваш код будет выглядить вот так:

// берем из таблицы список ip адресов (тут можно и WHERE добавить)
$result = mysqli_query($link, "SELECT ip_address FROM devices");
if ($result) { // если список есть
    $ok = $bad = Array(); // в этих массивах будем хранить списки адресов после проверки
    while ($row = mysql_fetch_assoc($result)) { // прокручиваемся через него по одному
        exec("ping -c 1 " . $row["ip_address"], $output, $result); // пингуем адрес
        if ($result == 0) { // в зависимости от результатов пинга заносим адрес в массив
            $ok[] = $row["ip_address"];
        } else { 
            $bad[] = $row["ip_address"];
        }
    }
    // если есть OK адреса - обновлем все сразу одним запросом
    if (Count($ok)) {
        mysqli_query($link, "UPDATE test SET `status`=1 WHERE `ip_address` IN ('".implode("', '", $ok)."')");
    }
    // если есть BAD адреса - обновлем все сразу одним запросом
    if (Count($bad)) {
        mysqli_query($link, "UPDATE test SET `status`=0 WHERE `ip_address` IN ('".implode("', '", $bad)."')");
    }
}

А если Вас удивляет то, что ошибки показываются на одном сервере и не показываются на другом, так это вполне могут быть просто настройки. Отключено показывание сообщений об ошибках.
Поставьте вот это

error_reporting(E_ALL & ~E_NOTICE);
ini_set('display_errors', '1');

в первых строчках кода. И тогда ошибки будут отображаться везде.

26
  • так я попробывал foreach($devices as $value) и так, foreach($test as $value) , даже так foreach($test => $devices as $value)... а ошибка не меняется... не могли подсказать как надо? чтобы он работал, а то 3 день как мучаюсь :(
    – TriSTaR
    23 окт 2015 в 5:46
  • 2
    нет-нет, синтакс у Вас правильный. Система просто на знает, что такое $devices. У Вас нигде этого в коде нет.
    – cyadvert
    23 окт 2015 в 5:48
  • а как его объявлять? О.о Я ещё новичок в php, а то что я базами данных указал test ничего не меняет?
    – TriSTaR
    23 окт 2015 в 5:52
  • @TriSTaR foreach это последовательный обход коллекции, я так понимаю у Вас есть список "устройств" который Вы хотите пропинговать, я Вас правильно понял?! если да то создайте массив/список и заполните его необходимыми данными, например ip адресами, а дальше у Вас уже все есть
    – Bald
    23 окт 2015 в 5:54
  • Да! на скрине не достаточно показано но есть столбец ip_address где хранятся все айпи адреса всех пользователей кто присоединён к нашим сетям, так вот я хотел бы пинговать каждые 3 мин. (автоматически) и узнавать они в сети или нет!
    – TriSTaR
    23 окт 2015 в 6:01

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.