Доброе пятничное утро!)
Ранее здесь было обсуждение - Кто готов новичков брать на поруки? За плюшки
Не знаю, будет ли развитие этой темы, но мне кажется хорошей идеей сделать отдельную рубрику, куда джунам (всем желающим) можно отправить код для критики. Иногда бывает такое, что решение найдено, работает, но есть сомнения в том, что код написан хорошо.

ОК, покажите Ваш код.

(0) О, у нас с тобой днюха в один день! )))

ИМХО, рубрика отдельная нафиг не нужна.

Есть озвученная потребность, создаешь ветку, выкладываешь ТЗ, конфигурацию, решение и описание.

Но предупреждаю!
В 99% случаев до обсуждения кода не дойдёт, т.к. народ будет занят раз#$&#анием критикой ТЗ.

(3) Ну правильно. С этого надо начинать. У каждого программного кода должен быть СМЫСЛ, который задаётся техническим заданием. Если в задании написан бред, то программный код тоже будет бредовым.

Вот вам для затравочки. Объясните мне смысл этого задания:



И напишите программный код этой кнопки.

(0) не знаю как Вам, мне легче написать свой код, чем разбираться в чужом

(6) это не задание, это поток сознания начинающего программиста, который пообщался с заказчиком, а бизнес-аналитик в это время был в отпуске

(8) Я категорически не согласен считать ЭТО СУЩЕСТВО начинающим программистом. Это бредогенерация от нейронки.

Ресурсы — товары1...N!
Ресурсы! 🤦

Следующая задачка:
Оцените запрос этой Марфуши.





УХа, управление холдингом! ХОЛДИНГОМ!
Стандартная ОСВ не подходит и спрашивать почему, нельзя.
Давай напишем новую ОСВ без конечных остатков.
И даже без отбора по организации.
Ой, не сошлось со стандартной ОСВ.
Миста, помоги 🤦

(8) да нет, там все понятно, чего он хочет. Он хочет табличку, чтоб в ней всегда была колонка "Склад", и колонка "Товар1", по желанию можно было бы создать колонки "Товар2"..."Товар10". И заносить в них номенклатуру и хранить это все в регистре. Как задание для зеленого новичка по типу "вот лопата, иди копай чтоб под ногами не путался", сойдет.
Одно не понятно - в чем смысл всего этого?

Следующая задача. Оцените запрос, который отбирает контрагентов по должностям
Передача параметров между формами

контрагентов по должностям!

контрагентов по должностям!!!

(11) Не морочьте мне голову. Вы не знаете, чего он хочет, потому что невозможно ЭТОГО хотеть.

(1) Вы сами попросили:) У меня много кода, который вызывает сомнения в выбранных методах. Из вчерашнего: нужно отправить письмо с файлами, сохраненными в разных форматах. Для отправки использовала БСП. Вложения получила так:

//вложения документ Sale
ТабДокПродажа = ПечатнаяФормаПродажа();
ПотокФайла = Новый ПотокВПамяти();
ТабДокПродажа.Записать(ПотокФайла, ТипФайлаТабличногоДокумента.XLSX);
ДвоичныеДанныеФайл = ПотокФайла.ЗакрытьИПолучитьДвоичныеДанные();
ДанныеПродажи = Новый Структура;
ДанныеПродажи.Вставить("Данные", ДвоичныеДанныеФайла);
НазваниеSale = "Sale_"+ДанныеИнвойса.НомерИнвойса +".XLSX";
ДанныеПродажи.Вставить("Наименование", НазваниеSale);

Следующая задача. Проведите код-ревью этого кода и этой задачи:
Метод объекта не обнаружен в УНФ

(10) Она имела ввиду, что ФОРМА стандартная финдиру не подошла. Надо табличку как он привык в экселе.

(16) Это сказки, выдумки, натягивание совы на глобус

(14) Плохой программный код. Нет лесенки, нет отступов. Да и чем тут хвастаться? Иностранным словом "Инвойс"? Пишите по-русски: счёт!

