Си диез / Говнокод #3545 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 O_O, (Updated )

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

    • ava O_O # Ссылка на оригинал 0 ⇈

      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 # Ссылка на оригинал 0 ⇈

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

      • ava pushkoff # Ссылка на оригинал 0 ⇈

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

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

      • ava Webkill # Ссылка на оригинал 0 ⇈

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

        • ava Dmitriy # Ссылка на оригинал 0 ⇈

          Пиши на любом языке. Только пиши грамотно!
          Пиши на будь-якой мові. Тільки пиши грамотно!
          Write in any language. Just write competently!
          Ответить

  • ava tinynick # Ссылка на оригинал 0 

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

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

  • ava Dmitriy # Ссылка на оригинал 0

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

    • ava ReallyBugMeNot # Ссылка на оригинал 0 ⇈

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

        • ava Denis Popov # Ссылка на оригинал 0 ⇈

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

  • ava Lexey # Ссылка на оригинал 0

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

    • ava xeonix # Ссылка на оригинал 0 ⇈

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

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

    ↑
    8
    ↓