Ситуация: мы принимаем на работу нового сотрудника. На словах кодить умеет. Наш главный позвонил на его прежние места работы для получения рекомендаций.

На одном заводе отказались как либо комментировать его компетенцию и попросили e-mail. После на него выслали пример его кода.

Ниже код:
Г=ПолучитьИмяВременногоФайла();Д=Справочники.ВнешниеОбработки.НайтиПоКоду("000000097").ХранилищеВнешнейОбработки.Получить();Д.Записать(Г);К=ВнешниеОбработки.Создать(Г);в="в производство";ф=К.ПолучитьФорму(,,в);Ф.Заголовок=в;О=ф.Отбор;С=Справочники.Склады;З=Новый СписокЗначений;З.Добавить(С.НайтиПоКоду("00221"));З.Добавить(С.НайтиПоКоду("000000005"));Т=О.СкладОтправитель;Т.ВидСравнения=ВидСравнения.ВСпискеПоИерархии;Т.ИСпользование=Истина;Т.значение=З;З.Очистить();З.Добавить(С.НайтиПоКоду("00093"));З.Добавить(С.НайтиПоКоду("00100"));Т=О.СкладПолучатель;Т.ВидСравнения=ВидСравнения.ВСпискеПоИерархии;Т.ИСпользование=Истина;Т.Значение=З;Т=О.Дата;Т.ВидСравнения=ВидСравнения.ИнтервалВключаяГраницы;Д=ТекущаяДата();Т.ЗначениеС=НачалоГода(Д);Т.ЗначениеПо=КонецГода(Д);Т.Установить();ф.Открыть();

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

(0) Что смущает?

1. Найти по коду (что есть плохой признак)
2. Слишком короткие названия переменных, непонятно зачем так сжимать код, извращение какое-то.

3. Еще в коде нет переноса строки

(2) 1. Найти по наименованию - плохой тон, по коду как раз нормально все.
2. Название переменных - дело сугубо личное, если в вашей компании нет стандартов написания кода утвержденных.
3. Код написанный в 1 строку работает быстрее нежели код разбитый на строки.

Пример кода, приведенный в примере потребляет меньше памяти, и неизвестно в каких условиях он работал.
Быть может в условиях жестких ограничений по аппаратной части.

Вывод: Вы - учите матчасть, а Кандидат знает больше Вас.

(4) То есть по вашему мнению - существуют аппаратная часть - которая не повзолит с достаточной скоростью открыть форму списка документов с отборами, и в целях оптимизации был написан гоМнокод - абсолютно нечитабельный ...

(5) А зачем тема ? Вы же заранее определились.

(5) Да, к сожалению аппаратная часть это всегда проблема.
А язык 1с это очень тяжеловесная конструкция.
Код нормальный.
Читается нормально.
Если не соответствует вашим стандартам - ну так вы должны сначала свои стандарты огласить.
За длинные названия переменных я бы расстреливал без разговоров.
Хотя некоторым нравиться.

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

(5) Что-то мне намекает что в (0) приведен код после обфускации. Что неужели нельзя было на собеседовании просто попросить написать кусочек кода хотя бы просто на листочке. Это кстати гораздо больше дает понимания кандидата, когда кусочек кода пишут от руки без различных автозавершений и подсказок.

(4)
> 3. Код написанный в 1 строку работает быстрее нежели код разбитый на строки.

Это такой ночной бред? Или новое слово в оптимизации? Или тут должна быть табличка "Сарказм"?

(0) Мне бы с таким коллегой работать не хотелось. Т.к. поддерживать потом эту нечитаемую конструкцию сложнее, чем нормально структурированный код с осмысленными именами переменных.

(10) Спасибо за поддержку
(11) Спасибо за поддержку

(11) А откуда вы знаете что он так предпочитает писать?
Может он предпочитает структурировать код, просто требования к коду на прежней работе предписывают оформлять его так?

