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

    Всего: 6

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

    +2

    1. 1
    2. 2
    public bool Success { get; set; }
            public bool Failure { get { return !Success; } set { Success = !value; } }

    И не поспоришь!

    pipjaka, 10 Августа 2017

    Комментарии (0)
  3. JavaScript / Говнокод #22305

    −45

    1. 1
    2. 2
    3. 3
    4. 4
    var acceptCookies = CookiePopup.GetCookieValue();
      if (acceptCookies == '' || (acceptCookies != 0 && acceptCookies != 1)) {
        CookiePopup.ShowPopup();
      }

    Пример не уверенного в себе разработчика. Когда каждая его функция это "черный ящик" и результат не всегда предсказуем.

    pipjaka, 09 Февраля 2017

    Комментарии (49)
  4. C# / Говнокод #22055

    −41

    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
    public string GetValue(string value)
        {
            var displayOptions = (DisplayOptions)ViewBag.Options;
            var defaultCulture = ContentHelper.GetDefaultCulture();
            var currentCulture = UserHelper.GetSessionCulture() ?? defaultCulture;
    
            var domainValue = Model.Domain.Values.FirstOrDefault(v => v.Value.ToLower() == value.ToLower());
    
            var text = domainValue == null ? value : (domainValue?.Translations.Where(t => currentCulture.Equals(t.Culture)).Select(t => t.Text).FirstOrDefault() ?? domainValue.Text);
            var result = text;
    
            if (!string.IsNullOrEmpty(Model.Domain.DisplayPrefix))
            {
                result = string.Format("{0} {1}", Model.Domain.DisplayPrefix, text);
            }
    
            if (!string.IsNullOrEmpty(Model.Domain.DisplayPrefix) && !string.IsNullOrEmpty(Model.Domain.DisplaySuffix))
            {
                result = string.Format("{0} {1} {2}", Model.Domain.DisplayPrefix, text, Model.Domain.DisplaySuffix);
            }
    
            if (!string.IsNullOrEmpty(Model.Domain.DisplaySuffix))
            {
                result = string.Format("{0} {1} {2}", Model.Domain.DisplayPrefix, text, Model.Domain.DisplaySuffix);
            }
    
            return String.Format("{0}{1}{2}{3}"
                , displayOptions.StartTag != null && displayOptions.EndTag != null ? (displayOptions.StartTag ?? string.Empty) : string.Empty
                , displayOptions.ShowDomainAlternative ? @Html.DisplayForDomainAlternative(domainValue) : new MvcHtmlString(string.Empty)
                , displayOptions.ShowDomainText ? result : string.Empty
                , displayOptions.StartTag != null && displayOptions.EndTag != null ? (displayOptions.EndTag ?? string.Empty) : string.Empty);
        }

    Так безжалостно получали result чтобы в конце заменить его на пустую сроку. Не говоря уже про общее построение метода и как себя ненавидел автор, добавляя еще одну строку кода в String.Format.

    pipjaka, 27 Января 2017

    Комментарии (6)
  5. C# / Говнокод #19409

    +1

    1. 1
    bool isNoGoodCommentText = String.IsNullOrEmpty(this.txbxCommentCtrl.Text) || String.IsNullOrWhiteSpace(this.txbxCommentCtrl.Text);

    И действительно, is not good

    pipjaka, 05 Февраля 2016

    Комментарии (4)
  6. C# / Говнокод #13980

    +130

    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
    DataTable newTable = new DataTable();
    newTable.Columns.Add("id");
    newTable.Columns.Add("type");
    
    DataRow empty_row = newTable.NewRow();
    empty_row["id"] = 0;
    empty_row["type"] = "--- Все графики ---";
    newTable.Rows.Add(empty_row);
    
    for (int i = 0; i < tPayDays.Rows.Count; i++)
    {
          if (tPayDays.Rows[i]["id"].ToString() != "11")
          {
                 DataRow new_row = newTable.NewRow();
                 new_row["id"] = tPayDays.Rows[i][0].ToString();
                 new_row["type"] = tPayDays.Rows[i][1].ToString();
                 newTable.Rows.Add(new_row);
           }
    }

    Выкручивался как мог.
    Мало того, дальше в коде айдишник стал стрингового типа и стало еще веселей.

    pipjaka, 20 Октября 2013

    Комментарии (6)
  7. C# / Говнокод #13890

    +105

    1. 1
    2. 2
    3. 3
    4. 4
    if (row["Enable"].ToString().ToLower() == "true" || row["Enable"].ToString() == "1")
             chState.Checked = true;
     else if (row["Enable"].ToString().ToLower() == "false" || row["Enable"].ToString() == "0")
             chState.Checked = false;

    В начале метода такая вот проверка.
    Дальше да же читать не стал.

    pipjaka, 03 Октября 2013

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