douju2599 2011-10-20 11:58
浏览 78
已采纳

PHP ob_start调用两次,现在不会正确刷新

I have written a MVC-framework and I am using ob_start('error_handler') to be able to catch fatal errors aswell. This works great!

In boot.php

// Start the error-logging
ob_start( 'error_logging' );

// Run the request
Request::run( URL::getInstance()->init()->formatURL() );

// Flush and disable output-buffering
ob_end_flush();

error_logging function:

function error_logging( $output )
{

    // Get the last error
    $trace = error_get_last();
    // Initialize our final array of fields explaining the error
    $final = array();

    // Check if the error does not exist, return the original output unedited
    if( !isset( $trace ) ) return $output;

    // Loop through or error and build our array
    foreach( $trace as $info => $value ) $final[$info] = $value;

    // Initialize our Error-class
    $error = Error::factory();
    // Handle the error with the fields and build our error-message
    $newOutput = $error->handle( $final['type'] , $final['message'], $final['file'], $final['line'] );

    // Check if the new output is null, if yes set output to the original output, otherwise set it to the new output
    $output = ( $newOutput === null ) ? $output : $newOutput;

    // Return the output
    return $output;

}

Again, this works fine and dandy to handle all the errors! I don't use any flush or anything inside the Error-class.

Now, I have a Controller that uses ob_start since it's a fairly long running-time of a certain method and I want feedback to the user what's going on and what the script does. I'm using ob_flush() for this.

However, after implementing this ob_start('error_logging'); the ob_flush's don't seem to work. They wait until the entire script is finished(which takes about 8 minutes).

Why?

An example of how I'm flushing is like this: In Controller-method:

$this->login->execute();

In Login::execute:

public function execute( $site = 'mobile' )
{

    ob_start();

    $this->debug->log('--------------------------------------------<br />');
    $this->debug->log( 'Logging into site: ' . ucfirst( $site ) . '<br />' );

    // Initiate cookies
    if( $site == 'full' )
        $this->_initCookies();

    // Set up the URL
    $url = Config::get( $site ) . Config::get( 'url_login' );
    curl_setopt( $this->curl->get(), CURLOPT_URL, $url );

    // CURL-values
    curl_setopt( $this->curl->get(), CURLOPT_FOLLOWLOCATION, 1  );
    curl_setopt( $this->curl->get(), CURLOPT_RETURNTRANSFER, 1  );
    curl_setopt( $this->curl->get(), CURLOPT_POST,           1  );
    curl_setopt( $this->curl->get(), CURLOPT_POSTFIELDS,     Config::get( 'postfields' ) );
    curl_setopt( $this->curl->get(), CURLOPT_COOKIEFILE,     'resources/tmp/cookies.txt');
    curl_setopt( $this->curl->get(), CURLOPT_CONNECTTIMEOUT, 10 );
    curl_setopt( $this->curl->get(), CURLOPT_TIMEOUT,        40 );
    curl_setopt( $this->curl->get(), CURLOPT_USERAGENT,      'Mozilla/5.0 (Windows NT 6.1; WOW64; rv:6.0.1) Gecko/20100101 Firefox/6.0.1');

    // Get the page returned after we've logged in
    $this->login_html = str_get_html( curl_exec( $this->curl->get() ) );

    // Validate the page returned to see if we successfully logged in
    $this->_validateLogin( $site );

    $this->debug->log('--------------------------------------------<br />');

    ob_end_flush();

    // Return $this
    return $this;

}

Each $this->debug->log() call flushes the output like so:

public function log( $string )
{

    if( $this->active() || $this->logging() )
    {

        echo str_repeat("<!-- AGENT SMITH -->", 500);
        echo $string;

        ob_flush();
        flush();

    }

}

Any idea why it's not flushing correctly?? Tell me if you need to view some more code and I'll provide it!!

Thanks

::::::::SOLUTION::::::::::

I had to save the first buffer(the error_logging), end it, start the new one, do my things and then end that one, start the first buffer again and echo the saved output.

Did a class for it:

class Tanaxia_Buffer
{

    public static $instance = null;

    private $main = null;
    private $previousOutput = "";

    public static function getInstance()
    {

        if( Buffer::$instance === null )
            Buffer::$instance = new Buffer();

        return Buffer::$instance;

    }

    public function set_main( $ob )
    {

        $this->main = $ob;

    }

    public function start( $main = null )
    {

        if( $main !== null )
            $this->set_main( $main );

        ob_start( $this->main );

    }

    public function end( $type = 'clean' )
    {

        switch( $type )
        {

            case 'flush': ob_end_flush(); break;
            case 'clean': ob_end_clean(); break;

        }

    }

    public function start_separate()
    {

        $this->previousOutput = ob_get_contents();
        ob_end_clean();

        ob_start();

    }

    public function end_separate( $type = 'flush' )
    {

        switch( $type )
        {

            case 'flush': ob_end_flush(); break;
            case 'clean': ob_end_clean(); break;

        }

        ob_start( $this->main );
        echo $this->previousOutput;

        empty( $this->previousOutput );

    }

}

Usage:

boot.php

// Start the error-logging
Buffer::getInstance()->start( 'error_logging' );

// Run the request
Request::run( URL::getInstance()->init()->formatURL() );

// Flush and disable output-buffering
Buffer::getInstance()->end( 'flush' );

Login::execute():

public function execute( $site = 'mobile' )
{

    Buffer::getInstance()->start_separate();

    // Everything in between

    Buffer::getInstance()->end_separate();

    // Return $this
    return $this;

}

Works perfectly! Won't support several nested ob_start's though since it only saves 1 previous output. Can be fixed though but not neccessary right now!

  • 写回答

1条回答 默认 最新

  • duanliaouu965826 2011-10-20 13:49
    关注

    I'm guessing that it has something to do with the nested ob_start() calls. I'm assuming that you want to keep them nested, but does it flush the way you want it to if in your execute() function you either call ob_end_flush() or ob_end_clean() before you start the next output buffer? Something like this:

    public function execute( $site = 'mobile' )
    {
        // save existing contents of output buffer
        $previousOutput = ob_get_contents();
        // wipe the output buffer itself
        ob_end_clean();
        // start a new output buffer
        ob_start();
    
        /* SNIP all the code that wouldn't change */
    
        // flush the function's output buffer
        ob_end_flush();
        // restart the old output buffer
        ob_start("error_logging");
        // take whatever was in the old buffer and make sure it shows up in the new one
        echo $previousOutput;
    
        return $this;
    }
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论

报告相同问题?

悬赏问题

  • ¥15 对于这个复杂问题的解释说明
  • ¥50 三种调度算法报错 采用的你的方案
  • ¥15 关于#python#的问题,请各位专家解答!
  • ¥200 询问:python实现大地主题正反算的程序设计,有偿
  • ¥15 smptlib使用465端口发送邮件失败
  • ¥200 总是报错,能帮助用python实现程序实现高斯正反算吗?有偿
  • ¥15 对于squad数据集的基于bert模型的微调
  • ¥15 为什么我运行这个网络会出现以下报错?CRNN神经网络
  • ¥20 steam下载游戏占用内存
  • ¥15 CST保存项目时失败