Schleife funktioniert nicht richtig

LordEnte

Mitglied
Hi, bin neu hier und habe eine kurze und wahrscheinlich schnell zu beantwortende Frage. 😀
Habe versucht verschiedene Werte in 4 verschiedenen int-Arrays zu speichern, und diese dann mit einer for Schleife abzurufen. Falls ein eingebener Wert mit dem eines Arrays übereinstimmt, soll ein bestimmter Wert geladen werden. Am Besten schaut selbst: :-D
Java:
int op1 = Integer.parseInt(fieldSprunghoehe.getText());
int op2 = Integer.parseInt(fieldLandhoehe.getText());

int ergebnis = op1 - op2;

int[] ArrayLaufen = { 1, 2, 3, 4, 5};
int[] ArraySpringen = { 6, 7, 8, 9, 10 };
int[] ArrayBeides = { 11, 12, 13, 14, 15 };
int[] ArrayUnmoeglich = { 16, 17, 18, 19, 20 };


String LaufenT = "Laufen";
String SpringenT = "Springen";
String BeidesT = "Beides";
String UnmoeglichT =  "Unmoeglich";

Object endergebnis = null;

int i = 0;

for (i = 0; i > 200; i++) {
    if (ergebnis == ArrayLaufen[i]) {
        endergebnis = LaufenT;
    } else if (ergebnis == ArraySpringen[i]) {
        endergebnis = SpringenT;
    } else if (ergebnis == ArrayBeides[i]) {
        endergebnis = BeidesT;
    } else if (ergebnis == ArrayUnmoeglich[i]) {
        endergebnis = UnmoeglichT;

    }
}

JOptionPane.showMessageDialog(Gui2.this, "Ergebnis: " + endergebnis, "BERECHNET!",
    JOptionPane.INFORMATION_MESSAGE);
Das Problem ist dabei, dass z.B. bei der Differenz 5, statt normalerweise Laufen, Springen herauskommt.
Hier die Ergebnisse: 1, 2 und 3 ergeben "Laufen"
4, 5 und 6 ergeben "Springen"
7, 8 und 9 ergeben "Beides"
10, 11 und 12 ergeben "Unmoeglich"
Wobei 1-5 Laufen, 6-10 Springen, 11-15 Beides und 16-20 Unmoeglich ergeben sollen.

Habe schon ziemlich lange nachgedacht, komme aber auf keine Lösung.

Bei Vorschlägen der Fehlerbehebung würde ich euch bitten, euch anfängerfreundlich auszudrücken - Danke 😀 (Ich weiß nicht wie man den Code in ein Fenster packt, wär auch cool, wenn ihr mir das beantworten könntet)

Lg LordEnte :-D
 
Zuletzt bearbeitet von einem Moderator:
Hallo,

