Bild wird immer neu gemalt

[Exterminas]

hi alle zusammen,

ich möchte in meinem "programm" eine zufallszahl zwischen 1 und 15 erstellen und diese dann an hand eines bildes darstellen. es funktioniert alles nur werden in zufälliger reinfolge immer neue bilder gemalt irgendwann hört es dann auf und wenn ich meinen button drücke der dann eine neue zahl und ein neues bild machen soll dann geht es wieder von vorne los und es werden wieder mehrere bilder hintereinander gemalt.

code des applets:

import java.awt.*;
import java.applet.*;
import java.awt.event.*;

public class Orakel extends Applet implements ActionListener
{
  MSSOrakel o;
  Button neu;

  public void init()
  {
      this.setLayout(null);
      this.setBackground(Color.black);
      neu = new Button("Vorhersagen");
      neu.setBounds(130,20,70,20);
      add(neu);
      neu.addActionListener(this);
      o = new MSSOrakel(this);
  }

  public void paint(Graphics s)
  {
      o.Vorhersage();
      o.BildMalen();
  }
  
  public void actionPerformed(ActionEvent e)
  {   
      o.Vorhersage();
      o.BildMalen();
  }
}

code der klasse:

import java.awt.Graphics;
import java.awt.Image;
import java.applet.Applet;
import java.lang.Math;

public class MSSOrakel extends Wuerfel
{
    protected Graphics stift;
    protected Image img;
    protected Applet Orakel;
    protected int augenzahl;
    
    public MSSOrakel(Applet a)
    {
        Orakel = a;
        stift = Orakel.getGraphics();
        augenzahl = 0;
    }
    
    public void Vorhersage()
    {
        augenzahl = (int) (Math.random() * 15) + 1;
    }
    
    public void BildMalen()
    {        
        if(augenzahl==1)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/1.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==2)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/2.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==3)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/3.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==4)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/4.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==5)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/5.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==6)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/6.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==7)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/7.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==8)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/8.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==9)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/9.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==10)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/10.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==11)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/11.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==12)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/12.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==13)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/13.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==14)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/14.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
        if(augenzahl==15)
        {
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/15.jpg");
            stift.drawImage(img,0,0,Orakel);
        }
    }
}

ich hoffe ihr könnt mir helfen schon mal danke im vorraus

mfg
Exterminas

 

[SlaterB]

ich weiß es nicht genau, vermute aber, dass
stift.drawImage(img,0,0,Orakel);
ein repaint des Applets auslöst welches wieder zum nächsten Bild führt usw.

----------
mit
irgendeine-java-Grafikkomponente.getGraphics();
geht alles schief, was Anfänger sich so ausdenken,
verwende NIEMALS getGraphics()

stattdessen etwa so:

public void paint(Graphics s) 
  { 
      o.Vorhersage(); 
      o.BildMalen(g); 
  } 
  
  public void actionPerformed(ActionEvent e) 
  {    
      repaint();
  }





+



public void BildMalen(Graphics g) 
    {        
        if(augenzahl==1) 
        { 
            img = Orakel.getImage(Orakel.getCodeBase(),"Bilder/1.jpg"); 
            g.drawImage(img,0,0,Orakel); 
        } 
....
}

---------

allgemeiner Tipp:
lade die Bilder anfangs in ein Image-Array und zeige die Bilder aus dem Array an statt sie immer wieder zu laden,

netter Nebeneffekt: du brauchst auch keine 15 if mehr,
mit Arrays geht alles leichter, Array-Index 0-14

wenn du dich mit Arrays noch gar nicht auskennst, was ich jetzt bei diesen 15 ifs vermute,
dann vielleicht auf die Schnelle zu schwer,
muss nicht unbedingt sein

 

[Exterminas]

danke für die verbesserung und den tip mit den arrays ich guck mir das mal an aber das problem ist nicht behoben das bild zeichnet sich immer noch neu.

 

[SlaterB]

vielleicht wird das repaint aus einem anderen Grund öfters ausgeführt (Browser oder so),
dann wähle in der actionPerformed einmal zufällig ein Bild aus
und in repaint zeichest du immer das gleiche (aktuell ausgewählte) Bild

 

[Guest]

also noch mal danke für die hilfe ich hab die lösung jetzt endlich gefunden. das problem muss dadurch entstanden sein dass ich die zufallszahl auf 0 gesetzt hab und dann beim starten des applets gleich eine zufallszahl generieren lies. ich hab der zufallszahl jetzt einen zufälligen startwert gegeben und eine neue zahl erst durch drücken des knopfes zuweisen lassen. jetzt funktioniert es auch und das bild wird nur einmal geladen.
hier noch mal der funktionierende quelltext:

meine klasse:

import java.awt.Graphics;
import java.awt.Image;
import java.applet.Applet;
import java.lang.Math;

public class MSSOrakel extends Wuerfel
{
    protected Graphics stift;
    protected Image img;
    protected Applet Orakel;
    protected Image picArray[] = new Image[15];
    
