Говнокод #9967 — C# — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

C# / Говнокод #9967

+113

for (int i = 0; i < retries; i++)
{
    if ((GetProductVersioningProvider() as IQueueProductVersion).IsVersionCommitted(version))
    {
        return true;
    }
    else
    {
        Thread.Sleep(checkDelay);
    }
}

Классика жанра же.

Запостил:

abatishchev, 17 Апреля 2012

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

absolut 17.04.2012 15:46 # +6

В любой непонятной ситуации ложитесь спать
Ответить
- SmackMyBitchUp 17.04.2012 17:56 # +1
  
  Жизненно. Спать лучше, нежели мозги ебать.
  Ответить
- rat4 17.04.2012 18:07 # 0
  
  В непонятной ситуации, если вы под веществами - делайте вид, что всё нормально; иначе - ложитесь спать.
  Ответить
  - abatishchev 17.04.2012 18:19 # 0
    if (blood.Contains(drugs)) throw new UnderDrugsException());
    Ответить
    - SmackMyBitchUp 17.04.2012 18:33 # 0
      
      скобак даухухухуууя
      Ответить
      - rat4 17.04.2012 18:42 # +3
        
        (if (contains blood drugs) (throw (new UnderDrugsException)))
        
        Так лучше?
        Ответить
        
        roman-kashitsyn 17.04.2012 20:55 # 0
        
        MUCH better.
        Ответить
        
        rat4 17.04.2012 20:59 # 0
        
        Я правильно усвоил скобочки, или это говнокод?
        Ответить
        
        FaiGovno 17.04.2012 21:27 # 0
        
        то был грустный смайлик начинаеющего микрософт проффесиоанала
        Ответить
        
        roman-kashitsyn 18.04.2012 09:54 # +1
        
        оно даже почти (лень писать генератор класса для кастомного исключения) работает:
        
        http://ideone.com/mwFBx
        Ответить
        
        wvxvw 18.04.2012 16:31 # +1
        
        Ну только в Лиспе традиционно работа с ошибками совсем другая. И код бы скорее всего выглядел как-то так:
        
        (define-condition fuzzy-error (error) ()) (define-condition i-dont-know (fuzzy-error) ((is-it-dark :initarg :is-it-dark :reader i-dont-know-is-it-dark) (are-you-blind :initarg :are-you-blind :reader i-dont-know-are-you-blind) (what-did-you-take :initarg :what-did-you-take :reader i-dont-know-what-did-you-take)) (:report (lambda (condition stream) (format stream "The suspect says:~& is it dark? - ~S, is he blind? - ~S, what did he take? - ~S" (i-dont-know-is-it-dark condition) (i-dont-know-are-you-blind condition) (i-dont-know-what-did-you-take condition))))) (defun see-whos-there () (labels ((ask () (format t "Who's there? [Y/n]~&") (finish-output t) (eval (read)))) (restart-case (error 'i-dont-know :is-it-dark nil :are-you-blind nil :what-did-you-take 'just-a-little-bit) (nobody (answer) :report "Can see no one!" :interactive ask (declare (ignore answer))) (dead-people (answer) :report "They are coming for me!" :interactive ask (declare (ignore answer))) (rainbow-unicorn (answer) :report "I see trees of green, red roses too I see them bloom for me and you And I think to myself what a wonderful world." :interactive ask (declare (ignore answer))) (recursive (answer) :report "I see me seeing you seeing me seeing you" :interactive ask (declare (ignore answer)))))) (see-whos-there)
        Ответить
        
        roman-kashitsyn 18.04.2012 16:39 # 0
        
        Это CL живёт с сигнальным протоколом, в Clojure используется модель исключений Java.
        Ответить
        
        wvxvw 18.04.2012 17:21 # 0
        
        Меня больше в этом смысле смущает то, что в ПХП очевидно есть метки, типа как в Бейсике, которые начинаются с двоеточия. Иначе я не могу придумать оправдания такой боевой раскраске кода :)
        
        EDIT: Ох, так они и взаправду существуют! http://php.net/manual/en/control-structures.goto.php Вот чего не знал... а оказывается! Ток двоеточие с неправильной стороны.
        Ответить
        
        wvxvw 18.04.2012 17:26 # +1
        
        Ой, там еще самое замечательное то, что его добавили в пятой версии! Раньше его не было.
        Ответить
        
        roman-kashitsyn 18.04.2012 18:28 # 0
        
        я пишу обычно [ code=lisp ] с рассчётом на то, что когда-нибудь поддержку запилят.
        Ответить
        
        wvxvw 18.04.2012 22:57 # 0
        
        В Гейши ж есть подсветка для Лиспа. Я как-то давно случайно закоммитил что-то на Лиспе в репозитори конторы, где тогда работал. Там СВН был подключен с каким-то модулем для "красивой" раскраски исходников, который эту самую Гейши использовал, так помнится, нормально покрасил. Хотя, хз, давно это было...
        Ответить
    - FaiGovno 17.04.2012 19:43 # +5
      
      AsParallel()
      Ответить
      - absolut 17.04.2012 20:35 # +5
        
        AsOrthogonal();
        Ответить
3.14159265 17.04.2012 18:00 # +6

>Классика жанра же.
Нет LINQ, штоле?
Ответить
TarasB 17.04.2012 20:41 # +6

Thread.AsParallel.Sleep(checkDelay);
Ответить
- absolut 18.04.2012 08:20 # 0
  
  Справедливо. Всем сестрам по серьгам.
  Ответить

Говнокод: по колено в коде.

C# / Говнокод #9967

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

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