Код мой, это отбор во внешней форме списка документов. Ограничение строки в 150 символов (я сам не знаю почему). Была необходимость добавить отбор по дате и вместить все в 150 символов.

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

Поиск по коду мне и самому не нравится, но возможность XMLЗначение вписать туда я не могу. Да и отбор по складам уже был до меня.

(15) кстати по поводу "распальцованного" - насторожило то  что у него на звонке телефона - играет группа Лесоповал.

Главное требование к коду одно - чтобы работал!
А все остальное уже зависит от ситуации и может сильно отличаться в разных случаях.
Кому то наплевать на поддержку, давай код который будет быстро работать.
Кому то наплевать на скорость работы - главное чтобы код был понятен даже идиоту.

(17) Если на звонке телефона стоит что-то отличное от обычных гудков это уже клиника.

(16) Еще можно было использовать английский язык. Там многие команды короче русских аналогов.

(0) Код работать то будет, но не факт что всегда.
В таких костылях НайтиПоКоду или аналоги поиска надо выносить в отдельною процедуру.
(14) Выпрыгнул как черт из табакерки - я аж испугался!

Что особенно интересно, так это то что оба оппонента зарегистрированы сегодня.
Самому с собой подискутировать это наверное прикольно ;)

(10) Английские синонимы отрабатывают быстрее. ;)

(21) Да это видимо тролль какой-то

(0) Дайте свое задание ему написать код.

Тролль зарегистрировался под ником "НовыйСотрудникНаФабр", вряд ли тот сотрудник мог так быстро узнать себя в этой теме.

(24) Видимо так и сделаем.

Финальный код выглядел так:
Г=ПолучитьИмяВременногоФайла();Д=Справочники.ВнешниеОбработки.НайтиПоКоду("000000097").ХранилищеВнешнейОбработки.Получить();Д.Записать(Г);К=ВнешниеОбработки.Создать(Г);в="в производство";ф=К.ПолучитьФорму(,,в);Ф.Заголовок=в;О=ф.Отбор;С=Справочники.Склады;З=Новый СписокЗначений;З.Добавить(С.НайтиПоКоду("00221"));З.Добавить(С.НайтиПоКоду("000000005"));Т=О.СкладОтправитель;Т.ВидСравнения=ВидСравнения.ВСпискеПоИерархии;Т.Использование=True;Т.значение=З;Р=Новый СписокЗначений;Р.Добавить(С.НайтиПоКоду("00093"));Р.Добавить(С.НайтиПоКоду("00100"));Т=О.СкладПолучатель;Т.ВидСравнения=ВидСравнения.ВСпискеПоИерархии;Т.ИСпользование=True;Т.Значение=Р;Т=О.Дата;Т.ВидСравнения=ВидСравнения.ИнтервалВключаяГраницы;Д=ТекущаяДата();Т.ЗначениеС=НачалоГода(Д);Т.ЗначениеПо=КонецГода(Д);Т.Установить();ф.Открыть()

Телефон без звонка, вибрация стоит.

(23) Да вы очень похожи на тролля.

(10)не-не-не, реально быстрее... т.е.
и1=0;
Пока и1<10000000 Цикл
и1=и1+3;
и1=и1-2;
КонецЦикла;

отработает раз в 20 медленнее чем

и1=0;  Пока и1<10000000 Цикл  и1=и1+3; и1=и1-2; КонецЦикла;

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

(30) да сколько можно повторять это, уже на инфостарте давно разоблачили это. Ну не бывает в рабочих система таких простых циклов, на которых заметна разница в производительности и не работают продакшены с включенным отладчиком.

(30) вот это откровение

(31) Про продакшн никто не говорит.

(0) из базовых это конечно поиск по наименованию, получение значений ссылочных объектов через точку, оформление кода (в будущем будет только усложнять поддержку), запуск внеш обработок возможен с версии 8.2 без записи во временный файл, который потом конечно же забывают удалить. Но основная претензия была бы в том, что в типовой уже есть такие механизмы и нужно вызывать их.

