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

    −110

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    СписокЗначенийТиповЦен = Новый СписокЗначений;
...
КоличествоЦен = 0;
РезультатВыбора = ФормаВыбораТиповЦен.ОткрытьМодально();
Для Каждого Элемент Из РезультатВыбора Цикл 
	СписокЗначенийТиповЦен.Добавить(Элемент.Значение);
	КоличествоЦен = КоличествоЦен + 1;
КонецЦикла;

Если КоличествоЦен = 0 Тогда
	Предупреждение("Не выбран ни один тип цен!!!");	
	Возврат;
КонецЕсли;

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

    Запостил: ingenuus, 14 Марта 2011

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

    • ava da4ever 14.03.2011 21:51 # 0

      программист с двадцатилетним опытом детектед

      императивочка во все поля
      Ответить

    • ava istem 15.03.2011 01:31 # −1 

      $listPriceTypeValues = new listValues();
...
$quantityPrice = 0;
$choices = $formSelectTypesOfPrices->viewModal ();
foreach ( $choices as $element) {
	$listPriceTypeValues->add($element);
	quantityPrice =  quantityPrice + 1;
}

If ( $quantityPrice = 0) {
	Warning ("Do not select any type of price !!!");
	echo "Do not select any type of price !!!";
	return;
}

      ...ни чё не напоминает?..
      Ответить

      • ava istem 15.03.2011 01:44 # −1

        Кстати, да... http://govnokod.ru/5974#comment78028...
        Ответить

      • ava da4ever 15.03.2011 01:55 # 0

        напоминает неудачную шутку.

        в одноэс какбы есть некоторое количество встроенных объектов с известными методами. говнокод в том, что вместо двух строчек:
        РезультатВыбора = ФормаВыбораТиповЦен.ОткрытьМодально();
если РезультатВыбора.Количество() < 1 тогда Die("I hate you");

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

    • ava sergeante 31.03.2011 17:19 # 0

      Из всего кода здесь нужно оставить только строку 4. Всё.
      Ответить

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