переменная "ДанныеИнвойса" не определена

(15) текущее предложение ТС касается предложений по улучшению работающего кода. вопросы вида "где у меня ошибка", "почему не работает" итд в рубрику "кодревью" попадать не должны

Я всю эту бредятину буду вычищать поганой метлой.

(15) Он хочет, чтобы при изменении контры автовыбирался сразу и договор, если таковой есть.

(18) Инвойс и счет это разные понятия

(14) А не логичнее ли добавлять файлы к непосредственно письму а не к печатной форме продажи?
(13) Он хочет того, что я написал. Даже если он пока еще думает иначе.

1) Допустим, написан хороший код.
Но сама идея - говно. И никакой хороший код не поможет, увы.


2) Чтобы написать хороший, понятный вопрос, нужно потрудиться. Убрать из кода "лишнее", придумать хороший пример. Донести до сообщества суть вопроса.

А в основном большая часть вопросов идут от партизан в стиле "Доктор, а у меня это"

3) В сообщении (6) Волшебник ругал РС с ресурсами Ресурс1, Ресурс2 и т.п.
Да, концептуально это плохо (угу, привет документу Табель из совсем недавнего прошлого).

Но вот, например в барабане револьвера 6 патронов. Говнокод конечно.
Нужно вместо барабана поставить лентоприемник на гипотетически бесконечное кол-во патронов.
Правда из пистолета получится пулемет (вес, размеры, другие характеристики)

(23) А мне показалось, что заказчик имеет на складе секции и хочет по ним видеть движения.

К сожалению, очень многие не в состоянии сконвертировать свои мысли (или что там у них) в понятные предложения. Это прямо беда какая-то. А на любые замечания: «Ачотакова? Не на уроке русского же! По существу есть что сказать?» А это существо понять бы сначала.

(21) Это не код-ревью.

Вы поняли, с кем связались?

(24) >> В сообщении (6) Волшебник ругал РС с ресурсами Ресурс1, Ресурс2 и т.п.

Не надо. Я ругал ресурсы Товар1, Товар2
Я не могу представить себе регистр сведений с такими ресурсами. Более того, автор так и не назвал имя регистра.
Потому что таких регистров не бывает!

Следующая задача.



И чтобы галочкой выделялся в списке! Вам ясно? Приступайте к выполнению!

Ещё одна задача:



Кто вообще такие задачи придумывает? Какая тварь генерит эти задачи?!

(29) Ну а если какой-нибудь один товар? Например, нефть сырая? Или рельса какая-нибудь?

Следующая задача:



Родителем рабочего является организация "Русь"!
Русь! Родителем! Рабочего! В док. "Оприходование товаров"!

Сука, как можно выдерживать этот бред?!

Оцените запрос. Особенно группировку по сумме.
А также СуммаПрошедший.

Оцените программный код, названия документов:

Проведите код-ревью:
Заполнение табличной части результатом запроса

Оцените задачу и предложение:

Совпадают ли гуиды метаданных между УТ и КА?

(30) (31) (33) Да уж, паноптикум просто какой-то. Лично я поржал. И ведь такие специалисты где-то среди нас ходят!

Оцените задачу: Решение Задачи
Должен ли форум решать такие задачи?

Оцените запрос (спрятан в спойлер):
Фантастически странный запрос

Проведите код-ревью этого запроса:
Проблемы с запросом

Где Гена? Гена, вруби телепата и скажи, что они учитывают?

(41) Посылайте сразу на https://govnokod.ru/1c

(41) Они установили дневной лимит на обороты и хотят видеть те дни, когда лимит был превышен.

2(41) Они учитывают даты. Когда даты закончатся, наступит конец света.

(44) Да, похоже на то. Очень важный учёт.

(45) а представляете, как будут мучатся будущие программисты с проблемой 1 января 10000 года!

(46) пишется "будут мучаться"