(33) так и вижу как все программирование циклов сводится к увеличению значения на единицу.

кто ясно мыслит, тот ясно излагает..
значит.. не стоит брать..

или запастись галоперидолом.. или попкорном..

(36)(37) это точно, будет постоянный цирк

(0)какая задача стояла известно?
какая поставка задачи  - такой и код.

Сотрудник не из (1)? )))

(0) Лажовый код.

(0) плохой, не красивый код
видно, что начинающий.

Вы серьёзно это обсуждаете? (0) (14) толстый тролль

Скорее всего, претендент на должность после общения поставил себе задачу проиграть этот конкурс. Тоже не хотел бы работать в конторе, выставляющей чужой код на осмеяние.

Естественно, всегда лучше выбрать программиста, который пишет красивый, легко читаемый код... Пусть он даже работать не будет! Зато конкурентом не будет...А если он пишет коряво, но работающий, хрен ошибку найдешь, чтобы перед начальством разоблачить!

(14) "Была необходимость добавить отбор по дате и вместить все в 150 символов."

Если бы мне дали такое задание - я бы сделал нормальный код (без учета последнего условия).  При сдаче работы написал "ограничение кода в 150 символов приведет к нечитаемости. Как следствие - к сложности дальнейшего сопровождения. Сделать, конечно, можно. Но не хотелось бы себе карму портить".

В итоге бы и задачу сделал, и карма бы не пострадала.

(45) + А самый главный вопрос...А приведенный код полностью решил поставленную перед программистом задачу с точки зрения пользователей?

(46) Хорошая у тебя карма, не страдает, даже если пользователи не получили желаемого...

(48) У меня пользователи всегда довольны.

(49) Но тут были поставлены конкретные условия - 150 символов. Клиент всегда прав...даже если он не прав

(2) (3) все верно смущает - это г-код. Вопрос в зарплате и в требованиях к программисту. Ну и можно же спросить кандидата лично, попросить привести пример когда, потом задать вопрос в всвязи с чем подобные автррские решения применяются и возможно ли от этого как-то отойти будет при работе в вашей компании

(50) "конкретные условия - 150 символов." - условие не имеет смысла.  Я бы понял, если задача стояла на ассемблере написать. И количество памяти было бы ограничено.  А для 1С это ограничение не имеет никакого смысла. Точнее так: это ограничение может иметь только рекомендательный характер.

(52) Но раз поставлено, значит какой то смысл был. У нас для дискуссии явно недостаточно информации.

+52 Заказчик в данном случае не понимает, о чем говорит. Разработчику нужно было этот момент обсудить на этапе постановки ТЗ.

А завтра придет заказчик и попросит воткнуть 1С в бортовую систему космического корабля. Тоже нужно будет делать?

(54) Если оплата соответствующая, то почему нет? Будем делать!

(55) Главное договор хорошо составить!

(52) Имеет смысл, не имеет...Но поставленную задачу ты же не выполнил. Просто отказался ее выполнять, правда обосновав отказ! Но заказчик может получить свои хотелки, найдя менее щепетильного программиста...И будет доволен!

(0) "На одном заводе отказались как либо комментировать его компетенцию и попросили e-mail. После на него выслали пример его кода."

У вас нет специалистов провести собеседование?

(47) Самый главный вопрос чей на самом деле это код.

Да его сократили, чтобы СМС-кой переслать...

(0) Автор явно на инфостарте азы хлебал.

(60) Кстати, вариант.  А на принимающей стороне как? Руками внесли?

Прикольная тема )))))

(0) Гений1С?

(0),(14) А что случилось с вашим коллегой FigNikPredumaesh ?

(57) Туда ему и дорога.
ИМХО если ставят такие задачи, то надо бежать оттуда и не оглядываться.

(0) где-то я такое видел,соискателя не Лехой зовут?

