C# - "obiekt" klasy jako parametr funkcji

Mam sobie 2 klasy, monsterka i playera, w kazdej z nich mam wartosc:

Kod:

---

public int health,
public int maxHealth

---

no i tworze sobie te kreatury i chce zrobic funkcje ktora pobeira ich hp czyli:

Kod:

---

        public int getHealth()
        {
                return health;
        }

---

i niby wszystko dziala i jest ok, ale skoro w obu klasach mam taka sama zmienna to czy nieda sie moze stworzyc funkcji 1 raz np w oddzielnej klasie z funkcjami tak zeby dzialalo to na zasadzie

Kod:

---

        public int getHealth(creature)
        {
                return creature.health;
        }

---

Bo nie bardzo wiem jak moge przekazac te creature do funkcji, tj jako co to okreslic :D
Tak dla wizualizacji, postac tworze w nastepujacy sposob;

Kod:

---

Character Postac = new Character("Arkam", 100, 100, 0, 1,1,1,1,0,-1);

---

Wiec tak jak mowilem, problem glownie z przekazaniem tej kreatury, czy id do funkcji. Z gory dzieki :)


##edit
zrobilem tak xd juz chyba nvm

Kod:

---

public class Functions {
       
        public int getHealth(Character creature)
        {
                return creature.health;
        }
       
        public int getHealth(Enemy creature)
        {
                return creature.health;
        }
}


//Przyklad_1
Functions funkcje = new Functions ();
                funkcje.getHealth (Postac);

//Przyklad_2
Functions funkcje = new Functions ();
                funkcje.getHealth(Przeciwnik);

---

Przedstawione przez Ciebie jest nieprawidłowe, może i działa, ale tak się nie pisze. Poczytaj sobie o dziedziczeniu class.

Poczytaj sobie o polimorfizmie - w skrócie, używamy dziedziczenia bo skoro obie klasy mają te same pola to można zrobić klasę (matkę?) np Creature i żeby Player i Monster po niej dziedziczyły (tj. kopiowały wszystkie pola i funkcje które są w niej do siebie)

Tutaj przykładowy kod:

Kod:

---

using System;

namespace NaTorga
{
    class Creature
    {
        public int health;
        public int maxHealth;
    }
    class Player : Creature
    {
        //dodatkowe rzeczy tylko dla playera
    }
    class Monster : Creature
    {
        // dodatkowe rzeczy dla monstera
    }

   

    class Program
    {
        public static int getHealth(Creature c)
        {
            return c.health;
        }

        static void Main(string[] args)
        {
            Player p1 = new Player();
            Monster m1 = new Monster();

            p1.health = 100;
            m1.health = 200;

            Console.WriteLine(getHealth(p1));
            Console.WriteLine(getHealth(m1));

            Console.Read();
        }
    }
}

---

edit: tutaj dobry artykuł na msdn'nie https://msdn.microsoft.com/pl-pl/lib...v=vs.120).aspx

@Yakuza ; wonchaj masło

Cytuj:

---

sw.5 napisał

Poczytaj sobie o polimorfizmie - w skrócie, używamy dziedziczenia bo skoro obie klasy mają te same pola to można zrobić klasę (matkę?) np Creature i żeby Player i Monster po niej dziedziczyły (tj. kopiowały wszystkie pola i funkcje które są w niej do siebie)

Tutaj przykładowy kod:

Kod:

---

using System;

namespace NaTorga
{
    class Creature
    {
        public int health;
        public int maxHealth;
    }
    class Player : Creature
    {
        //dodatkowe rzeczy tylko dla playera
    }
    class Monster : Creature
    {
        // dodatkowe rzeczy dla monstera
    }

   

    class Program
    {
        public static int getHealth(Creature c)
        {
            return c.health;
        }

        static void Main(string[] args)
        {
            Player p1 = new Player();
            Monster m1 = new Monster();

            p1.health = 100;
            m1.health = 200;

            Console.WriteLine(getHealth(p1));
            Console.WriteLine(getHealth(m1));

            Console.Read();
        }
    }
}

---

edit: tutaj dobry artykuł na msdn'nie https://msdn.microsoft.com/pl-pl/lib...v=vs.120).aspx

---

JA CI ZARAZ DAM TE GETTERY Z DUPY

Po pierwsze

Kod:

---

public int getHealth()
        {
                return health;
        }

---

w c#

robi się tak

Kod:

---

public int Health
{
get
{
return health;
}
}

---

PO DRUGIE W C# używasz PascalCase

PO 3 naucz się języka zanim zaczniesz pisać bo gwałcisz tylko jakoby piękny język c# ...

