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

    +133.1

    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
    /// <summary>
/// General handler for all buttons
/// </summary>
private void FormButtons_Click(object sender, EventArgs e)
{
    Control control = (Control) sender;

    if (control.Handle == btnCreateInvoices.Handle)
        ExportOrders();
    else if (control.Handle == btnFirstUsageInvoices.Handle)
        ExportFirstUsageInvoices();
    else if (control.Handle == btnImportCustomers.Handle)
        ImportCustomers();
    else if (control.Handle == btnImportProdcuts.Handle)
        ImportProducts();
    else if // и так далее...
}

    WinForms приложение, на все кнопки навешен 1 обработчик события OnClick.
    А внутри вот....

    Запостил: vleschenko, 12 Августа 2009

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

    • ava guest 12.08.2009 12:12 # 0

      Не вижу говнокода. разве что сравнивание по хэндлам - это лишнее...
      Ответить

    • ava guest 12.08.2009 12:14 # −1

      хы опять if / elseif. нет бы свитч
      Ответить

      • ava guest 12.08.2009 12:16 # +1

        в свитч нельзя вставлять неконстанты, учите матчасть!
        Ответить

        • ava guest 12.08.2009 12:24 # −3

          я то матчасть знаю)
          switch (control.Name)
          {
          case "btnCreateInvoices":
          {
          ExportOrders();
          break;
          }
          default:
          break;
          }
          Так сделать религия не позволяет?
          Ответить

          • ava guest 12.08.2009 12:38 # 0

            это больше смахивает на говно год, а Вам if / elseif использовать религия не позволяет? ;)
            Ответить

          • ava megaprogrammer001 12.08.2009 12:40 # 0

            любитель стрингов? это говнокод ++
            Ответить

          • ava guest 12.08.2009 13:01 # +1

            А если изменю название кнопки в дизайнере? в ваш говнохэндлер тоже придется лезть и править...
            Ответить

            • ava guest 12.08.2009 13:31 # 0

              и так и так придется
              Ответить

              • ava Nagg 12.08.2009 14:09 # 0

                в данном примере - не придется. а со строками - придется.
                Ответить

          • ava guest 12.08.2009 15:02 # −3

            бля пиздец сами говнокодят хуже всех индусов вместе взятых но пиздят какие они блять супер кодеры повсеместно

            case ваще говно лучше его никогда не юзать.
            Ответить

    • ava guest 12.08.2009 12:40 # −1

      Гы, а так почему нельзя?
      Control control = (Control) sender;
      switch (control)
      {
      case btnCreateInvoices:
      ExportOrders();
      break;
      ...
      Ответить

    • ava Kuros 12.08.2009 12:44 # 0

      А все же сделать отдельные обработчики не судьба?
      Ответить

      • ava guest 12.08.2009 13:02 # −1

        Если в них будет по одной строчке как в примере - то это имхо ещё больший говнокод
        Ответить

        • ava Kuros 12.08.2009 13:20 # 0

          Пример уж точно не образец для подражания
          Ответить

        • ava Dimarius 12.08.2009 16:12 # 0

          Почему по одной строчке? Там в каждом if вызывается метод. Возможно, код этого метода переедет сразу в обработчик.
          Ответить

    • ava guest 12.08.2009 17:15 # +1

      Отдельные обработчики - стандартный подход Windows Forms.
      А switch/elseif - пережиток оконных процедур winapi
      Ответить

      • ava guest 12.08.2009 20:57 # −1

        Стандартный подход - неверно сказано, скорее подход IDE, предназначенный как раз для быдлокодеров, которым лень код писать.
        Ответить

    • ava guest 12.08.2009 19:06 # 0

      автор кода наверное любит бейсик с его контрол арреями))))
      Ответить

    • ava guest 12.08.2009 22:51 # 0

      Человеки, вообще-то для этих целей надо было создавать разные обработчики событий, а не собирать все в кучу.
      Ответить

    • ava guest 14.08.2009 11:26 # 0

      Ну видимо задалбало человека по всем эти обработчикам туда-сюда скролить, вот он и впихнул всё в одну функцию. Плюс можно контролы делать копи-пасте.
      Ответить

    • ava пъянь 15.08.2009 13:43 # −1

      ваще мысль интересная надо попробовать так пописать ради експеримента.

      все блять тока пиздять что плохо плохо плохо хоть ктоб сказал почему?
      Ответить

    • ava ikovalyov 16.08.2009 16:48 # 0

      разное событие для каждой кнопки и не нужна функция сразу.
      Ответить

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