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

    +2

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if (_viewModel.ShowColumnChooser && _viewModel.UseDefaultColumnChooser)
    //...
else {
    if (!_viewModel.ShowColumnChooser)
        return;
    //...
}

    Сэмпл платного фреймворка Syncfusion. Да, на этом коде они объясняют, как использовать их классы. А в поддержке отвечают сплошные косноязычные Раджеши Кутрапалли.

    Запостил: gogishvilli001, 03 Апреля 2016

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

    • ava kerman 04.04.2016 00:30 # +2

      Индусы, бля.
      Ебибедские скобочки, подчёркивание в приватных членах (вот за это отхвачу), хотя стоп... 
      if (a &&b) {
...}
else (!a) {
...
}
      Ответить

      • ava gogishvilli001 04.04.2016 00:39 # 0

        Вы имели ввиду 'else if(!a)'?
        Ответить

        • ava kerman 04.04.2016 00:49 # 0

          Ога.
          Тут упрощается до 
          if (a && b) {
...}
else if (b) {
...
}
          Ответить

          • ava tucvbif 04.04.2016 01:13 # +2

            нет.
            Ответить

            • ava kerman 04.04.2016 01:30 # 0

              Таки да, не упрощается. Ну и хуй с ним.
              Ответить

              • ava tucvbif 04.04.2016 01:35 # 0

                А оригинал упрощается до
                if (!_viewModel.ShowColumnChooser)
        return;
else if (_viewModel.UseDefaultColumnChooser)
    //...
else {
    //...
}
                Ответить

                • ava kerman 04.04.2016 01:51 # 0

                  else лишний. А вообще такое упрощение ещё спорно. Может родное тоже было ничего )
                  Ответить

                  • ava tucvbif 04.04.2016 01:52 # 0

                    какой из?
                    Ответить

                    • ava kerman 04.04.2016 01:56 # 0

                      А тебе решарпер не подсвечивает сереньким? Я в студию не загонял, но else после return уже рвотные позывы вызывает.
                      Ответить

                      • ava tucvbif 04.04.2016 02:24 # 0

                        Точно, затупил.
                        Ответить

                      • ava nihau 04.04.2016 12:19 # +4

                        Мы из поколения программистов выращенных продуктами джетбрейнс (решарпером), поможет ли другой продукт джетбрейнс в решении наших проблем?
                        Ответить

          • ava gogishvilli001 04.04.2016 01:25 # −3

            Вы и в первом случае неправильно прочли.
            if (a &&b) {statement1}
            else {
            if(!a)return
            statement2
            }
            трансформируется в
            if (a &&b) {statement1}
            else if (a){statement2}
            затем в
            if (a){
            if (b){statement1}
            else{statement2}
            }
            либо
            if(!a)return
            if (b){statement1}
            else{statement2}
            либо
            if(!a)return
            if (!b){
            statement2
            return
            }
            statement1
            Ответить

            • ava kerman 04.04.2016 01:50 # +2

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

              • ava gogishvilli001 04.04.2016 11:48 # 0

                Только ты уж расскажи, как тут отступы делать. А то пробелы оно убирает, возможности редакции нет. Табуляции использовать?
                Ответить

                • ava Vasiliy 04.04.2016 12:02 # +4

                  отступы только для платных аккаунтов. Для получения платного акаунта отправь СМС со словом Аккаунт на номер 7880. Если не поможет. Отправь туда же номер банковской карты и CCV
                  Ответить

                  • ava gogishvilli001 06.04.2016 01:03 # 0

                    "Отправил деньги по е-мэйлу". Теперь скажи, как делать отступы, ведь у меня платный аккаунт.
                    Ответить

                    • ava Vasiliy 06.04.2016 01:30 # 0

                      банк не потвердил перевод. отправь номер карты ccv Юскан паспорта , девичью фамилию матери,
                      кличку собаки, любимое блюдо. на емейл что бы банк пошевелился
                      Ответить

                    • ava 1024-- 06.04.2016 10:27 # +1

                      Спасибо, деньги поступили в банк (тут, чтобы не убить сервера, деньги идут либо в мой банк, либо в банк Василия. Масштабирование и распределение нагрузки, все дела).
                      Мы добавили Вам следующий элемент интерфейса рядом с полем ввода комментария:
                      А не использовать ли нам bbcode?
                      Ответить

                      • ava Vasiliy 06.04.2016 10:36 # 0

                        Как то странно нагрузка распределяется в мой банк ни разу ничего не пришло. Нагрузка наверное на мой банк большая.
                        Ответить

              • ava gogishvilli001 04.04.2016 13:40 # 0

                if(!a)return
                if (!b){
                statement2
                return
                }
                statement1
                Ответить

    • ava Elvenfighter 04.04.2016 16:18 # 0

      Если бы не было return, то нормальный код.
      А так, тут в принципе return стоило бы вынести из if:
      if (! _viewModel.ShowColumnChooser) 
    return;
if (_viewModel.UseDefaultColumnChooser) {
    // ...
} else {
   // ...
}

      ваш кеп
      Ответить

      • ava gogishvilli001 04.04.2016 16:44 # 0

        В том-то и дело. Хотя я привык делать ещё один return, как в комментарии выше.
        Ответить

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