Неблагодарное это дело – выдавать сообщения об ошибках




Принцип обмена данными из 1С с сайтом (на MySQL) и выдачи (публикации) этих данных по запросу.
PHP-Скрипт автоматической загрузки данных из файла данных в формате CSV в базу данных сайта работающего на WordPress.

В продолжение моей темы: 1С:Альфа-Авто Автосалон Автосервис: обмен с сайтом.
С помощью данного скрипта можно загружать в автоматическом режиме, по расписанию, данные сервисных книжек (ремонтов авто) из 1С:Альфа-Авто Автосалон Автосервис.
Также можно загружать данные в ручном режиме: для этого делается скрытая страница, где размещается специальная кнопка.
Комментарии размещенные внутри скрипта разъяснят логику и порядок действия.
Комментарии с "/////    echo" использовались для отладки.
Дополнительно создана таблица для журналирования результатов загрузки данных.
Скрипт включает в себя защиту от SQL инъекций (думаю безопасность соблюдена в полной мере).
В кратце:
1. Пишется скрипт, который запускает этот.
2. Создается регламентное задание в WordPress, по которому запускается скрипт из п.1. 
3. Этот скрипт осуществляет проверку на существование файла обмена в папке.
4. Если данные не новые, загрузка не производится.
5. Если данные новые, очищается таблица сервисных книжек.
6. Загружаются новые данные.

Собственно сам скрипт:

<?php // Полная загрузка сервисных книжек, создан 2026-01-05 12:44:55

global $wpdb2;
global $failure;
global $file_hist;

/////  echo '<H2><b>Старт загрузки</b></H2><br>';

$failure=FALSE;
//подключаемся к базе
$wpdb2 = include_once 'connection.php'; ; // подключаемся к MySQL
// если не удалось подключиться, и нужно оборвать PHP с сообщением об этой ошибке
if (!empty($wpdb2->error))
{
/////   echo '<H2><b>Ошибка подключения к БД, завершение.</b></H2><br>';
$failure=TRUE;
wp_die( $wpdb2->error );
}

$m_size_file=0;
$m_mtime_file=0;
$m_comment='';
/////проверка существования файлов выгрузки из 1С
////файл выгрузки сервисных книжек
$file_hist = ABSPATH.'/_1c_alfa_exchange/AA_hist.csv';
if (!file_exists($file_hist))
{
/////   echo '<H2><b>Файл обмена с сервисными книжками не существует.</b></H2><br>';
$m_comment='Файл обмена с сервисными книжками не существует';
$failure=TRUE;
}

/////инициируем таблицу лога
/////если не существует файла то возврат и ничего не делаем
if ($failure){
///включает защиту от SQL инъекций и данные можно передавать как есть, например: $_GET['foo']
/////   echo '<H2><b>Попытка вставить запись в лог таблицу</b></H2><br>';
$insert_fail_zapros=$wpdb2->insert('vin_logs', array('time_stamp'=>time(),'last_mtime_upload'=>$m_mtime_file,'last_size_upload'=>$m_size_file,'comment'=>$m_comment));
wp_die();
/////    echo '<H2><b>Возврат в начало.</b></H2><br>';
return $failure;
}
/////проверка лога загрузки, что бы не загружать тоже самое
$masiv_data_file=stat($file_hist);   ////передаем в массив свойство файла
$m_size_file=$masiv_data_file[7];    ////получаем размер файла
$m_mtime_file=$masiv_data_file[9];   ////получаем дату модификации файла
////создаем запрос на получение последней удачной загрузки
////выбираем по штампу времени создания (редактирования) файла загрузки AA_hist.csv, $m_mtime_file

/////   echo '<H2><b>Размер файла: '.$m_size_file.'</b></H2><br>';
/////   echo '<H2><b>Штамп времени файла: '.$m_mtime_file.'</b></H2><br>';
/////   echo '<H2><b>Формирование запроса на выборку из лога</b></H2><br>';
////препарируем запрос
$text_zaprosa=$wpdb2->prepare("SELECT * FROM `vin_logs` WHERE `last_mtime_upload` = %s", $m_mtime_file);
$results=$wpdb2->get_results($text_zaprosa);

