Antonova D.V. Lab4 #4

Closed
AntonovaDaria wants to merge 3 commits from Lab4 into Lab3
No description provided.
AntonovaDaria added 3 commits 2022-12-16 23:02:16 +04:00
AntonovaDaria requested review from eegov 2022-12-16 23:02:37 +04:00
eegov approved these changes 2022-12-22 21:58:31 +04:00
eegov left a comment
Collaborator

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

Зачтено с понижением балла(ов)
@ -0,0 +51,4 @@
if (map == null || string.IsNullOrEmpty(name)) return;
// Если совершается попытка добавить карту с уже существующим именем, то выходим
if (_mapStorages.ContainsKey(name)) return;
Collaborator

Это действие должно быть первым в методе

Это действие должно быть первым в методе
@ -0,0 +64,4 @@
{
// Если имя удаляемой карты не задано - выходим
if (string.IsNullOrEmpty(name)) return;
_mapStorages.Remove(name);
Collaborator

Нет проверки, что такой ключ уже есть в словаре

Нет проверки, что такой ключ уже есть в словаре
@ -0,0 +77,4 @@
get
{
if (string.IsNullOrEmpty(ind)) return null;
return _mapStorages[ind];
Collaborator

Нет проверки, что такой ключ уже есть в словаре

Нет проверки, что такой ключ уже есть в словаре
@ -38,3 +40,1 @@
{
return Insert(jet, 0);
}
//public int Insert(T jet)
Collaborator

Закомментированного кода быть не должно

Закомментированного кода быть не должно
eegov closed this pull request 2022-12-22 21:58:41 +04:00

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: AntonovaDaria/ISEbd-22_Antonova_D.V._AirBomber._Base#4
No description provided.