Klasse in eine extra Java-Datei

[wildbret]

Hallo,

ich wollte mein Programm etwas übersichtlicher gestalten und habe deswegen eine Klasse aus der Datei in eine neue Java-Datei verlagert.
Ich habe die imports und extends gemacht und es kompiliert fehlerfrei.

Aber bei Ausführung des Programmes gibt es jetzt eine Exception die vorher nicht auftrat.
Es handelt sich um ein Programm, dass Daten über die serielle Schnittstelle sendet.
Die ausgelagerte Klasse dient dazu die Daten zu empfangen und in ein TextField auszugeben.

Exception in thread "Win32SerialPort Notification thread" java.lang.NullPointerException
at Receiver.serialEvent(Receiver.java:11)
at com.sun.comm.Win32SerialPort.sendDataAvailEvent(Win32SerialPort.java:649)
at com.sun.comm.NotificationThread.run(Win32SerialPort.java:878)

Hier die ausgelagerte Klasse, es wurde sonst nichts verändert weder an dieser Klasse noch an der von der sie erbt.
Was habe ich falsch gemacht, stimmt mit der Vererbung etwas nicht, oder darf ich die Klasse in diesem Fall nicht auslagern?

import java.io.IOException;
import javax.comm.SerialPortEvent;
import javax.comm.SerialPortEventListener;

@SuppressWarnings("serial")
public class Receiver implements SerialPortEventListener{
	  public void serialEvent(SerialPortEvent event) { 
		  if(event.getEventType()==SerialPortEvent.DATA_AVAILABLE){
		        byte[] readBuffer = new byte[256];
		        try {
		          while (in.available() > 0) {in.read(readBuffer);} 		          		         
		       
		          ausgabe.append("\n");
		          for (int i=0;i<readBuffer.length;i++) {
		        	  if(readBuffer[i]==0x02 && readBuffer[i+2]==0x55){        		  
		        		  byte laenge = readBuffer[i+1];
		        		  byte abc[] = new byte[laenge-1];
		        		  for (int j=0;j<(laenge-1);j++){
		        			  abc[j] = readBuffer[i+3+j];		        			  
		        		  }
		        		  String bca = new String(abc);
		        		  ausgabe.append(bca);
		        	  }  
		        	  if(readBuffer[i]==0x02 && readBuffer[i+2]!=0x55){
		        		  byte laenge = readBuffer[i+1];
		        		  for (int j=0;j<(laenge+4);j++){		        			  
		        			  int intumw = unsignedByteToInt(readBuffer[i+j]);
		        			  ausgabe.append(Integer.toHexString(intumw)+" ");
		        		  }      	        		  
		        	  }
		          } 
		        }
		        catch (IOException e) {System.out.println("Fehler: "+e);}		        
		  }    	      
	  }
  }

Wäre nett wenn mir da jemand einen Tip geben könnte.

Danke

 

[Der Müde Joe]

dir Fehlen jetzt gewisse Dinge...zB

in.... in.avaiable()

was ist in?

in der anderen Klasse wahrscheinlich ein InputStream.
Aber in dieser Klasse nix.

 

[wildbret]

Ja genau, in der anderen Klasse ist in ein InputStream.
Erbe ich den nicht?

 

[Der Müde Joe]

naja..kann ich nicht sehen. hast ja das entfernt.
super.in.foo();

 

[wildbret]

Ich poste mal die Teile der anderen Klasse von denen ich glaube, dass sie relevant sind.
Der Rest sollte nur GUI und so sein.

static CommPortIdentifier portID;  
  InputStream in;
  BufferedOutputStream B_out;
  OutputStream out;
  static SerialPort SP;
