Разбор загрузки из Excel для джунов
Вставка
- Опубліковано 9 лип 2024
- Записывайтесь на следующий поток курса «Чистый код на 1С» по ссылке: codeway.school/?...
Отправим ссылку на сайт курса в телеграмм бота.
Показал, на что начинающим 1С программистам обращать внимание при написании кода на 1С.
Заявку на разбор кода оставляйте здесь: forms.gle/duwvQhoWNtbeZLBh9
НАВИГАЦИЯ
00:00 - Задача
00:30 - Стандарт разметки модуля
00:52 - Табуляция или Пробелы
01:28 - Имена переменных
02:55 - Вертикальное выравнивание
04:01 - Лучше использовать НСтр
05:25 - Присваивание сравнения
05:45 - Пустые ветви в условиях
06:58 - Реквизиты формы и реквизиты Объекта
07:58 - Комментарии в коде
09:04 - Присылайте код на разбор
ДОП. МАТЕРИАЛЫ:
Стрим с рефакторингом 1С кода с Дмитрием Решитко: ua-cam.com/users/liveNvvPO-qj...
Разбор для мидлов: • Разбор загрузки из Exc...
Стандарт по структуре модулей: its.1c.ru/db/v8std#content:45...
==========
Информационные площадки "Жёлтого клуба":
Телеграмм канал: t.me/+h2Ipfl1Gdms3OGYy
Телеграм чат: t.me/yellowclub_vrn
Группа ВКонтакте: vk: yellowclub_official
Подписывайся на канала Желтого клуба, чтобы не пропустить интересных гостей
/ @yellow_club
Посмотрел разбор Джуна, понравилось, но не понял один момент, почему ты сообщение для исключения описал без НСтр? Это же в интерфейс выведется? Или нет...
Мой косяк, спасибо 🙏
Это же интерфейсная строка и к ним мы всегда одинаково относимся
Вот! Наконец появилось то, что лично мне интересно. Разбор кода, это то, что нужно. Просто браво! Мужики, пожалуйста выкладывайте, ну по возможности конечно, чаще разбор всевозможного кода. Можно даже не только код новичков разбирать, код профессионалов так же интересен.
Рад, что полезно
Круто, побольше бы таких видео. Я бы прислал свой код, только очень стыдно. Я Равшан и Джамшут от мира 1с. 'Не ругайся, насяйника. Моя отладка делать на сервере, поэтому сапускать служба с ключом debug'.
Ахаха) присылай, не боись) таких как ты много. Всем польза будет. Только желательно что-нибудь с запросами присылать
Спасибо, отличный формат 😊
Рад, что полезно
спасибо. узнал несколько нового, чего не просят на работе, но что более структурирует. что в этом видео, что для миддлов
Рад, что было полезно
Классный формат!
Отличный формат
Рад, что полезно
Спасибо! Интересно!
Рад, что полезно
Про горизонтальное (вертикальное) выравнивание прям почти мой комментарий озвучен под постом со скриншотом )) p.s. Это видео посмотрел позже, чем оставил коммент под тем постом.
Всё бы хорошо, но момент, где затрагивается выравнивание, я считаю неверным с Вашей стороны, т.к. делаю эти действия которые отминут лишние 5 секунд времени, тем самым Вы поможете уже другим программистам быстрее разобраться в коде, чем бегать глазами в поисках начала и конца, да и выглядит куда бережнее и читабельнее
К тому же есть из бесплатного:
TNice
и
SmartConfigurator.ahk ( он же v8CfgAddsAhk)
Понятно, что лучше было бы приобрести СнегоПат или ТруКонф.
Но для моих задач выравнивания мне с головой хватает TNice.
А почему нельзя смешивать пробелы и символы табуляции? К чему это приведёт?
Не согласна насчет вертикального выравнивания. Два-три раза нажать "Таб" - не такая уж большая трата времени. Но читабельность существенно улучшает.
Ребята, выравнивание - это костыль с помощью которого пытаются улучшить плохой код. Но, это нормально. Все, кто пытается повысить свой уровень проходили через это.
Выравнивание это такой же способ повышения читабельности кода, и оно очень хорошо работает, особенно когда есть блок из однотипных строк. Аргументы уровня “ой мне лень выравнивать” очень смешны, по такой логике давайте еще пробелы и переносы строк не ставить, а чо, лень же, зачем тратить на это время. И потом глаза ломаешь, читая простыни кривого косого кода “по стандарту”.
по поводу вертикально выравнивания. Например если пользуешся ms sql там в редакторе очень удобно копипастить выравненные перечисления полей, кофигуратор такого насколько я знаю не умеет, поэтому да бесполезно
Почему ты не делишься файлами .cf примеров?
Зачем?
Для чего? По твоему мнению?
Раньше смотрела по вечерам видео от Хазина, но этот формат украл мое ❤
Это хорошо)
+
Почему автор выбрал реквизиты объекта? Возможно он планирует в будущем прикрутить внешний вызов.
Насчет последних комментариев: это автор писал для себя видать, чтоб лишний раз повторить концепцию работы платформы. Я, учась в нетологии на данный момент, тоже так иногда пишу комменты в коде , чтоб закрепить какие-то моменты чисто для СВОЕГО понимания...
Для таких целей лучше завести блокнот и туда записывать. В коде потом сложно найти умные мысли
Хороший разбор, понятный и правильный. Комментарии лишние появились, скорее всего, вот откуда: человек скопировал кусок кода откуда-то, где были комментарии, и их оставил. Насчет НСтр, ну не знаю, локализация на другие языки - штука сомнительная, особенно, если человек пишет конкретную задачу под себя/заказчика, а тратить время на них зачем? Ты на пробелы не хочешь тратить, а тут форматирование строк бесполезное. А код типовых решений - это вообще кровь из глаз, его разбирать - себя не уважать :)
НСтр хорошо отделяет интерфейсные строки. Сразу в коде видишь, что эта строка выводится пользователю и относишься к ней соответственно: пишешь более понятно
@@yellow_club ну, это да, согласен
@@yellow_club подскажите, а НСтр в вызове исключения писать не надо было ?
@msvdev надо
Почему так много Антонов программистов 1С
А я люблю вертикальное выравнивание! Для меня так читабельнее :)
Гемора с ними много)
@@yellow_clubЧитабельность кода повышает в разы на мой взгляд - ткнуть 2-3 раза Tab дело не сложное.
@@yellow_club Код читается много чаще, чем пишется. Поэтому с тезисом о экономи несколько секунд - не согласен. Платформа не поддерживает, но в EDT при желании можно плагин наваять. Интересно, а дядя Боб чем аргументирует?
@Gesperid сейчас в посте размещу скрин дяди)
автор все напутал:
1. скобки не нужны - только мешают - платформа их все равно никак не выделяет, как в любой другой IDE
2. выравнивание пробелами как раз таки облегчает чтение
Автор - лапочка, поэтому везде прав)
1. Как это не выделяет? Всё хорошо выделяется... Скобки, как любые другие "скобки" в виде условий, циклов... Быть может, нужно настроить конфигуратор? В EDT тоже всё выделяется...
Выравнивают вродь табами а ее пробелами. для автоматизации есть турбоконф.
Пересказ стандартов 1с и чистого коду. это лучше чем комменты работы платформы в коде или такой же моветон ?
Не могу понять. Это ты так наехал или похвалил?)
@@yellow_club Хлопальщиков и без меня на каналах хватает. Зачем пересказывать чужие книги и стандарты ? Осмысления и дополнения не было, было упрощенное копирование.
Является ли бездумное копирование чужих кусков показателем высшей точки проф развития ?
Сделай своё видео, тогда и поговорим
@@yellow_club Говорить будем когда про свои удачные проекты рассказывать начнешь, а не пробелы по чужим книжкам расставлять.
Ахаха) серьёзный парень)
Занимайся проектами, это тоже полезно. А я программистам буду помогать.
По нстр не согласен абсолютно, это мертвый код.. нстр не бесплатен с точки зрения ресурсов, у него то же есть стек вызова, есть время выполнения.. если решение не планируется локализовывать - он вредит больше чем от него пользы
Все что не захардкожено имеет стек вызова.
В других языках тоже интерфейсные тексты можно хардкодить, а можно в ресурсный файл выносить, чтение из которого это стек вызова не бесплатный.
С учётом текущего оборудования мы давно не боремся за лишний такт процессора или лишний килобайт памяти, удобство и скорость разработки берут верх. И это не относительно 1С а относительно всего рынка разработки (кроме инженеров занимающихся программированием микроконтроллеров, да и там уже с esp32 не сильно за этим следят).
@@yellow_club проблема в том что у нстр четко описанная функция в документации - локализация строк (про интерфейсные строки там ни слова нет кстати). Так зачем же наделять этот метод тем чем он не занимается? то есть вы даете ему скрытый смысл - выделяете строки интерфейса им, но эту логику понимаете только вы, то есть любой другой заглянув в ваш код не поймет, то ли это подготовка к локализации, то ли мертвый код... Возникают вопросы, а почему бы не сделать глобальный свой метод ИнтерфейснаяСтрока("ххх") которая будет возвращать параметр как результат. Тогда запись явно красивее Заголовок = ИнтерфейснаяСтрока("Мастер создания продажи"); Тогда уж... ПС: а бывают еще строки с локализацией но не интерфейсные.. например такое часто встречается в интеграциях, где нужно распозновать входящие данные в зависимости от источника, когда источник может выдавать данные на разных языках, а мы должны ошибки то же на разных языках возвращать.. либо загрузка из того же экселя, пользователь работающий на английском интерфейсе будет пользоваться таблицей на английском, а если мы при загрузке ищем колонки или данные в этом экселе, то нам надо использовать нстр не как интерфейсную строку. Тогда вся эта логика с интерфейсными строками рушится. Нстр - это только локализация, любых строк и ничего больше.
вы не правы про вертикальное выравнивание - это добавляет эстетики и красоты коду. и не нужно кучу пробелов вставлять - копируем строку и если нужно корректируем табуляцией.
А зачем все эти пляски с копированием, корректированием?
@@yellow_club никто не пишет новую строку руками, когда столбец строк однотипный. всегда копируют уже отформатированную строку. дальше либо она уже красивая, либо вставляют\убирают одну табуляцию. зачем уже написал - код лучше\быстрее читается. эстетика.
Ну ок, если нравится, то почему бы и нет.
А потом если вдруг новая строка оказывается длиннее, и вертикальное выравнивание нужно переносить правее, нужно будет геморроиться со всеми строками, которые выравнены таким образом @@user-kq9zl3ct7q
Мне раньше тоже оно нравилось, а потом надоело каждый раз все заново выравнивать, когда появляется строка с наименованием колонки длиннее всех остальных.
"Не реализовано" - так пишет только джун, надо писать вроде "реализована поддержка только экселя".