ISEbd21 Zinovev V.V. Lab Work 08 #21

Closed
vladimir_zinovev wants to merge 1 commits from TankBasicLabEighth into TankBasicLabSeventh
No description provided.
vladimir_zinovev added 1 commit 2023-12-14 22:02:34 +04:00
eegov approved these changes 2023-12-16 22:13:48 +04:00
eegov left a comment
Collaborator

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

Зачтено с понижением балла(ов)
@ -0,0 +23,4 @@
{
return x.EntityArmoredTransport.BodyColor.GetBrightness().CompareTo(y.EntityArmoredTransport.BodyColor.GetBrightness());
}
if (x.GetType().Name != y.GetType().Name)
Collaborator

Требовалось сортировать по критериям: цвет, скорость, вес

Требовалось сортировать по критериям: цвет, скорость, вес
@ -0,0 +17,4 @@
}
public bool Equals(ArmoredTransportsCollectionInfo? other)
{
return this.Name == other.Name;
Collaborator

Нет проверки, что other не равен null

Нет проверки, что other не равен null
@ -0,0 +41,4 @@
if (x is DrawningTank && y is DrawningTank)
{
EntityTank EntityX = (EntityTank)x.EntityArmoredTransport;
EntityTank EntityY = (EntityTank)y.EntityArmoredTransport;
Collaborator

Нет сравнения по дополнительному цвету

Нет сравнения по дополнительному цвету
eegov closed this pull request 2023-12-16 22:13:56 +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: vladimir_zinovev/ISEbd-21_Zinovev_V.V._Tank_Basic#21
No description provided.