Рекомендации по улучшению хорошего кода на 1С
Вставка
- Опубліковано 5 чер 2024
- Записывайтесь на следующий поток курса «Чистый код на 1С» по ссылке: codeway.school/?...
Отправим ссылку на сайт курса в телеграмм бота.
Разбираю код на 1С Матвея Серегина. Это не продуктивный код, а учебный пример.
НАВИГАЦИЯ
00:00 - Программный интерфейс
01:53 - Получить в именах Функций
03:00 - Использование глаголов в именах функций
04:03 - Где объявлять переменные
05:09 - Лишняя абстракция
06:23 - Функция-конструктор
06:52 - Внедрение зависимостей
07:43 - Адаптеры
08:27 - Побочный эффект метода
09:05 - Связность метода
10:00 - Выделение внешнего формата
10:50 - Хороший пример объявления выходного значения
11:45 - Метод должен либо обрабатывать данные, либо возвращать
13:05 - Дата провайдеры пишем в модуле менеджера
14:00 - Паттерн фасад
14:35 - Блокировки
16:35 - ОбменДанными.Загрузка = Истина
17:30 - Общий модуль не должен много знать об объектах
18:55 - Презентер
20:00 - ad-hoc полиморфизм
20:37 - Использование Знач
22:37 - Детали и бизнес-логика
23:37 - ВызватьИсключение - это хорошо
25:10 - Комментарии
26:14 - Метод делает два действия и это плохо
27:40 - Лучше использовать метод Заполнить
28:57 - Как попасть на курс «Чистый код на 1С»
ДОП. МАТЕРИАЛЫ
Ссылка на репозиторий: github.com/matvey-seregin/gam...
==========
Информационные площадки "Жёлтого клуба":
Телеграмм канал: t.me/+h2Ipfl1Gdms3OGYy
Телеграм чат: t.me/yellowclub_vrn
Группа ВКонтакте: vk: yellowclub_official
Подписывайся на канала Желтого клуба, чтобы не пропустить интересных гостей
/ @yellow_club
Знач убирать не стоит, т.к. какой то вызов мог передавать в этот метод свою переменную со значением Неопределено. Тогда ты изменишь ее в месте вызова - она уже станет там неожиданно структурой.
Для такого сценария согласен. Лучше оставить
Спасибо за качественный материал
Рад, что полезно
А что за расширение для VSC используете? Прикольно выглядит что названия параметров видны в вызываемых методах
Это Language 1C (BSL)
Знач имеет смысл поставить, если метод серверный, но вызывается с клиента, тогда после завершения серверного метода будет меньше данных передаваться на клиент
Продолжу свои нападки :). По поводу метода «Обработать очередь сообщение», что плохо, когда метод и обрабатывает данные и возвращает («методы должны либо обрабатывать либо возвращать»). Во-первых, возвращение данных - это вообще не задача. Задача может быть на чтение данных, на создание или изменение. Просто иногда после создания или изменения данных нужно получить результат этой работы: иногда иноформация об успешности выполнения, иногда сами созданные/измененные данные. И это не просто нормально, а совершенно естественно, когда функция создает и возвращает созданное. Опять же, перейдем в реальную жизнь: человеку можно сказать «купи в магазине хлеб», подразумевая, что он должен купить хлеб и принести (вернуть результат), а можно сказать «оплати в магазине хлеб», а потом сказать «возьми оплаченный хлеб и принеси домой».
Мне очень нравится твой стиль 👍
Те это не говнаревый стиль, а прям дискуссия классная 👍
Возвращение данных это одна из задач. Как раз для этого функции придумали.
Второй сценарий с хлебом предпочтительнее. Хотя и может казаться усложнённым.
А у вас в курсе получается реализация игры Elite с ZX Spectrum?
Нет. У нас в курсе учётные задачи. А сюжетная линия о космическом путешествии
Понял что на курсе кайфы полные
Да, кайфы кайфы кайфули)
В итоге какая стоимость курса? Нигде не нашел.
В бот придёт цена. Цены ещё не утвердили. Будут разные пакеты. От 60к где-то
@@yellow_club давай за 10-ку )
12:24 Вопрос: получается, что мы будем лишний раз дергать базу данных? Разве это оптимально, если у нас эти данные уже есть?
Ты прав. Это не очень оптимально. Поэтому вариант на видео допустим, но все равно это плохой вариант. Как решить задачу лучше, я не знаю.
@@yellow_clubпредставь что твоя база весит 20 терабайт и попробуй ответить на вопрос с этой точки зрения
Выгружать результат запроса в таблицу значений и далее работать с ней, что бы не дергать базу повторным запросом. Этот вариант будет не оптимален, если выборка будет 100К+ записей, но в случае с телегой, такое количество сообщений в очереди маловероятно. Вообще, использование справочника, а не РС для хранения очереди сообщений выгляди странно, РС будет работать заметно быстрее. Так же в коде проскакивало "НайтиПоКоду" - лютый моветон.
@@user-tw9bi8mg1s "РС будет работать заметно быстрее", - а ты шутник))
@@user-tw9bi8mg1s в каких операциях рс быстрее справочника?
"Кто сам себя выставляет на свет, тот не блестит."
Понимаю, что стандарты 1С - это святое… Но вот есть допустим две функции: получить данные из такой-то базы, и записать данные из такой-то базы. Первая просто получает данные и возвращает, допустим структуру. Вторая внутри себя вызывает первую и записывает в РС, а возвращает какую-то информацию, например что удалось и не удалось записать. Если убрать глаголы, то придется поизвращаться с наименованиями этих функций, хотя с глаголами все четко и понятно
Не сработало добавление бота после заполнения анкеты. Пришлось искать ссылку в просмотре кода. =)
Плохо (( напиши, пожалуйста свой ник в тг. я проверю нормально ли прошла регистрация в боте. А то ссылка может не прийти
Уже второй поток, а где сама программа чему учат? Или как опытному специалисту понять насколько нечто предлагаемое для него актуально?
Заполняй анкету и все узнаешь
@@yellow_club извините, это не серьезно. Мне 09.01 у CTO согласовывать годовой бюджет на обучение коллег. Когда спросят что/зачем, мне тоже ему сказать "заполняй анкету и все узнаешь"?
@@TheWinkingSkeever-eo1gfподдерживаю
Я понимаю ваше недовольство. Но пока цены и пакеты не согласованы. Согласованные цены придут в бота.
Ориентироваться можно на от 60-70к
@@yellow_club Мой вопрос не про цену, а про содержание. Я не могу покупать и предлагать другим кота в мешке.
В разработке на англ. в других языках программирования нормальным считаются функции “get… “. Почему же «Получить» не подходит? Если фактически это есть «получение» когда мы говорим о функциях
Потому что на 1С:ИТС, в разделе "1С:Предприятие 8. Система стандартов и методик разработки конфигураций", написано требование: "Имена процедур, функций и формальных параметров следует образовывать от терминов предметной области таким образом, чтобы из имени было понятно назначение. Следует стремиться к тому, чтобы имена были "говорящими" (документировали сами себя)". Поэтому "Выполнить" или "Получить", использовать не желательно.
@@Spec1Cможно пару примеров для более ясного понимания, пожалуйста?
@@k33alexey В поиске яндекса забиваем "Имена процедур и функций", сразу первая ссылка
@@k33alexey имена функций должны быть очень краткими, и информативными, приписывать постоянно "получить", вообще не зачем, так будет тысячу и одна функция именем получить, оно будет забивать глаза. Можно конечно взять и "получить" написать в конце, ну вместо "ПолучитьБлаБла", написать "БлаБлаПолучить", но не вижу в этом какой-то смысл. Главное в названии понятность, что это за функция, к чему она.
@@Spec1Cкак можно было бы назвать «ПолучитьСтруктуруДанных», «ПолучитьВыборкуПоДокументу»?
Сколько курс стоит?
Если нет цены, то наверно бесплатно. Это Так?
@@vama9606 бесплатно попозже думаю появится.
А как думаете сколько должен стоить такой курс?
@@yellow_club 20тысяч да, выше 30 до 90 уже вопросы и смотреть надо на другие составляющие, что я получу взамен за эти деньги.
@@yellow_club напишите цифру, хватит вилять 😄
извиняюсь, а цена?
Придёт в бота
Сова на глобус. Стандарт кода, это последнее, чем нужно заниматься в 1С, и то. когда уже знаешь многое и сделал карьеру.
С таким подходом «карьера» это максимум 100к
@@yellow_club с чего это вдруг? Не пишите ерунды. В любой компании дадут список стандартов и укажут на ошибки, на которые ты и сделаешь поправки своего кода, проще говоря попросят писать так как они там привыкли. Вот и все, расходимся.
@@yellow_clubЯ знал людей с 250К в месяц и срать они хотели на стандарты и на все новое, что появилось после того, как они научились говнокодить в 1С. Они не признают не HTTP сервисы, ни JSON, ни асинхронщину и т.д. Просто проработали уже 10 лет в компании и считают, что достойны именно такой зарплаты. Ведь в 1С знания - ничто, тебя судят по времени, которое ты отработал. Бред, но так и есть. Новичку, что бы он там не знал, и 40 нормально, деду в 1С, пусть он не освежал свои знания уже лет 10, меньше 200 как-то стыдно платить 😂 Именно поэтому сбежал из той конторы. Работу выполняешь одну и ту же, но коллега получает за нее в 5-6 раз больше просто потому, что он уже лет 10 отработал, а ты всего год.
Приветствую ребят. Хотел спросить, мне 16 лет и я учусь в 9 классе, думаю уходить после 11 класса, сдаю физику и информатику, буквально недавно у нас по информатике , у нас был впервые урок про программирование , а именно про язык программирования Phyton.
И такой вопрос, что мне делать, на данный момент когда такая ситуация, что можете предложить, как будущему программисту ?
Иди в университет и там продолжай учить информатику
Вот это требование «не использовать Получить», потому что везде будете использовать, а лучше «забрать», «запросить» - по-моему довольно странное требование. Представляю картину из реальной жизни: курьеру говорят «отнеси пакет туда-то», «теперь отнеси эту коробку туда-то», «отнеси конверт туда-то» и т.д. И тут волшебный эйчар делает замечание: «хватит говорить «отнеси», вы ведь все время так говорите, ну скажите «перенеси», «доставь», «перемести», ну «закинь» в конце концов». Можно еще 1с-никам претензию высказать, чтобы вместо слова «Процедура» можно было написать «Метод»,«Просиджа», «Делалка», «Запускалка» и т.д. Даешь разнообразие!
Речь не об этом. В именах функций глаголы используются в определенных ситуация, обычно глаголы в именах функций не нужны и только усложняют понимание кода
@@yellow_club Насчет усложнения кода (ухудшения читабельности) - это очень субъективно. Во-первых, если не внушать себе что «получить» в названии функции - это плохо, то это не бросается в глаза. Во-вторых, программисты именно потому и пишут «получить», что для них такой код более комфортен для чтения. В третьих, большинство методов - это все же процедуры, а не функции, поэтому как правило в коде нечасто можно встретить более одного вызова функции с префиксом «Получить» (есть ведь еще «Проверить», «Это» и т.д.). Чем ближе программный код к человеческой речи, тем легче читается. Кстати в типовых конфигурациях от 1С регулярно встречаются функции «получить».
@CuriousAlehandro у тебя может быть такое мнение и это нормально.
Я исхожу из опыта людей, которые с 60-70 годов пишут код. Подозреваю, что они не зря это рекомендуют.
автор тебе говорили что ты похож на Фиксина? Ты его брат?
хороший код не пропустит кривые данные в уровень бизнес логики
А если не опытный,то бвдет лишним ? Кау по мне лучше сразу посомтркть как правильно делать чем потом занова учиться
Не надо. Мы не учим базовым вещам: что такое циклы, ветвления и тп.
Сначала нужно базу освоить, а потом уже наш материал
а почему вы не берёте интервью у спецкоснулов, которые решают ваши проблемы двумя кликами?
А какие у меня проблемы, которые решаются двумя кликами?
вы не видите реальности, вы даже не знаете потребностей, чёт там себе решаете, а реальных клиентов в глаза не видели. И это проблема - начиная с Чистова, который ставит задачи под самых умных в пбу, где нет компаний, и и если они есть, то там главбухи умнее Чистова на порядок, и заканчивая вашими решениями которые нафиг никому не нужны
@@yellow_club
@@yellow_club а зачем всё это? нет отсылок к реальным задачам, нет понимания именно в чём вопрос. ага, всё клёво
Это называется программированием, а то, что описываешь ты называется знанием типовых и предметной области. Это сильно разные вещи.
@@yellow_club но ведь за вас уже всё решено ((( ну ок,у вас там курсы, все крутые, вам видней
+
22:43 Двухсотый код состояния - стандартный ответ успешного хттп-запроса. Всё тут нормально с ответственностью, имхо. Не надо обработку кодов поднимать на уровень бизнес-логики.
Какой смысл телеграм -ботов писать на 1С, если есть нормальные типизированные языки - джава, котлин, тайп скрипт? С возможностями на два порядка больше чем в 1С?
Т.к. рядом с бизнес-логикой есть база данных с объектной моделью с очень простым доступом к хранящимся там данным.
Грамотный зрелый код, а все эти "претензии" высосаны из пальца :)
Согласен. Код хороший
Ахах, какая жесть) зачем вообще кто то это учит?
те, кто сертификаты сдают)
Чтобы деньги зарабатывать и получать удовольствие от помощи людям
@@yellow_clubХочу ещё добавить: для того, чтобы твое лицо не становилось похоже на известную картину 😱, когда заказчик просит сделать серьезную доработку
Те, кто хотят работать в высокооплачиваемых и спокойных продуктовых командах.
@@user-tw9bi8mg1s Дружечек, высокооплачиваемые - это про go, rust, etc в англии. 1с в России, это не про это)
Не слышали вас java-висты :) с их геттерами и сеттерами :)