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

    +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
    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
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    95. 95
    abstract class Data
        {
            public string QueryText { get; protected set; }
            public BindingSource SelectResult { get; private set; }
            public object[] SelectedRow { get; protected set; } 
    
        
            /// <summary>
            /// Добавляет новую запись в таблицу
            /// </summary>
            protected virtual void InsertRecord()
            {
                if (IsConnected())
                {
                    SqlDataAdapter DA = new SqlDataAdapter();
                    DA.InsertCommand = new SqlCommand(QueryText, DB.Connection);
                    DA.InsertCommand.ExecuteNonQuery();
                }
    
            }
    
            protected virtual void UpdateRecord()
            {
                if (IsConnected())
                {
                    SqlDataAdapter DA = new SqlDataAdapter();
                    DA.UpdateCommand = new SqlCommand(QueryText, DB.Connection);
                    DA.UpdateCommand.ExecuteNonQuery();
                }
            }
    
            protected virtual void DeleteRecord()
            {
                if (IsConnected())
                {
                    SqlDataAdapter DA = new SqlDataAdapter();
                    DA.DeleteCommand = new SqlCommand(QueryText, DB.Connection);
                    DA.DeleteCommand.ExecuteNonQuery();
                }
            }
    
    
            /// <summary>
            /// Выбирает данные и записывает результат в переменную SelectResult
            /// </summary>
            protected virtual void Select()
            {
                DataSet DS = new DataSet();
                DS.Tables.Add("Tbl");
                SqlDataAdapter ADP = new SqlDataAdapter();
    
                if (IsConnected())
                {
                    ADP.SelectCommand = new SqlCommand(QueryText, DB.Connection);
                    ADP.Fill(DS.Tables["Tbl"]);
    
                    SelectResult = new BindingSource();
                    string s = DS.Tables[0].TableName;
                    SelectResult.DataMember = s;
                    SelectResult.DataSource = DS;
                }
    
            }
    
    
            /// <summary>
            /// Выбирает данные и записывает результат в Object массив SelectedRow
            /// </summary>
            public virtual void SelectRow()
            {
                DataSet DS = new DataSet();
                DS.Tables.Add("Tbl");
                SqlDataAdapter ADP = new SqlDataAdapter();
    
                if (IsConnected())
                {
                    ADP.SelectCommand = new SqlCommand(QueryText, DB.Connection);
                    ADP.Fill(DS.Tables["Tbl"]);
                }
    
                SelectedRow = DS.Tables[0].Rows[0].ItemArray;
    
            }
    
            private bool IsConnected()
            {
                if(DB.Connection.State == System.Data.ConnectionState.Open)
                {
                    return true;
                }
                else
                {
                    return false;
                }
            }

    Ate1st, 28 Апреля 2014

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

    +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
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    SqlConnection con = new SqlConnection(connectionString);
                string sql = "SELECT idea.Id, idea.Small_text, idea.Full_text, idea.Example_text, ";
                sql = sql + " ul.FullName as Avtor, ref.Name as Status, even.date_act, (select count (*) from events ev where ev.id_action=2 and ev.id_target=idea.Id and ev.type_target=1) as CountComm, (select count (*) from events ev where ev.id_action=57 and ev.id_target=idea.Id and ev.type_target=1) as Voit, (select max(ev.date_act) from events ev where ev.id_action in (1,2,4,5,7,8,9,10,58,60,61,62,63,64,65,66,67,68,69,70,71,72,74) and ev.id_target=idea.Id and ev.type_target=1) as Last_action";
                sql = sql + " FROM Idea idea, UserLists ul, Refbook ref, Events even ";
                sql = sql + " WHERE idea.Avtor=ul.Id and idea.Status_id = ref.Id and even.id_action=1 and even.type_target=1 and even.id_target=idea.id ";
                if (id >= 0)
                    sql = sql + "  and idea.Status_id=" + id;
                if (id_avtor >= 0)
                    sql = sql + "  and idea.Avtor=" + id_avtor;
                sql = sql + " ORDER BY Last_action desc";
    
                SqlCommand cmd = new SqlCommand(sql, con);
                cmd.CommandType = CommandType.Text;
                List<Idea> Ideas = new List<Idea>();
                using (con)
                {
                    con.Open();
                    SqlDataReader reader = cmd.ExecuteReader();
                    while (reader.Read())
                    {
                        Idea idea = GetIdeaInfo(Int32.Parse(reader["Id"].ToString()));
                        Ideas.Add(idea);
                    }
                    reader.Close();
                    return Ideas;
                }

    Принял еще один проект. ASP.NET MVC4. Автор уехал в Новую Зеландию.

    Vince, 28 Апреля 2014

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

    +130

    1. 001
    2. 002
    3. 003
    4. 004
    5. 005
    6. 006
    7. 007
    8. 008
    9. 009
    10. 010
    11. 011
    12. 012
    13. 013
    14. 014
    15. 015
    16. 016
    17. 017
    18. 018
    19. 019
    20. 020
    21. 021
    22. 022
    23. 023
    24. 024
    25. 025
    26. 026
    27. 027
    28. 028
    29. 029
    30. 030
    31. 031
    32. 032
    33. 033
    34. 034
    35. 035
    36. 036
    37. 037
    38. 038
    39. 039
    40. 040
    41. 041
    42. 042
    43. 043
    44. 044
    45. 045
    46. 046
    47. 047
    48. 048
    49. 049
    50. 050
    51. 051
    52. 052
    53. 053
    54. 054
    55. 055
    56. 056
    57. 057
    58. 058
    59. 059
    60. 060
    61. 061
    62. 062
    63. 063
    64. 064
    65. 065
    66. 066
    67. 067
    68. 068
    69. 069
    70. 070
    71. 071
    72. 072
    73. 073
    74. 074
    75. 075
    76. 076
    77. 077
    78. 078
    79. 079
    80. 080
    81. 081
    82. 082
    83. 083
    84. 084
    85. 085
    86. 086
    87. 087
    88. 088
    89. 089
    90. 090
    91. 091
    92. 092
    93. 093
    94. 094
    95. 095
    96. 096
    97. 097
    98. 098
    99. 099
    100. 100
    using System;
    using System.Collections.Generic;
    using System.Linq;
    using System.Text;
    using System.Text.RegularExpressions;
    using System.IO;
    
    namespace SimpleLex
    {
        class Lexical
        {
            //Сюда передается путь к файлу конфигурации
            public string Path = "";
            //Определяем ключевые слова 
            string regular = "Color|Size|Name";
            public void lexical()
            {
                //Создаем массив строк для дальнейшего заполнения
                string[] conf = new string[1024];
                //Переменная счетчик
                int i = 0;
                //Заполняем массив из файла 
                //Path мы присваиваем значение при создании
                //объекта нашего класса.
                conf = File.ReadAllLines(Path);
                //Запускаем цикл чтения 
                while (conf.Length - 1 >= i)
                {
                // Создаем новый объект класса Regex
                // и передаем ему в качестве конструктора
                // cписок ключевых слов
                Regex RegularExp = new Regex(regular);
                //Начинаем поиск совпадений в текущей строке
                Match match = RegularExp.Match(conf[i]);
                while (match.Success)
                {
                    //Ожидаем наличия совпадений
                    break;
                    //Если нашли то выпрыгиваем 
                }
    
                switch (match.Value) // Смотрим что же мы обнаружили и вызываем соответствующий метод
                {
                    case "Color":
                        //вызываем метод Color_
                        //и передаем ему текущую строку для разбора
                        //Предварительно удалив все пробелы с двух сторон
                        //если они были метод Trim()
                        Color_(conf[i].Trim());
                        i++;
                        break;
                    case "Size":
                        size_(conf[i].Trim());
                        i++;
                        break;
                    default:
                        //все другое пропускаем
                        i++;
                        break;
                }
                }
            }
            void Color_(string str)
            {
                int i = 0;
                while (str.Substring(i,1)!= "=")
                {
                    //Ищем разделитель в данном случае "="
                    i++;
                }
                //Находим и извлекаем нужную нам информацию
                string value_ = str.Substring(i + 1, str.Length - (i+1)).Trim();
                //Теперь все зависит от вашей фантазии хотите 
                //Создайте класс который будет устанавливать цвет шрифта
                //И.т.д
    
                Console.WriteLine(value_);
                Console.ReadKey();
            }
            void size_(string str)
            {
                /*
                 * В этом методе я хочу показать как быть если вы используете
                 * в своем файле для каждой команды разные разделители
                 * в теории конечно можно все это засунуть в один метод
                 * Но я советую в дальнейшем если вы будите применять
                 * именно такую структуру передавать в метод тип разделителя
                 * и соответственно искать именно его другое дело если вы имеете
                 * различный формат входных данных как у нас параметр size имеет
                 * вид shize X=100,Y=500
                 */
                //Есть прекрасный метод 
                int i = 0;
                while (str.Substring(i,1) != " ")
                {
                    i++;
                }
                str = str.Substring(i, str.Length - i).Trim();
                string[] commandMas = new string[3];
                commandMas = str.Split(',');

    (GetPrivateProfileString? Не, не слышал)
    Лютый песдеzz с сайта vbbook.ru

    brutushafens, 24 Апреля 2014

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

    +138

    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
    private static Dictionary<RoleEnum, string> Roles = new Dictionary<RoleEnum, string>
            {
                {RoleEnum.TeamMember, "Team Member"},
                {RoleEnum.ProjectManager, "Project Manager"},
                {RoleEnum.ProgramManager, "Program Manager"},
                {RoleEnum.PortfolioManager, "Portfolio Manager"},
                {RoleEnum.Executive, "Executive"},
                {RoleEnum.Undefined, "Undefined"}
            };
    
            public static RoleEnum ParseRole(string role)
            {
                RoleEnum result = RoleEnum.Undefined;
                Roles
                    .Where(_ => _.Value == role)
                    .ToList()
                    .ForEach(_ => result = _.Key);
    
                return result;
            }

    Странное использование дикшинари, очень странное, в обратную сторону можна сказать

    boades, 24 Апреля 2014

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

    +137

    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
    Index and length must refer to a location within the string.
    Parameter name: length
    Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
    
    Exception Details: System.ArgumentOutOfRangeException: Index and length must refer to a location within the string.
    Parameter name: length
    
    Source Error:
    
    
    Line 1239:    // СЕГОДНЯ И ВЧЕРА
    Line 1240:    string agent = Request.UserAgent;
    Line 1241:    string sid = DateTime.Now.ToShortDateString().Substring(0, 10).Replace(".", "").Replace("/", "").Replace("-", "");
    Line 1242:    string sid2 = DateTime.Now.AddDays(-1).ToShortDateString().Substring(0, 10).Replace(".", "").Replace("/", "").Replace("-", "");
    Line 1243:    //

    http://tltgorod.ru/news/theme-5/news-21502

    Хуяк-хуяк и в продакшн.

    bakagaijin, 24 Апреля 2014

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

    +137

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    public class Entity
    {
        // пример
        private DateTime _modified = new DateTime(2014, 4, 18, 23, 59, 59);
    
      public void SaveModified()
      {
                var date = DateTime.Parce(this._datetime.ToShortDateString());
                base.Save(date, ... );
      }
    }

    Продолжение проекта http://govnokod.ru/15780

    Насколько я понял, то это "операция усечения даты" 8-D
    которая должна была выглядеть так:
    var date = this._datetime.Date;

    CodeKiller, 19 Апреля 2014

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

    +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
    // где-то там ...  
    private int Kapital = 777;
    
    // ...
    
    void Save()
    {
    ...
                int kapital;
                try { kapital = (int)Double.Parse(Kapital.ToString()); }
                catch { kapital = 0; }
    ...
    }

    Проект из солнечной Швейцарии...
    И таких перлов много :)

    CodeKiller, 18 Апреля 2014

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

    +131

    1. 1
    2. 2
    3. 3
    4. 4
    DateTime dateEst = DateTime.Now.ToEstTime();
    StringBuilder responseTime = new StringBuilder();
    responseTime.Append(dateEst.ToString("MM/dd/yyyy"));
    responseTime.Append(dateEst.ToString("  h:mm:ss tt"));

    Отображаем время ответа от сервера в удобном формате.

    vertu, 17 Апреля 2014

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

    +137

    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
    model.PID = row["PID"].ToString();
    model.Rect = row["RECt"].ToString();
    model.Pubt = row["PUBt"].ToString();
    model.Ex = row["Ex"].ToString();
    model.Prev_Vol = row["Prev_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Prev_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
    model.Curr_Vol = row["Curr_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Curr_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
    model.Mov_Vol = row["Mov_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Mov_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
    decimal tonavgvol = Tools.ParseDecimalValue(row["Mov_Vol"].ToString());
    decimal currentvol = Tools.ParseDecimalValue(row["Curr_Vol"].ToString());
    if (row["Mov_Vol"].ToString() != "" && row["Curr_Vol"].ToString() != "" && tonavgvol != 0)
    	model.VolRat = (currentvol / tonavgvol).FormatAmount();
    model.Open_Vol = row["Open_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Open_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
    model.Close_Vol = row["Close_Vol"].ToString() != "" ? Tools.ParseDecimalValue(row["Close_Vol"].ToString()).FormatAmountWithoutDecimal() : "";
    model.AllTicks = row["AllTicks"].ToString() != "" ? Tools.ParseDecimalValue(row["AllTicks"].ToString()).FormatAmountWithoutDecimal() : "";
    model.ATR = row["ATR14"].ToString() != "" ? Tools.ParseDecimalValue(row["ATR14"].ToString()).FormatAmountFourDecimals() : "";
    decimal open = Tools.ParseDecimalValue(row["Open"].ToString());
    decimal close = Tools.ParseDecimalValue(row["Close"].ToString());
    if (row["Open"].ToString() != "" && row["Close"].ToString() != "" && open != 0)
    	model.PrevDay = (((close - open) / open) * 100).FormatPercent();
    model.PrevClose = row["PrevClose"].ToString() != "" ? Tools.ParseDecimalValue(row["PrevClose"].ToString()).FormatAmount() : "";
    model.DayOpen = row["DayOpen"].ToString() != "" ? Tools.ParseDecimalValue(row["DayOpen"].ToString()).FormatAmount() : "";
    decimal prevclose = Tools.ParseDecimalValue(row["PrevClose"].ToString());
    decimal dayopen = Tools.ParseDecimalValue(row["DayOpen"].ToString());
    if (row["DayOpen"].ToString() != "" && row["PrevClose"].ToString() != "" && prevclose != 0)
    	model.OpenGap = (((dayopen - prevclose) / prevclose) * 100).FormatPercent();
    model.SPYLast = row["TONSPLast"].ToString() != "" ? Tools.ParseDecimalValue(row["TONSPLast"].ToString()).FormatAmount() : "";
    model.TONOpen = row["TONOpen"].ToString() != "" ? Tools.ParseDecimalValue(row["TONOpen"].ToString()).FormatAmount() : "";
    model.TONHigh = row["TONHigh"].ToString() != "" ? Tools.ParseDecimalValue(row["TONHigh"].ToString()).FormatAmount() : "";
    model.TONLow = row["TONLow"].ToString() != "" ? Tools.ParseDecimalValue(row["TONLow"].ToString()).FormatAmount() : "";
    model.TONLast = row["TONLast"].ToString() != "" ? Tools.ParseDecimalValue(row["TONLast"].ToString()).FormatAmount() : "";
    model.EODHigh = row["EODHigh"].ToString() != "" ? Tools.ParseDecimalValue(row["EODHigh"].ToString()).FormatAmount() : "";
    model.EODLow = row["EODLow"].ToString() != "" ? Tools.ParseDecimalValue(row["EODLow"].ToString()).FormatAmount() : "";
    model.EODClose = row["EODClose"].ToString() != "" ? Tools.ParseDecimalValue(row["EODClose"].ToString()).FormatAmount() : "";
    model.SPYClose = row["EODSPClose"].ToString() != "" ? Tools.ParseDecimalValue(row["EODSPClose"].ToString()).FormatAmount() : "";
    model.PostVWAP = row["PostVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["PostVWAP"].ToString()).FormatAmount() : "";
    model.PreVWAP = row["PreVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["PreVWAP"].ToString()).FormatAmount() : "";
    model.MainVWAP = row["MainVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["MainVWAP"].ToString()).FormatAmount() : "";
    model.AllVWAP = row["AllVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["AllVWAP"].ToString()).FormatAmount() : "";
    model.EODVWAP = row["EODVWAP"].ToString() != "" ? Tools.ParseDecimalValue(row["EODVWAP"].ToString()).FormatAmount() : "";
    
    decimal tonlast = Tools.ParseDecimalValue(row["TONLast"].ToString());
    decimal eodhigh = Tools.ParseDecimalValue(row["EODHigh"].ToString());
    decimal eodlow = Tools.ParseDecimalValue(row["EODLow"].ToString());
    decimal tonhigh = Tools.ParseDecimalValue(row["TONHigh"].ToString());
    decimal tonlow = Tools.ParseDecimalValue(row["TONLow"].ToString());
    decimal eodclose = Tools.ParseDecimalValue(row["EODClose"].ToString());
    decimal oedspclose = Tools.ParseDecimalValue(row["EODSPClose"].ToString());
    decimal tonsplast = Tools.ParseDecimalValue(row["TONSPLast"].ToString());
    
    // еще где-то 100-150 строчек в таком же духе

    ADO.NET во все поля.
    Бизнес-логика? Что это такое?

    P.S. У класса model все свойства типа string, у всех decimal переменных потом тоже вызывается ToString().

    vertu, 11 Апреля 2014

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

    +135

    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
    internal enum IsScanned
        {
            Create = 0,
            Scan = 1,
            Complete = 2,
            Error = 3,
            NoAccess = 4,
        }
    
    ...
    
    public int IsScanned { get; set; }
    
    ...
    
    if (details.IsScanned != (int)IsScanned.Create)

    paulem, 11 Апреля 2014

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