<?php // Полная загрузка сервисных книжек, создан 2025-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='\
Действительно, нет предела совершенству…
Простите, а вот такие портянки
Показать
Зачем? И так несколько раз у вас. Разве нельзя написать так
Или на худой конец, преобразовать в число и уже его проверить. Я уже молчу про это:
Показать
часть условий бессмыслено так как везде стоит «нрег» и они не выполнятся ни когда. Можно было вместо этого списка сделать такую проверку
И дальше парсить (что лучше) или на худой конец поставить кучу СтрЗаменить вообще без условия. И если я не ошибаюсь (а я с 77 почти не работал) всё это можно и в ней.
(1)
Вот это точно негодное предложение, смысл проделанной работы не в том чтобы что-то тупо заменить, а в том, в какой последовательности. Суть предложения не в том, чтобы блеснуть своим хитро-микро-думо-мудро-программированием, а чтобы поделиться простым решением сложной проблемы. Спасибо, подумаю над первой частью, но перенос данных уже произошел, главное чтобы сделать качественный перенос данных. Оно работает достаточно быстро и понятно и без того.
(2) Ну так вашем случае часть условий не выполнима. То что делать чисто СтрЗаменить, согласен не лучший вариант (ибо можно заменить что-то в другом месте). Лучший вариант (на мой взгляд) это парсинг. Нашли через «Найти» точку отправления и от неё уже двигаться до запятой, этот кусок между отправной точкой и запятой, можно спокойно перевести в нижний регистр, удалить пробелы и сделать один СтрЗаменить и прибавить полученный результат к результирующей строке
(3) Мне лично нравится более наглядный код даже если я потеряю производительность, возможно это специфика моего взгляда на жизнь. Более медленный и наглядный код не хочется менять даже на более оптимальный но менее читаемый, пусть даже речь будет не об одной лишней секунде работы программы.
Если КодСимвола >= 48 и КодСимвола <= 57 Тогда
будет сложнее читать в контексте АдресКонецТрансляция3.
За совет по оптимизации спасибо!
(4) Речь не столько о производительности, сколько о том что часть условий ни когда не выполнится, вот посмотрите внимательно
В условии идет перевод в нижний регистр. И нрег(«,КВ.») не будет равно «,КВ.». Может конечно 77 на регистр плевать при поиске….
(5) согласен с (4), нрег в этом случае помешало сохранить регистр источника и достаточно первой ветки отквоченного условия, остальные не нужны.
Вот это:
очень плохая идея. Правильный вариант будет такой:
(6) и то можно сократить до такого:
я если честно не вижу ситуации, где двойной пробел был бы важен и его нельзя было убрать.
(6)
А в данном случае не достаточно двух условий?
Ну и перед проверкой не лишним будет сразу избавиться от двойных/тройных/четверных/пятерных пробелов
(8) по цитате 1 — только в совокупности с решением по цитате №2, по цитате 2 да, но в адресе могут быть не только пробелы с кодом 40. Благодарю!
(4) ну, все-таки исходим из того что код будет читать какой-никакой программист, а не менеджер
на крайняк можно юхать для читабельности чтото типа
Найти(«0123456789»,СимволИзСтроки)
(7)
также, надо понимать что такой код при, например, 4 пробелах подряд никогда не заменит 4 пробела на один пробел… или три пробела на 1 пробел
(10) безусловно все советы дельные, и это легко всё добавить в код, хоть мне в голову сразу не пришло. Спасибо!