Array deklarieren und befüllen; "]" expected

[kontur]

Hallo zusammen,

ich habe mir eine Klasse Planet geschrieben, in welcher per Applet ganz simple Wetter und Tageszeit dargestellt werden soll.
Da Applet simuliert auch Wolkenbewegung und Regenfall.
Da ich nun Nachts auch Sterne am Himmel haben will, möchte ich die Positionen jener in einem Array speichern. Da das Applet animiert ist werden die Sterne am Nachthimmel auch sekündlich neu gezeichnet, deswegen die Notwendigkeit, die Positionen einmal zu speichern und dann für die Restzeit des Appletts immer wieder die selben zu zeigen.

Nun habe ich ein Problem beim Kompilieren. Hier zunächst der Code:

private boolean starsCreated = false;
//habe "private int guessX[];  " editiert zu:
private int[] guessX;

// und "private int guessY[];" zu
private int[] guessY;

.
.
.

	public void drawStars(Graphics g){
		if (starsCreated!=true) createStars();
		else{
			for (int j=0; j<500; j++){
				int yellowVariation = (int)((Math.random()*100)+155); 
			  	Color rgbStar = new Color(255, 255, yellowVariation);
			    
			  	g.setColor(rgbStar);	    
			  	g.drawOval(guessx[j],guessy[j],2,2);		
			}			
		}
		
	}
			
	public void createStars(){
		for (int i=0; i<500; i++){	  	
	  		int guessx[i] = (int)(Math.random()*this.planetWidth);	  		
	  		int guessy[i] = (int)(Math.random()*this.planetHorizon);	  	
		}
	}

Die Kompilierfehlermeldung lautet "]" expected mit Zeiger auf die beiden i in createStars(), Linie 26 und 27 hier.
Eine weitere Frage von mir wäre, ob die Variablen evtl wie folgt deklariert werden müssen (auch wenn es am Kompilierfehler nichts ändert).

private int guessX[] = new int[500];
private int guessY[] = new int[500];

Wäre überaus dankbar, wenn mir Anfänger jemand weiterhelfen könnte
Gruß,
k.

 

[Caesar]

Ohne mir das genauer angeschaut zu haben, würde ich sagen, es liegt hierdran:

private int guessX[];
private int guessY[];

Die "[]" müssen immer nach dem Datentyp stehen, also so:

private int[] guessX;
private int[] guessY;

 

[kontur]

Also im ersten Moment dachte ich "nicht schon wieder ein threat geöffnet und es war nur ein Tippfehler"...aber
ich habe das nun geändert und bekomme noch immer gleichen Fehler.


Danke schonmal,
k.

 

[Caesar]

Okay, ich hab's noch ma genauer angeschaut. Versuch's ma damit:

private int[] guessx;
private int[] guessy;

und

private int[] guessx = new int[500];
private int[] guessy = new int[500];

also, das X und das Y am Ende klein geschrieben, denn unter dem Namen rufst du es im Code immer auf.
Obwohl mir das nicht erklärt, wieso er "]" erwartet.

 

[Lim_Dul]

Ändere

  public void createStars(){
      for (int i=0; i<500; i++){        
           int guessx[i] = (int)(Math.random()*this.planetWidth);           
           int guessy[i] = (int)(Math.random()*this.planetHorizon);        
      }
   }

in

  public void createStars(){
      for (int i=0; i<500; i++){        
           guessx[i] = (int)(Math.random()*this.planetWidth);           
           guessy[i] = (int)(Math.random()*this.planetHorizon);        
      }
   }

Allerdings solltest du vorher die Arrays definieren, wie du selbst geschrieben hast:

private int[] guessX = new int[500];
private int[] guessY = new int[500];

 

[kontur]

also mein Code jetzt:

	private boolean starsCreated = false;
	private int[] guessX = new int[500];
	private int[] guessY = new int[500]; 

       public void drawStars(Graphics g){
		if (starsCreated!=true) createStars();
		else{
			for (int j=0; j<500; j++){
				int yellowVariation = (int)((Math.random()*100)+155); 
			  	Color rgbStar = new Color(255, 255, yellowVariation);
			    
			  	g.setColor(rgbStar);	    
			  	g.drawOval(guessX[j],guessY[j],2,2);		
			}			
		}
		
	}
			
	public void createStars(){
		for (int i=0; i<500; i++){	  	
	  		int guessX[i] = (int)(Math.random()*this.planetWidth);	  		
	  		int guessY[i] = (int)(Math.random()*this.planetHorizon);	  	
		}
	}

Fehlermeldung ist noch immer die Gleiche.


Danke soweit,
k.

 

[Lim_Dul]

Du sollst

   int guessX[i] = (int)(Math.random()*this.planetWidth);           
   int guessY[i] = (int)(Math.random()*this.planetHorizon);

in

   guessX[i] = (int)(Math.random()*this.planetWidth);           
   guessY[i] = (int)(Math.random()*this.planetHorizon);

ändern.

Die int am Anfang der Zeile gehören da nicht, das Array ist ja schon deklariert.

 

[kontur]

ach....thanks funktioniert

k.