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

    +134

    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
    foreach (Admin _admin in _admins.Where(a => a.Login == _login))
                    {
                        if (_admin.Password == _password)
                        {
                            SessionObj = new Al2AllAuth
                            {
                                AccessId = _admin.Type,
                                Ename = _admin.Name,
                                Whoid = _admin.Id
                            };
                            var _type = (AdminTypes) _admin.Type;
                            switch (_type)
                            {
                                case AdminTypes.FirstType:
                                    return RedirectToAction(c_adminActionName, c_userControllerName);
                                case AdminTypes.SecondType:
                                    if (string.IsNullOrEmpty(model.Page) || string.IsNullOrEmpty(model.Uid))
                                        throw new ArgumentException(InternalResources.EmptyPageOrUid, "model");
                                    return RedirectToAction(model.Page, new { uid = model.Uid });
                                default: 
                                    break;
                            }
                        }
                        else
                        {
                            return RedirectToAction(_errorActionName);
                        }
                    }

    Лямбда в foreach очен смутила... Зачем?

    KonstantinK, 11 Апреля 2014

    Комментарии (23)
  2. C# / Говнокод #15704

    +142

    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
    List<EntryChange> changes = new List<EntryChange>();
    
    for (int index = 0;
                index < partsContainer.transform.childCount;
                        ++index)
    {
    
        if (partsContainer
                    .transform
                        .GetChild(index)
                                .GetComponent<castle_parts_container>()
                                    .inSave()
                        ==
                        true)
        {
    
            // id of current part on scene
            int partId =
                    partsContainer.transform
                        .GetChild(index).gameObject
                                .GetComponent<castle_parts_container>()
                                                        .partIdentificator;
    
            int partColor =
                    partsContainer.transform
                        .GetChild(index).gameObject
                                .GetComponent<castle_parts_container>()
                                                            .savedColor();
            int partNumber =
                    partsContainer.transform
                        .GetChild(index).gameObject
                                .GetComponent<castle_parts_container>()
                                                            .savedNumber();
    
            EntryChange change = new EntryChange();
    
            change.original =
                    partsContainer.transform
                                    .GetChild(index).gameObject;
            change.target =
                    partPrefabs.Where(
                        x => x.GetComponent<castle_parts_container>().partIdentificator
                                ==
                                partId
                                    &&
                                x.GetComponent<castle_parts_container>().partColor
                                    ==
                                    partColor
                                        &&
                                    x.GetComponent<castle_parts_container>().partNumber
                                        ==
                                        partNumber
                    ).First();
    
            changes.Add(change);

    Избыточный код с форматированием в стиле Маяковского

    arazect, 08 Апреля 2014

    Комментарии (121)
  3. C# / Говнокод #15675

    +128

    1. 1
    2. 2
    3. 3
    if (f > 1) { }
    else
    chart5.Series[1].Points.AddXY(i, f);

    Было замечено за коллегой

    goaquit, 05 Апреля 2014

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

    +131

    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
    xlApp = new Excel.Application();
                try
                {
                    xlApp.Workbooks.Add(Type.Missing);
    
                    xlApp.Interactive = false;
                    xlApp.EnableEvents = false;
    
                    xlSheet = (Excel.Worksheet)xlApp.Sheets[1];
                    xlSheet.Name = "Данные";
    
                    DataTable dt = GetData();
    
                    int collInd = 0;
                    int rowInd = 0;
                    string data = "";
    
                    for (int i = 0; i < dt.Columns.Count; i++)
                    {
                        data = dt.Columns[i].ColumnName.ToString();
                        xlSheet.Cells[1, i + 1] = data;
    
                        xlSheetRange = xlSheet.get_Range("A1:Z1", Type.Missing);
    
                        xlSheetRange.WrapText = true;
                        xlSheetRange.Font.Bold = true;
                    }
    
                    for (rowInd = 0; rowInd < dt.Rows.Count; rowInd++)
                    {
                        for (collInd = 0; collInd < dt.Columns.Count; collInd++)
                        {
                            data = dt.Rows[rowInd].ItemArray[collInd].ToString();
                            xlSheet.Cells[rowInd + 2, collInd + 1] = data;
                        }
                    }
    
                    xlSheetRange = xlSheet.UsedRange;
    
                    xlSheetRange.Columns.AutoFit();
                    xlSheetRange.Rows.AutoFit();
                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.ToString());
                }
                finally
                {
                    xlApp.Visible = true;
    
                    xlApp.Interactive = true;
                    xlApp.ScreenUpdating = true;
                    xlApp.UserControl = true;
    
                    releaseObject(xlSheetRange);
                    releaseObject(xlSheet);
                    releaseObject(xlApp);
                }

    orozov, 04 Апреля 2014

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

    +136

    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
    private Action<Action, Action<Exception>> ToCallbackAction(Action a)
            {
                return (sc, ec) =>
                {
                    try
                    {
                        a();
                        sc();
                    }
                    catch (Exception e)
                    {
                        ec(e);
                    }
                };
            }

    толково или говно? Мнения разделились

    taburetka, 01 Апреля 2014

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

    +135

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    void ConvertingExpressions()
    
    {
    
    float.TryParse(transform.position.x.ToString(), out x);
    
    float.TryParse(transform.position.y.ToString(), out y);
    
    float.TryParse(transform.position.z.ToString(), out z);
    
    }

    Я просто искал решение проблем и нашел это:
    http://itech.do.am/publ/7-1-0-58

    yegorf1, 27 Марта 2014

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

    +135

    1. 1
    this.xlWorkSheet.get_Range("V" + (i + 3).ToString(), Missing.Value).Value2 = Convert.ToDouble(sample_info.Rows[i]["id"].ToString().Replace(Program.separator, Program.new_separator));

    Классика. Меняем точку на запятую.

    redrick, 26 Марта 2014

    Комментарии (1)
  8. C# / Говнокод #15514

    +133

    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
    try
                {
                    using (var context = new vbEntities())
                    {                   
                        
                        var DBF = new DBF();
                        var dt = DBF.GetAll(@"C:\Users\.........\Documents\BNKSEEK.DBF");
    
                        int i = 0;
                        progressBar1.Maximum = dt.Rows.Count;
    
                        foreach (DataRow dr in dt.Rows)
                        {
                            var row = new Bank();
                            row.BIK = dr["RKC"].ToString();
                            row.CityName = dr["NNP"].ToString();
                            row.Address = dr["ADR"].ToString();
                            row.Name = dr["NAMEP"].ToString();
                            row.KS = dr["KSNP"].ToString();
                            row.Phone = dr["TELEF"].ToString();
                            row.OKPO = dr["OKPO"].ToString();  
                            context.Bank.Add(row);
                            progressBar1.Value = i++;
                        }
                        context.SaveChanges();
                    }
                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                }
            }

    orozov, 18 Марта 2014

    Комментарии (13)
  9. C# / Говнокод #15504

    +129

    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
    class WorkDBF
        {
            private OdbcConnection _connection = null;
            public DataTable Execute(string command)
            {
                DataTable dt = null;
                if (_connection != null)
                {
                    try
                    {
                        _connection.Open();
                        dt = new DataTable();
                        System.Data.Odbc.OdbcCommand oCmd = _connection.CreateCommand();
                        oCmd.CommandText = command;
                        dt.Load(oCmd.ExecuteReader());
                        _connection.Close();
                    }
                    catch (Exception e)
                    {
                        MessageBox.Show(e.Message);
                    }
                }
                return dt;
            }
            public DataTable GetAll(string dbpath)
            {
                return Execute("SELECT * FROM " + dbpath);
            }
            public WorkDBF()
            {
                this._connection = new System.Data.Odbc.OdbcConnection();
                _connection.ConnectionString = @"Driver={Microsoft dBase Driver (*.dbf)};" +
                    "SourceType=DBF;Exclusive=No;" +
                    "Collate=Machine;NULL=NO;DELETED=NO;" +
                    "BACKGROUNDFETCH=NO;";
            }
        }

    orozov, 17 Марта 2014

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

    +129

    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
    [System.Runtime.Serialization.DataMember(Name = "o")]
            public OnlineOperator[] AvailableOperators
            {
                get
                {
                    List<OnlineOperator> o = new List<OnlineOperator>();
    
                    if (this.AllOperators != null)
                    {
                        Boolean flag = false;
                        List<OnlineOperator> o2 = new List<OnlineOperator>();
    
                        foreach (OnlineOperator oo in this.AllOperators)
                        {
                            if (oo.Status != OperatorStatus.Offline)
                            {
                                if (oo.Priority == 0)
                                {
                                    if (oo.Status == OperatorStatus.Online)
                                    {
                                        flag = true;
                                    }
                                    o.Add(oo);
                                }
    
                                if (!flag)
                                {
                                    o2.Add(oo);
                                }
                            }
                        }
    
                        if (!flag)
                        {
                            o = o2;
                        }
                    }
    
                    //if (o.Count < 1)
                    {
                        o.Add(new OnlineOperator()
                        {
                            Id = 0,
                            UserDisplayName = String.Format(Resources.pattern_offline, this.OfflineNote),
                            Status = OperatorStatus.None
                        });
                    }
    
                    return o.ToArray();
                }
                set
                {
                    ;
                }
            }

    Логика конечно странновато реализована, но реализация ReadOnly свойства меня просто убила

    dens, 13 Марта 2014

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