Klassen Kleine Übung zum Thema Klassen

[Tyor]

Ich habe mir in den letzten Tagen mit Hilfe einer Website die Grundlagen von Java beigebracht und wollte nun wissen ob ich grundlegende Fehler oder unnötige Mehrarbeit mache. Vor allem bei der Verwendung von Objekten bin ich mir nicht sicher. Hab mich an einem mini RPG versucht. Ein kurzes Feedback wäre nett.

import java.util.*;

public class Main {
    public static void main(String[] args) {
        Random rand = new Random();
        Char mainChar = new Char();
        Char enemy = new Char();
        mainChar.setStartHP(100);
        mainChar.setMaxHP(mainChar.getStartHP());
        mainChar.setCurrentHP(mainChar.getMaxHP());
        mainChar.setBaseAttack(10);
        mainChar.setAttack(mainChar.getBaseAttack());
        Fight kampf = new Fight();

        while (mainChar.getCurrentHP() > 0) {
            enemy.setBaseAttack(2 + mainChar.getLvl() * 2); // gegner generieren
            enemy.setAttack(enemy.getBaseAttack());//waffe noch einfügen
            enemy.setMaxHP(mainChar.getLvl() * mainChar.getLvl());
            enemy.setCurrentHP(enemy.getMaxHP());
            enemy.setGold(rand.nextInt(mainChar.getLvl() * 20));
           
            while (enemy.getCurrentHP() > 0 & mainChar.getCurrentHP() > 0)// kampfhandlung
            {
                enemy.setCurrentHP(kampf.kampfhandlung(
                        enemy.getCurrentHP(), mainChar.getAttack() + rand.nextInt(mainChar.getLvl())));
                mainChar.setCurrentHP(kampf.kampfhandlung(
                        mainChar.getCurrentHP(), enemy.getAttack()));
                System.out.println("Dein Leben: " + mainChar.getCurrentHP()
                        + ". \tDas Leben deines Gegners: " + enemy.getCurrentHP());
            }
           
            // belohnung für besiegten gegner
            if (mainChar.getCurrentHP() > 0) {
                mainChar.setLvl(mainChar.getLvl() + 1);
                mainChar.setMaxHP(mainChar.getMaxHP() + rand.nextInt(mainChar.getLvl()));//neues MaxHP
                System.out.println("Level UP! Du bist nun Level "
                        + mainChar.getLvl() + "!");
                mainChar.setGold(enemy.getGold() + mainChar.getGold());
                // heilung
                mainChar.setCurrentHP(mainChar.getCurrentHP()
                        + mainChar.getLvl() * 5);
                if (mainChar.getCurrentHP() > mainChar.getMaxHP())
                    mainChar.setCurrentHP(mainChar.getMaxHP());
                System.out.println("Dein Leben zu Begin des Kampfes beträgt: "
                        + mainChar.getCurrentHP());
            } else
                System.out.println("Dein maximales Level war "
                        + mainChar.getLvl() + " Du hattest "
                        + mainChar.getGold() + " Gold!");
        }

    }
}

​

 

[black swan]

while (enemy.getCurrentHP() > 0 & mainChar.getCurrentHP() > 0)// kampfhandlung
                                ^

Logische Verknüpfungen immer mit && nicht &.

Empfehlung (nicht Muss): Überlade Deinen Konstruktor zum Erzeugen von Charakteren. Beispiel:

    // Überladener Konstruktor
    public void Char (int hp, int ba,int mxhp, int sthp)  {
    this.hitpoint = hp;
    this.baseattack = ba;
    this.maxhp = mxhp;
    this.starthp = sthp;
    ...
}

Das führt dann von

    Char mainChar = new Char();      
    mainChar.setStartHP(100);
    mainChar.setMaxHP(mainChar.getStartHP());
    mainChar.setCurrentHP(mainChar.getMaxHP());
    mainChar.setBaseAttack(10);
    mainChar.setAttack(mainChar.getBaseAttack());

zu

    Char mainChar = new Char(100, 10, 100, 100); 
    mainChar.setAttack(mainChar.getBaseAttack()); //?

Wobei die letzte Zeile keinen Sinn ergibt: Du erzeugst ein Objekt, und weisst ihm einen Parameter zu, den Du vorher aus dem selbigen ausgelesen hast. Das sieht aus, als könnte es im Konstruktor erledigt werden (zumindest beim initialen Aufruf).

Ansonsten finde ich den Code ok. Bestimmt hat der ein oder andere User noch weitere Vorschläge, ich selbst programmiere Java auch noch nicht zu lange.

 

[Tyor]

Super vielen Dank! Hab das in meinen Code übernommen. Hab das Programm dementsprechend umgeschrieben. Habe vor allem versucht die main übersichtlich zu halten

import java.util.Random;

public class Main {
    public static void main(String[] args) {
        Random rand = new Random();
        Char mainChar = new Char();
        Char enemy = new Char();
        mainChar.iniChar(100, 10, 0, 1);
        Fight kampf = new Fight();

        while (mainChar.getCurrentHP() > 0) {
            enemy.iniEnemy(mainChar, rand);

            while (enemy.getCurrentHP() > 0 && mainChar.getCurrentHP() > 0)// kampfhandlung
            {
                kampf.kampfhandlung(mainChar, enemy, rand);
            }

            // belohnung für besiegten Gegner (LevelUP)
            if (mainChar.getCurrentHP() > 0) {
                mainChar.lvlUP(enemy);
                mainChar.healing();
            } else
                System.out.println("Dein maximales Level war "
                        + mainChar.getLvl() + " Du hattest "
                        + mainChar.getGold() + " Gold!");
        }

    }
}

 

[Joose]

Hab das in meinen Code übernommen. Hab das Programm dementsprechend umgeschrieben. Habe vor allem versucht die main übersichtlich zu halten

i
        while (mainChar.getCurrentHP() > 0) {.......

            while (enemy.getCurrentHP() > 0 && mainChar.getCurrentHP() > 0)// kampfhandlung
            {
                kampf.kampfhandlung(mainChar, enemy, rand);
            }
            // belohnung für besiegten Gegner (LevelUP)
            if (mainChar.getCurrentHP() > 0) {
                mainChar.lvlUP(enemy);
                mainChar.healing();
            } else
                System.out.println("Dein maximales Level war "
                        + mainChar.getLvl() + " Du hattest "
                        + mainChar.getGold() + " Gold!");
        }

Kleiner Tipp: Auch wenn es sich beim else Zweig um eine einzeilige Anweisung handelt, verwende am Besten immer Klammern bei Schleife und if/else Anweisungen. Es passieren recht schnell Fehler wo man vergisst die Klammern zu setzen und dann passieren ungewöhnliche Dinge.

Ansonsten habe ich gesehen das du 4x die Überprüfung "getCurrentHP > 0" im Code hast. Hier könnte man in der Klasse Char eine eigene Methode "isAlive()" definieren, welche man dann jeweils aufruft.
Vorteil die Bedingung kann man in Zukunft zentral an einer Stelle bearbeiten, es ist lesbarer.

Von anderen Spielen kenn ich es das beim einem LvlUp der Character automatisch geheilt wird, wenn das auch deine Absicht wäre dann könnte man den Aufruf von "healing()" direkt in der Methode "lvlUp" erledigen.