Variable ändert sich nicht

[_antje_]

Hallo,

ich habe in einer Klasse folgende Methoden um die Variabel anzVar zu manipulieren. anzVar selber ist im Konstruktor als int auf 1 gesetzt.

 public void setAnzVar(int anz)	
	 {
	 	anzVar=anz; 
		AlgFenster.text.append("setAnzahlVar: "+anzVar+"\n");
	 }
	 
public int getAnzVar()
	 {
		AlgFenster.text.append("GetAnzVar: "+anzVar+"\n");
	 	return anzVar;
	 }
	 
public void inkrementAnzVar()
	 {
	 	setAnzVar(anzVar++);
		AlgFenster.text.append("inkremtentAnzVar: "+anzVar+"\n");
	 }

In meiner Dialogklasse greife ich auf diese Variable anzVar zu, um diese zu erhöhen. Aber egal wie ich sie auch verändere, egal welche Methode ich einsetze, die Variable anzVar ändert sich einfach nicht. Ich kann sie nicht hochzählen. Sie bleibt immer auf dem Wert im Kontruktor. Ich habe auch schon probiert, sie außerhalb vom Kontruktor zu initialisieren, aber auch bleibt sie bei dem initialisierten Wert.
Was habe ich da falsch gemacht?

Kann mir bitte jemand helfen?
Danke

Tschau
Antje

 

[Mizus]

hmm... also ich sehe in deinem code keine Fehler.
Aber kann es sein das du beim du die Variable mit final deklariert hast???
Ansonsten ist es auch noch möglich das du an irgendeiner anderen Stelle die Variable wieder auf den default Wert setzt.

 

[Pulvertoastman]

Die Benutzungs eines Debuggers mag hier vielleicht aufschlussreich sein...

 

[meez]

Hast du nicht immer das gleiche Objekt?
Setzte sie mal static...

 

[L-ectron-X]

setAnzVar(anzVar++);

Du benutzt hier den Postimkrement-Operator, dass heißt, Du übergibst erst die Variable anzVar an setAnzVar() und erst danach wird die Variable anzVar um eins erhöht. Danach wird die Methode verlassen und die lokale Variable zerstört.
Versuche es doch mal so:

setAnzVar(++anzVar);

Nun wird die Variable anzVar vor der Übergabe an die setAnzVar()-Methode inkermentiert. (Preinkement)

 

[bummerland]

mach mal aus

setAnzVar(anzVar++);

setAnzVar(anzVar+1);

edit: huch, seid ihr schnell...

 

[Isaac]

public void inkrementAnzVar() 
    { 
       setAnzVar(anzVar++); 
      AlgFenster.text.append("inkremtentAnzVar: "+anzVar+"\n"); 
    }

Diese Methode muss anzVar verändern. Du führst hier ein Postinkrement durch. mit SetAnzVar wird also noch der alte Wert gesetzt und danach inkrementiert. Die Ausgabe ins Fenster muss also die inkrementierte anzVar anzeigen. Es sei denn du hast anzVar als Final deklariert oder du versteckst anzVar unter einer lokalen Variable. In Eclipse kann man das auf Error oder Warning stellen damit sowas nicht passiert "Local variable declaration hides another field or variable".

Andere Möglichkeiten fallen mir im Moment nicht ein.

 

[_antje_]

Der Debugger bringt keine Fehler, es werden auch keine Exceptions geworfen.
In der Klasse Dialog verwende ich immer das gleiche Objekt, um auf anzVar zuzugreifen.
Ich habe auch schon gesucht, ob ich die Variable vielleicht wieder auf default setze, habe aber nichts gefunden.
Die static Variante hatte ich erst versucht, dass ging auch nicht, deshalb habe ich es dann mit dem Objekt versucht.
Ich weiß echt keinen Rat mehr.

 

[Isaac]

Dann poste doch mal den kompletten Code (wenn er denn nicht 10 Seiten lang ist). Dann kann ich das mal durch meine Compiler jagen.

 

[Pulvertoastman]

Man kann einen Debugger auch benutzen, um haltepunkte zu setzen, um dann die Ausführung schrittweise fortzusetzen und das Verhalten der Variablen beobachten.

 

[_antje_]

Ich habe keine final Variable,

beim Inkrementieren wird anzVar eins höher angezeigt, wenn ich danach aber getAnzVar aufrufe, ist anzVar wieder default.

++anzVar, geht auch nicht

 

[Pulvertoastman]

Greifst du da auch auf die gleichen Objekte zu?

 

[_antje_]

Hier ist der Qellcode.

Das Programm ist schon umfangreicher, da es sich um eine Projektarbeit handelt.

Im Dialog habe ich ein Objekt von Algorithmus angelegt. Der betreffende Quellcode steht unter type=5 und in der Ereignisbehandlung unter "Variable hinzufügen".
Ich hoffe, du siehst durch.

Klasse Algorithmus:

public class Algorithmus 
{
	boolean standard;
	double x,y,z;
	int step;
	int[] folge;
	
	private int anzVar;
	public String[] varBez;
	public double[] varWert;

	static boolean varDef;
	
	public Algorithmus()
	{
		standard=true;
		varDef=false;
		step=0;		
				
		anzVar=0;		
				
		varBez=new String[anzVar];
		varWert=new double[anzVar];	
		for(int i=0;i<anzVar;i++)
		{
			varBez[i]="";
			varWert[i]=0;		
		}
				
		if(standard==true){	x=10;y=5;z=0;}
		else{	x=10;	y=5; z=0;}
				
	}

	 public void setAnzVar(int anz)	
	 {
	 	anzVar=anz; 
		AlgFenster.text.append("setAnzahlVar: "+anzVar+"\n");
	 }
	 
	 public int getAnzVar()
	 {
		AlgFenster.text.append("GetAnzVar: "+anzVar+"\n");
	 	return anzVar;
	 }
	 
	 public void inkrementAnzVar()
	 {
	 	setAnzVar(++anzVar);
		AlgFenster.text.append("inkremtentAnzVar: "+anzVar+"\n");
	 }
	 
	public void setStandard(boolean b) {standard=b;	}

	//public void setFolge(int wert){folge[step]=wert; step++;}
		
/*	static String getVar()
	{
		String str=new String("Variablen: ");
		
		for(int i=0;i<anzVar;i++)
				str=" "+var[i]; 
				
			
		return str;
	}*/
	
	 void initialVar()
	{
		varBez=new String[anzVar];
		varWert=new double[anzVar];
	}
	
	 void setVariablen(String var, double wert)
	{
		for(int i=0;i<anzVar;i++)
			{
				varBez[i]=var; 
				varWert[i]=wert;
				AusgabeFenster.text.append("Variable:"+varBez[i]+"  Wert: "+varWert[i]+"\n");
		
		} 
	}
	
	
	
	double[] getVarWerte()
	{
		return varWert;
	}
	
	 String[] getVarBez()
	{
		return varBez;
	}
	
		
}

Klasse Dialog:

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;

public class Dialog extends JDialog implements ActionListener
{
	private JPanel buttonPanel=new JPanel();
	private JPanel dialogPanel=new JPanel();
	private JTextField eingabe1=new JTextField();
	private JTextField eingabe2=new JTextField();
	private JTextField eingabe3=new JTextField();
	private JTextField eingabe4=new JTextField();	
	private Label lb1,lb2,lb3,lb4;
	private JButton okButton=new JButton("OK");
	private JButton abrButton=new JButton("Abrechen");	
	private JButton addVar=new JButton("Variable hinzufuegen");
	
	static int dlg=0;
		
	Algorithmus alg=new Algorithmus();
		
	JTextField[] text;
	JTextField[] textWert;
	Choice[] typ;
		
	public Dialog(String titel,boolean modal,int type)
	{
		setTitle(titel);
		setSize(300,200);
		eingabe1.setText("");
		eingabe2.setText("");
		eingabe3.setText("");
		eingabe4.setText("");
		eingabe1.setBounds(100,100,50,25);
		eingabe2.setBounds(100,140,50,25);
		eingabe3.setBounds(100,180,50,25);
		buttonPanel.add(okButton);
		buttonPanel.add(abrButton);
	
		
		
		
		if(type==5)//Variablen eingeben
	            {
	                    int anz=alg.getAnzVar();
					
		Label[] label=new Label[anz];
		Label[] labelWert=new Label[anz];
		Label[] labelType=new Label[anz];
				
		text=new JTextField[anz];
		textWert=new JTextField[anz];
				
		setSize(350,100+anz*40);
		buttonPanel.add(addVar);			
		
		dialogPanel.setLayout(new GridLayout(anz*2,2));

		for(int i=0;i<anz;i++)
		{
                		label[i]=new Label("Variable "+(i+1)+": ");
			labelWert[i]=new Label("Wert "+(i+1)+": ");
			//labelType[i]=new Label("Type "+(i+1)+": ");
			text[i]=new JTextField();
			textWert[i]=new JTextField();
			text[i].setBounds(100,100+(40*i),50,25);
			textWert[i].setBounds(100,100+(40*i),50,25);
			//text[i].setText(alg.getVarBez(i));
			//textWert[i].setText(""+alg.getWert(i));
										 
			//typ[i].add("int");
			//typ[i].add("double");
			//typ[i].add("String");
			
		//	dialogPanel.add(labelType[i]);
			dialogPanel.add(label[i]);
			dialogPanel.add(labelWert[i]);
										
			//dialogPanel.add(typ[i]);typ[i].show();
			dialogPanel.add(text[i]);
			dialogPanel.add(textWert[i]);
							
		}
		dlg=5;
	}
	
                if(type==6) //Anzahl Variablen
	{
	    	if(Algorithmus.varDef==true)
		{
			
		}
		else
		{
		lb1=new Label("Wieviele Variablen?");
		dialogPanel.setLayout(new GridLayout(2,0));
		dialogPanel.add(lb1);
		dialogPanel.add(eingabe1);	
		dlg=6;
	}
			
}
		

	
            getContentPane().add(dialogPanel,BorderLayout.CENTER);
            getContentPane().add(buttonPanel,BorderLayout.SOUTH);
	okButton.addActionListener(this);
	abrButton.addActionListener(this);
	addVar.addActionListener(this);
	setVisible(true);
	}
	
	public void actionPerformed(ActionEvent e)
	{
		String kommando=e.getActionCommand();
		
		if(kommando.equals("OK"))
		{
			
			if(dlg==5)
			{	
				int anz=alg.getAnzVar();
				
				String[] str=new String[anz];
				double[] wert=new double[anz];
									
				for(int i=0;i<anz;i++)
				{
					str[i]=text[i].getText();
					wert[i]=Double.parseDouble(textWert[i].getText());
					
					AlgFenster.text.append("Variable:"+ str[i]+" = ");
					AlgFenster.text.append("Wert"+wert[i]+"   "+i+"\n");
					
	//				
				}
	
				alg.initialVar();
								
				//for(int i=0;i<anz;i++)				
				//alg.setVariablen(str[i],wert[i],i);				
								
	//			for(int i=0;i<anz;i++)
	//			AlgFenster.text.append(Algorithmus.getWert(i)+""+Algorithmus.getVarBez(i)+"\n");
			}
			
			if(dlg==6)
			{
				int  var=Integer.parseInt(eingabe1.getText());
				alg.setAnzVar(var);
				AlgFenster.statusleiste.setText("Variablen: "+var);
				Dialog dlg=new Dialog("Variablen",false,5);
			
			}
			
			
			
			setVisible(false);
			dispose();
		}
		
		
		
		if(kommando.equals("Variable hinzufuegen"))
				{
					alg.inkrementAnzVar();
										
					Dialog dlg=new Dialog("Variablen definieren",false,5);
					
					setVisible(false);
					dispose();
					
				}
	}
}

 

[Mick]

Hey,

      if(kommando.equals("Variable hinzufuegen")) 
            { 
               alg.inkrementAnzVar(); 
                               
               Dialog dlg=new Dialog("Variablen definieren",false,5); 
                
               setVisible(false); 
               dispose(); 
                
            }

hier erstellst Du eine neue Instanz von Dialog und schliesst Deinen. Damit
ist auch das Algo Objekt weg! Da kann dann auch nix inkrementiert bleiben!

Lösungsvorschlag: Dem Konstrutor von Dialog einen Algo mitgeben.

Nochwas, was soll das bezwecken?

      anzVar=0;       
             
      varBez=new String[anzVar]; 
      varWert=new double[anzVar];    
      for(int i=0;i<anzVar;i++)  // -> anzVar ist 0, da wird die Schleife garnicht duchlaufen!!!
      { 
         varBez[i]=""; 
         varWert[i]=0;       
      }

Grüße,
Mick

 

[Isaac]

Also anhand der geposteten Sourcefragmente kann ich keinen Fehler finden auch wenn mir die Klasse Algorithmus sehr seltsam vorkommt da sie in meinen Augen kaum bis garkeinen Sinn macht.
Jedenfalls kann es nicht sein das wenn du

alg.inkrementAnzVar();
alg.inkrementAnzVar();
alg.getAnzVar();

Aufrufst anzVar immer noch die selbe Zahl ist.

@Mick

alg ist Klassenglobal in Dialog definiert und ist danach nicht weg.
Edit: Ich Schelm Du hast natürlich recht. Er wirft sich ja selbst auf den Müll. Ashe auf mein Haupt :autsch:

 

[alpi]

mach doch "anzVar" static und entferne die Initialisierung aus dem Konstruktor dann läufts