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

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

+129
1. 1
2. 2
3. 3
4. 4
5. 5
6. 6
7. 7
8. 8
```
if (enabled)
                {
                    tabPanel.Tabs[FilesTabKey].Enabled = filesAndDocumentsForm.HaveReadPermission(EstateId);
                }
                else
                {
                    tabPanel.Tabs[FilesTabKey].Enabled = false;
                }
```
Запостил: zxxc, 26 Июля 2014

Tweet
Комментарии (69) RSS
- zxxc 26.07.2014 09:29 # 0
  
  посмотрел filesAndDocumentsForm.HaveReadPermission (EstateId);
  это мегатяжелая фигня, видимо человек боялся что
  enabled && filesAndDocumentsForm.HaveReadPermission (EstateId)
  приведет к запуску filesAndDocumentsForm.HaveReadPermission
  Ответить
  - guest 26.07.2014 14:43 # −1
    
    В сишке ксть такая забавная штука: `while (i++ < 200);` является одновременно и валидной конструкцией, и высером ебанутого автора-мудака. Поэтому чтобы подчеркнуть, что они не ебанутые, авторы зачастую прибегают к следующему форматированию:
    
    while (i++ < 200) { ; }
    
    Что подчеркивает, что данный блок был вставлен сюда специально, и не смей, сукаблять, его трогать.
    Так и здесь, у меня есть подозрение, что автор либо хотел подчеркнуть самую массивную часть методи, либо просто не любит короткую нотацию и тернарные операторы.
    Ответить
    - bormand 26.07.2014 14:58 # +4
      
      // как это видел программист while (i++ < 200) { ; } // как это видит оптимизатор крестов i = 201
      Ответить
      
      guest 26.07.2014 15:27 # −1
      
      увборманд, вы же понимаете, в while может оказаться любая хуйня, включая вызовы функций или, например, поиск конца строки, это пример для наглядности
      Ответить
      
      kegdan 26.07.2014 16:33 # +1
      
      И это хорошо. Еще бы был анализатор кода, который бы выделял полное говно в коде. Было бы весело.
      Кстати есть ли обучалки, которые дают задачу, а потом ставят твоему коду оценки типа - скорость, изящность, красота? И взрывают комп, если оценка дно-дно-дно
      Ответить
      
      bormand 26.07.2014 19:28 # 0
      
      code hunt или как там его звали. Если хочется покачать LINQ - само то.
      Ответить
      
      kegdan 26.07.2014 19:30 # 0
      
      Там и жаба есть. Милая вещь
      Ответить
      
      bormand 26.07.2014 19:32 # 0
      
      Жаба там в неравных условиях. Ее запилили через жопу трансляцию в шарп. И там, емнип, нету кучи классов. 3.14159265, помнится, на это ругался.
      
      Так что юзабельно только для изучения шарпика.
      Ответить
      
      kegdan 26.07.2014 19:37 # 0
      
      ну так она не для опытных дядь, а для новичков. Для них - самое то. Мне так кажется
      Ответить
      
      kegdan 26.07.2014 20:52 # 0
      
      Что мне там не нравится, так то, что без линка даже с помощь байтоебства нельзя получить 3 балла
      Ответить
      
      bormand 28.07.2014 05:27 # +2
      
      Дык этот сайт как раз и запилен для того, чтобы показать как жаба отсасывает у шарпика с его LINQ... Реклама ;)
      Ответить
      
      kegdan 28.07.2014 06:08 # +1
      
      линк можно самому написать. с нормальными именами map filter и тд там ведь даже никакой оптимизации нет
      Ответить
      
      3.14159265 28.07.2014 13:26 # 0
      
      >то без линка даже с помощь байтоебства нельзя получить 3 балла
      Я не знаю, как тут вообще можно его применить
      6.12 = return s+new StringBuffer(s).reverse(); //1 кирпич
      7.01 = return b+c+a+a+c+b; //2 кирпича
      12.12 = java.util.Collections.sort(a); return a; //1 кирпич
      Тупо однострочнки.
      Ответить
      
      kegdan 28.07.2014 13:39 # 0
      
      >>Я не знаю, как тут вообще можно его применить
      В жабе то?)
      Как тебе ебанутые уровни в стиле - "угадай формулу!"?
      Я периодически захожу, пишу на 3 плюшки пару заданий. Иногда довольно много времени уходит на понять какой путь хотели авторы сего дива
      Ответить
      
      3.14159265 28.07.2014 13:46 # 0
      
      >В жабе то?)
      В шарпе. Как применить LINQ к такой вот задаче:
      7.01 = return b+c+a+a+c+b; если есть простое и лаконичное решение.
      Ответить
      
      kegdan 28.07.2014 14:04 # 0
      
      Видимо как то так
      https://ideone.com/RmZZal
      Ответить
      
      TauSigma 28.07.2014 15:08 # 0
      
      И хардкором без линкю :)
      
      String[] arr = new String[] { "a", "b", "c", }; StringBuilder result = new StringBuilder(); result.Append(String.Join(String.Empty, arr)); Array.Reverse(arr); result.Append(String.Join(String.Empty, arr)); Console.WriteLine(result.ToString());
      Ответить
      
      kegdan 28.07.2014 15:49 # 0
      
      result.Append(String.Join(String.Empty, arr));
      
      знатное говнецо
      Ответить
      
      bormand 28.07.2014 16:23 # 0
      
      А стрингбилдер случаем не умеет кушать Enumerabl'ы?
      Ответить
      
      kegdan 28.07.2014 16:26 # 0
      
      ноуп
      Ответить
      
      TauSigma 28.07.2014 16:59 # 0
      
      От чего-же?
      Array.Reverse использует нативный код.
      
      // System.Array [ReliabilityContract(Consistency.MayCorruptInstance, Cer.MayFail), SecurityCritical] [MethodImpl(MethodImplOptions.InternalCall)] private static extern bool TrySZReverse(Array array, int index, int count);
      
      String.Join манипулирует указателями и так-же, через нативный код:
      
      // string [SecurityCritical] [MethodImpl(MethodImplOptions.InternalCall)] internal static extern string FastAllocateString(int length);
      
      + UnSafeCharBuffer
      Ответить
      
      kegdan 28.07.2014 17:05 # 0
      
      https://ideone.com/VOsKv2
      Ответить
      
      TauSigma 28.07.2014 17:09 # 0
      
      Так производительность не замеряют:
      http://ideone.com/SP3CRy
      Ответить
      
      defecate-plusplus 28.07.2014 17:34 # 0
      
      а так?
      http://ideone.com/gm7yzn
      Ответить
      
      kegdan 28.07.2014 17:38 # 0
      
      сорри, попутал поля
      
      http://ideone.com/ee6QgC
      
      Действительно выходит, что метод тауСигмы чуууууть-чуть быстрее
      Ответить
      
      bormand 28.07.2014 17:47 # 0
      
      > чуууууть-чуть быстрее
      Пример не маловат ли для бенча? :) Всего полсекунды на оба теста...
      Ответить
      
      kegdan 28.07.2014 17:49 # 0
      
      если мало 1048576 строк - добавь к цифирке в цикле чутка
      Ответить
      
      defecate-plusplus 28.07.2014 17:50 # +2
      
      если ты не голубой
      дорисуй вагон другой
      Ответить
      
      bormand 28.07.2014 17:50 # 0
      
      > добавь к цифирке в цикле чутка
      На 22 еще пашет, дальше уже падать по таймауту начинает. Разница процентов на 10 есть, да.
      Ответить
      
      kegdan 28.07.2014 17:51 # 0
      
      Туплю. щас процент найду
      Ответить
      
      TauSigma 28.07.2014 17:58 # 0
      
      http://ideone.com/JAspBd
      Так проще будет.
      Странно, а у меня на винде форыч быстрее получается...
      Ответить
      
      bormand 28.07.2014 18:02 # 0
      
      > на винде форыч быстрее получается
      Намного?
      Ответить
      
      TauSigma 28.07.2014 18:06 # 0
      
      >Намного?
      
      Форыч: Elapsed: 00.0191 s, MemDelta: 10276.39 KB, GC count: 2 Join: Elapsed: 00.0315 s, MemDelta: 16384.07 KB, GC count: 1
      
      Хотя у меня машина засрана всякими кошмарскими и остальными богомерскими штуками...
      Или в моне не грамотно реализовали String.Join(...).
      Ответить
      
      kegdan 28.07.2014 18:21 # 0
      
      Эх, Форыч, Форыч.
      Форыч Форыч Форыч - это Фио
      Ответить
      
      Abbath 28.07.2014 23:20 # 0
      
      фио с двумя равными кругами
      Ответить
      
      kegdan 28.07.2014 23:25 # +1
      
      я НИХУЯ НЕ ПОНЯЛ
      Ответить
      
      Abbath 29.07.2014 00:06 # 0
      
      что такое 8? 8 - это цифра
      цифра с двумя ровными кругами
      8 на боку бывает тупо бесконечность
      8, что же завтра будет с нами?
      Ответить
      
      kegdan 29.07.2014 06:50 # 0
      
      Восемь! Восемь! Восемь и три!
      Восемь! Восемь ты набери!
      Там ответит дядя Равшан!
      Такси - жигуль-баклажан!
      Ответить
      
      kegdan 28.07.2014 18:04 # 0
      
      http://ideone.com/dLFyxG
      
      в 2 раза.
      Хуяcе я перегрелся на солнышке
      Ответить
      
      kegdan 28.07.2014 13:45 # 0
      
      Например возведение в степень. Я написал классический быстрый алгоритм - 2 плашки. Думаю - может линк хотят - нет, тоже 2 плашки. Цикл - 2 плашки. А рекурсия - 3! И надпись "вы написали элегантный код!"
      Ответить
      
      3.14159265 28.07.2014 13:47 # 0
      
      >Например возведение в степень.
      Зона/Номер?
      Ответить
      
      kegdan 28.07.2014 13:53 # 0
      
      3.01
      Ответить
      
      3.14159265 28.07.2014 18:53 # +1
      
      >классический быстрый алгоритм - 2 плашки
      >Цикл - 2 плашки. А рекурсия - 3!
      Нахера циклы? Используй классический быстрый FPU:
      return (int)((long)Math.pow(number,power));
      Только там задание ебанутое - надо повторить ублюдочное переполнение лонга в int. Отсюда и дурацкие касты.
      Ответить
      
      kegdan 28.07.2014 18:56 # 0
      
      Да я уже все подряд юзал. Было интересно на чем он сдастся
      Ответить
      
      kegdan 28.07.2014 19:15 # 0
      
      ты 6.03 сделал на 3 плашки?
      Ответить
      
      3.14159265 28.07.2014 19:51 # 0
      
      >ты 6.03 сделал на 3 плашки?
      Угу.
      
      public static String Puzzle(String s) { String r=""; for (int i = 1; i < s.length(); ++i){ char ch=s.charAt(i-1); if (' '==s.charAt(i)) ch = Character.toUpperCase(ch); r+=ch; } r+=Character.toUpperCase( s.charAt(s.length()-1) ); return r; }
      
      Как ни странно многие решения циклами высираются на три кирпича.
      Ответить
      
      kegdan 28.07.2014 19:56 # 0
      
      Я сейчас писал почти тоже самое - 2 кирпича . Что за хуйня? Мне кажется его придумали сатанисты
      Ответить
      
      3.14159265 28.07.2014 19:57 # +1
      
      >Что за хуйня?
      У меня была точно такая же реакция когда я проходил CODEHURT.
      
      Впрочем уровни с шифрованием мне понравились.
      Ответить
      
      kegdan 28.07.2014 20:04 # 0
      
      http://download.hdd.tomsk.ru/preview/voejptak.jpg
      
      ЧЯДНТ?!
      
      var output=""; for (int i = 1; i < s.Length; ++i) output+=' '==s[i]?Char.ToUpper(s[i-1]):s[i-1]; return output+char.ToUpper( s[s.Length-1] );
      Ответить
      
      3.14159265 29.07.2014 13:26 # 0
      
      Хм. Вроде идентичная херь. Может оно для шарпа по-другому считает кирпичи?
      Ответить
      
      kegdan 29.07.2014 13:28 # 0
      
      плохо представляю как сделать лучше
      Ответить
      
      3.14159265 29.07.2014 13:57 # 0
      
      LINQ. Сделает ваш код лучше.
      Ответить
      
      3.14159265 29.07.2014 14:00 # 0
      
      >плохо представляю как сделать лучше
      http://govnokod.ru/16428#comment241295
      У меня этот код выдает 3 кирпича. Что я делаю так?
      
      Ответить
      
      kegdan 29.07.2014 14:44 # 0
      
      Я уже все возможные прыжки с линком сделал, я тупо не могу придумать способ. Заебало. Я уверен, что на 3 балла надо сделать какой то ебанутый неоптимальный прыжок в сторону, но мне уже похуй. Гори в аду code hunt
      Ответить
      
      bormand 29.07.2014 14:50 # +3
      
      > но мне уже похуй. Гори в аду code hunt
      Да у тебя же CODE HURT!
      Ответить
      
      kegdan 29.07.2014 14:51 # 0
      
      не то слово. У меня пена фонтанирует.
      Ответить
      
      kegdan 28.07.2014 19:45 # +1
      
      06.09
      return a.Length==b.Length?a+b:a.Length>b.Length ?a:b; - 3 плашки
      
      return a.Length<b.Length?a:a.Length>b.Length?b: a+b; - 2 плашки
      
      В чем, БЛЕААААААТЬ!!!, разница?!!! == быстрее < ?!!)
      Ответить
      
      3.14159265 28.07.2014 19:56 # 0
      
      Лол, у меня три, причем ветка a+b в самом конце
      int al=a.length(),bl=b.length(); if (al>bl) return a; if (bl>al) return b; return a+b;
      Ответить
      
      TarasB 28.07.2014 20:14 # 0
      
      разница в порядке обхода ветвей
      Ответить
      
      kegdan 28.07.2014 20:15 # 0
      
      А в чем принципиальная разница?
      Ответить
      
      3.14159265 28.07.2014 13:20 # 0
      
      >Так что юзабельно только для изучения шарпика.
      Я её почти всю на жабе прошёл. Вспомнил время лаб, т.к. много фич из стандартной библиотеки не запилили - велосипедил всё сам.
      Только циклы, только массивы, только хардкор.
      Ответить
    - kegdan 26.07.2014 16:21 # 0
      
      >>`while (i++ < 200);` является одновременно и валидной конструкцией
      
      пример можно?
      Ответить
      
      bormand 26.07.2014 16:24 # 0
      
      Бессмысленно - да, не спорю. Но чем она невалидна? С точки зрения языка и компилятора там всё норм.
      Ответить
      
      kegdan 26.07.2014 16:26 # 0
      
      Валидна-то - спору нет. но гость сказал
      "В сишке ксть такая забавная штука: `while (i++ < 200);` является одновременно и валидной конструкцией, и высером ебанутого автора-мудака. Поэтому чтобы подчеркнуть, что они не ебанутые"
      В каком месте не ебанутые ?
      Ответить
      
      guest 26.07.2014 20:58 # +4
      
      самый тупой пример на свете (мне лень переписывать нахождение квадратного корня).
      
      while(!matches_required_precision(x = improve_precision(x)));
      Ответить
      
      Xom94ok 26.07.2014 23:16 # 0
      
      Как вычмат-то тяжело вспоминается :)
      http://ideone.com/x6rkHb
      На хабре проскакивал пример, в котором аналогичным циклом коммитили выделенную память.
      ИМХО логику лучше писать в теле цикла.
      Ответить
      
      kegdan 27.07.2014 11:30 # +1
      
      while(!matches_required_precision(x = improve_precision(x)));
      
      нифига не
      
      while (i++ < 200);
      
      короче вы пошутили а я не понял
      Ответить
      
      guest 29.07.2014 05:53 # 0
      
      сами вы пошутили
      
      http://govnokod.ru/16428#comment241109
      Ответить
      
      bormand 29.07.2014 05:54 # +2
      
      Ну так пример можно было и понаглядней выбрать... Вот такой например:
      while (*dst++ = *src++);
      Имхо, отличная иллюстрация для того, о чем ты писал в том комменте.
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

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

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

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