warhate / tl410-rias Goto Github PK
View Code? Open in Web Editor NEWAutomatically exported from code.google.com/p/tl410-rias
Automatically exported from code.google.com/p/tl410-rias
1. Необходимо добавить выпадающий список
логических операторов (=,>,>=,<,<=,!=)
2. Необходимо сделать так, что б выбор
логических операторов (и, или) относился от
первого значения к следующему. Таким
образом, что бы нельзя было добавить
логическое условие к последнему выражению.
К примеру: 1)высота = 3 ИЛИ 2) время = 4. После 2
условия мы не должны добавить логическое
условие.
3. Необходимо решить проблему допустимости
введения пустых ячеек. Нужно сделать, что
бы происходила проверка заполнености
ячеек данных.
Original issue reported on code.google.com by byakuya%[email protected]
on 10 Nov 2011 at 7:50
Attachments:
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. В коде компонента необходимо больше
комментировать сложные операции и
описание сложных методовдля простоты
чтения и понимания кода.
Original issue reported on code.google.com by byakuya%[email protected]
on 10 Nov 2011 at 8:29
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.