(66) Ага - ставят задачу написать онокод а потом всем говорят - смотри какой онокод написали.

(66) Зачем? Нормальная задача.
(52) Почему условие в 150символов не имеет смысла?

(68) =)

Мне код не нравится.
По одной простой причине.
Мне с быстрого взгляда непонятно его целевое назначение. А это не только проблема дальнейшей поддержки этого кода но и явная проблема с исполнителем этого кода. Код писался быстро, неаккуратно, прог не задумывается о дальнейшей жизни кода.
Представляю какой ужас будет если этот прог будет ваять не код, а форму. Возможно что прог - выкормыш стшников иже с ними, а это значит что не работал плотно на прикладных задачах бизнеса. Как-то так я себе представляю.
Код может быть мамнокодом, но он должен быть понятен

Некто недоброжелатель прислал некий код приписанный соискателю. Типа, а тестовое задание он не делал что ли?

(31) Сделай вывод текущего времени до и после цикла в обоих случаях. - Даже отладчика не нужно

(0) смотря что за программист вам нужен. Если недорогой падаван, которому надо ставить четкие задачи и потом проверять, то берите. Если вам нужен самостоятельный профессионал, то - нет.

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

Предлагаю теорию заговора.
1) Вводятся новые правила - преследование за критику 1С
2) Начинаются вбросы с обсуждением качества кода для вызова этой самой критики.
N) ...
N+1) Профит?

По зарплате сколько Вы ему предлагаете?

(0) Косячище. Переменные использованы не в алфавитном порядке.

(0) Если приходится использовать НайтиПоКоду("000000097"), то писать надо так:
НайтиПоКоду("000000097"); //Такое вот наименование

Чтобы легче было разбираться в дальнейшем. Кто его знает что там под этим кодом.

(0) Возможно он так привык. Можно принять с испытательным сроком и разъяснить требования к нему и к результатам и качеству его работы.
ЗЫ Бывают варианты пострашнее и тупее. ИМХО чувак либо прикалывается, либо так привык. Это все поправимо. Главное чтобы шарил в темах.

(0) Спрашивать код у завода и надеяться, что дадут именно его код?
Да вы шутите? Вы не думали, что вас могли обмануть? :)

Вы бы лучше дали ему написать какой либо отчет и посмотрели его код.

А так, вы просто омерзительны. А вам вообще нужен программист? :)

(78) Согласен, комментарий лучше писать, а то через месяц уже не понять, что за элемент был :)

(2) >> 1. Найти по коду (что есть плохой признак)
Был бы плохим, его бы не было в 1С.

>> 2. Слишком короткие названия переменных, непонятно зачем так сжимать код, извращение какое-то.
Чувак перелез в 1С с ЯП низкого уровня, он так привык, все норм.

(0) Я бы не принял. Судя о отрывку и реакции бывшего работодателя - он пишет быдлокод, которым бывший работодатель крайне недоволен.  Вы тоже будете им недовольны.

(3) >>  3. Еще в коде нет переноса строки
Действительно, косяк :)))

(8) Выложи здесь свой код, сейчас похвалим.

(10) Учите матчасть. Не зря люди циклы в одну строку пишут.

(17) Хоть один ровный пацан будет в фирме. Все вам завидовать будут, берите.

Коллеги, спасибо за комментарии. Большинство из вас таки бы не взяли этого ****кодера на работу.

Сейчас главному скину ссылку на эту тему и он уже решение примет.

(74) Вон оно как.
Значит эти товарищи которые пишут на ассемблере все поголовно любители. Ни одного профессионала.

Почитала - прикололо. Т.е. код можно оптимизировать, тупо собрав в одну строку?

Насчет переменных - жестковато, конечно, тяжело другим разбирать этот код. Хотя бы буковок чуток побольше...

Периодически почитаешь, так многим руки бы оторвали за сокращения переменных.
Я вот не могу себе отказать, например, в СтрокаТЗ (СтрокаТаблицыЗначений) и СтрокаТЧ (СтрокаТабличнойЧасти)

