Klassen fragen zur getter und setter methode

[neitro]

Guten Morgen Community,
ich beschäftige mich seit einigen Tagen mit dem programmieren eines Spiel.
Mein Problem sieht wie folgt aus ich hab eine Klasse namens "add_gold" denke erklärt sich von selbst denk ich und die ruft pet getter Methode aus der Klasse "player" den Gold stand und den Wert um den es sich erhöhen soll. wenn ich dies per Setter versuch zu setzen änder sicht der Wert nicht der bleibt bei konstanten 100 wie die Variable deklariert wurder.

package Game;

public class add_gold extends
{	
	publicvoid addgold()
	{
		player_1 p1 = new player_1();
		System.out.println(p1.get_p1_gold());
		p1.set_p1_gold(p1.get_p1_gold()+p1.get_p1_goldprod());
	}
}
;

das ist die Golde erhöhung Methode und in der anderen Klasse sind nur die Attribute, Getter und Setter

package Game;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

public class player_1 implements KeyListener
{
	// Klassen  Variabeln
	public int p1_gold = 100;
	public int p1_leben = 20;
	public int p1_angriff = 10;
	public int p1_goldprod = 10;
	public double p1_ausbildung = 30f;
//Constructor
	public player_1()
	{
	}
	//Setter
	void set_p1_gold(int set_p1_gold)
	{	
		this.p1_gold = set_p1_gold;
	}
	//Getter
	int get_p1_gold();

Ich bedank mich schon mal im vorraus.
Mit freundlichen grüßen neitro

 

[VfL_Freak]

Moin,

ich hab eine Klasse namens "add_gold" denke erklärt sich von selbst denk ich

na ja .....

Aber vielleicht solltest Du dann "set_p1_gold" auch irgendwo aufrufen ???:L

[EDIT]
BTW: dies hier "void set_p1_gold(int set_p1_gold)" ist auf Dauer eine ganz miese Idee!! Stell' Dir vor, Du hast irgendwann 1000 oder 5000 Zeilen Code .... meinst Du, Du blickst dann noch durch, was was ist ???:L
[/EDIT]

Gruß
Klaus

 

[neitro]

Das Programm bzw. Spiel wird nicht all zu groß aber mein prob ich bekomm jedes mal wenn ich add_gold aufrufe den startwert ausgeben also 100 hab ich per system.out.println überprüft ich versteh nur nicht ganz warum.

package Game;

public class timer 
{
	long thistime;
	long lasttime;
	long dauer;
	boolean timer;
	
	public timer()
	{
		dauer = 1000;
	}
	
	public void time()
	{
		add_gold addg = new add_gold();
		move_p1 m1 = new move_p1();
		move_p2 m2 = new move_p2();
		
		lasttime = System.currentTimeMillis();
		thistime = 0;
		timer = true;
		while (timer)
		{
			thistime = System.currentTimeMillis();
			if (dauer<= (thistime - lasttime))
			{
				System.out.println(true);
				add_gold.addgold();
				m1.moveforward_p1();// 
				m2.moveforward_p2();//
				timer = false;
			}
		}
	}
}

 

[VfL_Freak]

Moin,

public class add_gold extends
{	
	public void addgold()
	{
		player_1 p1 = new player_1();
		System.out.println(p1.get_p1_gold());
		p1.set_p1_gold(p1.get_p1_gold()+p1.get_p1_goldprod());
	}
}

Sorry, habe jetzt den Aufruf von "set_p1_gold" gefunden ....

Das Problem wird sein, dass Du bei jedem Aufruf von "addgold" einen NEUEN Spieler p1 anlegst, der dann natürlich initial wieder '100' hat!

Zudem finde ich aber die Methode "get_p1_goldprod()" nicht ...

Gruß
Klaus

 

[neitro]

ach sorry danke das könnt es sein probiers gleich mal. ja beim kopieren wurd die ausversehen weggeschnitten hier:

	//Setter
	void set_p1_gold(int set_p1_gold)
	{	
		this.p1_gold = set_p1_gold;
	}
	//Getter
	int get_p1_gold()
	{
		return p1_gold;
	}
	int get_p1_goldprod()
	{
		return p1_goldprod;
	}

 

[VfL_Freak]

Moin,

habe es befürchtet ... :lol:

Sollte die Ausgabe dann nicht theoretisch 110 sein ??
Schau' mal mit 'nem Debugger drauf!

Gruß
Klaus

 

[neitro]

Ne ich krieg immer genau 100 raus wie kann ich den jetzt die klasse so einstellen das sie nicht als neuen spieler anleg wie gesagt ich lern erst seit 4 tagen java und weiß manche sachen noch nicht

 

[neitro]

Ne ich krieg immer genau 100 raus wie kann ich den jetzt die klasse so einstellen das ich nicht als ein neuen spieler anlege? sorry für doppel post hat grad irgendwie rumgesponnen

 

[CerO]

public class add_gold extends //<-----
{   
    public void addgold()
    {
        player_1 p1 = new player_1();  //<--- Mit dieser Zeile legst du immer ein neues Objekt an
        System.out.println(p1.get_p1_gold());
        p1.set_p1_gold(p1.get_p1_gold()+p1.get_p1_goldprod());
    }
}

Das extends kannst du weglassen, wenn add_gold von keiner anderen Klasse erbt.

player_1 p1 = new player_1(); Diese Zeile sollte nur einmal aufgerufen werden, wenn es nur einen spieler gibt. Deshalb die da rausnehmen und irgendwo hinschieben, wo sie nicht jedesmal aufgerufen wird, sondern nur einmal am Anfang.

 

[neitro]

public class add_gold extends //<-----
{   
    public void addgold()
    {
        player_1 p1 = new player_1();  //<--- Mit dieser Zeile legst du immer ein neues Objekt an
        System.out.println(p1.get_p1_gold());
        p1.set_p1_gold(p1.get_p1_gold()+p1.get_p1_goldprod());
    }
}

Das extends kannst du weglassen, wenn add_gold von keiner anderen Klasse erbt.

player_1 p1 = new player_1(); Diese Zeile sollte nur einmal aufgerufen werden, wenn es nur einen spieler gibt. Deshalb die da rausnehmen und irgendwo hinschieben, wo sie nicht jedesmal aufgerufen wird, sondern nur einmal am Anfang.

Ja aber wie mach ich das wenn ich sie außerhalb schreib krieg ich p1. ein fehler und will das ich auf static setzt. wie kann ich die klasse aufrufen ohne das sie dauerhaft neues objekt anlege oder besser das objekt aufrufen?

 

[CerO]

Vielleicht in die main Methode, bevor du mit deiner Gameschleife anfängst. Kann halt nur raten wenn ich nicht das ganze Programm kenne.

 

[gescom]

wie schon gesagt wurde

 player_1 p1 = new player_1();

erstellst du ein neues Objekt der Klasse player_1

schreibe die Klasse in Player um und schreibe einen Konstruktor der nicht leer ist. So kannst du beliebig viele Spieler erstellen. add_gold sollte/ könnte dann eine Methode der Klasse Player sein

public class Player{

    //Deine Klassenvariablen
    private int gold = 0;

    //Dein Konstruktor
    public Player(int gold){
       this.gold = gold;
    }

    public void setGold(int gold){

    }

    public int getGold(){
 
    }

    public void addGold(int gold){
       
    }

}