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

    +120

    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
    18. 18
    19. 19
    protected void GridView1_PreRender(object sender, EventArgs e)
    {         
         System.Drawing.Color OddGridColor = new System.Drawing.Color();
         OddGridColor = System.Drawing.Color.FromArgb(0xD3, 0xDE, 0xEF);
         System.Drawing.Color EvenGridColor = new System.Drawing.Color();
         EvenGridColor = System.Drawing.Color.FromArgb(0xEC, 0xE9, 0xD8);

            for (int i = 0; i < GridView1.Rows.Count; i++)
            {
                  if (i % 2 == 1)
                  {
                      GridView1.Rows[i].BackColor = OddGridColor;
                  }
                  else
                  {
                      GridView1.Rows[i].BackColor = EvenGridColor;
                  }
             }
    }

    the way of change GridView color through one ...
    Why do not use <AlternatingRowStyle> ?!!

    Запостил: O_O, 23 Июня 2010

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

    • ava ReallyBugMeNot 23.06.2010 12:47 # 0

      Почему бы не писать по-русски, если с "инглишем" проблемы?
      Ответить

      • ava O_O 23.06.2010 13:45 # +3

        because i cannot write russian without mistakes.
        i am just from Ukraine and i had writed some posts in ukrainian but not everybody understanded me, that's why i use my bad english :)

        sorry for possible mistakes ...
        Ответить

        • ava Dmitriy 23.06.2010 14:47 # +2

          Офигеть :) Первый раз сталкиваюсь с подобным подходом к языку. Несмотря на изначальное недоумение, не могу не приветствовать "сокамерника". Буду применять аналогичный подход в отношении украинского, если потребуют на нем писать :))))
          Ответить

        • ava pushkoff 23.06.2010 15:58 # 0

          посвящяю тред ненависти к укр. яз.

          from now i will write english too... sometimes...
          Ответить

        • ava Lennis 23.06.2010 16:21 # 0

          ja tezh budu pysaty anglijskoju :)
          Ответить

        • ava Webkill 23.06.2010 17:12 # −9

          показать все, что скрытоа я вось хоч и агресывный ымперськый москаль, але украиньскою володию малёк такый ололо
          Ответить

    • ava tinynick 23.06.2010 12:47 # 0 

      GridView1.Rows[i].BackColor = (i % 2 == 1)? OddColor: EvenColor;

      Товарищ наверное думал примерно так: "Читать доку некогда, нужно скорее работать работу".
      Ответить

    • ava Мистер Хэнки 23.06.2010 12:58 # +1

      хард код детектед
      Ответить

    • ava Dmitriy 23.06.2010 13:00 # −3

      Вообще-то, далеко не все гриды поддерживают Альтернативную раскраску. Тернарная логика, конечно, уместнее, но, во-первых, все равно компилер преобразует ее к тому что представлено в кода, а во-вторых, как правило, тернарной логикой начинают пользоваться не с первого года программирования.
      Ответить

      • ava ReallyBugMeNot 23.06.2010 14:08 # −3

        Не соглашусь, что здесь тернарная логика здесь лучше. А если мы захотим менять не только BackColor, а и что-то ещё? Дублировать условие 100500 раз?
        Ответить

        • ava turdman 23.06.2010 14:15 # 0

          >тернарная логика
          >логика
          Ответить

          • ava Denis Popov 23.06.2010 14:24 # −4

            Это мое ноухау - тернарная логика.
            Кстати исключительные права на тернарную логику тоже принадлежат мне.
            Ответить

    • ava Lexey 23.06.2010 17:43 # 0

      Меня тут больше радуют 2 бессмысленных new в начале.
      Ответить

      • ava xeonix 24.06.2010 01:08 # 0

        вот тока их никто не заметил.
        мне тож доставили именно эти строки кода.
        а вот раскрашивать таким образом гридвью вполне нормально, например если вариантов цветов больше 2-х.
        а так, говнокод конешно...
        Ответить

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