(90) "СтрокаТЗ " - это еще по божеский

(91) Но всё же - г*овнокод?!

(89) Язык 1С - это не низкоуровневый язык.
И оформлять код нужно примерно так, как в типовых конфигурациях. Стандарты же в каждом языке есть, как без них.

(92) Сама по себе СтрокаТЗ не делает код быдлокодом. Тут надо код в целом смотреть. Если он читаемый - то все ок.

(88) На самом деле многое может зависеть от условий работы и принятых норм. К примеру: если будут загонять по времени из серии "мне по-фигу как ты это сделаешь, но чтобы к 10:00 это было", то почему бы и не по-фигу, как это сделано. Работает, код правильный, оформление не по стандартам 1С.... тут уж по ситуации.

В общем, если сотрудник откажется писать по стандартам ("мне и так удобно"), тогда нет смысла брать.

о, мой код с прошлой работы

(2) Ой не надо. Есть такая ситуация, как специфика ведения учета в программе. Не он виновен, он лишь продолжил традицию :)

Как обычно, такой код сводится либо к появлению предопределенных элементов, либо появление некого справочника / регистра, где бы вводились подобные параметры :)

В принципе, если у Вас в фирме платят за объём кода (за количество строк) - берите!

(83) Работодателю не код надо смотреть, а результат работы кода :)

Тут недавно посмотрел про оптимизацию запросов по Составные типы http://catalog.mista.ru/public/184361/

Так там писать запрос нужно так, что любой мало образованный подумает, что писал было кодер :)

(95) В такой ситуации адекватный программист должен сказать, что "вы ставите не реальные сроки" и установить свой срок. Либо потребовать чтобы начальник поручил установить срок независимому компетентному эксперту, если он не доверят мнению программиста.

(4) "Код написанный в 1 строку работает быстрее нежели код разбитый на строки. "
Нет. Он быстрее при замере производительности в отладчике, и только)

(101) т.е. вам отладчик уже не показатель? :DDDD

(100) Тогда уж поменять работу! А то мне - нормальному программисту - не дали времени красиво оформить код!

(95) Нет проблем, по стандарту, так стандарту. Мы только срок кодинга увеличим в двое, оки :)

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

Имхо, и программист и компания достойны друг друга)))
Дай бог не пересекаться с такими

(105) А вы я смотрю знатный извращенец. Замеряете работу в нано-секундах :)
Я вот думал, что замер времени всегда делать необходимо на рабочих объемах данных с учетом транзакций и других коллизий :)

(104) Я о том же. ;)))

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

(0) Мы не знаем какие требования к качеству кода были на заводе? Какие требования к управлению информацией? Если у него стояла задача - писать так что б работало и так что б потом было сложно поддерживать, то код вполне себе хорош. Поиск по коду или наименованию плохой стиль, но опять же, если Справочник закрыт от редактирования, почему бы т нет?
ИМХО. Поговорите правила написания кода сразу. Если человек адекватный то подстроится. За 3 месяца это выяснить можно.

Ветка - развод. Есть большое подозрение, что КурлыкКурлык, НовыйСотрудникНаФабр и Дзенн - это один человек.

(110) Программист должен нести ответственность за свой код - несмотря на любые сложные условия работы.

Зато интрига! ;)))

(112) Мне почему-то казалось, что он должен нести ответственность за результат работы кода. Оказалось, что казалось...

Еще недавно другой коллега совершил каминг аут - теперь мы думаем - кем его заменить /:

(112) Он признался что пробовал писать на ООП? :)

ЗЫ Есть 1С-нику дать задачу на ООП он никогда не напишет класс RecordSet и прочие для работы с БД

(110) Судя о отрывку кода - либо никаких, либо программер на них забил. Со вторым случае все ясно, я  первом случае адекватный программист стал бы придерживаться хоть мало мальсики, но стандартов 1с и элеметарной культуры кода.

