Vererbung Frage zu objektübermittelung

[charged]

Ich bekomm es einfach nicht hin in GLOOP die Objekte für eine Kollisions-Abfrage in eine andere Klasse zu übertragen. Wahrscheinlich bin ich einfach nur zu dumm

Hier der Code aus der Hauptklasse

   import GLOOP.*;


public class Spiel
{
    
     
    ...
    Avatar meinAvatar;
    
    baum[] derBaum;
    
    
      
    
    public Spiel()
    {
        
        ....

        
        
        for(int k=0; k<1000; k++)
       {
           derBaum[k] = new baum();
            
       }
.....
for(int k=0; k <1000; k++)
            {
             if(meinAvatar.meinBauch.gibX() + 70 == derBaum[k].Stamm.gibX() ||       meinAvatar.meinBauch.gibX() - 70 == derBaum[k].Stamm.gibX() && meinAvatar.meinBauch.gibZ() + 70 == derBaum[k].Stamm.gibZ() || meinAvatar.meinBauch.gibZ() - 70 == derBaum[k].Stamm.gibZ())
             {
               beende = 2 ;
               
             }
           
            }

Hier der Baum
[Java]

import GLOOP.*;

class baum{


GLQuader Stamm;
GLWuerfel Busch;


public baum()
{
double rnx;
double rnz;
rnx = (Math.random()*500000)-250000;
rnz = (Math.random()*500000)-250000;
Stamm = new GLQuader(rnx ,300,rnz,100,600,100) ;
Busch = new GLWuerfel(rnx,1000,rnz,800);


...


}


}

[/code]
und der AVATAR

  import GLOOP.*;


public class Avatar 
    // Bezugsobjekte
    {
    
  
    GLQuader meinBauch;
   ...
    // Attribute

    // Konstruktor
    
    
    
    
    public Avatar()
    {
  
        
        meinBauch = new GLQuader(50000,210,50000,40,120,80);
       ...
        
       

... 
        

    }

Ich bekomm es einfach nicht hin den Bauch und den Stamm für eine Kollisionsabfrage zu übertragen. Irgendjemand eine Idee was ich falsch mache???? :L

 

[turtle]

[EDIT]Wow, welch ein Kuddel-Muddel.[/EDIT]

Ich rate DRINGEND den Code lesbarer zu machen, denn ich verstehe ihn nicht. Ich hab mal eine Ausgabe eingebaut, was da verglichen wird.

		System.out.println(meinAvatar.meinBauch.gibX());//50000.0
		System.out.println(derBaum.getStamm().gibX()); // -199207.77
		if (meinAvatar.meinBauch.gibX() + 70 == derBaum.getStamm().gibX()
				|| meinAvatar.meinBauch.gibX() - 70 == derBaum.getStamm().gibX()
				&& meinAvatar.meinBauch.gibZ() + 70 == derBaum.getStamm().gibZ()
				|| meinAvatar.meinBauch.gibZ() - 70 == derBaum.getStamm().gibZ()) {
			beende = 2;
		}

Und dann ist dir wohl klar, (hoffentlich, weil mir nicht) warum beende nie 2 werden kann, oder?

Diese 70 nennen wir magic value und sollten NICHT tief im Code versteckt sein. Was bedeutet 70? Was ändert sich, wenn ich hier 230 eintrage? Und ich muss diese Zahl VIER Mal ändern, wenn ich merke, das die falsch ist und geändert werden muss.

Aber wie ich schon sagte, dieses if- solltest du dringend neu programmieren.

Variable beginnen mit einem Kleinbuchstaben.

Man lässt normalerweise anderen Klassen keinen Zugriff auf interne Attribute, habe daher einen getter für Stamm eingebaut.

 

[charged]

Erstmal vielen Dank für deine Antwort, trotz meines unlesbaren codes!

Die 70 ist der kleinst mögliche Abstand zwischen Avatar und Baum, bevor der Avatar, welcher gesteuert werden kann, berührt wird. Wenn der Avatar also einen Baum berührt soll sich das programm beenden (also beende = 2). Die Abfrage ist in meinem ersten Post aufgrund eines Tippfehlers falsch. Richtig wäre:

    if((meinAvatar.gibX() + 70 == meinStamm[k].gibX()
             || meinAvatar.gibX() - 70 == meinStamm[k].gibX()) 
             && (meinAvatar.gibZ() + 70 == meinStamm[k].gibZ() 
             || meinAvatar.gibZ() - 70 == meinStamm[k].gibZ()))
             {
              beende = 2 ;
              }

(habe die klammern vergessen)

Das Problem liegt aber darin, dass ich nicht weiß wie ich an Stamm[k] komme. Falls du es nicht oben verstanden hast :

In der Hauptklasse ruf ich eine Funktion 1000 mal auf, welche den Baum an zufälligen orten deklariert

 for(int k=0; k<1000; k++)
       {
           derBaum[k] = new baum();
            
       }

Und in dieser anderen Klasse also baum :

import GLOOP.*;

class baum{
    
    
    GLQuader  Stamm;
    GLWuerfel Busch;
    
    
    public baum()
    {
       double rnx;
       double rnz;
       rnx = (Math.random()*500000)-250000;
       rnz = (Math.random()*500000)-250000;
      Stamm = new GLQuader(rnx ,300,rnz,100,600,100)   ;    
      Busch = new GLWuerfel(rnx,1000,rnz,800);
      
      
      Stamm.setzeTextur("Stamm.jpg");
      Busch.setzeTextur("Busch.jpg");
            
    }
       
}

an die Koordinaten des Avatarbauches komm ich leicht durch eine unterfunktion gibx und gibz in der Avatar Klasse.
Allerdings brauch ich für eine Kollisionsabfrage auch die Koordinaten jedes Baumes und ich weiß nicht wie ich an diese Komme.


Falls du es jetzt verstanden hast, hast du jetzt eine Idee?

 

[turtle]

Wie ich schon geschrieben habe, getStamm() implementieren,oder ich verstehe dich immer noch nicht

class Baum {
	GLQuader stamm;
	GLWuerfel busch;

	public Baum() {
		double rnx;
		double rnz;
		rnx = (Math.random() * 500000) - 250000;
		rnz = (Math.random() * 500000) - 250000;
		stamm = new GLQuader(rnx, 300, rnz, 100, 600, 100);
		busch = new GLWuerfel(rnx, 1000, rnz, 800);

	}

	public GLQuader getStamm() {
		return stamm;
	}

}

 

[charged]

Entweder sehe ich das falsch oder Ich bekomme mit

          public GLQuader getStammX() {
         return stamm.gibX();
         }

und

         public GLQuader getStammZ() {
         return stamm.gibZ();
         }

nur die Koordinaten des letzten Stamms.
Ich brauche aber die Koordinaten aller 1000 zufällig platzierter Stämme, welche ja alle anders sind.

Dein Beispiel habe ich ja genauso bei dem Avatar da es von diesem nur 1 gibt.

Ich bin (wie man an meinem code sieht ) ziemlich neu in Java also nimm bitte das was ich sage nicht zu ernst.

 

[turtle]

Aber genau so, wie du in einer Schleife 1000 Objekte erzeugst, kannst du doch über alle Bäume iterieren und von jedem baum[k] auf dessen Stamm zugreifen

for(int k=0; k<1000; k++) {
Stamm stamm = baum[k].getStamm();
}

 

[charged]

Ich habe es jetzt folgendermaßen geregelt :

    for(int k=0; k <1000; k++)
           
             if((meinAvatar.gibX() + 70 ==  derBaum[k].Stamm.gibX()
             || meinAvatar.gibX() - 70  ==  derBaum[k].Stamm.gibX())
             && (meinAvatar.gibZ() + 70 ==  derBaum[k].Stamm.gibZ() 
             || meinAvatar.gibZ() - 70  ==  derBaum[k].Stamm.gibZ()))
             {
               beende = 2 ;
               
             }

Allerdings läuft der Avatar immernoch durch die Bäume Irgendjemand eine Idee?

 

[charged]

Hier nochmal der ganze code

Hauptklasse:

import GLOOP.*;


public class Spiel
{
    
     
    GLKamera dieKamera;
     
    GLLicht dasLicht;
    GLTastatur dieTastatur;
    GLBoden    derBoden;
    GLTextur   texSand;
    GLTextur   texCloud;
    GLHimmel   derHimmel;
    Avatar meinAvatar;
    GLKugel   dieSonne;
    GLVektor  vector;
    baum[] derBaum;
    
      
    // Attribute

    // Konstruktor
    public Spiel()
    {
        int i = 1;
        int j = 46;
        
        
        
        
       
        vector = new GLVektor(10000,10000,10000);
        dasLicht = new GLLicht(vector);
        dieTastatur = new GLTastatur();
        
        texSand = new GLTextur("Sand.jpg");
        texCloud = new GLTextur("Himmel.jpg");
        derBoden = new GLBoden(texSand);
        derHimmel = new GLHimmel(texCloud);
        
        meinAvatar = new Avatar();
        dieSonne = new GLKugel(100,100000,100,10000);
        dieSonne.setzeSelbstleuchten(230,255,0);
        derBaum = new baum[1000];
        int beende = 1;
        
      
       
        
        for(int k=0; k<1000; k++)
       {
           derBaum[k] = new baum();
            
       }
       
        
            while (!dieTastatur.esc() && beende != 2 ){
                
                
            while(dieTastatur.links()  ||   dieTastatur.istGedrueckt('a')  ||  dieTastatur.istGedrueckt('A') || dieTastatur.rechts() ||  dieTastatur.istGedrueckt('d')  ||  dieTastatur.istGedrueckt('D') || dieTastatur.oben()   ||  dieTastatur.istGedrueckt('w')  ||  dieTastatur.istGedrueckt('W') || dieTastatur.unten()  ||  dieTastatur.istGedrueckt('s')  ||  dieTastatur.istGedrueckt('S'))
            {
            
            
            if (dieTastatur.links()  ||  dieTastatur.istGedrueckt('a')  ||  dieTastatur.istGedrueckt('A') )
            {
            meinAvatar.bewegeLinks();
           }
            
            if (dieTastatur.rechts() ||  dieTastatur.istGedrueckt('d')  ||  dieTastatur.istGedrueckt('D') )
            {
            meinAvatar.bewegeRechts();
            } 
            if (dieTastatur.oben()   ||  dieTastatur.istGedrueckt('w')  ||  dieTastatur.istGedrueckt('W') )
            {
            meinAvatar.bewegeVor();
            }
            if (dieTastatur.unten()  ||  dieTastatur.istGedrueckt('s')  ||  dieTastatur.istGedrueckt('S') )
            {
            meinAvatar.bewegeZurück();
            }
            if (i < 91 && i > 0)
            {
            meinAvatar.rotierep( );
            i++;
            }
            if (i == j) i = -1;
            if (i > -91 && i <0)
            {
            meinAvatar.rotierem();
            i--;
            }
            if (i == -91)
            {
            i = 1;
            j = 91;
            }
             
             for(int k=0; k <1000; k++)
             {
             if((meinAvatar.gibX() + 70 ==  derBaum[k].Stamm.gibX()
             || meinAvatar.gibX()  - 70  ==  derBaum[k].Stamm.gibX())
             || (meinAvatar.gibZ() + 70 ==  derBaum[k].Stamm.gibZ() 
             || meinAvatar.gibZ()  - 70  ==  derBaum[k].Stamm.gibZ()))
             {
               beende = 2 ;
               
             }
            }
            Sys.warte(5);
            }
            
            meinAvatar.rotationnull();
            j=46;
            i=1;
            }
            
           
            
           Sys.beenden();
        }
        
}

und die BaumKlasse

import GLOOP.*;

class baum{
    
    
    GLQuader  Stamm;
    GLWuerfel Busch;
    
    
    public baum()
    {
       double rnx;
       double rnz;
       rnx = (Math.random()*500000)-250000;
       rnz = (Math.random()*500000)-250000;
      Stamm = new GLQuader(rnx ,300,rnz,100,600,100)   ;    
      Busch = new GLWuerfel(rnx,1000,rnz,800);
      
      
      Stamm.setzeTextur("Stamm.jpg");
      Busch.setzeTextur("Busch.jpg");
        
      
    }
    
     
}

Ich hab keine Ahnung was falsch ist aber der Avatar kkann durch bäume laufen