if ($results)
{   foreach ( $results as $r)
{
////если штамп времени и размер файла совпадают, возврат
if (($r->last_mtime_upload==$m_mtime_file) && ($r->last_size_upload==$m_size_file))
{////echo '<H2><b>Возврат в начало, т.к. найдена запись в логе.</b></H2><br>';
$insert_fail_zapros=$wpdb2->insert('vin_logs', array('time_stamp'=>time(),'last_mtime_upload'=>$m_mtime_file,'last_size_upload'=>$m_size_file,'comment'=>'Загрузка отменена, новых данных нет, т.к. найдена запись в логе.'));
wp_die();
return $failure;
}
}
}
////если данные новые, пишем в лог запись о начале загрузки
/////echo '<H2><b>Попытка вставить запись о начале загрузки в лог таблицу</b></H2><br>';
$insert_fail_zapros=$wpdb2->insert('vin_logs', array('time_stamp'=>time(),'last_mtime_upload'=>0, 'last_size_upload'=>$m_size_file, 'comment'=>'Начало загрузки'));

////очищаем таблицу
$clear_tbl_zap=$wpdb2->prepare("TRUNCATE TABLE %s", 'vin_history');
$clear_tbl_zap_repl=str_replace("'","`",$clear_tbl_zap);
$results=$wpdb2->query($clear_tbl_zap_repl);
/////   echo '<H2><b>Очистка таблицы сервисных книжек</b></H2><br>';
if (empty($results))
{
/////   echo '<H2><b>Ошибка очистки таблицы книжек, завершение.</b></H2><br>';
//// если очистка не удалась, возврат
$failure=TRUE;
wp_die();
return $failure;
}

////загружаем данные
$table='vin_history';         // Имя таблицы для импорта
//$file_hist Имя CSV файла, откуда берется информация     // (путь от корня web-сервера)
$delim=';';          // Разделитель полей в CSV файле
$enclosed='"';      // Кавычки для содержимого полей
$escaped='\