(107) В (105) сказано о том-же, замерять время тестовой игрушки со счётчиком, записанной в одну строку и в несколько без смысла, даже на больших количествах итераций. Более того, инструмент "замер производительности" предназначен для получения именно оценки времени выполнения кода на существующих объёмах данных (код приемлемый для БД из 100 записей на 10000 по времени выполнения может повести себя непредсказуемо) и может быть использован для принятия первичного решения об оптимизации

(116) напишет, только первый раз медленно, как и любой другой программист

(116)
class MyRecordSet : public CRecordSet {
public:
  MyRecordSet() : CRecordSet() {};
  ~MyRecordSet();
}

118+ справедливо и в обратную сторону

(119) похоже на С# или это Java?

(121) Я пытался изобразить с++, насколько я его помню

(0) УГ, и вот почему:
найтипокоду - леща бы отвешивал за такое без разговоров и вот почему: однажды один франч на большом лесозаготовительном предприятии моей области написал обработочку, которая вся была заполонена командой найтипокоду. потом была сделана свертка и найтипокоду перестал работать, вместе с обработкой.
итог - если есть какое то постоянно использующееся значение, то надо его либо в константу, либо в регистр сведений.
код без переносов? этот прог такой важный, что не может нажать enter? надо думать о других людях, кто будет поддерживать ЭТО.
любителям коротких переменных - отдельная сковородка с кипящим маслом в аду. почему я должен догадываться чем отличается а от а1? у него там автозаполнение сломалось что-ли?
в итоге - простой код превращается в абракадабру, которую следующий программист (если он не такой же поехавший) должен будет приводить к УДОБОЧИТАЕМОМУ виду и тратить свое время на пустое занятие.

(111) IP-адрес один и тот же?

(112) А работодатель обязан платить все налоги и выплачивать все в белую, а не в конверте... Оплачивать больничные. Давать отпуск на 28 дней, а не на 14 :)

(123) Но до свертки то она работала? В чем тогда проблема?
Можно написать более универсально.
Но это займет больше времени, и будет стоить дороже.
А готов ли за это заплатить заказчик - надо спросить у заказчика.

(115) Человек вам раскрылся, а вы его уже увольнять :DDDD

(123) После свертки изменились коды? Ого, мощная видать свертка прошла.
Менять коды можно запретить.

(121) Это явно не джава.
Дальнейшее действие, что именно наш класс должен вернуть таблицузначений (или хотя бы массив)

public class Record {

    private int id;
    private Date datePost;
    private String textPost;

    public Record(int id, Date datePost, String textPost) {
        this.id = id;
        this.datePost = datePost;
        this.textPost = textPost;
    }

    Override
    public String toString() {
        DateFormat df = new SimpleDateFormat("MM/dd/yyyy HH:mm:ss");
        return Integer.toString(this.id) +" "+ df.format(this.datePost)+" "+this.textPost;
    }
}

Насчет НайтиПоКоду.
Данный механизм надо лечить за счет ПланаВидовХарактеристик "Константы" или хотя бы справочника такого.

(112) Он и несет. Пришел начальник, сказал - писать так, что б сложно было поддерживать. Вот и результат.

(123) Очень интересно. А расскажи-ка, как быть если конфа на поддержке и снимать её нельзя. Какие такие регистры и константы использовать? Перед тем как делать свертку надо думать головой о последствиях и все заранее предотворотить. И не будет никаких проблем. Нет ничего плохого в НайтиПоКоду(), если использовать по уму.

(131) Ну не обязательно так прямо сказал.
Может сказать так - чтоб до завтра написал, пофиг как но сроки горят.
И рассусоливать тут для красоты кода и удобства поддержки некогда, ибо срыв сроков, и вон с работы, поганой метлой за косяки.

(126) И правда, дилемма. Либо код, либо быстро :)
(130) И какая разница, где искать код?

