Diskuze: Co dělám špatně??
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
Ziki:4.2.2015 0:10
Ono celej ten způsob vyřešení útoku se mi zdá špatnej, ty při
každým volání funkce Attack() snížíš jeho damage (takže po několika
voláních může být i záporný 
). a ten další řádek pod tím mi taky nepřijde správně a jak už
pako poznamenal, nikde se nesnižuje hp.
druhého bojovníka ti to nevypíše, protože celej ten while cyklus (ten s live > 0) se provede v rámci jednoho volání orc.Attack(elf). a protože se hodnota proměnné live nemění tak se ti to nekonečně zacyklí.
Kafac:4.2.2015 16:04
Přesně nad tím jsem taky přemýšlel,jestli udělat abstraktní třídu a
z toho zdědit Mága a Bojovníka.Nakonec jsem to udělal takhle,jelikož jsem
si říkal,že mágovi nebudu dávat manu a nepotřeboval by nic navíc oproti
válečníkovi,kdyby jo,tak bych to kdyžtak udělal s polymorfismem a v mágovi
bych to přepsal ten Attack,ale asi s tou abstraktní třídou to bude
lepší.Určitě to zkusím 
.
Kafac:4.2.2015 22:15
abstraktní třída:
class PlayerBase
{
protected string name;
protected int life;
protected int damage;
protected int defense;
public PlayerBase(string name,int life,int damage,int defense)
{
this.name = name;
this.life = life;
this.damage = damage;
this.defense = defense;
}
public void Attack(PlayerBase rival)
{
rival.life = rival.life + rival.defense;
damage = rival.life - this.damage;
Console.WriteLine("Hráč {0} zaútočil na hráče {1} a sebral mu životy za {2} hp",this.name,rival.name,damage);
}
}Warrior :
class Warrior:PlayerBase
{
public Warrior(string name,int life,int damage,int defense):base (name,life,damage,defense)
{
}
}Mage :
class Mage:PlayerBase
{
public Mage(string name,int life,int damage,int defense):base (name,life,damage,defense)
{
}
}a Program :
{
Warrior orc = new Warrior("Baltazar",100,25,50);
Mage shadow = new Mage("Blood",50,80,10);
orc.Attack(shadow);
shadow.Attack(orc);
Console.ReadKey();
}už to v pohodě útočí,bere i druhý hráč,zkusím tam dát ještě ten
cyklus,+ při 0 hp se přestane bojovat
sadlomaslox25:5.2.2015 7:48
no tak zrovna v tomhle pripade by mela byt metoda bud metoda Attack virtualni
a z podedenych tridach by se mela upravit protoze warrior asi utoci jinak nez
mage . pripadne udelat par
protected metod jako OnAttack a CalculateDamage a Attack bych prepsal na neco
jako
public void Attack(PlayerBase rival)
{
if(rival.OnAttack(this))
{ int damage=CalculateDamage(); rival.TakeDamage(damage);
this.DoDamage(damage); }
else
//nemuze utocit, (miss,dodge,parry,aura...)
hanpari:9.2.2015 20:00
První věc, kterou bych na tvém místě nedělal, je toto:
Console.WriteLine("Hráč {0} zaútočil na hráče {1} a sebral mu životy za {2} hp",this.name,name,writeDMG)Místo toho vrať hodnotu(ideální varianta) nebo si alespon udelej
jednoduchou třídu render, přes kterou budeš vypisovat. To má tu výhodu,
že ti stačí změnit kód na jednom místě, kdežto teď musíš vyeditovat
všechna místa v kódu, kde máš Console.WriteLine. Což je opravdu špatně
Zkus zauvažovat nad tím, že bys svou hru dodělal a pak chtěl místo textového výstupu grafický. Anebo jen jiné formátování. Kolik změn by to pro tebe znamenalo?
Další věc, kterou bych ti doporučil, je konsistentní řádkování. A hlavně maximálně cca 60-70 znaků na řádek. Řádky přes celý monitor opravdu nejsou pravé ořechové, nehledě na známou skutečnost, že dlouhé řádky se špatně čtou.
sadlomaslox25:9.2.2015 20:48
"Rád bych někoho požádal,jestli by přepsal aspoň 1 třídu"
no ty chyby se musi pochopit a nejlip je pochopis tak ze je udelas abys pak
vedel proc je nemas delat
.Kafac:10.2.2015 20:53
Zatím jsem nic neupravoval,ale zkusil jsem to takhle a útok proběhl bez chyby: PlayerBase:
public class PlayerBase
{
public string name;
public int life;
public int damage;
public int defense;
public PlayerBase(string name,int life,int damage,int defense)
{
this.name = name;
this.life = life;
this.damage = damage;
this.defense = defense;
}
public void Attack(PlayerBase rival)
{
rival.life = rival.life + rival.defense;
damage = rival.life - this.damage;
Console.WriteLine("Hráč {0} zaútočil na hráče {1} a sebral mu životy za {2} hp",this.name,rival.name,damage);
}Warrior:
class Warrior:PlayerBase
{
public Warrior(string name,int life,int damage,int defense):base (name,life,damage,defense)
{
}Mage :
class Mage:PlayerBase
{
public Mage(string name,int life,int damage,int defense):base (name,life,damage,defense)
{
}IronArmor :
class IronArmor
{
public bool Iarmor;
private int defense = 25;
public void IArmorWearing(PlayerBase user)
{
if (Iarmor == true)
{
user.defense = user.defense + this.defense;
}
else
{
defense = user.defense;
}
}Program.cs :
class Program
{
static void Main(string[] args)
{
Warrior orc = new Warrior("Star",250,80,120);
Mage elf = new Mage("Shadow",150,130,70);
IronArmor iron = new IronArmor();
iron.IArmorWearing(orc);
orc.Attack(elf);
elf.Attack(orc);
Console.ReadKey();
}Budu rád za všechnu kritiku .
Kafac:10.2.2015 21:12
Děkuji,tohle ještě určitě není finální,taky jsem si všiml,že to je k ničemu,ale jestli tam ještě něco přidám (chci tam přidat mágovi magickou útok a manu a válečníkovi něco s útokem),tak se to bude hodit.Ale zatím to můžu takhle předělat,jelikož teď asi nebudu mít moc času,tak aby to tam zbytečně nebylo.
Ještě nějaká kritika nebo rada?
Zobrazeno 34 zpráv z 34.