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

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

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

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

+130

class Users : Data
    {
        public int ID { get; set; }
        public string Name { get; set; }
        public int PostID { get; set; }
        public int SubdivID { get; set; }
        public QueryType qType { get; set; }
        public static QueryState qState { get; set; }
        
        /// <summary>
        /// Типы запроса для QueryBuilder
        /// </summary>
        public enum QueryType : int
        {
            Insert = 1,
            Delete,
            Update,
            SelectUsers,
            SelectUser,
            SelectUsersbyName
        }

        /// <summary>
        ///  Статусы открытия формы (добавление или редактирование)
        /// </summary>
        public enum QueryState : int
        {
            Insert = 1,
            Update
        }

        public Users()
        {
            ID = -1;
            Name = null;
        }

        /// <summary>
        /// Формирует запрос на основе значения переменной qType и осуществляет выборку данных
        /// </summary>
        public void QueryBuilder()
        {
            switch (qType)
            {
                case QueryType.SelectUsers:
                    {
                        QueryText = "Select Users.ID, Users.Name, Posts.Name, Subdivisions.Name from Users, Posts, Subdivisions Where Users.SubdivID = subdivisions.ID and Users.PostID = Posts.ID";
                        Select();
                        break;
                    }
                case QueryType.SelectUser:
                    {
                        if (ID != -1)
                        {
                            QueryText = "Select Users.ID, Users.Name, Posts.Name, Subdivisions.Name, Posts.ID, Subdivisions.ID From Users, Posts, Subdivisions " +
                                        " Where Users.SubdivID = subdivisions.ID and Users.PostID = Posts.ID and Users.ID = " + ID;
                            SelectRow();
                        }
                        else
                        {
                            MessageBox.Show("ID пользователя не найден");
                        }
                        break;
                    }
                case QueryType.Insert:
                    {
                        QueryText = "Insert Into Users(Name, PostID, SubdivID) Values('" + Name + "', '" + PostID + "', '" + SubdivID + "')";
                        InsertRecord();
                        break;
                    }
                case QueryType.Delete:
                    {
                        QueryText = "Delete From Users Where ID = " + ID;
                        DeleteRecord();
                        break;
                    }
                case QueryType.Update:
                    {
                        QueryText = "Update Users set Name = '" + Name + "', PostID = '" + PostID + "', SubdivID = '" + SubdivID + "'  Where ID = " + ID;
                        UpdateRecord();
                        break;
                    }
                case QueryType.SelectUsersbyName:
                    {
                        QueryText = "Select Users.ID, Users.Name, Posts.Name, Subdivisions.Name from Users, Posts, Subdivisions " +
                                    "Where Users.SubdivID = subdivisions.ID and Users.PostID = Posts.ID and Users.Name like " + "'%" + Name + "%'";
                        Select();
                        break;
                    }
            }
        }

15856 Продолжение...

Ate1st, 28 Апреля 2014

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

+133

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