Бухгалтерия / Говнокод #5983 Ссылка на оригинал

0

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

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

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

Запостил: ingenuus ingenuus, (Updated )

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

  • ava da4ever # Ссылка на оригинал 0

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

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

  • ava istem # Ссылка на оригинал 0 

    $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 da4ever # Ссылка на оригинал 0 ⇈

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

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

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

  • ava sergeante # Ссылка на оригинал 0

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

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

Где здесь C++, guest?!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