die Bedingung in dieser for-Schleife wird nie wahr:
Java:
for(i = 0; i > 200; i++) {
(i kann nicht gleichzeitig 0 sein und größer als 200
 
Hallo,

die Bedingung in dieser for-Schleife wird nie wahr:
Java:
for(i = 0; i > 200; i++) {
(i kann nicht gleichzeitig 0 sein und größer als 200

Ich dachte "i = o" ist der Anfangswert und "i > 200" die Bedingung, also so, dass die Schleife solange gehen soll, bis 200 erreicht wurde, wie würde sie anders funktionieren?

btw wenn das der gesamte Code sein sollte, würde da gar nichts stehen, es fehlt nämlich die main 😉
Hier:
package mlgRechner;

public class Objects {

public static void main(String[] args) {

Gui MLGRechner = new Gui("Rechner");
}

}

Ich habe einen kleinen Fehler behoben(Irgendwie hat die Klasse auf eine Andere zugegriffen und deren Daten genommen, keine Ahnung wie), nur jetzt kommt dauernd "null" heraus, wie BuddaKaeks es schon geschrieben hat.
Jetzt weiß ich nicht wieso, da ich noch Anfänger bin... 😀

Danke an die schnellen Antworten, Lg LordEnte
 
Wenn ich die Bedingung auf z.B. "i < 30" setze, kommen in der Konsole Fehlermeldungen und es wir auf diese Zeile hingewiesen: if(ergebnis == ArrayLaufen) {
 
@Lamajockey
So ist es.
Da sein Ausgangspost aber teilweise kursiv geschrieben ist lässt sich annehmen dass er es bereits so macht und einfach ArrayLaufen[i] geschrieben hat. Wenn man das ohne Code-Tags macht dann wird [i] als kursiv interpretiert.
 
Moin,
Ich dachte "i = o" ist der Anfangswert und "i > 200" die Bedingung, also so, dass die Schleife solange gehen soll, bis 200 erreicht wurde, wie würde sie anders funktionieren?
In dem du die Bedingung richtig setzt!! 😉
Es muss natürlich 'i < 200' heißen ... oder ugs.: 'solange i kleiner als 200 ist' !!
bei dir steht sowas wie: 'solange i GRÖSSER als 200 ist' !!

VG Klaus
 
Da es zum Lösen des Problems einfacher ist, poste ich den gesamten Code mit den dabei kommenden Errors:
Gui:
Java:
package mlgRechner;

import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Arrays;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JTextField;

public class Gui extends JFrame {

 private JLabel labelSprunghoehe;
 private JLabel labelLandhoehe;
 private JLabel Operator;

 private JTextField fieldSprunghoehe;
 private JTextField fieldLandhoehe;

 private JButton buttonRechnen;

 public Gui(String titel) {

  setTitle(titel);
  setDefaultCloseOperation(EXIT_ON_CLOSE);
  setLayout(new FlowLayout());

  setSize(250, 110);
  setResizable(false);

  initComponents();

  add(labelSprunghoehe);
  add(labelLandhoehe);
  add(fieldSprunghoehe);
  add(Operator);
  add(fieldLandhoehe);
  add(buttonRechnen);

  setLocationRelativeTo(null);
  setVisible(true);
 }

 private void initComponents() {

  Operator = new JLabel("auf");

  labelSprunghoehe = new JLabel("Sprunghöhe                 ");
  labelLandhoehe = new JLabel("Landhöhe   ");

  fieldSprunghoehe = new JTextField(8);
  fieldLandhoehe = new JTextField(8);

  buttonRechnen = new JButton("Rechnen!");
  buttonRechnen.addActionListener(new ActionListener() {

   @Override
   public void actionPerformed(ActionEvent e) {

    int op1 = Integer.parseInt(fieldSprunghoehe.getText());
    int op2 = Integer.parseInt(fieldLandhoehe.getText());   
   
    int ergebnis = op1 - op2;

    int[] ArrayLaufen = new int[] { 1, 2, 3, 4, 5};

    int[] ArraySpringen = new int[] { 6, 7, 8, 9, 10};
 
    int[] ArrayBeides = new int[] {11, 12, 13, 14, 15};

    int[] ArrayUnmoeglich = new int[] {16, 17, 18, 19, 20};

    String LaufenT = new String("Laufen");
    String SpringenT = new String("Springen");
    String BeidesT = new String("Beides");
    String UnmoeglichT = new String("Unmoeglich");
  
    int i = 0;

    String endergebnis = "";
   
    for (i = 0; i < 30; i++) {
     if(ergebnis==ArrayLaufen[i]) {
      endergebnis = LaufenT;
     } else if (ergebnis==ArraySpringen[i]) {
      endergebnis = SpringenT;
     } else if (ergebnis==ArrayBeides[i]) {
      endergebnis = BeidesT;
     } else if (ergebnis==ArrayUnmoeglich[i]) {
      endergebnis = UnmoeglichT;
     }
    }
    JOptionPane.showMessageDialog(Gui.this, "Ergebnis: " + endergebnis, "BERECHNET!",
      JOptionPane.INFORMATION_MESSAGE);

    fieldSprunghoehe.setText("");
    fieldLandhoehe.setText("");
   }
  });
 }
}

Objects:
Java:
[B]package[/B] mlgRechner;



[B]public[/B] [B]class[/B] [B]Objects[/B] {



[B]public[/B] [B]static[/B] [B]void[/B] [B]main[/B]([B]String[/B][] args) {



[B]Gui[/B] [B][U]MLGRechner[/U][/B] = [B]new[/B] Gui("Rechner");

}
}
und die dabei enstehenden Error:
Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 5

at mlgRechner.Gui$1.actionPerformed(Gui.java:87)


at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)

at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)

at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

wobei Zeile 87 diese ist:
Java:
[B]if[/B](ergebnis==ArrayLaufen[i]) {

Würde mich über anfängerfreundliche Lösungsvorschläge freuen. 😀
Vielen Dank im Vorraus,
Lg LordEnte
 
Könnte es vllt zusammenhängen, das deine Arrays fünf Einträge haben, du bis Eintrag 30 gehst, und die Exception java.lang.ArrayIndexOutOfBoundsException: 5 ist?
 
Könnte es vllt zusammenhängen, das deine Arrays fünf Einträge haben, du bis Eintrag 30 gehst, und die Exception java.lang.ArrayIndexOutOfBoundsException: 5 ist?

Ich habe jetzt die Bedingung auf "i < 5" gesetzt, und es funktioniert 😀
Wusste nicht, dass das von Bedeutung ist xD
Danke für die schnelle Antwort, wäre niemals selbst draufgekommen 😀
Lg LordEnte
 

Zurück
Oben