...
public class connectionListener implements ActionListener{	
	public void actionPerformed(ActionEvent e) {
		String command = e.getActionCommand();	
		
		if(command.equals("Verbindung aufbauen")){
			String PortNr = Gruppe.getSelection().getActionCommand();
			try{portID = CommPortIdentifier.getPortIdentifier(PortNr);
	        	SP = (SerialPort) portID.open("Framesend",2000); 
	        	ausgabe.append("\nPort "+PortNr+" geöffnet");
			}
			catch(Exception exc){
				System.out.println("Fehler :"+exc);
				ausgabe.append("\nFehler: ungültige Portauswahl!");
			}		
			try {in = SP.getInputStream();				
				out = SP.getOutputStream();
				//B_in = new BufferedInputStream(in);
				B_out = new BufferedOutputStream(out);
	    		SP.addEventListener(new Receiver());
			}
			catch (Exception e1) { System.out.println("Fehler: "+e1);}
				SP.notifyOnDataAvailable(true);
				int Baud = Integer.valueOf(Baudrate.getSelectedItem()).intValue();
			try {SP.setSerialPortParams(Baud,
	                                  SerialPort.DATABITS_8,
	                                  SerialPort.STOPBITS_1,
	                                  SerialPort.PARITY_NONE);}
			catch (UnsupportedCommOperationException e1) {}
		}

 

[Der Müde Joe]

Was erbt denn der receiver? oder von wem besser gefragt?

Oder mach halt:

class Receiver implements...{

private InputStream in;

public Receiver( InputStream in){
this.in = in;
}

public void serialEvent....
}

dann hast du denn in auch ohne super aufzurufen.
wenn InpSt. in der super Klasse existiert, kannste auch
eine getter machen oder eben super

EDIT:
code angepasst
Erben = extends Irgendwas

 

[wildbret]

Oh sorry, ich hab in dem was ich kopiert habe ne Zeile falsch.

public class Receiver extends FrameSend implements SerialPortEventListener {

Ich dachtre der Receiver erbt alles was in FrameSend steht, das ist die Java-Datei wo die Klasse zuvor drin stand.
Die Klasse connectionListener die ich gepostet habe ist eine Unterklasse von FrameSend.

 

[wildbret]

Ok soweit so gut, ich habe deinen Konstruktor eingebaut und den Aufruf des Receivers in der anderen Klasse angepasst.
Die Exception ist weg und es gibt keine Fehler.
Seine Aufgabe erfüllen, nämlich ankommende Daten in das TextField zu schreiben tut der Receiver allerdings nicht.

Ich versuche mal das TextField (ausgabe) auch mit dem Konstruktor zu übergeben.

 

[Der Müde Joe]

Die Klasse connectionListener die ich gepostet habe ist eine Unterklasse von FrameSend.

nein ist sie nicht...sie erbt ja nix. falls du Innere Klasse meinst, das kann sein, ist aber keine Vererben.

InputStream und so.....wo ist das? kapiers jetzt nicht ganz.

PS: KlassenNamen inner mit Grossbuchstaben beginnen, varialblenname immer klein

 

[Der Müde Joe]

mach doch einfach mal ein System.out.println da rein,
dann siehst du auch was abläuft.

Oder noch besser...debugger anwerfen und step by step
durch

 

[wildbret]

Stimmt du hast Recht, erinnere mich an meine Java Vorlesung, innere Klasse nennt man das
Der InputStream wird in der Hauptklasse FrameSend ganz am Anfang angelegt.

public class FrameSend extends Frame{	 
...
InputStream in;
...

Dann kommen einige Methoden und dann die innere Klasse ConnectionListener, in der der InputStream dem Serial Port zugewiesen wird.

public class connectionListener implements ActionListener{	
...
try {in = SP.getInputStream();			
...

Der Receiver greift diesen InputStream dann wieder auf.

Durch das hinzufügen der TextArea in deinen Konstruktor funktioniert das Programm jetzt wieder !
Danke dir!

public Receiver(InputStream in,TextArea ausgabe){
		this.in = in;
		this.ausgabe = ausgabe;
	}

 

[Der Müde Joe]

so macht aber die Vererbung keinen Sinn mehr und du kannst "extends FrameSend" weglassen.

in hättest du via super erreicht...also

super.in.avaiable()
oder in der superKlasse getIn() definnieren
und dies in der erbenden Klasse so aufrufen:
getIn().avaiable()
oder halt 1 mal
InputStream in = getIn();

 

[wildbret]

Hmm, ich habs mal versucht.

In der Superklasse habe ich folgende Methoden geschrieben.

public static InputStream getIN() {
	  return in;
  }
  public static TextArea getAusgabe() {
	  return ausgabe;
  }

Im Receiver habe ich das extends und den Konstruktor entfernt.

InputStream in = FrameSend.getIN();
	TextArea ausgabe = FrameSend.getAusgabe();

Ist das richtig und so wie du es dir gedacht hast?

Aber extends FrameSend ist mir aufgefallen muss ich trotzdem mache, weil ich eine Umformmethode aus der superklasse nutzen möchte.

*edit* nur leider gibt der Receiver jetzt wieder nichts mehr in ausgabe aus.

 

[Der Müde Joe]

war eigentlich mit Vererbung ohne static gedacht.

dh:

protected InputStream getIn(){ return this.in;)

...
InputStream in = getIn();

die getIn() wird ja an die Unterklasse vererbt

 

[Der Müde Joe]

Als Beispiel:

public class SuperKlasse {

	private String s = "hello";	
	protected String ss = "hello2";
	
	protected String getS(){
		return this.s;
	}
}

public class SubKlasse extends SuperKlasse{

	public static void main(String [] args){		
		SubKlasse sub = new SubKlasse();
		
		String s = sub.getS();		
		System.out.println(s);
		
		System.out.println(sub.ss);
		
		sub.test();
		sub.test2();
	}
	
	public void test(){
		System.out.println(super.ss);
	}
	
	public void test2(){
		System.out.println(getS());
	}
}

EDIT:
Beispiel erweitert...
getS() kann halt nicht im static kontext aufgerufen werden..darum im test drin

 

[wildbret]

So habe ich es jetzt umgesetzt, müsste genau so sein wie im Beispiel, aber jetzt kommt wieder die Exception.
Bisher hat es nur mit dem Konstruktor funktioniert, seltsamerweise.

Wofür ist eigentlich das protected. Selbst wenn ich (protected InputStream in schreibe klappt es nicht.

public class FrameSend extends Frame{	 
        InputStream in;
        TextArea ausgabe;

        protected InputStream getIn(){ return this.in;}
  
        protected TextArea getAusgabe() { return this.ausgabe;}
...
}

public class Receiver extends FrameSend implements SerialPortEventListener {	
	InputStream in = getIn(); 
	TextArea ausgabe = getAusgabe();
...
}

 

[Der Müde Joe]

Exception? poste mal

public protected private sind für die Sichtbarkeit.

public....überall sichtbar
protected ...im selber package sichtbar
private ..nur in der klasse Sichbar

 

[wildbret]

die gleiche Exception wie am Anfang in der Zeile

while (in.available() > 0) {in.read(readBuffer);}

Exception in thread "Win32SerialPort Notification thread" java.lang.NullPointerException
at Receiver.serialEvent(Receiver.java:18)
at com.sun.comm.Win32SerialPort.sendDataAvailEvent(Win32SerialPort.java:649)
at com.sun.comm.NotificationThread.run(Win32SerialPort.java:878)

 

[wildbret]

Ich kann das eigentlich gut nachvollziehen was du mir gezeigt hast und kann nicht verstehen warum es mit den get-Methoden nicht geht mit dem Konstruktor vom Anfang aber schon.

 

[Der Müde Joe]

ach...ich bin schuld......

so gehts nat nicht....die Vererbung geht schon, aber
receiver erbt nur und nimmt nicht automatisch die
Werte von einem anderen Objekt, auch wenn es
eine Superklasse ist.

Am besten du machts das mit Receiver(in, Ta)

EDIT:
new Receiver() ...macht ein neues Objekt....mit allem was auch in der Superklasse ist
wobei in und ta und so nat null sind für diese neue Objekt

Eigentlich ist Vererbung hier Fehl am Platze, da Receiver nix gemeinsam hat
mit FrameSend oder Frame

 

[wildbret]

Ok ich danke dir erstmal.
Habe einiges gelernt und die Klasse ist erfolgreich in eine neue Datei gewandert.
Jetzt versteh ich auch, warum sie in der Originaldatei ohne Konstruktor funktioniert hat. Weil sie eine innere Klasse war.

Da der Empfänger nun erfolgreich ausgelagert ist, werde ich morgen evtl. mal versuchen den Sender (auch eine innere Klasse von FrameSend) auch auszulagern.
Sinn und zweck des ganzen ist es die einzelnen Funktionen von der GUI zu trennen und das ganze übersichtlicher zu machen.

 

[Der Müde Joe]

nennt sich MVC ....

einfach mal googeln
zB
http://www.javaworld.com/javaworld/jw-04-1998/jw-04-howto.html
http://www.csis.pace.edu/~bergin/mvc/mvcgui.html