    public MSSOrakel(Applet a)
    {
        Orakel = a;
        augenzahl = (int) (Math.random() * 15) + 1;
        picArray[0] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/1.jpg");
        picArray[1] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/2.jpg");
        picArray[2] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/3.jpg");
        picArray[3] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/4.jpg");
        picArray[4] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/5.jpg");
        picArray[5] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/6.jpg");
        picArray[6] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/7.jpg");
        picArray[7] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/8.jpg");
        picArray[8] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/9.jpg");
        picArray[9] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/10.jpg");
        picArray[10] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/11.jpg");
        picArray[11] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/12.jpg");
        picArray[12] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/13.jpg");
        picArray[13] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/14.jpg");
        picArray[14] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/15.jpg");
    }
    
    public void Vorhersage()
    {
        augenzahl = (int) (Math.random() * 15) + 1;
    }
    
    public void BildMalen(Graphics g)
    {        
        if(augenzahl==1)
        {
            g.drawImage(picArray[0],1,1,Orakel);
        }
        if(augenzahl==2)
        {
            g.drawImage(picArray[1],1,1,Orakel);
        }
        if(augenzahl==3)
        {
            g.drawImage(picArray[2],1,1,Orakel);
        }
        if(augenzahl==4)
        {
            g.drawImage(picArray[3],1,1,Orakel);
        }
        if(augenzahl==5)
        {
            g.drawImage(picArray[4],1,1,Orakel);
        }
        if(augenzahl==6)
        {
            g.drawImage(picArray[5],1,1,Orakel);
        }
        if(augenzahl==7)
        {
            g.drawImage(picArray[6],1,1,Orakel);
        }
        if(augenzahl==8)
        {
            g.drawImage(picArray[7],1,1,Orakel);
        }
        if(augenzahl==9)
        {
            g.drawImage(picArray[8],1,1,Orakel);
        }
        if(augenzahl==10)
        {
            g.drawImage(picArray[9],1,1,Orakel);
        }
        if(augenzahl==11)
        {
            g.drawImage(picArray[10],1,1,Orakel);
        }
        if(augenzahl==12)
        {
            g.drawImage(picArray[11],1,1,Orakel);
        }
        if(augenzahl==13)
        {
            g.drawImage(picArray[12],1,1,Orakel);
        }
        if(augenzahl==14)
        {
            g.drawImage(picArray[13],1,1,Orakel);
        }
        if(augenzahl==15)
        {
            g.drawImage(picArray[14],1,1,Orakel);
        }
    }
}

und hier das applet:

import java.awt.*;
import java.applet.*;
import java.awt.event.*;

public class Orakel extends Applet implements ActionListener
{
  MSSOrakel o;
  Button neu;

  public void init()
  {
      this.setLayout(null);
      this.setBackground(Color.black);
      neu = new Button("Vorhersagen");
      neu.setBounds(130,20,70,20);
      add(neu);
      neu.addActionListener(this);
      o = new MSSOrakel(this);
  }

  public void paint(Graphics g)
  {
      o.BildMalen(g);
  }
  
  public void actionPerformed(ActionEvent e)
  {   
      o.Vorhersage();
      repaint();
  }
}

es war die rede von einem array. so schwer wars eigentlich gar nicht ich fands eigentlich recht simpel war es ungefär das was du meintest??

 

[SlaterB]

picArray[0] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/1.jpg");
picArray[1] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/2.jpg");
picArray[2] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/3.jpg");
picArray[3] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/4.jpg");
picArray[4] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/5.jpg");
picArray[5] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/6.jpg");
picArray[6] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/7.jpg");
picArray[7] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/8.jpg");
picArray[8] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/9.jpg");
picArray[9] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/10.jpg");
picArray[10] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/11.jpg");
picArray[11] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/12.jpg");
picArray[12] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/13.jpg");
picArray[13] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/14.jpg");
picArray[14] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/15.jpg");

->

static int MAX = 15;


for (int i=0; i< MAX; i++) {
   picArray[i] = Orakel.getImage(Orakel.getCodeBase(),"Bilder/"+(i+1)+".jpg");
}

-----------

if(augenzahl==1)
{
g.drawImage(picArray[0],1,1,Orakel);
}
if(augenzahl==2)
{
g.drawImage(picArray[1],1,1,Orakel);
}
if(augenzahl==3)
{
g.drawImage(picArray[2],1,1,Orakel);
}
....

->

    g.drawImage(picArray[augenzahl-1],1,1,Orakel);

 

[Exterminas]

so letzte frage:
was bringt das i++??
ich weiß das wenn ich das schreib die variable +1 gerechnet wird und das ohne das i++ das applet nicht funktioniert aber was es jetzt bringt weiß ich nich.

 

[Leroy42]

was bringt das i++??
ich weiß das wenn ich das schreib die variable +1 gerechnet

... und i als neuem Wert zugewiesen wird.

i wird also nach jedem Schleifendurchlauf um 1 erhöht.

 

[Exterminas]

ahh ok dange damit wären dann alle meine fragen beantworted

mfg
Exterminas