Cytuj:

---

Piżmak Terrorysta napisał

PO DRUGIE W C# używasz PascalCase

---

a co microsoft tak nakazał że w C# tylko pascal case? xD

Raczej spolecznosc, i ma racje. Blagam trzymac sie standardow, to na prawde ulatwia prace.

Aj dobra all up, sorry xd za dużo bez snu i nie wiem co próbowałem zrobić. Właściwie to jak się połozyłem spać to ogarnalem co zrobilem xd problem rozwiązany, przepisalem to z dziedziczeniem, temat do zamkniecia.
//Anyway thz za chęć pomocy ;p

@Mickey Mouse ; szacun dla Ciebie że jako laik zobaczyłeś potrzebe korzystania z dziedziczenia :)

Cytuj:

---

joozioom napisał

a co microsoft tak nakazał że w C# tylko pascal case? xD

---

ucz .net deva jak pisać c# weź ty lepiej wracaj oglądać demotywatory

Cytuj:

---

Bazan napisał

Raczej spolecznosc, i ma racje. Blagam trzymac sie standardow, to na prawde ulatwia prace.

---

Jaka firma, taki standard.

@topic
Zapraszamy na IRC :)

Cytuj:

---

Piżmak Terrorysta napisał

bo gwałcisz tylko jakoby piękny język c#

---

A Tobie zdecydowanie polecam dokształcić się co oznacza słowo "jakoby" bo widocznie słabo Ci to idzie.

Oglądałem nad narnią wiem o co w tym słowie chodzi

