MajorDoMo

igk1972 писал(а):Запуск циклов стартует startup_maintenance.php (в нем делает бекап бд, бекап файлов, и еще ко-что)
Спрашивается зачем файл fix_linked_properties.php в котором основная часть кода из startup_maintenance.php (или наоборот)?
И еще есть файл periodical_db_save.php

sergejey писал(а):сам модуль рудимент и код его писался лет 10 назад наверное

LutsenkoDenis писал(а):Файл /lib/perfmonitor.class.php
1. В функции EndMeasure условие if ($save_to_db) и все что после него никогда не выполняется. Вопрос, а нафиг это в коде тогда?

LutsenkoDenis писал(а):2. В той же функции еще есть ошибка: вот так $perf_data[$mpoint]['NUM']++; делать нельзя. Т.к. нельзя прибавлять что-либо к не объявленной переменной или к не существующему элементу массива.

sergejey писал(а):Чтобы было
Это функционал сбора статистики производительности в базу данных, в продакшене не используется, но оставлен для профилирования.

sergejey писал(а):Ну не красиво немножко, это да, но не так критично, что прямо нельзя -- работает же

LutsenkoDenis писал(а):Файл: /modules/application.class.php
Ошибка: Нужно UPDATE вместо UPDAE

1. Данная функция не всегда возвращает результат. По идее она должна в блоке где записывается ошибка возвращать пустой массив.
2. Зачем в цикле каждый раз вычисляется кол-во строк данных из запроса к бд?

function Select($query)
   {
      if ($result = mysql_query($query, $this->dbh)) 
      {
         $res = array();
         
         for($i = 0; $i < mysql_num_rows($result); $i++) 
         {
            $rec = mysql_fetch_array($result, MYSQL_ASSOC);
            $res[] = $rec;
         }
         
         return $res;
      }
      else 
      {
         $this->Error($query);
      }
   } 

LutsenkoDenis писал(а):Файл: /lib/mysql.class.php

1. Данная функция не всегда возвращает результат. По идее она должна в блоке где записывается ошибка возвращать пустой массив.
2. Зачем в цикле каждый раз вычисляется кол-во строк данных из запроса к бд?
3. Зачем в массив res записываются в цикле одни и те же данные?

• а) смотрим описание функции(написано если что не мной )
  Это публичная функция (access public), которая на вход принимает строковый параметр с запросом к бд(string $query) и возвращает массив(return array).

  /**
  * Execute SQL SELECT query and return all records
  *
  * This method returns records as array of assosiated arrays (by field names)
  *
  * @param string $query SQL SELECT query
  * @return array execution result
  * @access public
  */

• б) Исходя описания выше, в случае ошибки функция должна вернуть пустой массив.
• в) ошибка выборки... если имеется в виду это $this->Error($query); - ничего не прерывает. То что в ней return 1 ни о чем не говорит, т.к. далее нигде не используется.