auf klasse zugreifen in einer anderen klasse

[lenny]

Ich habe im ordner java/io die datei Vec3D.java gespeichert. (Sie sieht folgendermassen aus

class Vec3D {
	double v1, v2,v3;
	Vec3D (double v1, double v2, double v3) {this.v1=v1; this.v2=v2; this.v3=v3;}
	Vec3D add(Vec3D v) {v1=v1+v.v1; v2=v2+v.v2; v3=v3+v.v3;}
	Vec3D multiply(double s) {v1=v1*s; v2=v2*s; v3=v3*s;}
	double dotProduct (Vec3D v) {double x=v1*v.v1+v2*v.v2+v3*v.v3; return x;}
	double norm () {double x=Math.sqrt(v1*v1+v2*v2+v3*v3); return x;}
	public String toString(Vec3D v) {String x="("+v1+"/"+v2+"/"+v3+")"; return x;}
}

nun schreibe ich ein Programm, welches die vektoroperationen von der obigen klasse durchführen kann:

import java.io.*;
class Vektoren{

	static String readln(){
		InputStreamReader isr = isr = new InputStreamReader (System.in);
		BufferedReader br = new BufferedReader (isr);
		String s=null;
		try {
			s= br.readLine();
		}
		catch (IOException ioe) {
			System.err.println(ioe);
		}
		return s;
	}

	public static void main (String[]arg){
	double x=Double.parseDouble(arg[0]);
	double y=Double.parseDouble(arg[1]);
	double z=Double.parseDouble(arg[2]);
	Vec3D v1= new Vec3D(x,y,z);
	System.out.println("Was wollen sie tun? Addition=1, skal. Multipl=2, dotP=3, Länge=4");
	int op=Integer.parseInt(readln());
	switch (op) {
		case 1: {System.out.println("Geben sie noch einen Vektor an");
                                            Vec3D v2= new Vec3D(3,4,5);  Stringout=toString(v1.add(v2)); break;}
		default: System.out.println("Error"); break;
		}
	//System.out.println(out);
	
	
	
}}

nun gibt es aber probleme beim zugriff auf vec3d und probleme bei der funktion tostring. Was mache ich falsch?

 

[L-ectron-X]

Der Code ist, was die Lesbarkeit angeht, eine Zumutung. Wo hast Du so was gelernt? :?
Beim Kompilieren der Klasse Vec3D werden Fehler vom Compiler angezeigt, die auf fehlende return statements hinweisen. Ansonsten wären genaue Fehlermeldungen/-beschreibungen schön gewesen.
So weit erst mal.

 

[lenny]

gelertn beim selbststudium, fehlermeldungen:
im stile:
symbol: class vec3d
locatation: class vektoren
mit pfeil auf new >Vec3D(x,y,z)...

ps: wie soll ich den code lesbarer machen/schreiben?

 

[L-ectron-X]

Die Lesbarkeit des Codes kannst Du positiv beeinflussen, in dem Du z.B. die Codezeilen vernünftig einrückst und auf jeden Fall nur eine Zeile je Anweisung schreibst. Die obige Klasse Vec3D ist ein schlechtes Beispiel für Codestyle.
Bleibt immer noch der Fehler mit den Rückgabewerten. Du hast dort nur drei Variablen gesetzt.

Vec3D add(Vec3D v) {
  v1=v1+v.v1;
  v2=v2+v.v2;
  v3=v3+v.v3;
}

Die Methode soll ein Vec3D-Objekt zurück geben. Da fehlt ein Rückgabewert. Ebenso in der multiply()-Methode.

 

[Beni]

ps: wie soll ich den code lesbarer machen/schreiben?

Zum Beispiel könntest du sowas schreiben. Du siehst sicher den Unterschied: Aussagekräftigere Namen und Einrückungen. Ich hab mir mal die Kommentare gesparrt, und nur für add ein Beispiel gemacht.

public class Vector3D {
	private double x, y, z;
	Vector3D (double x, double y, double z) {
        	this.x=x;
		this.y=y;
		this.z=z;
	}

	public double getX(){
		return x;
	}

	public void setX( double x ){
		this.x = x;
	}

	/*
	* Zählt einen anderen Vektor komponentenweise zu diesem hinzu.
	* @param v Der zweite Summand.
	*/
	public void add(Vector3D v){
		x = x + v.x;
		y = y + v.y;
		z = z + v.z;
	}

	public void multiply(double s){
		x *= s;
		y *= s;
		z *= s;
	}


	public double dot( Vector3D v ){
		return x*v.x + y*v.y + z*v.z;
	}

	public double norm (){
		return Math.sqrt( dot( this ) );
	}

	public String toString(){
		return "("+x+"/"+y+"/"+z+")";
	}
}

 

[lenny]

Ich habe nun die Die Klasse Vec3D so umgeschrieben:

class Vec3D {

	double v1, v2,v3;

	Vec3D (double v1, double v2, double v3) {
		this.v1=v1;
		this.v2=v2;
		this.v3=v3;
	}

	Vec3D add(Vec3D v) {
		v1=v1+v.v1;
		v2=v2+v.v2;
		v3=v3+v.v3;
		x= new Vec3D(v1,v2,v3);
		return x;
	}

	Vec3D multiply(double s) {
		v1=v1*s;
		v2=v2*s;
		v3=v3*s;
		x= new Vec3D(v1,v2,v3);
		return x;
	}

	double dotProduct (Vec3D v) {
		double x=v1*v.v1+v2*v.v2+v3*v.v3
		return x;
	}

	double norm () {
		double x=Math.sqrt(v1*v1+v2*v2+v3*v3)
		return x;
	}

	public String toString() {
		return "("+v1+"/"+v2+"/"+v3+")";
	}

}

Doch es kommen immer noch die gleichen fehlermeldungen:

C:\java\3>javac Vektoren.java
Vektoren.java:21: cannot resolve symbol
symbol : class Vec3D
location: class Vektoren
Vec3D v1= new Vec3D(x,y,z);
^
Vektoren.java:21: cannot resolve symbol
symbol : class Vec3D
location: class Vektoren
Vec3D v1= new Vec3D(x,y,z);
^
Vektoren.java:25: cannot resolve symbol
symbol : class Vec3D
location: class Vektoren
case 1: {System.out.println("Geben sie noch einen Vektor an"); V
ec3D v2= new Vec3D(3,4,5); v1.add(v2); break;}
^

Vektoren.java:25: cannot resolve symbol
symbol : class Vec3D
location: class Vektoren
case 1: {System.out.println("Geben sie noch einen Vektor an"); V
ec3D v2= new Vec3D(3,4,5); v1.add(v2); break;}

^
4 errors

Wie gesagt, Vec3D.java liegt im Java inst.Verzeichnis/java/io/Vec3D.java

 

[Griffin]

Warum gibst du es nicht ins selbe Verzeichnis wie die Klasse Vektoren?
Macht doch auch bischen wenig Sinn die eigene Klasse ins java/io-Verzeichnis zu tun. Erstens: Das IO Package ist für In- und Output und damit hat deine Klasse nichts zu tun. Zweitens: Deine Klassen solltest du immer zusammen aufbewahren.

 

[lenny]

ok, habe nun die klasse zurückkopiert. nun erhalte ich folgende fehlermeldungen:

C:\java\3>javac Vektoren.java
.\Vec3D.java:15: cannot resolve symbol
symbol : variable x
location: class Vec3D
x= new Vec3D(v1,v2,v3);
^
.\Vec3D.java:16: cannot resolve symbol
symbol : variable x
location: class Vec3D
return x;
^
.\Vec3D.java:23: cannot resolve symbol
symbol : variable x
location: class Vec3D
x= new Vec3D(v1,v2,v3);
^
.\Vec3D.java:24: cannot resolve symbol
symbol : variable x
location: class Vec3D
return x;
^
4 errors

 

[L-ectron-X]

Die Variable x in Vec3D ist nicht definiert. So wird's schon eher was:

  ...
  ...
  ...
  Vec3D x= new Vec3D(v1,v2,v3);
  return x;
}

