warhate / tl410-rias Goto Github PK

1. В классах интерфейса необходимо 
использовать только простейшие логические 
конструкции(и то при условии 
необходимости) и вызовы из экземпляров 
других классов разноплановые обработчики 
информации.

Form1.cs
{{{
private void AddButton_Click(object sender, EventArgs e)
        {
            AddEvent newEvent = new AddEvent();
            newEvent.ShowDialog();

            // получаем данные о событии и записываем в таблицу
            EventData evdata = newEvent.getEventData();
            if (evdata.name != null)
            {
                // додаем новые столбцы
                for (int i = 0; i < evdata.listOfRows.Count; i++)
                {
                    if (this.dataEventsTable.Columns.Contains(evdata.listOfRows[i].Cells[0].Value.ToString()) == false)
                    {
                        DataGridViewTextBoxColumn newColumn = new DataGridViewTextBoxColumn();
                        newColumn.DataPropertyName = evdata.listOfRows[i].Cells[0].Value.ToString();
                        newColumn.HeaderText = evdata.listOfRows[i].Cells[0].Value.ToString();
                        newColumn.Width = 70;
                        newColumn.Name = evdata.listOfRows[i].Cells[0].Value.ToString();

                        this.dataEventsTable.Columns.Add(newColumn);
                    }
                }

                this.dataEventsTable.Rows.Add();

                // заполняем данными
                this.dataEventsTable.Rows[this.dataEventsTable.RowCount - 2].Cells[0].Value = evdata.name;
                String Umova = "";
                for (int i = 0; i < evdata.listOfRows.Count; i++)
                {
                    if (evdata.listOfRows[i].Cells[2].Value == null)
                        Umova = Umova + " " + "'" + evdata.listOfRows[i].Cells[0].Value.ToString() + "' ";
                    else
                        Umova = Umova + evdata.listOfRows[i].Cells[2].Value.ToString() + " " + "'" + evdata.listOfRows[i].Cells[0].Value.ToString() + "' ";
                }
}
}}}
Это все должна исполнять отдельная 
функция, а не обработчик интерфейса.



2. Имена переменных необходимо подбирать из 
английского языка для простоты понимания 
кода.

Form1.cs
{{{
String Umova = "";
}}}

Umova нужно заменить на Condition.



3. Для упрощения чтения кода, необходимо 
длинные конструкции кода заменять на 
ссылки, которые будут более компактными и к 
ним будет проще обратится, или методы get/set.

Form1.cs
{{{
Umova = Umova + evdata.listOfRows[i].Cells[2].Value.ToString() + " " + "'" + 
evdata.listOfRows[i].Cells[0].Value.ToString() + "' ";
}}}

public List<DataGridViewRow> listOfRows;
Нельзя передавать кусок интерфейса вместе 
с информацией. В свою очередь нужно 
передать определенную информацию.
evdata.listOfRows[i].Cells[2].Value.ToString() 
Нужно заменить на метод get, с двумя 
параметрами (номер столбца и номер строки)



4. Для хранения данных с последующей 
возможностью добавления новых параметров 
или значений, списковые данные нужно 
хранить в отдельном классе-оболочке.

AddEvent.cs
{{{
 public AddEvent()
        {
            InitializeComponent();


            // заполения списка событий
            List<String> listEv = new List<string>();
            listEv.Add("Отказ левого двигателя");
            listEv.Add("Отказ правого двигателя");
            listEv.Add("Отказ системы управления");
            listEv.Add("Отказ системы охлаждения двигателей");
            listEv.Add("Отказ срабатываня шасси");
            listEv.Add("Турбулентность");
            listEv.Add("Отказ датчика высоты");
            listEv.Add("Отказ датчика скорости");
            listEv.Add("Терорист на самолете");
            SetListEvents(listEv);

            // заполнение списка факторов
            List<String> listFac = new List<string>();
            listFac.Add("Время (мин)");
            listFac.Add("Высота (м)");
            listFac.Add("Далность (км)");
            SetListFactors(listFac);

            // заполнение списка умов
            List<String> listUm = new List<string>();
            listUm.Add("и");
            listUm.Add("или");
            //listUm.Add("не");
            SetListYmov(listUm);


        }
}}}

В данном случае, необходимо добавить 
класс-контейнер который бы вмещал в себе 
все возможные условия и данные. Для 
хранения было бы целесообразно 
использовать структуру данных Dictionary, что 
позволило бы иметь как и русский так и 
английский еквивалент событий и условий, 
что в будущем бы значительно упростило 
оброботу данных



5. Как и в пункте 2. проблема названия 
структурных едениц. В этом случае методов.

AddEvent.cs
{{{
 public void SetListYmov(List<String> list)
        {
            for (int i = 0; i < list.Count; i++)
                this.Column3.Items.Add(list[i]);
        }
}}}



6. Не целесообразное использование 
структуры. Необходимо поменять на другую 
структуру данных.

AddEvent.cs
{{{
public struct EventData
    {
        public String name;
        public List<DataGridViewRow> listOfRows;
    }
}}}

Структуру необходимо заменить на List или 
масив.




7. Спецификаторы доступа отвечающие за 
инициализацию интерфейса при открытии 
окна должны быть private, во избежание проблем 
с инкапсуляцией и несанкционированом 
доступе.

AddEvent.cs
{{{
        public void SetListEvents(List<String> list)
        {
            for (int i = 0; i < list.Count; i++)
                this.ListOfEvents.Items.Add(list[i]);
        }
        public void SetListFactors(List<String> list)
        {
            for (int i = 0; i < list.Count; i++)
                this.Column1.Items.Add(list[i]);
        }
        public void SetListYmov(List<String> list)
        {
            for (int i = 0; i < list.Count; i++)
                this.Column3.Items.Add(list[i]);
        }
}}}



8. В коде компонента необходимо больше 
комментировать сложные операции и 
описание сложных методовдля простоты 
чтения и понимания кода.

1. Необходимо добавить выпадающий список 
логических операторов (=,>,>=,<,<=,!=)

2. Необходимо сделать так, что б выбор 
логических операторов (и, или) относился от 
первого значения к следующему. Таким 
образом, что бы нельзя было добавить 
логическое условие к последнему выражению. 
К примеру: 1)высота = 3 ИЛИ 2) время = 4. После 2 
условия мы не должны добавить логическое 
условие.

3. Необходимо решить проблему допустимости 
введения пустых ячеек. Нужно сделать, что 
бы происходила проверка заполнености 
ячеек данных.