C# — Говнокод.ру

C# / Говнокод #1107

+145.9

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

public void newMessageTrySafe(Message message)
{
    if (SeparateThread.Wait(100, 5, new ExitWaitDelegate(this.)))
    {
        try
        {
            this.newMessage(message);
        }
        catch (Exception exception)
        {
            Utils.log("DataCache.newMessage() error: " + exception.Message, new object[0]);
        }
        try
        {
            Monitor.Exit(this.FRecalcSynchronizer);
            return;
        }
        catch (SynchronizationLockException)
        {
            return;
        }
    }
    Utils.log("newMessageTrySafe(): Deadlock! Unable to call newMessage bacause it's locked by this thread", new object[0]);
}

Решение проблемы дедлоков в многопоточной программе.

guest, 29 Мая 2009

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

C# / Говнокод #1096

+134.6

01
02
03
04
05
06
07
08
09
10
11
12

private bool _isProcessingThreadExited = false;

public bool IsShutDownComplete() 
{
   if (!_isProcessingThreadExited) 
   {
       // a thread hasnt exited yet
       return false;
   }

   return true;
}

... осталось еще bool на true проверить через ToString().Length > 4 - и это будет верх говнокодерства...

guest, 27 Мая 2009

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

C# / Говнокод #1090

+128.2

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15

public static string ValidateEmail(string Email)
        {
            try
            {
                if (Email.Trim() == string.Empty)
                {
                    return "Empty Parameter - Email";
                }
            }
            catch
            {
                return "Empty Parameter - Email";
            }
            return null;
        }

guest, 26 Мая 2009

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

C# / Говнокод #1071

+143.7

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43

/*Hide deleted button*/
   if (((UF.FileTypeID == FP.FileType1) && (FP.FileType1Edit == 1))
 || ((UF.FileTypeID == FP.FileType2) && (FP.FileType2Edit == 1))
 || ((UF.FileTypeID == FP.FileType3) && (FP.FileType3Edit == 1))
 || ((UF.FileTypeID == FP.FileType4) && (FP.FileType4Edit == 1))
 || ((UF.FileTypeID == FP.FileType5) && (FP.FileType5Edit == 1))
 || ((UF.FileTypeID == FP.FileType6) && (FP.FileType6Edit == 1)) 
|| ((UF.FileTypeID == FP.FileType7) && (FP.FileType7Edit == 1)) 
|| ((UF.FileTypeID == FP.FileType8) && (FP.FileType8Edit == 1)) 
|| ((UF.FileTypeID == FP.FileType9) && (FP.FileType9Edit == 1)) 
|| ((UF.FileTypeID == FP.FileType10) && (FP.FileType10Edit == 1))
 || ((UF.FileTypeID == FP.FileType11) && (FP.FileType11Edit == 1)))
   {
             if (CBL.GetListOfButtons(CPF.ID, 1) == true)
                            {
                                        bool ButtonVisible = true;
                                        String BText = "";
                                        foreach (ConfirmButtons CB in CBL.Items)
                                        {
                                            BText = "";
                                            if ((CB.ActionID > 0)&&(aAction.GetActionInfo(CB.ActionID) == true))
                                            {
                                                if (DTS.isStepAllowed(CB.ActionID, sysUser.GetID(), UserRoleID, aRequest.ID) == true)
                                                {
                                                    ButtonVisible = true;
                                                    #region Exeptions
                                                    if (CB.TypeName == "Confirm")
                                                    {
                                                        /*----------Check parallel process status--------------*/
                                                        if (aRequest.IsParent == 1)
                                                        {
                                                            if (CheckBP.CheckParallelBP(aRequest.ID, aRequest.aReqStatus.ID) == true)
                                                            {
                                                                if ((CheckBP.IsNecessary == 1) || ((CheckBP.ChildID > 0) && (CheckBP.ChildCurState > 0)))
                                                                {
                                                                    if (((CheckBP.IsNecessary == 1) && ((CheckBP.ChildID == 0))) || ((CheckBP.PositiveEndState != CheckBP.ChildCurState) && (CheckBP.NegativeEndState != CheckBP.ChildCurState)))
                                                                    {
                                                                        ButtonVisible = false;
                                                                        CFTitleText.Text = "В данный момент вы не можете cогласовать заявку. Незавершен параллельный процесс: '" + CheckBP.Name.ToString() + "'!";
                                                                    }
                                                                }
                                                            }
                                                        }

Кусок, начиная со строки 881(из 1307) метода Page_Load. Мастурбация мозга..

guest, 20 Мая 2009

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

C# / Говнокод #956

+952

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19

try
                    {
                        db.OpenDB();
                    }
                    catch { }
                    try
                    {
                        db.adapter.Fill(tableTemp);
                    }
                    catch { }
                    finally
                    {
                        db.command.Parameters.Clear();
                        try
                        {
                            db.CloseDB();
                        }
                        catch { }
                    }

Писать могут даже так.

guest, 27 Апреля 2009

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

C# / Говнокод #949

+151

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

public static DataSet ExecuteStoredProcedure(StoredProcedures procedure, Hashtable parameters, int nTries)
		{
			int retries = 0;

			while(retries<nTries)
			{
				try
				{
					return ExecuteStoredProcedure(procedure, parameters);
				}
				catch(Exception)
				{
					retries++;
				}
			}
			return null;
		}

Вот так вот екзекутились некоторые процедуры в проекте под CF. Возможно так хотели решить проблему с потерей коннекшена у девайса при врадении в слип? :)

guest, 24 Апреля 2009

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

C# / Говнокод #926

+152

1
2
3

SqlDataReader reader;
...
int id = int.Parse(reader["ID"].ToString());

Нет, ID - не какой-нибудь nvarchar, а вполне себе integer

guest, 23 Апреля 2009

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

C# / Говнокод #909

+113.8

01
02
03
04
05
06
07
08
09
10
11

public object DeepCopy(object value)
        {
            try
            {
                return value;
            }
            catch (Exception ex)
            {
                throw ex;
            }
        }

такой код повсюду... Источник: http://ayende.com/Blog/archive/2009/04/13/there-are-so-many-things-wrong-with-this-codehellip.aspx

guest, 21 Апреля 2009

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

C# / Говнокод #906

+137.6

1

public static readonly int    daysIn2Weeks = 14;

guest, 20 Апреля 2009

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

C# / Говнокод #876

+149

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16

protected void SubmitOk_ServerClick(object sender, EventArgs e)
    {
      ----- здесь код

      bool lOk = true;
      try
      {
        lSurvey.Save(int.Parse(mId.Value));
      }
      catch (Exception ex)
      {
        lOk = false;
        Util.ShowAdminError(ex.Message, this.Page);
      }
      if (lOk) Response.Redirect("AdminSurvey.aspx");
     }

Поступило предложение написать свою СМСКУ (ЦМС), блин ну спаришиветсо зачем???

guest, 16 Апреля 2009

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

Говнокод: по колено в коде.