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

    +113

    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
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    public partial class EditorForm : Form
{
	private List<IEditObject> _readOnlyObjects;
	public IEditObject EditValue {get; set;}
	
	public EditorForm(IEditObject value, List<IEditObject> readOnlyObjects)
	{
		if (value != null)
		{
			EditValue = value;
		}
		
		if (readOnlyObjects != null && readOnlyObjects.Count > 0)
		{
			_readOnlyObjects = readOnlyObjects;
		}
	}
}

    Во избежание лишних присвоений.

    Запостил: ICELedyanoj, 23 Февраля 2012

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

    • ava absolut 24.02.2012 08:55 # +2

      Несмотря на проверки, если вызвать EditorForm(null, null) внутри будут null.
      Ответить

      • ava ICELedyanoj 24.02.2012 09:21 # 0

        Ну так именно это меня и повеселило.
        Это же не какой-нибудь метод Init, а конструктор.
        Ответить

        • ava absolut 24.02.2012 09:24 # 0

          надо кидать ArgumentNullException
          Ответить

          • ava martin 25.02.2012 13:00 # 0

            возможно, что дальнейшую логику наличие там null'ов не смущает и тогда кидатся лишний раз исключениями смысла нет.
            Ответить

            • ava absolut 25.02.2012 13:59 # 0

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

              • ava ICELedyanoj 27.02.2012 09:48 # 0

                Нет. Уже больше месяца работаю над расшифровкой тонн кода, доставшегося после этого автора в наследство, и могу смело утверждать, что это просто такой авторский стиль.
                Чем больше проверок и трайкотчев - тем надежнее - так он считал.
                Доводилось видеть методы, состоящие из одной строчки - подпись на событие UIшного элемента, заключенной в трайкотч, в обработчике идет просто запись в лог. И таких было немало.
                Просто больная фантазия, не ищите смысла :)
                Ответить

                • ava absolut 27.02.2012 10:41 # 0

                  Так ведь я и говорил не про смысл. А про смущение.
                  Ответить

    • ava guest8 09.04.2019 12:44 # −999

      показать все, что скрытоvanished
      Ответить

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