Еще раз повторю - есть только одно обязательное требование к коду.
Код должен работать и выполнять поставленную задачу.
Если это так, значит код хороший.

Дополнительные требования возможны, и зачастую есть.
Но они индивидуальны для ситуации и заказчика.

(123) Тут говорит о том, что свертку проводил быдло кодер. Без анализа вообще кодинга и специфики конфигурации :)

(132) НайтиПоКоду("0043") - это отвратительно
НайтиПоКоду("0043"); //наш склад, с которым работает филиал в Мухосранске
- это более-менее

(123) Почему-то свято верилось, что свертка - это для уменьшения количества документов со всеми вытекающими, но никак не уменьшение количества элементов справочников.

+(135) Аминь. Но жадный работодатель думает, что код должен содержать в себе картинки и желательно смешные, что бы смотреть код было удобнее :)

(135) "Код должен работать и выполнять поставленную задачу"
При все уважении, но вы ошибаетесь. Надо всегда смотреть чуть дальше чем текущая ситуация. Хороший программист от новичка этим и отличается.
Проблема франчей, например, как раз в том, что они пишут код, по принципу - быстрее сделать, а после меня хоть потоп.

(135) Пишите код так, как будто сопровождать его будет склонный к насилию психопат, который знает, где вы живёте». Джон Ф. Вудс

(133) Тоже вариант.

(123) Существует и обратное мнение "руки оторвать за счётчики вида СчитаемСтрокиТоваров", разумная длина переменной - наше всё.
Нервный какой - если в (0) весь код внутри процедуры, то не критично, всё и так перед глазами, а вот если это часть процедуры на 2-3 экрана, тогда ой.

(130) И чем это лучше, ты всё равно не знаешь "Кто есть Кто". Альтернатива в 137

(140) не полностью, если стоит задача "держать клиента на крючке" то вполне себе метод "в моём коде Чёрт ногу сломит, так что - я ваша на веки"

(139) Вот если бы там был комментарий, например: "погода не летная, в трамвае отдавили ногу, носки разные, бл.., одел... Потому написал так:" - тогда бы все поняли и простили.

Вывод: коллеги, пишите комментарии - так занимательно читать! ;0)

(144) то же по уму, не надо а=а+1; комментировать на 5 строк почему именно так, а не иначе. Бо будешь вникать в комментарии вместо кода

А еще свою фамилию нужно писать и дату (в комментариях к своим допискам).

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

(137) Друже, у нас 5 филиалов-фирм, в Справочнике фирм запрещено менять коды. Справочник содержит ок.150 фирм. Как к этим 5 обращаться, если они в разных группах, а справочник огромный?
И обращение к ним во многих местах. Как вариант - Счета по фирмам и дефицит Товаров по ним, с указанием, сколько заказано, сколько товара в пути, сколько на складах этих фирм..
Справочник Склады - отдельный, без связи с фирмами.

Использую "НайтиПоКоду()", поскольку другой способ затратный, к примеру новый справочник связки "Фирмы-Склады" утяжелит и без того чугунную базу.

И не трудно было догадаться, что кому-нибудь, да захочется поменять коды в Справочниках. Форточку для замены кодов оставил для себя, для остальных - только чтение.

ЗЫ. Если долговременной памяти на короткие переменные у читающего хватает только на одну строчку, а на второй он забывает, о чем шла речь - надо гнать проверяющего, а не прогера.

ЗЫЫ - вот комменты надо заставить писать, однозначно.

(145) Судя по наблюдениям, особо в комментарии начинают читать только когда хотят избавится от человека. Ища повод :)
Так же комментарии для кодеров особо не важны, т.е. на них много внимания не заостряют. Радостно, если вашу функцию где либо вставили :)

...Печально, когда вашу функцию использую с вашим же комментарием... ;)

А никто не обратил внимание на то, почему при требовании в "150 символов" приведенный код занимает 796 символов?