Dzięki za pomoc :D jeszcze jeden problem, pewnie znów coś machnąłem albo przez niewiedzę źle zrobiłem ;(
Mianowicie problem polega na tym, że nie bardzo wiem jak mogę wywołać wewnątrz klasy metodę która znajduje się w klasie niżej.

Poniżej kawałek kodu:

Kod:

---

public class Creature {
        public int health;
        public int Health
        {
                get
                {
                        return health;
                }
                set
                {
                        this.health = value;
                        //Jak tutaj wywolac funkcje HealthBar.ChangeGreenFill()?
                }
        }
       
        public void spawn()
        {
                //Tutaj troche kodu odpowiedzialnego za spawnowanie, tj. dobieranie mu grafiki, dodawanie do obiektu BoxCollidera itd.  +
               
                GameObject s_Object = new GameObject (); //Obiekt do ktorego pozniej przypinam ww. collidery etc.
                //HealthBar
                HealthBar healthBar = new HealthBar (s_Object);
                healthBar.Create ()
               
        }
       
        //Obsluga graficznego hp bara
        public class HealthBar
        {
                public void ChangeGreenFill()
                {
                        //Tutaj kod zmieniajacy ilosc zielonego pola na pasku hp                       
                }
        }
       

}

---

Z góry dzięki

tak z ciekawosci, po co trzymasz klasę w klasie?

@edit
jak sie wczytałem w kod to widze ze kompletnie nie ogarniasz tematu ;/
po co Ci 2 publiczne pola health?

Zobacz jak powinny wyglądać właściwości w C# (bo chyba o to Ci chodzilo) - http://www.cezarywalenciuk.pl/post/2...iwosci-15.aspx

Ponadto nie trzymaj klasy HealthBar tylko obiekt tej klasy wewnątrz klasy Creature.

Cytuj:

---

Kusterek napisał

tak z ciekawosci, po co trzymasz klasę w klasie?

@edit
jak sie wczytałem w kod to widze ze kompletnie nie ogarniasz tematu ;/
po co Ci 2 publiczne pola health?

Zobacz jak powinny wyglądać właściwości w C# (bo chyba o to Ci chodzilo) - http://www.cezarywalenciuk.pl/post/2...iwosci-15.aspx

Ponadto nie trzymaj klasy HealthBar tylko obiekt tej klasy wewnątrz klasy Creature.

---

W sumie to sam nie wiem czemu tak zrobiłem, ale czy coś to zmienia w moim problemie jeżeli wyciągnę ja poza klasę Creatures?

Noooo, wtedy w klasie Creatures będziesz miał obiekt, jak będziesz miał obiekt to będziesz mógł wywołać jego metodę, a czy to nie jest rozwiązanie problemu?

#UP wielkie dzięki xd wyrzuciłem klasę hp bara poza obecną klasę, wrzuciłem w klasie creatures

Kod:

---

public HealthBar zycie;

---

+ w funkcji spawnujacej

Kod:

---

                zycie = new HealthBar (s_Object);
                zycie.Create ();

---

a następnie w funkcji health

Kod:

---

        public int Health
        {
                get
                {
                        return health;
                }
                set
                {
                        this.health = value;
                        zycie.arkam();
                }
        }

---

//te arkam to tak 4test :D

Ja polecam zeby ta Twoja funkcja spawn była konstruktorem, poczytaj w necie czym jest konstruktor.

@Kusterek ; wiem czym jest konstruktor tylko w klasie Creature tymczasowo wrzuciłem to jako void bo ogarniałem jeszcze kilka innych rzeczy i jeszcze nie jestem do końca pewien co gdzie najlepiej mi wetknąć :D ale dzięki za pomoc jeszcze raz.

To zadawaj dużo pytań a najlepiej wchodź na naszego IRCa (najwiecej osob w godz 9-17), bo pisząc zdupy możesz sobie narobic złych nawyków i w przyszlości mieć problemy

do seta sobie sprawdzanie dodaj xd

Kod:

---

private int _health;
public int Health
        {
                get
                {
                        return _health;
                }
                set
                {
                    if(_health != value)
                    {
                      _health = value;
                      zycie.arkam();
                    }
                }
        }

---

Jeszcze takie pytanie mam :D czy to co opisze niżej jest dobrym sposobem czy lepiej jak zrobię to jakoś inaczej?

Stworzyłem klasę Creatures, ma ona metodę:

Kod:

---

    public virtual void FillStatistics()
    {
        Debug.Log("ERROR: Fill the statistics of " + this.name); // tu bedzie critic error pozniej
    }

---

Podległe klasie Creatures mam m.in dwie inne klasy tj. Player oraz Monster
I tutaj moje pytanie, czy wypełnianie statystyk w taki sposób:

Kod:

---

 
    public override void FillStatistics()
    {
        this.Mana = 200;
        this.SpecialAttackPoints = 0;
        this.Sila = 1;
        this.Magic = 1;
        this.Agility = 1;
        this.Defense = 1;
        this.Position = new Vector3 (0, -1, 0);
        this.Outfit = 1;
    }

---

Jest dobrym pomysłem? tj. dane te będą pobierane z pliku zapisu ale jest to przykładowe wypełnienie. Czy powinienem użyć jeszcze czegoś innego żeby to lepiej zapisać? :D z góry dzięki.

//Uprzedzam od razu, że wcześniej miałem wszystko w konstruktorach ale było trochę mało czytelne więc postawiłem na metody.
//W konstruktorach zostawiłem tylko nick i startowe hp

Nie widzę w tym problemu żeby tak zrobić,

wywołanie tej metody będzie w konstruktorze?

Cytuj:

---

Mickey Mouse napisał

//Uprzedzam od razu, że wcześniej miałem wszystko w konstruktorach ale było trochę mało czytelne więc postawiłem na metody.

---

@Mickey Mouse ;
Jaki sens (poza tym że wziales kawałek kodu z jednego miejsca, i zrobiles z tego metode) ma ta metoda? Rownie dobrze mogloby to byc 10 metod, ktore robia 1 z tych rzeczy. Przenies to spowrotem do konstruktora (czesc do konstruktora base), bo to jest miejsce inicjalizowania obiektu. Pozatym, to co zrobiles to two-steps initialization (http://stackoverflow.com/q/29209922/4200455).

Cytuj:

---

szulak napisał

@Mickey Mouse ;
Jaki sens (poza tym że wziales kawałek kodu z jednego miejsca, i zrobiles z tego metode) ma ta metoda? Rownie dobrze mogloby to byc 10 metod, ktore robia 1 z tych rzeczy. Przenies to spowrotem do konstruktora (czesc do konstruktora base), bo to jest miejsce inicjalizowania obiektu. Pozatym, to co zrobiles to two-steps initialization (http://stackoverflow.com/q/29209922/4200455).

---

Trochę źle się wyraziłem :D w sumie racja niepotrzebnie to wyciągnąłem poza konstruktor. Obecnie wrzuciłem tyle ile mogłem do konstruktorów tj np.

Kod:

---

        public Player(string Name)
        {
        this.Mana = 200;
        this.SpecialAttackPoints = 0;
        this.Sila = 1;
        this.Magic = 1;
        this.Agility = 1;
        this.Defense = 1;
        this.Position = new Vector3(0, -1, 0);
        this.Outfit = 1;
        }

---

Kod:

---

    public Creature()
    {
        this.maxHealth = 155;
        this.health = this.MaxHealth;
    }

---

Tutaj przynajmniej narazie zakończe temat klas itd. Zaraz siadam nad napisaniem obsługi plików xml w tym to sobie ułatwię późniejsze aktualizacje.

//Moglibyście mi doradzić czego powinienem się teraz uczyć? o klasach, polimorfizmie itd. obejrzałem już masę tutoriali i została mi chyba tylko praktyka.

Praktyka, a w miedzyczasie wzorce projektowe.

a tak z Waszego doświadczenia, opłaca się kupować jakiekolwiek książki z C#? jeżeli tak to dajcie jakieś ciekawe tytuły, bo z C++ kupowałem kilka i tylko 1 była ogarnięta dość dobrze.

Cytuj:

---

Mickey Mouse napisał

a tak z Waszego doświadczenia, opłaca się kupować jakiekolwiek książki z C#? jeżeli tak to dajcie jakieś ciekawe tytuły, bo z C++ kupowałem kilka i tylko 1 była ogarnięta dość dobrze.

---

nie wiem czy się opłaca ale nie czytałem nigdy, lepiej ucz się się z neta do tego i tak najwięcej się w pierwszej pracy naumiesz

ok to przynajmniej na teraz do końca miesiąca pomęczę jeszcze te klasy w każdym wariancie i spróbuje ogarnąć jakieś tutoriale z tymi wzorcami projektowymi. Widziałem są jakieś video-tutki dla javy ale z tego co wyczytałem wzorce są niezależne od języka programowania a java tak czy tak jest bardzo podobna do C# to zawsze się czegoś naumiem :D

Cytuj:

---

Mickey Mouse napisał

a tak z Waszego doświadczenia, opłaca się kupować jakiekolwiek książki z C#? jeżeli tak to dajcie jakieś ciekawe tytuły, bo z C++ kupowałem kilka i tylko 1 była ogarnięta dość dobrze.

---

@Wonsz Zabujca ;
Dlaczego "lepiej ucz sie z neta" ? Jakies argumenty za tym?

@Mickey Mouse ;
Oczywiscie, ze sie oplaca. Zakladajac, ze ksiazka to koszt 100zl, a swoja prace wycenisz na 25zl/h, to praca 4h zwroci Ci ksiazke. Anyway.. przetestowalem kilka sposob 'nauki' programowania, i w moim wypadku, najbardziej sprawdza sie ksiazka + praktyka - tj., czytam rozdzial (lub subrozdział), i na podstawie zdoybtej wiedzy cos kodze. W prakryce wyglada to tak, ze 2 dni czytam - 1 dzien kodze. A pozatym, zapraszamy na IRCa.

Rusz Głową C# ;D polecam już nty raz ;P

Cytuj:

---

Kusterek napisał

Rusz Głową C# ;D polecam już nty raz ;P

---

ok zaraz kupię ebooka :D
@szulak ; jutro wejdę na irc

Cytuj:

---

szulak napisał

@Wonsz Zabujca ;
Dlaczego "lepiej ucz sie z neta" ? Jakies argumenty za tym?

@Mickey Mouse ;
Oczywiscie, ze sie oplaca. Zakladajac, ze ksiazka to koszt 100zl, a swoja prace wycenisz na 25zl/h, to praca 4h zwroci Ci ksiazke. Anyway.. przetestowalem kilka sposob 'nauki' programowania, i w moim wypadku, najbardziej sprawdza sie ksiazka + praktyka - tj., czytam rozdzial (lub subrozdział), i na podstawie zdoybtej wiedzy cos kodze. W prakryce wyglada to tak, ze 2 dni czytam - 1 dzien kodze. A pozatym, zapraszamy na IRCa.

---

duża część wiedzy jest dostępna na necie szczególnie podstaw do tego widziałem po znajomych ich naukę z książek i zawsze kończyło się przepisywaniem z książki na żywca więc wg mnie jak książki to raczej nie do podstaw

Szkoda mi trochę zakładać nowy temat a nadal problem z C# więc odkopię ten. Miałem sporą przerwę z C# i mam mały problem.
Mam 3 praktycznie identyczne funkcje zwracające 3 różne wartości ale nie mam pomysłu jak zrobić z tych trzech jedną która będzie zwracała tablice z 3 różnymi wartościami. Bo bez sensu robić 3x to samo przy późniejszym ustawianiu tych spelli skoro można krócej. Z góry dzięki za pomoc.

Kod:

---

  static float SPELL_TIME(ATTACKS attacks)
    {
        switch(attacks)
        {
            case ATTACKS.ATTACK_MELEE:
            case ATTACKS.BLUE_BALL:
            case ATTACKS.NIGHTMARE_BALL:
            case ATTACKS.THUNDERSTORM:
                return 5.0f;
            default:
                return 1.0f;
        }
    }

    static float SPELL_SLIDER_SPEED(ATTACKS attacks)
    {
        switch (attacks)
        {
            case ATTACKS.ATTACK_MELEE:
            case ATTACKS.BLUE_BALL:
            case ATTACKS.NIGHTMARE_BALL:
            case ATTACKS.THUNDERSTORM:
                return 3.0f;
            default:
                return 1.0f;
        }
    }

    static int SPELL_BASE_DMG(ATTACKS attacks)
    {
        switch (attacks)
        {
            case ATTACKS.ATTACK_MELEE:
            case ATTACKS.BLUE_BALL:
            case ATTACKS.NIGHTMARE_BALL:
            case ATTACKS.THUNDERSTORM:
            default:
                return 100;
        }
    }

---

lepiej zebys w enumie/klasie przy okazji przetrzymywał takie dane jak baseDmg, sliderSpeed itd

Cytuj:

---

Terr napisał

lepiej zebys w enumie/klasie przy okazji przetrzymywał takie dane jak baseDmg, sliderSpeed itd

---

W klasie nie bardzo mogę. A jak mam to zrobić w enumie?

Cytuj:

---

Mickey Mouse napisał

Szkoda mi trochę zakładać nowy temat a nadal problem z C# więc odkopię ten. Miałem sporą przerwę z C# i mam mały problem.
Mam 3 praktycznie identyczne funkcje zwracające 3 różne wartości ale nie mam pomysłu jak zrobić z tych trzech jedną która będzie zwracała tablice z 3 różnymi wartościami. Bo bez sensu robić 3x to samo przy późniejszym ustawianiu tych spelli skoro można krócej. Z góry dzięki za pomoc.

---

Kod:

---

using System;
using System.Collections.Generic;
using System.Reflection;

public enum AttackType
{
        Melee,
        BlueBall,
        NightmareBall,
        Thunderstorm
};

public struct Spell
{
        public AttackType attackType;
        public double time;
        public double sliderSpeed;
        public int baseDmg;
       
        public Spell(AttackType attackType, double time, double sliderSpeed, int baseDmg)
        {
                this.attackType = attackType;
                this.time = time;
                this.sliderSpeed = sliderSpeed;
                this.baseDmg = baseDmg;
        }
}


public class Test
{
       
        static T GetSpellProperty<T>(String name, Spell spell)
        {
                FieldInfo field = typeof(Spell).GetField(name);
        return (T)(field.GetValue(spell));
        }
       
       
        public static void Main()
        {
                List<Spell> knownSpells = new List<Spell>();
               
                knownSpells.Add(new Spell(AttackType.Melee, 1.0f, 0.5f, 20));
                knownSpells.Add(new Spell(AttackType.BlueBall, 2.0f, 0.5f, 40));
                knownSpells.Add(new Spell(AttackType.NightmareBall, 4.0f, 0.5f, 50));

                Console.WriteLine(GetSpellProperty<int>("baseDmg", knownSpells[0]));
        }
}

---

Cytuj:

---

szulak napisał

Kod:

---

using System;
using System.Collections.Generic;
using System.Reflection;

public enum AttackType
{
        Melee,
        BlueBall,
        NightmareBall,
        Thunderstorm
};

public struct Spell
{
        public AttackType attackType;
        public double time;
        public double sliderSpeed;
        public int baseDmg;
       
        public Spell(AttackType attackType, double time, double sliderSpeed, int baseDmg)
        {
                this.attackType = attackType;
                this.time = time;
                this.sliderSpeed = sliderSpeed;
                this.baseDmg = baseDmg;
        }
}


public class Test
{
       
        static T GetSpellProperty<T>(String name, Spell spell)
        {
                FieldInfo field = typeof(Spell).GetField(name);
        return (T)(field.GetValue(spell));
        }
       
       
        public static void Main()
        {
                List<Spell> knownSpells = new List<Spell>();
               
                knownSpells.Add(new Spell(AttackType.Melee, 1.0f, 0.5f, 20));
                knownSpells.Add(new Spell(AttackType.BlueBall, 2.0f, 0.5f, 40));
                knownSpells.Add(new Spell(AttackType.NightmareBall, 4.0f, 0.5f, 50));

                Console.WriteLine(GetSpellProperty<int>("baseDmg", knownSpells[0]));
        }
}

---

---

Dzięki wielkie za pomoc.

Cytuj:

---

Mickey Mouse napisał

Dzięki wielkie za pomoc.

---

@Mickey Mouse ;
Tutaj masz live: https://ideone.com/k75KtV