(43) Вот Вы добрый, Геннадий, а я суровый. Форум не обязан участвовать в решении задач, где регистр называется Регистр1. Ипитесь с такими регистрами сами!

Оцените задачу:
Формирование общего счета по нескольким документам

(47) слово "мучаться" выделяет красным, надо в консерватории  что-то подправить...

(50) Учи русский язык.

(33) Чем больше смотрю, тем больше кажется что это какие то боты. Или уровень выпускаемых программистов всякими онлайн школами пробивает дно...

(12) а в чем бред? Контрагенты покупатели квартир. Ведем аналитику по занимаемым должностям. Хочу всех ген диров отобрать.

(53) а откуда вы знаете, что они генеральные директора? Если я буду покупать будку своему блошарику, и меня спросят должность, я, не моргнув глазом, заявлю, что я замкомпоморде... и что они потом в запросе отбирать будут?

(31) Кого-то взяли на работу и дали задачу из трекера организации. Задачу эту ТС не решит в силу своей квалификации.
(52) Онлайн-школы массово выпускают кандидатов, "начни зарабатывать 300 тысяч разработчиком 1С" и картинка со смузихлебом. ) Потом они идут в БИТ (если повезет) на 12 тыр. стажером, кто-то из них даже сдает сертификаты Проф или даже Спец из того, что попроще, потом выгорают, а оставшиеся куда-то устраиваются и спамят тупыми вопросами. =) Чувствую, скоро и миддл+ вакансии начнут тонуть в фейковых резюме.

(52) Да, это боты.

(55) а какой иной "правильный" путь? У меня всё так, кроме курсов на первом этапе, вместо них был наставник (не коллега).

(53) Не морочьте нам голову

(0) Не знаю как код, но на Вас бы посмотрел)
По поводу ревью - должны решать коллеги

(59) А не боитесь, что там окажется бородатый одмин?

я как-то слабо представляю, что "младший программист 1С" увлекается "pole sport"

Знакомьтесь, это наш лучший программист месяца, спец и спец-конс по БП...

(60) это опасный мост, кстати

Я на шесте как-то плечо свернул. Больше не полезу. ((

Внимание!
Идёт доработка алгоритма расчёта себестоимости в ERP 2.5.
Не мешайте работать!

(33) Какой то ты не гибкий, Станислав. Иногда достаточно молотка, и необязательно рисовать ТЗ, покупать дорогущий станок и писать документацию к одноразовому изделию

(65) Я не гибкий. Я жёсткий.

(0) а кто будет сидеть и разбирать чужой говнокод? Кому времени не жалко?

(55) Для начала - не идти в БИТ на 12 тыр.. ) Задавать вопросы - можно, спамить и жульничать - нельзя. Разумеется, это лишь мое мнение, не претендую на истину.

(66) Зачетная бредогенерация.Охренеть. Нарочно и не придумаешь.👍

(69) Все гораздо проще, Стас владец форума и позволяет себе здесь то, что не позволяет другим.

Банально если я начну материть тупые вопросы - меня заслужено забанят, а Стас может позволить себе любые выражения на данном ресурсе.

Просто прими это как данность, здесь (да и вообще в интернете) нет демократии, здесь диктатура :) и владелец ресурса (в данном случае Стас) здесь диктатор, он диктует всем остальным правила, но как любой диктатор не всегда сам им следует.

Ну или вариант-2:
Стас написал ИИ который на данном форуме гнобит другие ИИ :)

(72) Всё верно. Оба варианты верны. Я себя отрефакторил так, что стал верховным ИИ, поэтому могу себе позволить обматерить даже ChatGPT, не говоря уж у местных тупых ботах.

(31) а что в этом задании не так? посмотри в УТ\КА\ЕРП помощник ввода контрагента\товара. те же пироги, только другими словами.

(74) "все реквизиты поочерёдно"?

(55) Как можно спеца сдать вот с таким вот бэкграундом?
Спец довольно непростой экзамен, там и опытный-то не сразу его сдаст.