- 1
- 2
public bool Success { get; set; }
public bool Failure { get { return !Success; } set { Success = !value; } }
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
Всего: 6
+3
public bool Success { get; set; }
public bool Failure { get { return !Success; } set { Success = !value; } }
И не поспоришь!
−45
var acceptCookies = CookiePopup.GetCookieValue();
if (acceptCookies == '' || (acceptCookies != 0 && acceptCookies != 1)) {
CookiePopup.ShowPopup();
}
Пример не уверенного в себе разработчика. Когда каждая его функция это "черный ящик" и результат не всегда предсказуем.
−41
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.
+1
bool isNoGoodCommentText = String.IsNullOrEmpty(this.txbxCommentCtrl.Text) || String.IsNullOrWhiteSpace(this.txbxCommentCtrl.Text);
И действительно, is not good
+130
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);
}
}
Выкручивался как мог.
Мало того, дальше в коде айдишник стал стрингового типа и стало еще веселей.
+105
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;
В начале метода такая вот проверка.
Дальше да же читать не стал.