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

    +115

    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
    public bool checkUser(UserModel model = null)
{
	check:
	if(model != null)
		if(model.authenticated != false)
			if(model._id > 0)
				return true;
			else
				goto check;
		else
			goto check;
	else
		goto check;
	return false;
}

    а вдруг?!

    Запостил: d3n4, 19 Декабря 2012

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

    • ava krypt 19.12.2012 01:47 # +1

      Жесть какая
      Ответить

    • ava krypt 19.12.2012 06:08 # +2

      Могу предположить, что модель работает в другом потоке и это такая "синхронизация"... Но goto в сравнении с nil...
      Ответить

      • ava Vindicar 19.12.2012 09:02 # +4

        А даже если и в другом потоке - как другой поток изменит значение model если model==null? Просто мина замедленного действия...
        Ответить

    • ava bormand 19.12.2012 06:34 # +8

      Метод, который просто не может закончиться неудачно.
      Ответить

    • ava vladthesparrow 19.12.2012 09:02 # +1

      Смеялся долго, хороший юморист у вас работает.
      Ответить

    • ava LispGovno 19.12.2012 09:58 # +2

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

      • ava bormand 19.12.2012 12:54 # +3

        Если authenticated и _id еще могут измениться, то сам model - никогда, и цикл внезапно может оказаться вечным...
        Ответить

      • ava TarasB 19.12.2012 13:39 # +1

        Кстати, это не из эклипса код?
        Ответить

    • ava RusAlex 19.12.2012 14:37 # 0

      спасибо. с таким кодом , точно и муха не пролетит.
      Ответить

    • ava TarasB 19.12.2012 14:46 # 0

      Было бы неплохо ещё обернуть трайкатчами пожирнее.
      Ответить

      • ava LispGovno 19.12.2012 22:39 # 0

        Зачем, чтобы ловить NullReferenceExeption?
        Ответить

        • ava d3n4 19.12.2012 23:45 # 0

          да нет, просто на всякий случай, а вдруг что.
          Ответить

      • ava bormand 20.12.2012 06:25 # +1

        try { код из топика } catch (Exception e) { goto check; } ?
        Ответить

        • ava TarasB 20.12.2012 09:36 # +1

          Во, типа того. Это компилируется вообще, кстати?
          Ответить

    • ava expl 29.01.2013 21:05 # 0

      Более читабельная версия, по идее, выглядит так:
      public bool checkUser(UserModel model = null)
{
	while(true) 
	    if(model != null && model.authenticated && model._id > 0)
                 return true;
	return false;
}
      Ответить

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