+(149) >>>Печально, когда вашу функцию использую

Использую в своем коде и при этом возникает ошибка, как правило сразу ищут комментарии, и как говорится разобрались, наказали кого попало :)

(150) Даже не стал вникать. И так видно, что код липовый. :)

В рамках консолидации холдинга и приведения к единому виду каких-либо справочников НайтиПоКоду() выйдет очень сильно боком.
Использование Константы самое разумное решение.

(146) + номер телефона и, очень желательно, домашний адрес

(148) конторе надо предоставить стандарт: код оформлять так, комментарии писать так и так. Предъявить его соискателю и спросить "Справишься", а потом предъявлять претензии, чт оде ко не соответствует.

(150) скорее всего код не вмещается на экран большого начальника (типа там маленький нет бук и читать плохо - отсюда требование: строки делать не более 150 символов)

(153) в рамках консолидации холдинга ищи по полному коду, а гроздья констант, в которых сам запутаешься сомнительное решение

(153) Мда.. "КонстантЫ"..
Стало-быть, мне надо было 5 констант для фирм, 10 констант для Складов на этих фирмах?
Итого 15 новых констант..
Интересный способ..

(155) + мало 15-ти-то.. Мне нужно еще Константы для связи Фирмы и пары складов с ними.. Интересно, кто-нть предложит способ завалить константами базу, чтобы решить задачу логистов из (148)?

(140) Я думаю ошибаетесь как раз вы.
Работодатель, заказчик ставит задачу, и ее надо выполнить.
Иногда полезно посмотреть дальше, и сделать получше.
Но это иногда, если есть такая возможность.

Если вы пригласили специалиста, и дали задачу написать код, четко обозначили задачу - он должен написать.
Если он видит что код по ТЗ способен принести проблемы в будущем - ну неплохо предупредить об этом заказчика.
Но написать надо именно по ТЗ.

(154) (155) Если проявить должное усилие можно изучить механизм ЗначениеСвойствОбъектов
На ИС есть описание использования аналогичного механизма для констант
Вместо громоздей используете иерархический вид, типа
Справоники
--Номенклатура
----КакаяТоВажнаяПозиция (Код"ВажнаяПозиция", тип значения Номенклатура)

(141) Мне вообще нет дела до того кто будет сопровождать код.
До этого может быть дело заказчику, тогда он предъявляет особые требования к коду.

(158) У меня 7.7. Можно, конечно, поуродовать справочники, и решить задачу без поиска по коду. Например, запоминать склады в СЗ - переменной в спр.Фирм. Была такая идея. Есть кому запретить непонятные оному изменения. Потому и сделано, что сделано: запрет изменения кодов.

(158) А вы вкурсе, что ваше свойство значения работает из ряда вон медленней, нежели иметь 100 простых реквизитов? :)

По месту на диске это не затратнее, чем свойство ;)

(158) а где константы и как сей механизм применим к ним?
(157) универсального решения этой задачи нет (выполнения неких действий именно для некоторых записей справочника, причём для разных записей могут выполняться разные действия). Прямой способ НайтиПоКоду - и делаем, что нужно (относительно универсальный). Остальные решения только в рамках конкретных условий.

+(158) По поводу констант. Их лучше не использовать, часто. Транзакции возникают. Даже в 8.ххх 1С с константами перемудрило :)

Ну, дела.. г-код, однозначно..
http://i.paste.pics/1dd7f4f4595ecdd9e514e1f3aaede399.png

И ведь, зараза такая, работает-же! :)
http://i.paste.pics/41ba3e778c44fa7ca06578da1faed4e9.png

вроде не пятница )))
ЗЫ вы б  еще код после закирпичевателя привели и спросили ))))

Ветка утоплена, глаза не мозолит.

(166) Есть тн Веблюжая нотация.
тзСоответствия (тз - тип значений ТаблицаЗначений, Соответствия - краткое описание что там внутри)

(167) Польская нотация