Probleme mit Ausgabe von Werten

[Acura]

Hallo zusammen,

hab jetzt im Eigenstudium mal angefangen mit Java, bin aber auf folgendes Problem gestossen... Will in folgendem Beispiel das er bei ausgabe.leseX() den zuvor eingegebenen Wert ausgibt, es kommt jedoch nur 0 dabei raus..
Hoff ihr könnt mir weiterhelfen.

import java.io.*;
public class Rechteck {

	
	public static void main(String[] args) throws IOException {
		
		InputStreamReader eingabeStrom = new InputStreamReader(System.in);
		BufferedReader tastatur = new BufferedReader(eingabeStrom);
		
		
		// Einlesen
		System.out.println("Bitte x Wert eingeben ");
		int xWert = Integer.parseInt( tastatur.readLine() );
		
		System.out.println("Bitte y Wert eingeben ");		
		int yWert = Integer.parseInt(  tastatur.readLine() );
		
		
		Punkt pos1 = new Punkt();
		pos1.setzePunkt( xWert, yWert);
		
		Punkt ausgabe = new Punkt();
		System.out.println("X-Wert: " + ausgabe.leseX() );

	}

}


class Punkt
{
	int xPos, yPos;
	
	void setzePunkt(int x, int y)
	{
		this.xPos   = x;
		this.yPos   = y;
	}
	
	int leseX()
	{
		return this.xPos;
	}
	int leseY()
	{
		return this.yPos;
	}
}


class Linie
{
	
	
	
}

 

[seejay]

Sei froh, dass es so ist

hier ist dein Fehler:

      Punkt pos1 = new Punkt();
      pos1.setzePunkt( xWert, yWert);
      
      Punkt ausgabe = new Punkt();
      System.out.println("X-Wert: " + ausgabe.leseX() );

Du setzt den x wert von pos1 aber gibst den x wert von ausgabe aus
die beiden sind verschiedene Objekte einer Klasse. Wie 2 Autos sagen wir mal Opel Corsa. Wenn du jetzt bei einem den Motor anmachst, ist bei dem anderen der Motor immer noch aus.

also so richtig:

      Punkt pos1 = new Punkt();
      pos1.setzePunkt( xWert, yWert);
      System.out.println("X-Wert(pos1): " + pos1.leseX() );   

      Punkt ausgabe = new Punkt();
      ausgabe.setzePunkt( xWert*2, yWert*2);
      System.out.println("X-Wert(ausgabe): " + ausgabe.leseX() );

 

[Acura]

vielen Dank... man bin ich dämlich.. jetzt klingts alles logisch

 

[Acura]

Hi, hab da noch mal so ein ähnliches Problem... beim Befehl showFlaeche kommt auch nur null raus... mach ich da bei Übergabewerten was falsch? komm einfach nich drauf

import java.math.*;
public class Rechteck5 {

	
	public static void main(String[] args) {
		 
		Rechteck seiteAein = new Rechteck();
		seiteAein.setSeiteA(5);
		
		Rechteck seiteBein = new Rechteck();
		seiteBein.setSeiteB(10);
		
		Rechteck seiteABein = new Rechteck();
		seiteABein.setSeiteAB(5, 10);
		
		Rechteck Flaecheaus = new Rechteck();
		Flaecheaus.showFlaeche();
		
		Rechteck getUmfangaus = new Rechteck();
		getUmfangaus.getUmfang();
		
		Rechteck Quadrat = new Rechteck();
		Quadrat.isQuadrat();
		
		
		
		
		
		
		
		

	}

}
class Rechteck
{
	double seiteA, seiteB;
	
	
	void setSeiteA(int a)
	{
		this.seiteA = a; 
	}
	void setSeiteB(int b)
	{
		this.seiteB = b;
	}
	void setSeiteAB(int a,int b)
	{
		this.seiteA = a;
		this.seiteB = b;
	}
    void setSeiteAB(String a,String b)
    {
    	this.seiteA = Integer.parseInt(a);
    	this.seiteB = Integer.parseInt(b);
    }
    double getSeiteA()
    {
    	return this.seiteA;
    }
    double getSeiteB()
    {
    	return this.seiteB;
    }
    double getFlaeche()
    {
    	return this.seiteA * this.seiteB;
    }
    void showFlaeche()
    {
    	System.out.println("Der Flächeninhalt beträgt: " + this.seiteA * this.seiteB);
    }
    double getUmfang()
    {
    	return this.seiteA * 2 + this.seiteB * 2;
    }
    /*double getDiagonale()
    {
    	return this.seiteA
    }*/
    boolean isQuadrat()
    {
    	if(seiteA == seiteB)
    	{
    		return true;
    		
    	
    	}
    	else
    	{
    		return false;
    		
    	}
    }
    
}

 

[mikachu]

