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

    +94.5

    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
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    /// Из aspx файла

//<asp:Repeater ID="Repeater1" runat="server">
//    <HeaderTemplate>
//        <table width="100%" cellspacing="5">
//    </HeaderTemplate>
//    <ItemTemplate>
//        <tr>
//            <td>
//                ...
//            </td>
//            <asp:Label runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol2") %>'>
//                <td>
//                    ...
//                </td>
//            </asp:Label>
//            <asp:Label ID="LCol3" runat="server" Visible='<%# DataBinder.Eval(Container.DataItem, "visibleCol3") %>'>
//                <td>
//                    ...
//                </td>
//            </asp:Label>
//        </tr>
//    </ItemTemplate>
//    <FooterTemplate>
//        </table>
//    </FooterTemplate>
//</asp:Repeater>

// Из cs файла

DataTable dtres = // получаем товары
if (dtres != null && dtres.Rows.Count > 0)
{
    DataTable dt = new DataTable();
	/*...*/
    
    for (int i=0;i<dtres.Rows.Count;)
    {
        DataRow dr=dtres.Rows[i];
        /*...*/

        int col2Index = i + 1;
        if (col2Index < dtres.Rows.Count)
        {
            /*...*/
            i++;
            int col3Index = i + 1;
            if (col3Index < dtres.Rows.Count)
            {
                /*...*/
                i++;
            }
            else { /*...*/ }
        }
        else { /*...*/ }

        dt.Rows.Add(newRow);
        i++;
    }

    /*...*/
}

    Человеку нужно было сделать товары в сетке 3x6.
    От того что он сделал у меня пропал дар речи. (Чтобы тут очень много кода не бы большую (не нужную для понимания) часть заменил на "...")
    Вобщем, в двух словах, он поместил вторую и третью ячейки таблицы в серверные контролы (причем в Label), а в коде берёт по три товара и засовывает в один RepeaterItem, ну а если количество товаров не делится нацело на три, то в последний RepeaterItem засовываются пустые данные, а Label делается невидимым.
    Как-то так.
    И самое главное, я не знаю, как человеку объяснить, что так нехорошо делать, т.к он главный программист...

    Запостил: Ordos, 27 Сентября 2009

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

    • ava guest 27.09.2009 20:18 # 0

      OMG. Плюсую.
      Ответить

    • ava zerkms 28.09.2009 01:02 # +4

      Пока ты будешь молчать и бояться - ты будешь продолжать поддерживать его говнокод :-)
      Ответить

      • ava Ordos 28.09.2009 15:41 # 0

        Я жду, когда на сайтах настанет полная жопа, чтобы потереть руки, сказать "ну я же говорил", ехидно усмехнуться и уволиться нахрен :о)
        Ответить

    • ava FofanovIS 28.09.2009 12:04 # 0

      DataTable dtres = // получаем товары
      if (dtres != null...

      В строчке с DataTable уже создаётся, небось, ссылка, нафига он в следующей строке проверяет на null?
      Ответить

      • ava Ordos 28.09.2009 15:28 # 0

        нет, там что-то типа:
        DataTable dtres = Products.GetVisibleProducts(categoryid);
        При ошибке в базе данных метод возвращает null;
        Ответить

        • ava FofanovIS 28.09.2009 16:32 # 0

          а, ну если return null, то, тогда да... Эх а было бы весело, если бы это было не так)))
          Ответить

    • ava tonic 28.09.2009 12:21 # +4

      Лечить таких главных программистов надо. Желательно электричеством.
      Ответить

    • ava guest 29.09.2009 02:40 # 0

      Бери кирпич и дай им сдачи!
      Ответить

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