1. Список говнокодов пользователя Progreso

    Всего: 3

  2. C# / Говнокод #3727

    +112

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    private void UpdateLayerLOBStatus(bool allDisable)
    {
      bool enable = true;

      if (allDisable)
      {
        enable = false;
      }

      SetStatuses(luLOBusiness, enable, true);
...........
 }

    Я не понимаю не то, что не я не говорю.

    Progreso, 17 Июля 2010

    Комментарии (5)

  3. C# / Говнокод #3322

    +119

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (dataTank.UpdateSQLTable(addressTbl, (int)SharedConstants.FocusAdapterTables.Address, SharedConstants.FocusAdapterTables.Address.ToString()) > 0)
          {
            return true;
          }
          return false;

    Yeah, baby!
    Похоже, ребята специально создавали себе места для рефакторинга - набудущее, чтобы было чем заняться.

    Progreso, 26 Мая 2010

    Комментарии (12)

  4. C# / Говнокод #3320

    +124

    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
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    switch ((int)row["State"])
          {
            case 0:
              /*** Get Contact Row for Company ***/
              if (contactTbl != null)
              {
                contactRow = null;
                DataRow[] contactRows = contactTbl.Select("Company_Id='" + row["Id"].ToString() + "'");
                if (contactRows != null && contactRows.Length > 0)
                {
                  contactRow = contactRows[0];
                  if ((bool)contactRow["fDontMailshot"] == false)
                  {
                    thisMailShot = true;
                  }
                  else
                  {
                    thisMailShot = false;
                  }
                }
              }

              if ((bool)row["fManagementCompany"] == true)
              {
                if (row["Id"].ToString() == "00-20181" || row["Id"].ToString() == "00-60838")
                {
                  myEntityID = Entity(2, row);
                  if (!SubContractor(myEntityID, row)) { return false; }
                }
                else
                {
                  myEntityID = Entity(10, row);
                }
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
              }
              else if ((bool)row["fSubcontractor"] == true || ((bool)row["fSupplier"] == true && (bool)row["fCustomer"] == false))
              {
                myEntityID = Entity(2, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!SubContractor(myEntityID, row)) { return false; }
              }
              else if ((bool)row["fHousingAssociation"] == true)
              {
                myEntityID = Entity(7, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              else if ((bool)row["f2ndOwnerTenant"] == true && (bool)row["fCustomer"] == false)
              {
                myEntityID = Entity(8, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              else if ((bool)row["fCustomer"] == true && (bool)row["f2ndOwnerTenant"] == false)
              {
                myEntityID = Entity(7, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              else if ((bool)row["fCustomer"] == true && (bool)row["f2ndOwnerTenant"] == true)
              {
                myEntityID = Entity(7, row);
                if (myEntityID < 0) { return false; }
                if (!Address(myEntityID, row)) { return false; }
                if (!Contact(myEntityID, contactRow, SOURCETYPE_ENTITY)) { return false; }
                if (!Customer(myEntityID, row, thisMailShot)) { return false; }
              }
              break;
            default:
              break;
          }

    Отличный пример использования конструкций ветвления. Имеем switch, в нём один case, а в кейсе много-много иф-элсов. И что очень характерно для этого проекта с тремя классами по 9000 строк, никакого повторного использования кода. Реализация принципа "зачем писать меньше?".

    Progreso, 26 Мая 2010

    Комментарии (10)