oder diese Variante sollte auch funktionieren:

return new Vec3D(v1,v2,v3);

 

[Griffin]

Vec3D add(Vec3D v) {
      v1=v1+v.v1;
      v2=v2+v.v2;
      v3=v3+v.v3;
      //Der Typ muss festgelegt werden
      Vec3D x= new Vec3D(v1,v2,v3);
      return x;
   }

   Vec3D multiply(double s) {
      v1=v1*s;
      v2=v2*s;
      v3=v3*s;
      //hier das selbe
      Vec3D x= new Vec3D(v1,v2,v3);
      return x;
   }

 

[lenny]

So, jetzt hab ichs glaub fast geschafft. Nur eins ist mit noch nicht ganz klar: ich kann ja jetzt zum beispeil a.multiply(b) machen. nun will ich das am bildschirm ausgeben. wie muss ich jetzt das machen. String out=toString(a.multiply(b)), geht ja nicht. Wie muss ich dann toString anwenden?
Vielen Dank Lenny.

 

[L-ectron-X]

Jetzt überleg mal. Wenn Du eine Methode eines Objekts aufrufen möchtest, wie machst Du das?
Genau!

referenz.methode();

Also ruft die Referenz auf ein Objekt vom Typ Vec3D die Methode toString() auf.
Wenn wir bei obigen Bsp. bleiben sieht das so aus:

x.toString();

Und was in diesem Fall auch funktioniert:

System.out.println(x);