der fehler liegt doch auf der hand.
du erstellst ein neues Recheck-objekt. dieses hat die werte auf null gesetzt.
da du die werte nicht belegst, bleiben die nicht definiert -> also kommt ein null raus.

setz einfach im konstruktor der klasse Recheck die werte schon auf -1 oder so

class Rechteck 
{ 
   double seiteA = -1, seiteB = -1;
//...
}

 

[sliwalker]

Hoi,

Du hast wieder das gleiche Problem wie vorher.
Wenn Du ein Objekt erstellst und es mit Werten befüllst. Dann darfst Du nicht ein neues Objekt erstellen, um die Werte ausgeben zu lassen.

Mal ein greifbares Beispiel passend zur Jahreszeit:
Du schneidest Dir aus Karton einen Weihnachstmann aus, um mit diesem Kunstschneesprühzeug einen Weihnachtsmann auf Dein Küchenfester zu sprühen.
Dann gehst u in die Küche und sprühst da den Weihnachtsmann drauf. Dann gehst Du zu Deiner Freundin und sagt, geh mal in Wohnzimmer un guck Dir den Weihnachtsmann auf der Scheibe an.

Resultat?

Deine freundin hält Dich für bescheuert, weil da ja gar kein Weihnachtsmann ist -.- Du hast den ja in die Küche gemacht...

So auch mit Objekten.
Wenn Du ein Objekt namens XYZ erstellst und dem Werte hinzufügst, dann nimm auch das Objekt XYZ um die Werte wieder auszugeben. Sonst wird das nix.

Was mein Vorrdener sagte ist nicht die Lösung deines Problems.
Das hilft nur andere fehler zu vermeiden, aber nicht Dein jetziges so zu beheben, wie Du erwartest...

greetz
Der Weihnachtsman...äh..SLi

 

[Acura]

ah ok vielen dank für die Hilfe... jetzt versteh ich hab das ja an ner völlig falschen Stelle gesetzt.... bin echt ein ne ziemliche Java Null hoff bloss ich krieg das irgendwann richtig gebacken...
hab jetzt den ganzen abend dran rumprobiert und noch mal neu und in seperate Klassen geschrieben.... und nun is das bei rausgekommen:

import java.io.*;
public class Rechteck5{
	
	private double seiteA, seiteB;
	

	public void setSeiteA()throws IOException
	{
		System.out.println("Bitte Seite A eingeben: ");
		InputStreamReader eingabeStrom = new InputStreamReader(System.in);
		BufferedReader tastatur = new BufferedReader(eingabeStrom);
		seiteA = Double.parseDouble(tastatur.readLine());
	}
	public void setSeiteB() throws IOException
	{	
		System.out.println("Bitte Seite B eingeben: ");
		InputStreamReader eingabeStrom = new InputStreamReader(System.in);
		BufferedReader tastatur = new BufferedReader(eingabeStrom);
		seiteB = Double.parseDouble(tastatur.readLine());
	}
	public void setSeiteAB(double a,double b)
	{
		seiteA = a;
		seiteB = b;
	}
	public void setSeiteAB(String a, String b)
	{
		seiteA = Double.parseDouble(a);
		seiteB = Double.parseDouble(b);
	}
	public void setSeiteA(String a)
	{
		seiteA = Double.parseDouble(a);
	}
	public void setSeiteB(String b)
	{
		seiteB = Double.parseDouble(b);
	}
	public double getSeiteA()
	{
		return seiteA;
	}
	public double getSeiteB()
	{
		return seiteB;
	}
	public double getFlaeche()
	{
		return seiteA*seiteB;
	}
	public void showFlaeche()
	{
		System.out.println("Der Flächeninhalt beträgt "+ getFlaeche());
	}
	public double getUmfang()
	{
		return seiteA*2+seiteB*2;
	}
	

	
}

die Werte werden jetzt auch richtig ausgegeben... hab jedoch mal gesehen dass,das auch komfortabler geht und ich nich in jede Methode das reinschreiben muss sondern das ich quasi ne extra Eingabe Methode mach jedoch hab ich keine Ahnung wie die Werte übergeben werden...

 

[Leroy42]

und in seperate Klassen geschrieben

Du meinst wohl seperate Methoden

public double getDouble(String name) { 
  double d = 0;
  try {
      System.out.println("Bitte Seite "+name+" eingeben: "); 
      InputStreamReader eingabeStrom = new InputStreamReader(System.in); 
      BufferedReader tastatur = new BufferedReader(eingabeStrom); 
      d = Double.parseDouble(tastatur.readLine()); 
  } catch (Exception e) {
     System.out.println(e);
  }
  return d;
}

und dann z.B.:

public void setSeiteAB()  { 
      seiteA = getDouble("A"); 
      seiteB = getDouble("B"); 
   }