Zyzf/PIbd-22_Kalyshev_Y_V_MotorBoat_Base
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Laba3 PIbd-22 Kalyshev Y V #6

Merged
eegov merged 2 commits from Laba3 into Laba2 2022-10-14 09:11:25 +04:00
Conversation 0 Commits 2 Files Changed 15 +725 -340
Zyzf commented 2022-10-08 15:53:03 +04:00
Owner
Copy Link
No description provided.
Zyzf added 2 commits 2022-10-08 15:53:03 +04:00
Zyzf requested review from eegov 2022-10-08 15:53:25 +04:00
eegov approved these changes 2022-10-14 09:11:18 +04:00
eegov left a comment
Collaborator
Copy Link

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

Зачтено с понижением балла(ов)
PIbd-22_Kalyshev_Y_V_MotorBoat_Base/SetBoatsGeneric.cs
@ -0,0 +33,4 @@
/// <returns></returns>
public int Insert(T boat)
{
for (int i = 0; i < _places.Length; i++)
eegov commented 2022-10-14 09:10:49 +04:00
Collaborator
Copy Link

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

Правильнее было вызвать Insert(T obj, 0);
PIbd-22_Kalyshev_Y_V_MotorBoat_Base/SetBoatsGeneric.cs
@ -0,0 +49,4 @@
/// <param name="boat">Добавляемый автомобиль</param>
/// <param name="position">Позиция</param>
/// <returns></returns>
public bool Insert(T boat, int position)
eegov commented 2022-10-14 09:11:05 +04:00
Collaborator
Copy Link

Операция должна возвращать не bool тип

Операция должна возвращать не bool тип
eegov merged commit a61811fb0a into Laba2 2022-10-14 09:11:25 +04:00
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: Zyzf/PIbd-22_Kalyshev_Y_V_MotorBoat_Base#6
No description provided.
Delete Branch "Laba3"

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?