15 Comments

  1. maxx

    Идея хорошая, я бы еще добавил в объект СообщениюПользователя путь к данным объектов или мест где возникла ошибка, чтобы навигация работала по списку ошибок, т.е нажал по ошибке и перешел сразу к месту где ошибка

    Reply
  2. Abadonna

    (0)

    // Возможно пользователь использовал Символы.ПС в своих нуждах, поэтому

    // временно заменим все Символы.ПС на другую комбинацию символов.

    // Потом вернем назад

    ВместоПС = «{;}»;

    Рекомендую: (ALT+0134) (ALT+0135) — никогда простой юзверь такое на клавиатуре не наберет 😉

    Reply
  3. Yashazz

    Да, уже витает в воздухе.

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

    Reply
  4. ufo58

    все бы ничего

    и идея хороша, и посыл нормальный

    но … выводить «Сообщением», тем более в управляемых формах … бррр

    да и Дерево … тяжеловато

    подумай, может дать вариант формирования тип Текст ?

    с последующим просмотром

    или сохранением …

    Reply
  5. TitanLuchs

    Присоединюсь к предыдущему оратору. При возможности тоже стараюсь избегать использование «Сообщить» и «СообщениеПользователю» в УФ, потому что перекореживание формы в этот момент — это иногда зрелище не для слабонервных. Одно время в своей конфе открывал отдельное окно с сообщениями, на быстродействии это не сказывалось (потому что форма все равно перерисовывается при изменении размера), но юзеров не так бесило.

    Reply
  6. pakill

    Вообще-то я старался сделать легкую процедуру, не зависящую ни от конфигурации, ни от контекста.

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

    Насчет вывода ошибки с помощью Сообщить():

    А как ее еще выводить на стороне сервера?

    Это можно сгладить лишь частично:

    #Если Сервер Тогда

    Сообщить(…);

    #Иначе

    Предупреждение(…);

    #КонецЕсли

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

    Reply
  7. Yashazz

    (6) Вот я именно и заменил «Сообщить» своей процедурой. И УФ не корёжит, и на сервере работает, в протокол пишет.

    Reply
  8. devel0per

    Хоть комментарий к процедуре и прекрасен, но из него не ясно чем плох отладчик, ведь

    нам никто не запрещает им пользоваться на сервере.

    Не лучше ли будет вместо:

    Коорд    = СокрЛП(КоординатыОшибки);

    и

    Если Коорд <> «» Тогда

    писать:

    Если ПустаяСтрока(Коорд) Тогда

    Вот не помню в клюшках функц. ПустаяСтрока была?

    Reply
  9. WalterMort

    Автор изобрел велосипед, вместо грамотной работы с исключениями.

    Reply
  10. Evil Beaver

    (9) WalterMort, ну тогда уж приводите пример «грамотной» работы с исключениями в рамках поставленной задачи. Что вы понимаете под «грамотной» работой?

    Reply
  11. Bukaska

    (9) WalterMort, Ой да ладно.. ну что же вы все как эти… даже не знаешь как назвать… Я конечно понимаю что нужно реализовываться, знаю что и инициатива бывает наказуема. Но кто не работает, тот не ест)))

    Reply
  12. WalterMort

    (10) Обычная передача ошибок снизу наверх. Классов исключений в 1С нет, но описания ошибки для замысла автора достаточно:

    Процедура Внешняя()

    Попытка

    Внутренняя();

    Исключение

    ВызватьИсключение «Невозможно выполнить внешняя: » + ОписаниеОшибки(); // наверх

    КонецПопытки;

    КонецПроцедуры

    Процедура Внутренняя()

    Попытка

    а = б/0;

    Исключение

    ВызватьИсключение «Невозможно выполнить внутренняя»; // передаем наверх

    КонецПопытки;

    КонецПроцедуры

    Результат: «Невозможно выполнить внешняя: Невозможно выполнить внутренняя»

    Ну а исключение клиентского (самого высокого) уровня уже можно как угодно отработать. Сообщением, предупреждением, или записью в лог.

    Reply
  13. Evil Beaver

    (12) WalterMort, вот поскольку классов исключений нет, то все, что могут передать исключения — это строки. Отсюда, шибко «грамотной» работы с исключениями все равно не будет.

    Reply
  14. pakill

    (8). Чем же плох отладчик? Чтож. Дайте пользователю отладчик. Пусть ищет и исправляет ошибки в данных (подчеркиваю в данных, а не в коде). Ну а я, как-то не готов давать пользователю отладчик.

    Коорд = СокрЛП(..) // это чтобы вместо: «текст , текст» всегда было так: «текст, текст»

    Если Коорд <> «» Тогда

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

    Но поскольку она все же есть, то не вижу ничего плохого чтобы сравнить <> «»

    (12). Насчет Попытка-Исключение.

    А Вы попробуйте воспроизвести в точности тот пример из комментария с помощью попытки-исключения.

    И сразу же выяснится, что вы будете заниматься разбором строк в каждой процедуре, где используете эту конструкцию. Кроме того, у Вас возникнут трудности при передаче списка ошибок. И чтобы облегчить жизнь придется написать свою процедуру, занимающуюся разбором строк и накоплением ошибок.

    Но скорее всего, Вы просто используете ту самую процедуру ОшибкуОтдать(), приведенную выше.

    Так что, попытка-исключение — это всего лишь, один из способов ее применения.

    >>

    Ну а исключение клиентского (самого высокого) уровня уже можно как угодно отработать. Сообщением, предупреждением, или записью в лог.

    Заметьте, принципиальную разницу: сообщение может быть выдано не только на верхнем уровне, а на любом из уровней. В примерах как раз показан вывод из разных процедур. И это зависит от того, как вызвана процедура.

    В Вашем случае — это наличие или отсутствие конструкции попытка-исключение в процедурах верхнего уровня.

    Это значит, что «ВызватьИсключение» — это и есть Ваш способ вывода ошибки, т.е. Вы с этим уже заранее определились.

    Reply
  15. monkbest

    С пользователем на УФ надо разговаривать правильно 🙂

    http://prosto1s.ru/index.php/59-soobshcheniya-polzovatelyu-v-upravlyaemykh-formakh-eshche-raz

    Reply

Leave a Comment

Ваш адрес email не будет опубликован. Обязательные поля помечены *