dyakonovr/PIbd-23_Dyakonov_R_R_Tank
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

All done #3

Closed
dyakonovr wants to merge 1 commits from lab3 into lab2
pull from: lab3
merge into: dyakonovr:lab2
Conversation 0 Commits 1 Files Changed 11 +631 -66
dyakonovr commented 2023-10-31 12:44:22 +04:00
Owner
Copy Link
No description provided.
dyakonovr added 1 commit 2023-10-31 12:44:22 +04:00
eegov approved these changes 2023-11-05 15:50:03 +04:00
eegov left a comment
Collaborator
Copy Link

Пул правильно называть надо, с указанием группы, ФИО и номера лабораторной

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

Пул правильно называть надо, с указанием группы, ФИО и номера лабораторной Зачтено с понижением балла(ов)
ProjectTank/FormTankCollection.Designer.cs
@ -0,0 +113,4 @@
private PictureBox pictureBoxCollection;
private Label label1;
private Button button3;
eegov commented 2023-11-05 15:49:09 +04:00
Collaborator
Copy Link

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

У элементов форм, с которыми работаем в логике должны быть логичные имена
ProjectTank/FormTankCollection.Designer.cs
@ -0,0 +114,4 @@
private PictureBox pictureBoxCollection;
private Label label1;
private Button button3;
private Button button2;
eegov commented 2023-11-05 15:49:06 +04:00
Collaborator
Copy Link

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

У элементов форм, с которыми работаем в логике должны быть логичные имена
ProjectTank/FormTankCollection.Designer.cs
@ -0,0 +115,4 @@
private Label label1;
private Button button3;
private Button button2;
private Button button4;
eegov commented 2023-11-05 15:49:04 +04:00
Collaborator
Copy Link

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

У элементов форм, с которыми работаем в логике должны быть логичные имена
ProjectTank/Generics/SetGeneric.cs
@ -0,0 +29,4 @@
/// <returns></returns>
public int Insert(T tank)
{
if (_places[Count-1] != null)
eegov commented 2023-11-05 15:49:45 +04:00
Collaborator
Copy Link

А откуда такое условие?

А откуда такое условие?
eegov closed this pull request 2023-11-05 15:50:08 +04:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
eegov
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
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: dyakonovr/PIbd-23_Dyakonov_R_R_Tank#3
No description provided.
Delete Branch "lab3"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?