Проблема бесконечного цикла PHP

Question

function httpGet( $url, $followRedirects=true ) {
    global $final_url;
    $url_parsed = parse_url($url);
    if ( empty($url_parsed['scheme']) ) {
        $url_parsed = parse_url('http://'.$url);
    }
    $final_url = $url_parsed;

    $port = $url_parsed["port"];
    if ( !$port ) {
        $port = 80;
    }
    $rtn['url']['port'] = $port;

    $path = $url_parsed["path"];
    if ( empty($path) ) {
        $path="/";
    }
    if ( !empty($url_parsed["query"]) ) {
        $path .= "?".$url_parsed["query"];
    }
    $rtn['url']['path'] = $path;

    $host = $url_parsed["host"];
    $foundBody = false;

    $out = "GET $path HTTP/1.0\r\n";
    $out .= "Host: $host\r\n";
    $out .= "User-Agent:      Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1) Gecko/20061010 Firefox/2.0\r\n";
    $out .= "Connection: Close\r\n\r\n";

    if ( !$fp = @fsockopen($host, $port, $errno, $errstr, 30) ) {
        $rtn['errornumber'] = $errno;
        $rtn['errorstring'] = $errstr;

    }
    fwrite($fp, $out);
    while (!@feof($fp)) {

        $s = @fgets($fp, 128);
        if ( $s == "\r\n" ) {
            $foundBody = true;
            continue;
        }
        if ( $foundBody ) {
            $body .= $s;
        } else {
            if ( ($followRedirects) && (stristr($s, "location:") != false) ) {
                $redirect = preg_replace("/location:/i", "", $s);
                return httpGet( trim($redirect) );
            }
            $header .= $s;
        }
    }

    fclose($fp);

    return(trim($body));
}

Этот код иногда проходит бесконечный цикл. Что здесь не так?

Answer 1

В документации feof() имеется большое красное предупреждение :

Внимание

Если соединение, открытое fsockopen(), не было закрыто сервером, feof() будет зависать. Чтобы обойти это, см. Пример ниже:

Пример # 1 Обработка таймаутов с помощью feof ()

    <?php
        function safe_feof($fp, &start = NULL) {
             $start = microtime(true); 
             return feof($fp);
        }   
        /* Assuming $fp is previously opened by fsockopen() */

        $start = NULL;
        $timeout = ini_get('default_socket_timeout');

        while(!safe_feof($fp, $start) && (microtime(true) - $start) < $timeout)
        {
          /* Handle */
        }
    ?>

---

Кроме того, вы должны только писать или читать из файлового указателя, если он действителен (что вы не делаете, вы просто устанавливаете сообщение об ошибке): Это приводит ко второму большому красному предупреждению:

Внимание

Если переданный файловый указатель недействителен, вы можете получить бесконечный цикл, потому что feof() не может вернуть TRUE.

Лучше было бы:

$result = '';

if ( !$fp = @fsockopen($host, $port, $errno, $errstr, 30) ) {
    $rtn['errornumber'] = $errno;
    $rtn['errorstring'] = $errstr;
}
else {
    fwrite($fp, $out);
    while (!@feof($fp)) {
        //...
    }
    fclose($fp);
    $result = trim(body);
}
return $result;

---

Последнее замечание: если вы выполните перенаправление с

 if ( ($followRedirects) && (stristr($s, "location:") != false) ) {
     $redirect = preg_replace("/location:/i", "", $s);
     return httpGet( trim($redirect) );
 }

вы никогда не закрываете указатель файла. Я думаю, что лучше это:

 if ( ($followRedirects) && (stristr($s, "location:") != false) ) {
     $redirect = preg_replace("/location:/i", "", $s);
     $result = httpGet( trim($redirect) );
     break;
 }
 // ...
 return $result;

Answer 2

function httpGet( $url, $followRedirects=true ) {
[...]
                return httpGet( trim($redirect) );
}

Ничто не мешает вам снова и снова получать один и тот же URL.

Answer 3

feof вернет false, если соединение все еще открыто в потоке tcp / ip.