PIbd-21-MalafeevL.S._Cruiser_Base_Lab3 #3

Closed
mallafeev wants to merge 1 commits from lab3 into lab2
Owner
No description provided.
mallafeev added 1 commit 2023-10-21 14:13:59 +04:00
mallafeev requested review from eegov 2023-10-21 14:14:04 +04:00
eegov approved these changes 2023-10-29 14:56:52 +04:00
eegov left a comment
Collaborator

Зачтено с понижением балла(ов)

Зачтено с понижением балла(ов)
@ -144,0 +177,4 @@
DialogResult = DialogResult.OK;
}
private void FormCruiser_Load(object sender, EventArgs e)
Collaborator

Пустых методов быть не должно

Пустых методов быть не должно
@ -0,0 +117,4 @@
private GroupBox groupBoxTools;
private TextBox textBoxNumber;
private Button buttonAddCruiser;
private Button button3;
Collaborator

У элементов форм, с которыми работаем в логике должны быть логичные имена

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +118,4 @@
private TextBox textBoxNumber;
private Button buttonAddCruiser;
private Button button3;
private Button button2;
Collaborator

У элементов форм, с которыми работаем в логике должны быть логичные имена

У элементов форм, с которыми работаем в логике должны быть логичные имена
@ -0,0 +92,4 @@
private void FormCruiserCollection_Load(object sender, EventArgs e)
Collaborator

Пустых методов быть не должно

Пустых методов быть не должно
@ -0,0 +35,4 @@
/// </summary>
/// <param name="cruiser">Добавляемый лайнер</param>
/// <returns></returns>
public int Insert(T cruiser)
Collaborator

Правильнее было вызвать Insert(T obj, 0);

Правильнее было вызвать Insert(T obj, 0);
eegov closed this pull request 2023-10-29 14:57:05 +04:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mallafeev/PIbd-21_MalafeevL.S._Cruiser_Base#3
No description provided.