Обновление документации по подсистеме загрузки файла через табличный документ #147
Conversation
WalkthroughВ этом запросе на извлечение кода были внесены значительные изменения в документацию по подсистеме загрузки файлов Excel через табличный документ. Документ теперь более подробно описывает процесс загрузки, включая этапы чтения файла и сопоставления его колонок с предопределенным макетом. Добавлены новые разделы, описывающие свойства колонок и правила конвертации, а также примеры использования. Введены новые функции для получения структуры загружаемого объекта и параметров чтения файла. Changes
Possibly related PRs
Suggested reviewers
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
This comment has been minimized.
This comment has been minimized.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Outside diff range and nitpick comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
31-31: Исправьте форматирование Markdown документа.Для улучшения читаемости и соответствия стандартам оформления:
- Используйте последовательные уровни заголовков (h1 -> h2 -> h3).
- Замените дефисы на звездочки в маркированных списках.
- Используйте пробелы вместо табуляции для отступов.
Also applies to: 271-271, 174-192
🧰 Tools
🪛 Markdownlint
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time(MD001, heading-increment)
31-31: Punctuation: ':'
Trailing punctuation in heading(MD026, no-trailing-punctuation)
11-11: Исправьте грамматические ошибки в тексте.Необходимо внести следующие исправления:
- "Для того, чтобы" -> "Для того чтобы"
- "не смотря на" -> "несмотря на"
- Добавьте запятую перед "необходимо"
- "в сервером" -> "в серверном"
- Добавьте запятую между однородными сказуемыми
- "шт" -> "шт."
Also applies to: 25-25, 28-28, 159-159, 172-172, 204-204, 438-438
🧰 Tools
🪛 LanguageTool
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...(dlya_togo_chtoby_2)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~159-~159: Предлог «в» предполагает употребление существительного в винительном или предложном падеже: «в сервере» или «в сервер»
Context: ...убрать лидирующие нули в колонке. Тогда в сервером общем модуле ("ТестЗагрузкиФайлаИзТабли...
(PREP_V_and_Noun)
[uncategorized] ~172-~172: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~204-~204: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~438-~438: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
271-271: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
174-174: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
175-175: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
176-176: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
177-177: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
179-179: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
180-180: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
181-181: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
182-182: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
184-184: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
185-185: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
186-186: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
189-189: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
190-190: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
191-191: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
192-192: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
174-174: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
175-175: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
176-176: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
177-177: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
179-179: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
180-180: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
181-181: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
182-182: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
184-184: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
185-185: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
186-186: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
189-189: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
190-190: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
191-191: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
192-192: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 30
Hard tabs
(MD010, no-hard-tabs)
162-162: Column: 1
Hard tabs
(MD010, no-hard-tabs)
162-162: Column: 90
Hard tabs
(MD010, no-hard-tabs)
168-168: Column: 23
Hard tabs
(MD010, no-hard-tabs)
197-197: Column: 23
Hard tabs
(MD010, no-hard-tabs)
412-412: Column: 1
Hard tabs
(MD010, no-hard-tabs)
424-424: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
| Процедура ЗагрузитьXSL(ДвоичныеДанныеФайла, Расширение) | ||
|
|
||
| СтруктураКолонок = ПолучитьСтруктуруЗагружаемогоОбъекта(); | ||
|
|
||
| // Инициализируем параметры чтения напрямую, для дальнейшей передачи | ||
| ПараметрыЧтения = ПолучитьПараметрыЧтенияФайла(); | ||
|
|
||
| // Помещаем файл в хранилище на сервере | ||
| АдресВХранилище = ПоместитьВоВременноеХранилище(ДвоичныеДанныеФайла, Новый УникальныйИдентификатор); | ||
| ТаблицаДанных = пбп_ЗагрузкаФайлаЧерезТабличныйДокументСервер | ||
| .КонвертироватьДанныеТабличногоДокументаВТаблицуЗначений( | ||
| АдресВХранилище, Расширение, ТаблицаСвойств, ПараметрыЧтения); | ||
|
|
||
| // Для не найденных значений в итоговой таблице будут Null. Уберем их | ||
| пбп_ЗагрузкаФайлаЧерезТабличныйДокументСервер.ТаблицаЗначенийБезNull(ТаблицаДанных); | ||
|
|
||
| // Пост-обработка полученных данных | ||
| // ... | ||
|
|
||
| КонецПроцедуры |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Добавьте обработку ошибок при загрузке файла.
Необходимо добавить обработку исключений при:
- Чтении двоичных данных файла
- Помещении файла во временное хранилище
- Конвертации данных
| НоваяСтрока.ИмяВТабДоке = "Номенклатура (Артикул)"; | ||
| НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг"; | ||
| НоваяСтрока.ПолеПоиска1 = "Артикул"; |
There was a problem hiding this comment.
Исправьте тип значения для поля Номенклатура.
В текущей реализации указан неверный тип значения:
-НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг";
+НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";📝 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.
| НоваяСтрока.ИмяВТабДоке = "Номенклатура (Артикул)"; | |
| НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг"; | |
| НоваяСтрока.ПолеПоиска1 = "Артикул"; | |
| НоваяСтрока.ИмяВТабДоке = "Номенклатура (Артикул)"; | |
| НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура"; | |
| НоваяСтрока.ПолеПоиска1 = "Артикул"; |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
31-31: Исправьте форматирование Markdown документа.Для улучшения читаемости и соответствия стандартам оформления:
- Используйте последовательные уровни заголовков (h3 вместо h5)
- Замените дефисы на звездочки в маркированных списках
- Используйте пробелы вместо табуляции для отступов
-##### С клиента: +### С клиента: -##### С сервера: +### С сервера: - - 1 - разделитель "."; + * 1 - разделитель ".";Also applies to: 37-37, 174-192
🧰 Tools
🪛 Markdownlint
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time(MD001, heading-increment)
31-31: Punctuation: ':'
Trailing punctuation in heading(MD026, no-trailing-punctuation)
11-11: Исправьте грамматические ошибки в тексте.
- "Для того, чтобы" -> "Для того чтобы" (запятая не нужна)
- "не смотря на" -> "несмотря на" (слитное написание)
- Добавьте запятую перед "необходимо"
- "в сервером" -> "в серверном"
- Добавьте запятую после "разделитель"
- "шт" -> "шт." (точка после сокращения)
Also applies to: 25-25, 28-28, 159-159, 172-172, 204-204, 438-438
🧰 Tools
🪛 LanguageTool
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...(dlya_togo_chtoby_2)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~159-~159: Предлог «в» предполагает употребление существительного в винительном или предложном падеже: «в сервере» или «в сервер»
Context: ...убрать лидирующие нули в колонке. Тогда в сервером общем модуле ("ТестЗагрузкиФайлаИзТабли...
(PREP_V_and_Noun)
[uncategorized] ~172-~172: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~204-~204: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~438-~438: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
271-271: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
174-174: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
175-175: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
176-176: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
177-177: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
179-179: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
180-180: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
181-181: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
182-182: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
184-184: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
185-185: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
186-186: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
189-189: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
190-190: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
191-191: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
192-192: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
174-174: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
175-175: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
176-176: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
177-177: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
179-179: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
180-180: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
181-181: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
182-182: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
184-184: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
185-185: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
186-186: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
189-189: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
190-190: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
191-191: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
192-192: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 30
Hard tabs
(MD010, no-hard-tabs)
162-162: Column: 1
Hard tabs
(MD010, no-hard-tabs)
162-162: Column: 90
Hard tabs
(MD010, no-hard-tabs)
168-168: Column: 23
Hard tabs
(MD010, no-hard-tabs)
197-197: Column: 23
Hard tabs
(MD010, no-hard-tabs)
412-412: Column: 1
Hard tabs
(MD010, no-hard-tabs)
424-424: Column: 1
Hard tabs
(MD010, no-hard-tabs)
477-477: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (1)
392-392: Исправьте тип значения для поля Номенклатура.
Используется неверный тип значения:
-НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг";
+НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";| ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение); | ||
| ДвоичныеДанныеФайла.Записать(ПутьКФайлу); |
There was a problem hiding this comment.
Добавьте обработку ошибок при работе с файлами.
Необходимо добавить обработку исключений при:
- Получении имени временного файла
- Записи двоичных данных в файл
+Попытка
ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение);
ДвоичныеДанныеФайла.Записать(ПутьКФайлу);
+Исключение
+ ЗаписьЖурналаРегистрации("ЗагрузкаCSV",
+ УровеньЖурналаРегистрации.Ошибка,,,
+ ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+ ВызватьИсключение;
+КонецПопытки📝 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.
| ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение); | |
| ДвоичныеДанныеФайла.Записать(ПутьКФайлу); | |
| Попытка | |
| ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение); | |
| ДвоичныеДанныеФайла.Записать(ПутьКФайлу); | |
| Исключение | |
| ЗаписьЖурналаРегистрации("ЗагрузкаCSV", | |
| УровеньЖурналаРегистрации.Ошибка,,, | |
| ПодробноеПредставлениеОшибки(ИнформацияОбОшибке())); | |
| ВызватьИсключение; | |
| КонецПопытки |
🧰 Tools
🪛 Markdownlint
477-477: Column: 1
Hard tabs
(MD010, no-hard-tabs)
| НоваяСтрока.ПолеПоиска2 = "Дата"; | ||
| НоваяСтрока.ОператорСравнения = "И"; | ||
| ``` | ||
| Также оператор "И" может использоваться для инициализации "виртуальных" полей ссылочных типов, т.е. которых нет в исходном файле. Например, если колонки "Номенклатура" в файле нет, но есть одно из полей поиска номенклатуры, допустим "Артикул", то используя оператор "И" в итоговой таблице мы получим ссылку на номенклатуру, полученную поиском по артикулу: |
There was a problem hiding this comment.
Можно ли просто написать:
НоваяСтрока = ТаблицаСвойств.Добавить();
НоваяСтрока.ИмяКолонки = "Артикул";
НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";
НоваяСтрока.ПолеПоиска1 = "Артикул";
Т.е. в чем плюс использования оператора И в таких случаях?
There was a problem hiding this comment.
Можно, но тогда имя колонки артикул будет уже зарезервировано для номенклатуры. Если нужно и артикул, и найти номенклатуру по артикулу, то нужно объявлять два разных свойства
| НоваяСтрока.ТипЗначения = "СправочникСсылка.УпаковкиЕдиницыИзмерения"; | ||
| НоваяСтрока.ПолеПоиска1 = "Наименование"; | ||
| ``` | ||
| Для конвертации значений "булево" необходимо указывать строковое представление ИСТИНА, для конвертации даты в параметрах чтения файла ключ КонвертироватьДаты должен быть Истина (подробно см. в главе [Параметры чтения файла](#параметры-чтения-файла)). Для конвертации ссылочных типов необходимо заполнить поля поиска. В указанном выше примере единица хранения будет искаться по полю "наименование". |
There was a problem hiding this comment.
Тут нужно написать про то что нужно в параметрах чтения файла указать что будет воспринято как истина для булево
| НоваяСтрока = ТаблицаСвойств.Добавить(); | ||
| НоваяСтрока.ИмяКолонки = "ЕдиницаХранения"; | ||
| НоваяСтрока.ТипЗначения = "СправочникСсылка.УпаковкиЕдиницыИзмерения"; | ||
| НоваяСтрока.ПолеПоиска1 = "Наименование"; |
There was a problem hiding this comment.
А если поле поиска будет не заполнено, то он не отработает или есть какой-то поиск по умолчанию по коду/наименованию?
There was a problem hiding this comment.
Нет, поиска по умолчанию не предусмотрено. Я его убрал, чтобы еще сильнее не путать ни код, ни юзеров
| ``` | ||
| Для конвертации значений "булево" необходимо указывать строковое представление ИСТИНА, для конвертации даты в параметрах чтения файла ключ КонвертироватьДаты должен быть Истина (подробно см. в главе [Параметры чтения файла](#параметры-чтения-файла)). Для конвертации ссылочных типов необходимо заполнить поля поиска. В указанном выше примере единица хранения будет искаться по полю "наименование". | ||
| 5) **ПолеПоиска1**, **ПолеПоиска2**, **ПолеПоиска3** - название реквизита объекта, по которому будет произведен поиск значения ссылочного типа. | ||
| 6) **ОператорСравнения** - оператор сравнения в условии соединения таблицы с типом значения между полями поиска. По-умолчанию "ИЛИ", также может равняться "И". Например, если в колонке файла "Контрагент" может быть указан либо ИНН, либо ОКПО, либо наименование, и, соответственно, поиск должен производится по этим полям объекта с указанным приоритетом (высший ИНН, низший - наименование), то указываем свойства в следующем порядке: |
There was a problem hiding this comment.
ОператорСравнения по-умолчанию ИЛИ?
There was a problem hiding this comment.
Да
Тут так и написано)
| НоваяСтрока.ПолеПоиска2 = "Дата"; | ||
| НоваяСтрока.ОператорСравнения = "И"; | ||
| ``` | ||
| Также оператор "И" может использоваться для инициализации "виртуальных" полей ссылочных типов, т.е. которых нет в исходном файле. Например, если колонки "Номенклатура" в файле нет, но есть одно из полей поиска номенклатуры, допустим "Артикул", то используя оператор "И" в итоговой таблице мы получим ссылку на номенклатуру, полученную поиском по артикулу: |
There was a problem hiding this comment.
Вообще конечно логично, но запутано очень. Мб там третий какой-то оператор сравнения добавить, ЗАМЕНА или что-то такое?
There was a problem hiding this comment.
Можно, надо подумать, как его обозвать только
| НоваяСтрока.ПолеПоиска1 = "Артикул"; | ||
| НоваяСтрока.ОператорСравнения = "И"; | ||
| ``` | ||
| 7) **ФункцияПреобразования** - имя экспортной функции в программном интерфейсе, которая будет вызвана для конвертации значения. Допустим, нам необходимо убрать лидирующие нули в колонке. Тогда в сервером общем модуле ("ТестЗагрузкиФайлаИзТабличногоДокумента") создаем экспортную функцию УбратьЛидирующиеНули с параметром "ВходящееЗначение" (в него придет строковое значение ячейки файла): |
| 1) **НазваниеЛиста** - Строка - название листа в исходном файле; | ||
| 2) **НомерСтроки** - Число - начальный номер строки, с которой начинается чтение файла. Обычно, это шапка таблицы. Строки до нее обрезаются; | ||
| 3) **СопоставлениеПоНаименованию** - Булево - если Истина, то колонки будут сравниваться по наименованию; если Ложь, то по порядку, указанном в таблице свойств колонок (подробно см. в начале главы [Описание свойств колонок файла](#описание-свойств-колонок-файла-макета)); | ||
| 4) **КонвертироватьДаты** - Булево - если Истина, то колонки с типом Дата будут конвертироваться автоматически из строкового значения в дату; |
There was a problem hiding this comment.
Сопоставление по-умолчанию - Ложь;
КонвертироватьДаты - Истина
| СтруктураКолонок = ПолучитьСтруктуруЗагружаемогоОбъекта(); | ||
|
|
||
| // Инициализируем параметры чтения напрямую, для дальнейшей передачи | ||
| ПараметрыЧтения = ПолучитьПараметрыЧтенияФайла(); |
There was a problem hiding this comment.
А с клиента не нужны параметры чтения? Я что-то запутался
There was a problem hiding this comment.
Они передаются как параметры функции
Наверное, нужно переделать, чтобы никого не путать
This comment has been minimized.
This comment has been minimized.
…х полей" This reverts commit 622c911.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (3)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (3)
11-12: Исправьте пунктуацию в сложном союзе.Уберите запятую в сложном союзе "Для того, чтобы":
-Для того, чтобы сопоставить колонки файла со значениями 1С +Для того чтобы сопоставить колонки файла со значениями 1С🧰 Tools
🪛 LanguageTool
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...(dlya_togo_chtoby_2)
104-104: Дополните документацию по конвертации булевых значений.Необходимо добавить информацию о том, какие значения воспринимаются как ИСТИНА при конвертации булевых значений. Например:
+Для конвертации значений "булево" можно указать строковое представление ИСТИНА в параметре ЗначениеИстина. +По умолчанию, если представление не указано, проверяется равенство строкам "да" или "истина" (без учета регистра).
463-463: Исправьте оформление сокращения.Добавьте точку после сокращения:
-измерения ("шт") +измерения ("шт.")🧰 Tools
🪛 LanguageTool
[uncategorized] ~463-~463: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...(ABREV_DOT)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~151-~151: Избыточные определения. В правильно составленном предложении не должно быть больше одного придаточного определения.
Context: ...ии "виртуальных" полей ссылочных типов (которые не входят в логику оператора "И"), т.е. которых нет в исходном файле. Например, если ко...
(OPREDELENIA)
[uncategorized] ~191-~191: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~223-~223: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~463-~463: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
290-290: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
193-193: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
194-194: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
195-195: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
196-196: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
198-198: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
199-199: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
200-200: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
201-201: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
203-203: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
204-204: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
205-205: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
208-208: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
209-209: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
210-210: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
211-211: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
193-193: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
194-194: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
195-195: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
196-196: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
198-198: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
199-199: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
200-200: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
201-201: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
203-203: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
204-204: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
205-205: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
208-208: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
209-209: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
210-210: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
211-211: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 28
Hard tabs
(MD010, no-hard-tabs)
169-169: Column: 23
Hard tabs
(MD010, no-hard-tabs)
170-170: Column: 24
Hard tabs
(MD010, no-hard-tabs)
171-171: Column: 24
Hard tabs
(MD010, no-hard-tabs)
172-172: Column: 28
Hard tabs
(MD010, no-hard-tabs)
175-175: Column: 23
Hard tabs
(MD010, no-hard-tabs)
176-176: Column: 24
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 1
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 90
Hard tabs
(MD010, no-hard-tabs)
187-187: Column: 23
Hard tabs
(MD010, no-hard-tabs)
216-216: Column: 23
Hard tabs
(MD010, no-hard-tabs)
437-437: Column: 1
Hard tabs
(MD010, no-hard-tabs)
449-449: Column: 1
Hard tabs
(MD010, no-hard-tabs)
502-502: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
340-342:
Исправьте тип значения для поля Номенклатура.
В примере указан неверный тип значения:
-НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг";
+НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";Likely invalid or redundant comment.
501-502:
Добавьте обработку ошибок при работе с файлами.
Необходимо добавить обработку исключений при работе с временными файлами:
+Попытка
ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение);
ДвоичныеДанныеФайла.Записать(ПутьКФайлу);
+Исключение
+ ЗаписьЖурналаРегистрации("ЗагрузкаCSV",
+ УровеньЖурналаРегистрации.Ошибка,,,
+ ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+ ВызватьИсключение;
+КонецПопыткиLikely invalid or redundant comment.
🧰 Tools
🪛 Markdownlint
502-502: Column: 1
Hard tabs
(MD010, no-hard-tabs)
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
11-12: Исправьте пунктуацию в предложении.Уберите запятую в конструкции "Для того, чтобы":
-Для того, чтобы сопоставить колонки файла со значениями 1С +Для того чтобы сопоставить колонки файла со значениями 1С🧰 Tools
🪛 LanguageTool
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...(dlya_togo_chtoby_2)
456-456: Исправьте оформление сокращения.-("шт") +("шт.")🧰 Tools
🪛 LanguageTool
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...(ABREV_DOT)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~151-~151: Избыточные определения. В правильно составленном предложении не должно быть больше одного придаточного определения.
Context: ...ии "виртуальных" полей ссылочных типов (которые не входят в логику оператора "И"), т.е. которых нет в исходном файле. Например, если ко...
(OPREDELENIA)
[uncategorized] ~191-~191: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~223-~223: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
290-290: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
193-193: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
194-194: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
195-195: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
196-196: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
198-198: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
199-199: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
200-200: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
201-201: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
203-203: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
204-204: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
205-205: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
208-208: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
209-209: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
210-210: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
211-211: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
193-193: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
194-194: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
195-195: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
196-196: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
198-198: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
199-199: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
200-200: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
201-201: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
203-203: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
204-204: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
205-205: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
208-208: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
209-209: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
210-210: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
211-211: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 28
Hard tabs
(MD010, no-hard-tabs)
169-169: Column: 23
Hard tabs
(MD010, no-hard-tabs)
170-170: Column: 24
Hard tabs
(MD010, no-hard-tabs)
171-171: Column: 24
Hard tabs
(MD010, no-hard-tabs)
172-172: Column: 28
Hard tabs
(MD010, no-hard-tabs)
175-175: Column: 23
Hard tabs
(MD010, no-hard-tabs)
176-176: Column: 24
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 1
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 90
Hard tabs
(MD010, no-hard-tabs)
187-187: Column: 23
Hard tabs
(MD010, no-hard-tabs)
216-216: Column: 23
Hard tabs
(MD010, no-hard-tabs)
495-495: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
340-342:
Исправьте тип значения для поля Номенклатура.
-НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг";
+НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";Likely invalid or redundant comment.
494-495:
Добавьте обработку ошибок при работе с файлами.
+Попытка
ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение);
ДвоичныеДанныеФайла.Записать(ПутьКФайлу);
+Исключение
+ ЗаписьЖурналаРегистрации("ЗагрузкаCSV",
+ УровеньЖурналаРегистрации.Ошибка,,,
+ ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+ ВызватьИсключение;
+КонецПопыткиLikely invalid or redundant comment.
🧰 Tools
🪛 Markdownlint
495-495: Column: 1
Hard tabs
(MD010, no-hard-tabs)
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Outside diff range and nitpick comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
31-37: Исправьте форматирование Markdown документа.Для улучшения читаемости и соответствия стандартам оформления:
- Используйте последовательные уровни заголовков (h1 -> h2 -> h3)
- Замените дефисы на звездочки в маркированных списках
- Исправьте отступы в списках
Also applies to: 193-211
🧰 Tools
🪛 Markdownlint
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time(MD001, heading-increment)
31-31: Punctuation: ':'
Trailing punctuation in heading(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading(MD026, no-trailing-punctuation)
11-11: Исправьте грамматические ошибки в тексте.
- Уберите запятую в конструкции "для того чтобы"
- Исправьте "не смотря на" на "несмотря на"
- Добавьте запятую перед "необходимо"
Also applies to: 25-25, 28-28, 223-223
🧰 Tools
🪛 LanguageTool
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...(dlya_togo_chtoby_2)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~151-~151: Избыточные определения. В правильно составленном предложении не должно быть больше одного придаточного определения.
Context: ...ии "виртуальных" полей ссылочных типов (которые не входят в логику оператора "И"), т.е. которых нет в исходном файле. Например, если ко...
(OPREDELENIA)
[uncategorized] ~191-~191: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~223-~223: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
290-290: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
193-193: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
194-194: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
195-195: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
196-196: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
198-198: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
199-199: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
200-200: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
201-201: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
203-203: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
204-204: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
205-205: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
208-208: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
209-209: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
210-210: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
211-211: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
193-193: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
194-194: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
195-195: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
196-196: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
198-198: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
199-199: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
200-200: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
201-201: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
203-203: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
204-204: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
205-205: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
208-208: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
209-209: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
210-210: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
211-211: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 28
Hard tabs
(MD010, no-hard-tabs)
169-169: Column: 23
Hard tabs
(MD010, no-hard-tabs)
170-170: Column: 24
Hard tabs
(MD010, no-hard-tabs)
171-171: Column: 24
Hard tabs
(MD010, no-hard-tabs)
172-172: Column: 28
Hard tabs
(MD010, no-hard-tabs)
175-175: Column: 23
Hard tabs
(MD010, no-hard-tabs)
176-176: Column: 24
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 1
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 90
Hard tabs
(MD010, no-hard-tabs)
187-187: Column: 23
Hard tabs
(MD010, no-hard-tabs)
216-216: Column: 23
Hard tabs
(MD010, no-hard-tabs)
495-495: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
340-342: Исправьте тип значения для поля Номенклатура.
494-495: Добавьте обработку ошибок при работе с файлами.
🧰 Tools
🪛 Markdownlint
495-495: Column: 1
Hard tabs
(MD010, no-hard-tabs)
| 6) **ОператорУсловия** - оператор сравнения в условии соединения таблицы с типом значения между полями поиска. По умолчанию "ИЛИ", также может равняться "И" или "ВИРТ". Например, если в колонке файла "Контрагент" может быть указан либо ИНН, либо ОКПО, либо наименование, и, соответственно, поиск должен производится по этим полям объекта с указанным приоритетом (высший ИНН, низший - наименование), то указываем свойства в следующем порядке: | ||
| ```bsl |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Уточните поведение оператора по умолчанию.
Добавьте явное указание, что оператор сравнения по умолчанию - "ИЛИ". Это важно для понимания логики поиска значений.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
3-6: Уточните поддерживаемые форматы файлов.Рекомендуется добавить информацию о:
- Поддерживаемых версиях Excel (.xls, .xlsx)
- Ограничениях на размер файла
- Требованиях к структуре CSV файлов (разделители, кодировка)
456-457: Улучшите описание обработки позиций колонок в CSV.Рекомендуется:
- Добавить визуальную схему разбора строки по позициям
- Привести примеры для разных форматов данных
- Описать особенности обработки пустых значений
🧰 Tools
🪛 LanguageTool
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...(ABREV_DOT)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (1)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~151-~151: Избыточные определения. В правильно составленном предложении не должно быть больше одного придаточного определения.
Context: ...ии "виртуальных" полей ссылочных типов (которые не входят в логику оператора "И"), т.е. которых нет в исходном файле. Например, если ко...
(OPREDELENIA)
[uncategorized] ~191-~191: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~223-~223: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
290-290: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
193-193: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
194-194: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
195-195: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
196-196: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
198-198: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
199-199: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
200-200: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
201-201: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
203-203: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
204-204: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
205-205: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
208-208: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
209-209: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
210-210: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
211-211: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
193-193: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
194-194: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
195-195: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
196-196: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
198-198: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
199-199: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
200-200: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
201-201: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
203-203: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
204-204: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
205-205: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
208-208: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
209-209: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
210-210: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
211-211: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 28
Hard tabs
(MD010, no-hard-tabs)
169-169: Column: 23
Hard tabs
(MD010, no-hard-tabs)
170-170: Column: 24
Hard tabs
(MD010, no-hard-tabs)
171-171: Column: 24
Hard tabs
(MD010, no-hard-tabs)
172-172: Column: 28
Hard tabs
(MD010, no-hard-tabs)
175-175: Column: 23
Hard tabs
(MD010, no-hard-tabs)
176-176: Column: 24
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 1
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 90
Hard tabs
(MD010, no-hard-tabs)
187-187: Column: 23
Hard tabs
(MD010, no-hard-tabs)
216-216: Column: 23
Hard tabs
(MD010, no-hard-tabs)
492-492: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (2)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (2)
340-342:
Исправьте тип значения для поля Номенклатура.
В примере указан неверный тип значения:
-НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг";
+НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";Likely invalid or redundant comment.
491-492:
Добавьте обработку ошибок при работе с файлами.
Необходимо добавить обработку исключений при:
- Получении имени временного файла
- Записи двоичных данных в файл
+Попытка
ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение);
ДвоичныеДанныеФайла.Записать(ПутьКФайлу);
+Исключение
+ ЗаписьЖурналаРегистрации("ЗагрузкаCSV",
+ УровеньЖурналаРегистрации.Ошибка,,,
+ ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+ ВызватьИсключение;
+КонецПопыткиLikely invalid or redundant comment.
🧰 Tools
🪛 Markdownlint
492-492: Column: 1
Hard tabs
(MD010, no-hard-tabs)
Analysis Details0 Issues
Coverage and Duplications
Project ID: PSSL |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (3)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (3)
11-12: Исправьте грамматическую ошибку в союзе.Уберите запятую в сложном союзе:
-Для того, чтобы сопоставить колонки файла со значениями +Для того чтобы сопоставить колонки файла со значениями🧰 Tools
🪛 LanguageTool
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...(dlya_togo_chtoby_2)
106-107: Уточните поведение оператора по умолчанию.Добавьте явное указание, что оператор сравнения по умолчанию - "ИЛИ". Это важно для понимания логики поиска значений.
456-456: Исправьте оформление сокращения.Добавьте точку после сокращения:
-("шт") +("шт.")🧰 Tools
🪛 LanguageTool
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...(ABREV_DOT)
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
📒 Files selected for processing (2)
README.md(1 hunks)docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- README.md
🧰 Additional context used
🪛 LanguageTool
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
[uncategorized] ~11-~11: Сложный союз в начале предложения. Запятые не нужны: «Для того чтобы».
Context: ...сание свойств колонок файла ("макета") Для того, чтобы сопоставить колонки файла со значениями...
(dlya_togo_chtoby_2)
[uncategorized] ~25-~25: Слово пишется слитно: «несмотря на».
Context: ...егда будет распознана как номенклатура, не смотря на то, что там может быть значение другого...
(Pravopisanie_slitno1)
[uncategorized] ~28-~28: Требуется запятая до конца предложения.
Context: ...ут доступны для последующей обработки. Чтобы инициализировать таблицу свойств необходимо вызвать функцию в зависимости от контек...
(Comma_Chtobi_2VERB)
[uncategorized] ~151-~151: Избыточные определения. В правильно составленном предложении не должно быть больше одного придаточного определения.
Context: ...ии "виртуальных" полей ссылочных типов (которые не входят в логику оператора "И"), т.е. которых нет в исходном файле. Например, если ко...
(OPREDELENIA)
[uncategorized] ~191-~191: Глаголы – однородные члены предложения: «показывает какой разделитель, используется» или «показывает какой разделитель и используется».
Context: ... "_" (например, "1_1_2"). Первый разряд показывает какой разделитель используется в представлении даты, второй разряд пок...
(Verb_comma_Verb)
[uncategorized] ~223-~223: Требуется запятая до конца предложения.
Context: ...ных из файла и их конвертацию. Для того чтобы указать параметры чтения файла необходимо вызвать функцию ПолучитьПараметрыЧтения...
(Comma_Chtobi_2VERB)
[uncategorized] ~456-~456: Точка должна стоять после сокращений: «шт.»
Context: ...уры и наименованием единицы измерения ("шт") отсутствует, но, очевидно, это разные...
(ABREV_DOT)
🪛 Markdownlint
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md
31-31: Expected: h3; Actual: h5
Heading levels should only increment by one level at a time
(MD001, heading-increment)
290-290: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
193-193: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
194-194: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
195-195: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
196-196: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
198-198: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
199-199: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
200-200: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
201-201: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
203-203: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
204-204: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
205-205: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
208-208: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
209-209: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
210-210: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
211-211: Expected: asterisk; Actual: dash
Unordered list style
(MD004, ul-style)
193-193: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
194-194: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
195-195: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
196-196: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
198-198: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
199-199: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
200-200: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
201-201: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
203-203: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
204-204: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
205-205: Expected: 2; Actual: 3
Unordered list indentation
(MD007, ul-indent)
208-208: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
209-209: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
210-210: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
211-211: Expected: 0; Actual: 3
Unordered list indentation
(MD007, ul-indent)
17-17: Column: 23
Hard tabs
(MD010, no-hard-tabs)
20-20: Column: 23
Hard tabs
(MD010, no-hard-tabs)
23-23: Column: 23
Hard tabs
(MD010, no-hard-tabs)
49-49: Column: 23
Hard tabs
(MD010, no-hard-tabs)
54-54: Column: 23
Hard tabs
(MD010, no-hard-tabs)
55-55: Column: 24
Hard tabs
(MD010, no-hard-tabs)
81-81: Column: 23
Hard tabs
(MD010, no-hard-tabs)
85-85: Column: 23
Hard tabs
(MD010, no-hard-tabs)
89-89: Column: 23
Hard tabs
(MD010, no-hard-tabs)
90-90: Column: 24
Hard tabs
(MD010, no-hard-tabs)
96-96: Column: 23
Hard tabs
(MD010, no-hard-tabs)
100-100: Column: 23
Hard tabs
(MD010, no-hard-tabs)
109-109: Column: 23
Hard tabs
(MD010, no-hard-tabs)
145-145: Column: 23
Hard tabs
(MD010, no-hard-tabs)
154-154: Column: 23
Hard tabs
(MD010, no-hard-tabs)
155-155: Column: 24
Hard tabs
(MD010, no-hard-tabs)
156-156: Column: 24
Hard tabs
(MD010, no-hard-tabs)
157-157: Column: 28
Hard tabs
(MD010, no-hard-tabs)
169-169: Column: 23
Hard tabs
(MD010, no-hard-tabs)
170-170: Column: 24
Hard tabs
(MD010, no-hard-tabs)
171-171: Column: 24
Hard tabs
(MD010, no-hard-tabs)
172-172: Column: 28
Hard tabs
(MD010, no-hard-tabs)
175-175: Column: 23
Hard tabs
(MD010, no-hard-tabs)
176-176: Column: 24
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 1
Hard tabs
(MD010, no-hard-tabs)
181-181: Column: 90
Hard tabs
(MD010, no-hard-tabs)
187-187: Column: 23
Hard tabs
(MD010, no-hard-tabs)
216-216: Column: 23
Hard tabs
(MD010, no-hard-tabs)
492-492: Column: 1
Hard tabs
(MD010, no-hard-tabs)
31-31: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
37-37: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (3)
docs/ЗагрузкаФайлаЧерезТабличныйДокумент.md (3)
239-443: Примеры корректны и информативны.
Раздел с примерами хорошо структурирован и содержит все необходимые детали для понимания процесса загрузки файлов.
🧰 Tools
🪛 Markdownlint
290-290: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time
(MD001, heading-increment)
340-342:
Исправьте тип значения для поля Номенклатура.
В текущей реализации указан неверный тип значения:
-НоваяСтрока.ТипЗначения = "СправочникСсылка.РеализацияТоваровУслуг";
+НоваяСтрока.ТипЗначения = "СправочникСсылка.Номенклатура";Likely invalid or redundant comment.
491-492:
Добавьте обработку ошибок при работе с файлами.
Необходимо добавить обработку исключений при:
- Получении имени временного файла
- Записи двоичных данных в файл
+Попытка
ПутьКФайлу = ПолучитьИмяВременногоФайла(Расширение);
ДвоичныеДанныеФайла.Записать(ПутьКФайлу);
+Исключение
+ ЗаписьЖурналаРегистрации("ЗагрузкаCSV",
+ УровеньЖурналаРегистрации.Ошибка,,,
+ ПодробноеПредставлениеОшибки(ИнформацияОбОшибке()));
+ ВызватьИсключение;
+КонецПопыткиLikely invalid or redundant comment.
🧰 Tools
🪛 Markdownlint
492-492: Column: 1
Hard tabs
(MD010, no-hard-tabs)
| НоваяСтрока.ТипЗначения = "СправочникСсылка.УпаковкиЕдиницыИзмерения"; | ||
| НоваяСтрока.ПолеПоиска1 = "Наименование"; | ||
| ``` | ||
| Для конвертации значений "булево" необходимо указывать строковое представление ИСТИНА (по умолчанию, если представление не указано, проверка на "да" и "истина"), для конвертации даты в параметрах чтения файла ключ КонвертироватьДаты должен быть Истина (подробно см. в главе [Параметры чтения файла](#параметры-чтения-файла)). Для конвертации ссылочных типов необходимо заполнить поля поиска. В указанном выше примере единица хранения будет искаться по полю "наименование". |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Дополните информацию о преобразовании булевых значений.
Необходимо указать:
- Какие значения воспринимаются как "Истина" по умолчанию
- Возможность настройки пользовательских значений через параметр ЗначениеИстина
- Примеры использования различных вариантов значений
#132
Summary by CodeRabbit