Почему не срабатывает функция?

Question

[maiskiykot]

Не могу понять причину неработающей функции:

function check_ip()
{
    $fil = file("ip.txt");
 
     foreach ($fil as $val)
    {
     if (!@in_array(trim($val),$_SESSION['ip']))
     {
      $found = 1;
      $_SESSION['ip'][]= trim($val);       
       return trim($val);    
       
     }     
    }
    if (!$found)
   {
      $_SESSION['ip'] = Array();
      include_once "func.php"; // переписывает файл ip.txt
      return trim($arr[1]); // массив $arr формируется в файле func.php
   }
}

Пока массив $fil отрабатывает - все отлично. Потом обнуляется массив $_SESSION['ip'], переписывается файл ip.txt и все останавливается. Никаких ошибок нет, по идее основной цикл должен продолжаться, который получает из этой функции ip, но все останавливается. Я грешил на продолжительность выполнения func.php (где-то минут 5), увеличил все возможные таймауты, но все равно останавливается скрипт. По идее ведь основной цикл не должен выполняться пока не получит ip от этой функции или это не так? Уже весь мозг поломал.

Comments

[Homr]

session_start(); перед $_SESSION['ip'] = Array();
И почему в сессию вы записываете пустой массив?

[maiskiykot]

Homr, это я обнуляю сессию. Сессии нормально работают

[Homr]

maiskiykot, почему бы просто сделать так: unset($_SESSION['ip']); и можно подробнее рассказать о том, что этот код вообще делает?

[maiskiykot]

Сессии только внутри функции работают. Функция отдает ip основному циклу.

[Homr]

я правильно понимаю что в файле "ip.txt" находится массив?)

[FanatPHP]

Homr, я могу рассказать не подробно, но точно: эта функция делает адъ. тут не знаешь с какой строчки начинать плакать

[Homr]

FanatPHP, не понимаю как вы сохраняете массив в текстовый файл, а потом пытаетесь его парсить? Может проще хранить в JSON формате?

[maiskiykot]

FanatPHP, начните плакать уже. Строчки можете не выбирать.

[Homr]

FanatPHP, т.е записать так

file_put_contents('ip.txt', json_encode($ip_array));

а читать и парсить так:

$json = json_decode(file_get_contents('ip.txt'));
foreach ($json as $val){
//
}

[Homr]

maiskiykot, лан пойду плакать (:

[maiskiykot]

Homr, Вы вопрос читали? Все нормально у меня и с записью и с чтением. Функция останавливается - вот в чем был вопрос.

[Homr]

maiskiykot, у меня сквозь слезы ничего не видно

[di]

if (!@in_array(trim($val),$_SESSION['ip']))
     {
      $_SESSION['ip'][]= trim($val);       
       return trim($val);    
      break; 
     }

Это останавливает цикл, функция возвращает значение trim($val); Потому что после оператора return вызванного внутри функции, она возвращает значение и дальше не работает. незачем там break еще писать, цикл уже остановился.

[maiskiykot]

Алексей Долгов, Однако вот эти две строчки отрабатывают. $_SESSION['ip'] = Array();
include_once "func.php"; // переписывает файл ip.txt Не работает только последняя или то, что принимает вывод этой функции

[ipokos]

Алексей Долгов, ну да.. это явно самая большая проблема этого кода))

[di]

maiskiykot, серьезно? Ну напиши var_dump("hello_world") после строки include_once "func.php";
И посмотри - сработает команда или нет.

[maiskiykot]

Алексей Долгов, А чего мне писать? Файл сам по себе переписывается что ли?

Answer 1

[maiskiykot]

Как обычно выкрутился сам. Плюнул на этот вариант и сделал вызов файла func.php курлом с таймаутом. Начало все крутиться! Всем спасибо за участие!

Answer 2

[ipokos]

Потом обнуляется массив $_SESSION['ip'], переписывается файл ip.txt и все останавливается

А что должно продолжаться если цикл закончил перебирать массив?

по идее основной цикл должен продолжаться, который получает из этой функции ip

что значит основной цикл? Вы эту функцию вызываете в цикле в другом месте?
Если да, то как он должен работать если вы обнуляете все данные из SESSION['ip'], а потом проверяете вхождение в него?

Зачем break после return ?
что бы точно дальше итерации не пошли? если да, то тогда лучше несколько ставить break; break; break;

По идее ведь основной цикл не должен выполняться пока не получит ip от этой функции или это не так?

И снова, что это за мистический " основной цикл" ?

Comments

[maiskiykot]

Основной цикл перебирает урлы, этот цикл IP, которые подставаляются для маскировки. В основном цикле десятки тысяч адресов, а этот список приходится обновлять, проверяя на живость IP. В итоге после прохода основного цикла обнуляется массив проверенных ip и создается новый файл с очередной порцией ip. Дальше функция должна отдать ip, который от нее все еще ждет(или нет?) основной цикл. Но вместо этого все останавливается. Ошибок никаких не выбрасывает. Где я накосячил - ХЗ.

[ipokos]

На словах логику как-то сложновато воспринимать.
1 - вы получаете список адресов $fil = file("ip.txt");
и перебираете, если в $_SESSION['ip'] нет такого ip, то вы обнуляете $_SESSION['ip'] (это суперглобальная переменная) и возвращаете непонятный $arr[5], а потом при следующей итерации "главного цикла", снова в нее смотрите.

2 - в том случае, если адреса в $_SESSION нет, (!@in_array(trim($val),$_SESSION['ip'])) , то "основному циклу" вы возвращаете $val.

3 - include_once "func.php"; // переписывает файл ip.txt
как переписывает? там функция или сразу исполняемый код?

4 - break в цикле, используется когда нужно прервать выполнение цикла, но продолжить выполнять код, который после него.

return $var - возвращает переданное значение $var

Включите отображение ошибок, смотрите логи, используйте xdebug

[maiskiykot]

ipokos, 3 - функция - мультипроверка ip. 4 - break убрал. Ошибки все включены вплоть до php.ini - пусто.

[ipokos]

3 - функция - мультипроверка ip

где вы ее вызываете?

[maiskiykot]

ipokos, В файле func.php. Все отрабатывает как надо. Переменные не конфликтуют с данной функцицией.

[Дмитрий]

php.net/manual/ru/control-structures.break.php
Про числовой необязательный аргумент посмотрите.

[maiskiykot]

Я уже и так и эдак все перевернул - стОит поставить return после вызова func.php и он не срабатывает. Видимо, все-таки проблема в том, что по таймауту отваливается сам цикл, не дождавшись ответа функции. Как это победить? Остальные вопросы прошу не обсуждать - код на коленке писался - ошибки были.