Как оптимизировать код?

Question

[Алексей Кос]

Как оптимизировать данный код:

<script type="text/javascript" src="http://code.jquery.com/jquery-latest.js"></script>
<script>
$(function(){
    var cnt = 0,
    timer = setInterval(function(){
        $.ajax({
            url:'console.php',
            type:'POST',
            dataType:'json',
            data: {some_params: 1},
            success:function(data){
                for(var key in data){
                    $('.'+key).text(data[key]);
                }
            }
        });
        if(cnt++ > 10000) clearInterval(timer);
    }, 1000);
});
</script>
<div class="console"></div>

console.php:

<?php
error_reporting(E_ALL);
if (!function_exists ("ssh2_connect")) {
    die ("function ssh2_connect doesn't exist");
}
 
if(!($con = ssh2_connect("localhost", 22))){
    die("unable to establish connection");
}
 
// авторизуемся по имени пользователя и паролю
if(!ssh2_auth_password($con, "root", "passwd")) {
    die("unable to authenticate");
}

// выполняем комманду
if (!($stream = ssh2_exec($con, "ls" ))) {
    die("unable to execute command");
}

stream_set_blocking($stream, true);
while($line = fgets($stream)) {
flush();
$result = array(
'console' => $line,
);
exit( json_encode($result) );
}
?>

Данный скрипт должен выводить ответ от ssh2 команды, но выводит только одну строку ответа. Что делать?
Google не помог.

Answer 1

[FanatPHP]

совершенно адский вопрос, я бы такие удалял.
Один заголовок чего стоит. "Помогите оптимизировать расходы, дайте 5000 рублей!"

Ну и код, конечно.

while($line = fgets($stream)) {
    flush();
    $result = array(
        'console' => $line,
    );
    exit( json_encode($result) );
}

А попробуй-ка сам посмотреть на этот код, и сказать, что в нем неправильно. Не бойся, это не так сложно, как тебе кажется.

Comments

[Алексей Кос]

FanatPHP Если я поменяю строку "exit( json_encode($result) );" на "print json_encode($result);"?, то нечего не выводит.

[Алексей Яхненко]

ты сам себе расскажи, построчно, что делает процитированный код. глядишь и догадаешься.

[FanatPHP]

Алексей Кос: пойми, этот код делает ровно то, что ты написал - выводит только первую строчку ответа и завершается. То есть это ошибка не в коде, а в задаче. Тебе нужны все строчки ответа? Ну так сначала получи их, все, а потом возвращай. Логично?

[Алексей Кос]

Так я получаю все строки если заменить строку "exit( json_encode($result) );" на "print $list.'
';" и открыть console.php в браузере то выводится полный ответ и все остальные строки.

[FanatPHP]

правильно. но тебе не нужны они в консоли - они нужны в джейсоне. поэтому их надо не выводить сразу, а копить в переменной, а потом ее отдать

[Anton Kolesnikov]

А может этого чувака сразу на уроки программирования за 7 класс отправить проще? А то боюсь что ваши ответы породят только еще больше вопросов...