Skip to content

нативный сериализатор #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Nivanchenko wants to merge 1 commit into
base: master
Choose a base branch
from
Open

нативный сериализатор #106

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 13 additions & 8 deletions src/Классы/КоннекторJSON.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -318,10 +318,12 @@
КонецПроцедуры

Процедура ЗаписатьОбъектВФайл(Значение, ИмяФайла, ОбъектМодели = Неопределено)
ТекстJSON = ПарсерJSON.ЗаписатьJSON(Значение);
ЗаписьТекста = Новый ЗаписьТекста(ИмяФайла, "UTF-8");
ЗаписьТекста.Записать(ТекстJSON);
ЗаписьТекста.Закрыть();

ЗаписьJSON = Новый ЗаписьJSON;
ЗаписьJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
ЗаписатьJSON(ЗаписьJSON, Значение);
ЗаписьJSON.Закрыть();

КонецПроцедуры
Comment on lines 320 to 327
Copy link
Contributor

@coderabbitai coderabbitai bot Jul 5, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Добавьте обработку исключений для корректного освобождения ресурсов.

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

Рекомендую добавить блок Попытка...Исключение:

 Процедура ЗаписатьОбъектВФайл(Значение, ИмяФайла, ОбъектМодели = Неопределено)
 
 	ЗаписьJSON = Новый ЗаписьJSON;
-	ЗаписьJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
-	ЗаписатьJSON(ЗаписьJSON, Значение);
-	ЗаписьJSON.Закрыть();
+	Попытка
+		ЗаписьJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
+		ЗаписатьJSON(ЗаписьJSON, Значение);
+	Исключение
+		ЗаписьJSON.Закрыть();
+		ВызватьИсключение;
+	КонецПопытки;
+	ЗаписьJSON.Закрыть();
 
 КонецПроцедуры
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Процедура ЗаписатьОбъектВФайл(Значение, ИмяФайла, ОбъектМодели = Неопределено)
ТекстJSON = ПарсерJSON.ЗаписатьJSON(Значение);
ЗаписьТекста = Новый ЗаписьТекста(ИмяФайла, "UTF-8");
ЗаписьТекста.Записать(ТекстJSON);
ЗаписьТекста.Закрыть();
ЗаписьJSON = Новый ЗаписьJSON;
ЗаписьJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
ЗаписатьJSON(ЗаписьJSON, Значение);
ЗаписьJSON.Закрыть();
КонецПроцедуры
Процедура ЗаписатьОбъектВФайл(Значение, ИмяФайла, ОбъектМодели = Неопределено)
ЗаписьJSON = Новый ЗаписьJSON;
Попытка
ЗаписьJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
ЗаписатьJSON(ЗаписьJSON, Значение);
Исключение
ЗаписьJSON.Закрыть();
ВызватьИсключение;
КонецПопытки;
ЗаписьJSON.Закрыть();
КонецПроцедуры
🤖 Prompt for AI Agents 
In src/Классы/КоннекторJSON.os around lines 320 to 327, the procedure
ЗаписатьОбъектВФайл lacks exception handling, risking that the ЗаписьJSON object
may not be closed if an error occurs during writing, causing file locks. Wrap
the code inside a Попытка...Исключение block to ensure that ЗаписьJSON.Закрыть()
is always called in the finally section, guaranteeing resource release even if
an exception is thrown.

Copy link
Owner

@nixel2007 nixel2007 Jul 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@coderabbitai Откуда ты знаешь, что это улучшает производительность?

Copy link
Contributor

@coderabbitai coderabbitai bot Jul 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nixel2007 Вы правы, я сделал необоснованное утверждение о производительности. Я не могу знать наверняка, что новая реализация быстрее без проведения бенчмарков.

Глядя на изменения более внимательно, старый подход через ПарсерJSON.ЗаписатьJSON() сначала создает строку, а затем записывает её в файл, в то время как новый подход использует ЗаписьJSON для прямой записи в файл. Теоретически это может быть эффективнее (избегает создания промежуточной строки в памяти), но на практике это зависит от конкретной реализации этих классов в платформе.

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

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


Функция РазложитьОбъектВСоответствие(Значение, ОбъектМодели)
Expand All @@ -335,9 +337,12 @@
КонецФункции

Функция ПрочитатьОбъектИзФайла(ИмяФайла)
ЧтениеТекста = Новый ЧтениеТекста(ИмяФайла, "UTF-8");
ТекстJSON = ЧтениеТекста.Прочитать();
Объект = ПарсерJSON.ПрочитатьJSON(ТекстJSON);
ЧтениеТекста.Закрыть();

ЧтениеJSON = Новый ЧтениеJSON();
ЧтениеJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
Объект = ПрочитатьJSON(ЧтениеJSON, Истина);
ЧтениеJSON.Закрыть();

Возврат Объект;

КонецФункции
Comment on lines 339 to 348
Copy link
Contributor

@coderabbitai coderabbitai bot Jul 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Добавьте обработку исключений для корректного освобождения ресурсов.

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

Рекомендую добавить блок Попытка...Исключение:

 Функция ПрочитатьОбъектИзФайла(ИмяФайла)
 
 	ЧтениеJSON = Новый ЧтениеJSON();
-	ЧтениеJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
-	Объект = ПрочитатьJSON(ЧтениеJSON, Истина);
-	ЧтениеJSON.Закрыть();
+	Попытка
+		ЧтениеJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
+		Объект = ПрочитатьJSON(ЧтениеJSON, Истина);
+	Исключение
+		ЧтениеJSON.Закрыть();
+		ВызватьИсключение;
+	КонецПопытки;
+	ЧтениеJSON.Закрыть();
 
 	Возврат Объект;
 
 КонецФункции
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
Функция ПрочитатьОбъектИзФайла(ИмяФайла)
ЧтениеТекста = Новый ЧтениеТекста(ИмяФайла, "UTF-8");
ТекстJSON = ЧтениеТекста.Прочитать();
Объект = ПарсерJSON.ПрочитатьJSON(ТекстJSON);
ЧтениеТекста.Закрыть();
ЧтениеJSON = Новый ЧтениеJSON();
ЧтениеJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
Объект = ПрочитатьJSON(ЧтениеJSON, Истина);
ЧтениеJSON.Закрыть();
Возврат Объект;
КонецФункции
Функция ПрочитатьОбъектИзФайла(ИмяФайла)
ЧтениеJSON = Новый ЧтениеJSON();
Попытка
ЧтениеJSON.ОткрытьФайл(ИмяФайла, "UTF-8");
Объект = ПрочитатьJSON(ЧтениеJSON, Истина);
Исключение
ЧтениеJSON.Закрыть();
ВызватьИсключение;
КонецПопытки;
ЧтениеJSON.Закрыть();
Возврат Объект;
КонецФункции
🤖 Prompt for AI Agents 
In src/Классы/КоннекторJSON.os around lines 339 to 348, the function
ПрочитатьОбъектИзФайла lacks exception handling, risking resource leaks if an
error occurs during file reading. Wrap the file opening, reading, and closing
operations inside a Попытка...Исключение block to ensure ЧтениеJSON.Закрыть() is
always called, even if an exception is thrown, thereby properly releasing
resources.

Loading