1. 1C / Говнокод #12748

    −121

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    Если Организация = Справочники.Организации.НайтиПоКоду("650532") Тогда	
    			Сумма2011 = 14271648;
    		ИначеЕсли Организация = Справочники.Организации.НайтиПоКоду("325316") Тогда
    			Сумма2011 = 1195741.30;
    		ИначеЕсли Организация = Справочники.Организации.НайтиПоКоду("670054") Тогда
    			Сумма2011 = 7782829;
    		ИначеЕсли Организация = Справочники.Организации.НайтиПоКоду("650533") Тогда
    			Сумма2011 = 72882366.11;

    Чем-то напоминает int rand() { return 4; };
    Тут не расстреливать, а живьем закапывать надо!

    Запостил: mrbus, 14 Марта 2013

    Комментарии (8) RSS

    • На кол сажать. Как кодят - так и казнить.
      Ответить
    • Я бы притормозил бы расстреливать программиста. Сам работаю за 1С, правда за 7.7, так вот что могу сказать: постановщики с которыми мне приходиться работать не знают даже основ программирования, и когда приходиться с ними спорить на счет поправок в ТЗ, а они упираются рогом, а за частую ТЗ и не бывает. Я сам делал отчет по расчетам с контрагентами так вот нужно было выводить не всех контрагентов базы а только "избранных". Так вот этих "избранных" вместо того чтобы поместить во внешний файл пришлось вводить в код по настаиванию постановщика, а там порядка 30 контрагентов, и теперь при добавлении нового приходиться лезть в обработку и добавлять в код. Просто бред.
      Ответить
      • Поэтому мы не знаем о структурах данных, а захардкоживаем лесенкой. Всё правильно сделали.
        Ответить
      • Для этого и придумали метаданные.
        Справочники, регистры сведений.
        Я выход нашел в том, что сделал регистр сведений и там храню все эти "особые значения", где им и место.

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

        Лет 18 назад я тоже начинал так программировать (это был бейсик и я пытался вывести картинку на экран). Так я и хардкодил каждый пиксель. Это - глупо.
        Ответить
      • Таких постановщиков - на кол! Не соглашается на нормальное решение? К директору! Объяснить, что каждый такой "чих" по добавлению стоить N-дцать бакинских... И так каждый раз! Заодно и сказать, что можно сделать так, что программер не будет нужен для таких ситуаций.
        Ответить
      • бред то, чем ТЫ занимаешься.. уже сколько не здравых клиентов попадалось, но никогда не писал цифры в код!
        Ответить
      • 0) постановщик вообще не должен лезть в реализацию. его дело - проверить, что всё работает как надо.
        1) про хардкод данных - всё сказано.
        Ответить
    • Я бы притормозил бы расстреливать программиста. Сам работаю за 1С, правда за 7.7, так вот что могу сказать: постановщики с которыми мне приходиться работать не знают даже основ программирования, и когда приходиться с ними спорить на счет поправок в ТЗ, а они упираются рогом, а за частую ТЗ и не бывает. Я сам делал отчет по расчетам с контрагентами так вот нужно было выводить не всех контрагентов базы а только "избранных". Так вот этих "избранных" вместо того чтобы поместить во внешний файл пришлось вводить в код по настаиванию постановщика, а там порядка 30 контрагентов, и теперь при добавлении нового приходиться лезть в обработку и добавлять в код. Просто бред.
      Ответить

    Добавить комментарий