[C#] Inferno Infinity

murkata86 10 Точки

[C#] Inferno Infinity

Здравейте,

Имам известни проблеми с тази задача и по - точно с тестовере в Judge. От всичките ми минава само първия и не разбирам защо не минават и останалите. И един въпрос относно END. В условието пише "Upon receiving the END command print the weapons in order of their appearance in the format provided below."

Ако няма Print на всеки Weapon, трябва ли да ги принтя, понеже, ако ги изпринтя нищо не минава в Judge. Има го и момента, че изчислявам stats за всеки Weapon, докато го създавам - имам клас за всяко оръжие - Common Axe, Rare Axe, Common Knife, Epic Knife, etc

.https://onedrive.live.com/embed?cid=0215B22FCDC531A6&resid=215B22FCDC531A6%21493&authkey=AIdlyxRdKy1NGNI

Поздрави,

Иван

Тагове:

29/07/2016 11:23:03 C# OOP Basics

Виж всички отговори

ArmenPotourlyan+deleted! 488 Точки

Измислих един тест и ето резултата като го пуснах по твоето и моето решение. Надявам се, че ще ти помогне да си намериш грешката като го дебъгнеш :)))

IN:

Create;Rare Sword;Harmony
Add;Harmony;0;Chipped Emerald
Add;Harmony;2;Flawless Ruby
Print;Harmony
END

OUT: (твоето решение)

Harmony: 1773-3087 Damage, +19 Strength, +17 Agility, +25 Vitality
Harmony: 1773-3087 Damage, +19 Strength, +17 Agility, +25 Vitality

OUT: (моето решение)

Harmony: 67-143 Damage, +19 Strength, +17 Agility, +25 Vitality

MIN DAMAGE:
4(DefaultMinDamage) * 3(Rarity) + 19(Strength) x 2 (StreghtToMinDamageConst) + 17(Agility) x 1 (AgilityToMinDamage)

MAX DAMAGE:
6(DefaultMaxDamage) * 3(Rarity) + 19(Strength) x 3 (StreghtToMaxDamageConst) + 17(Agility) x 4 (AgilityToMaxDamage)

29/07/2016 12:01:53

murkata86 10 Точки

След като получа новия MinDamage от Strength-а, го умножавам по Rarity (12 (minDamage) + (2 strength * 2) = 16 * Rare = 48 :)

Разбрах от къде ми идва грешката :)

Мерси за помощта!

Поздрави,

Иван

29/07/2016 12:55:19

stefankos 1 Точки

Здрасти Мурка.

Видях една хурка и искам да я споделя с теб :)

(това в кръга на шегата не се обиждай :) )

В Main ти инциализираш инвенторито си но си го експознал на всички.

Този код е напълно валиден!

IInventory inventory = new Inventory();
inventory.Weapons.Clear();

Видях че в интерфейса си оставил само get; , но това продължава да ти дава пълен достъп до списъка ти.

Поздрави

Стефан

01/08/2016 13:03:31 01/08/2016 13:47:18

murkata86 10 Точки

Между другото имах и такъв коментар на изпита, че листа ми остава експоузнат и все се каня да питам за това и все забравям...

Как всъщност се прави това?

01/08/2016 22:03:14

ArmenPotourlyan+deleted! 488 Точки

Един вариант е да го направиш IReadOnlyList<T>:

public IReadOnlyList<IWeapon> Weapons => this.weapons;

Друг вариант е да копираш листа в нов такъв:

public IList<IWeapon> Weapons => new List<IWeapon>(this.weapons);

01/08/2016 23:09:04

stefankos 1 Точки

Това първото е много яко но ми казаха че може да се кастне и полсе стигаш до същата ситуация:

public IReadOnlyList<IWeapon> Weapons => this.weapons;

var test = (List<Weapon>)weapons.Weapons;
test.Clear();

Второто с копирането би работило , но в ситуация с огромна база данни копирането би заело много памет и performance.

В тази конкретна задача аз съм си направил списъка private и съм експознал само методите/командите , които са ми дали в условието .

Честно казано и аз се чудя за по добър метод да се скрие базата данни.

02/08/2016 09:33:43

Виж всички отговори