php ошибка Cron

Тема в разделе "PHP", создана пользователем ManCuniAn, 18 июл 2010.

Статус темы:
Закрыта.
Модераторы: latteo
  1. ManCuniAn

    ManCuniAn

    Регистр.:
    5 фев 2009
    Сообщения:
    269
    Симпатии:
    32
    есть файл cron.rss.php
    прикрепляю.........файл разработчика Rss grabber собственно крон.

    вылетает ошибка на строке 215 с Else.потом вылетает на строке 296 и на строке 784 все с else...
    .кто разбирается с синтаксисом... помогите.....подправьте файл....
     

    Вложения:

    • cron.rss.rar
      Размер файла:
      7,5 КБ
      Просмотров:
      6
  2. latteo

    latteo Эффективное использование PHP, MySQL

    Moderator
    Регистр.:
    28 фев 2008
    Сообщения:
    1.405
    Симпатии:
    1.185
    В 215 строке else идет после цикла с for, я бы смело удалял такую глупость, в 300й после foreach - тоже в топку, 784 строки в выложенном скрипте не нашел вижу лишь 779 строк.
     
    ManCuniAn нравится это.
  3. chang

    chang

    Регистр.:
    20 ноя 2009
    Сообщения:
    364
    Симпатии:
    117
    в пхп, да и думаю в любом другом ЯП нет конструкций следующего вида
    PHP:
    for( *****  ) {
     
    //  *****
    }else {
     
    //  *****
    }
    foreach( *****  ) {
     
    //  *****
    }else {
     
    //  *****
    }
    /* самая четкая */
    function start_process( **** )
    {
      
    // **********
    }else{
     return 
    false;
    }
    а в вашем коде они используются (строка 215 и 300 ) ....
    скорее блок else должен относится к предыдущему ифу (тоесть нужно его поднять чуть выше к и согласовать скобки)... - но чтоб более точно сказать - нужно смотреть по смыслу написанного кода ...

    по поводу ошибки в 784 ( которой в коде нет) - так это из-за того что у вас не согласованы открывающиеся/закрывающиеся скобки ... скобка открылась но не закрылась ....

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


    PHP:
    // много кода 

    function xxx(){
    // много кода 
    }

    // много кода 
    написали бы ее уже вначале файла, а лучше вовсе в другой файл вынесли ...

    короче разбираться в коде нет никакого желание, так как там банально отсутствует даже самое примитивное форматирование кода
     
    ManCuniAn нравится это.
Статус темы:
Закрыта.