Skip to content

task5 is done#21

Open
South12309 wants to merge 2 commits intoFAANG-School:masterfrom
South12309:task_5
Open

task5 is done#21
South12309 wants to merge 2 commits intoFAANG-School:masterfrom
South12309:task_5

Conversation

@South12309
Copy link

task5 is done

try {
Thread.sleep(2000);
} catch (InterruptedException e) {
throw new RuntimeException(e);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Логируем

throw new BattleException(heroName + " dont have mana to use " + magicItem.getItemName());
}
Long l = itemUsages.get(magicItem.getItemName());
if (l==null) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Такого не может быть

throw new BattleException(heroName + " dont have mana to use " + magicItem.getItemName());
}
Long l = itemUsages.get(magicItem.getItemName());
if (l==null) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Получай long, а не Long

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants