Getter und Setter

[Fiction]

Hallo,
ich habe da ein Problem:

Ich habe eine Class A, in der ein String (storeName) definiert wird. Ich setze den mal auf "COM6" Über ein GUI Eingabefeld kann ich diesen Wert beliebig ändern. ZB. COM3. Mit einem Button "Click" bestätige ich die Änderung. Soweit so gut.

 public class BannerMotor extends JFrame implements ActionListener {

private String storeName = "COM6";

		enterName = new JLabel("Enter Port (eg. COM3): ");
		click = new JButton("Click");
		name = new JTextField();
		click.addActionListener(this);
		add(click);
		add(name);
		add(enterName);

public void actionPerformed(ActionEvent e) {
		
		SerialTest SerialOutput = new SerialTest();

		///////////Action listener for Serial Port///////////////////
		if (e.getSource() == click) {	
			storeName = name.getText();
			SerialTest main = new SerialTest();
			main.initialize();
		}
}

	public String getStoreName() {
		return storeName;
	}

	
	public void setStoreName(String storeName) {
		this.storeName = storeName;
	}

Problem:

Wenn SerialTest() in der If Schleife aufgerufen wird, sollte eine Verbindung mit "COM3" entstehen, aber in PORT_NAMES[0] steht noch immer der Wert "COM6". Was muss ich tun, damit in PORT_NAMES[0] der neue Wert drinnen steht?

public class SerialTest implements SerialPortEventListener {

String XY = ComPort.getStoreName();
PORT_NAMES[0] = XY;
}

Wo liegt mein denkfehler?
Danke an alle die sich die Zeit nehmen, mir zu helfen.

 

[Beni]

Du setzt in einer Klasse namens "BannerMotor" den String "storeName", und liest dann aus einer Klasse namens "ComPort" einen String aus. Haben diese beiden Klassen irgendeine Verbindung? Nur weil zwei Felder den gleichen Namen haben, haben sie noch lange nicht denselben Inhalt.

Du könntest den Konstruktor von SerialTest erweitern, so dass man den Port gleich übergeben muss (und ihn dann in BannerMotor auch übergeben).

 

[Fiction]

Hallo Beni,
danke für die rasche Antwort.
Kannst du mir bitte deine Lösung etwas genauer beschreiben? Ich verstehe leider nicht, was du meinst.

Ich habe versucht mit:

		 BannerMotor ComPort = new BannerMotor(); //get BannerMotor storeName
		 PORT_NAMES[0] = ComPort.StoreName;

und es klappt.

Aber das ist nicht die elegante Lösung oder?

 

[Beni]

Ich weiss ja nicht genau wo du was wie machen willst, aber mein Vorschlag würde auf sowas rauslaufen:

public class SerialTest{
  private String port;

  public SerialTest( String port ){  // hier den Port übergeben
    this.port = port;
  }

  public void initialize(){
    // "port" ansprechen und Dinge aufrufen...
  }
}

Und eine kleine Modifikation in BannerMotor:

   if (e.getSource() == click) {   
            storeName = name.getText();
            SerialTest main = new SerialTest( storeName ); // hier den Port übergeben
            main.initialize();
        }

 

[Fiction]

AHA,

danke vielmals.

Ich werds gleich ausprobieren........

Hat geklappt!