Java Code ausbessern

[Javanier (Gast)]

Hallo,

ich steige gerade in die Java Programmierung ein und baue seriell eine 
Verbindung auf. Eigentlich bekomme ich schon Daten und werden in eine 
Datei geschrieben, jedoch werden mir Warnungen angezeigt wo ich nicht 
Durchblicke woran das liegen könnte.

Somit wollte ich fragen wie ich die Warnungen beheben kann und wie ich 
den Code noch verbessern könnte. Es sind nur die grundlegenden 
Funktionen vorhanden und die Funktion wird nicht ausgeführt wann Daten 
kommen sondern führt ständig while(1) aus.

Für Tipps bin ich euch dankbar.

MfG

import java.io.*;
import gnu.io.CommPort;
import gnu.io.CommPortIdentifier;
import gnu.io.SerialPort;
import gnu.io.NoSuchPortException;
import gnu.io.PortInUseException;
import gnu.io.UnsupportedCommOperationException;

/**
 *
 * @author
 */
public class UsbStick {
    
   private static SerialPort serialPort = null;
    
    
    public void dateiBeschreiben(String zeile){
     ...
    }
    
    public void verbinden(){
        CommPortIdentifier portIdentifier = null;
        CommPort commPort = null;
        
        try{
            portIdentifier = CommPortIdentifier.getPortIdentifier("/dev/ttyUSB0");
        }catch(NoSuchPortException e){
            System.out.println(e);
        }
        
        //Open the port and give the port name and a timeout 
        try{
            commPort = portIdentifier.open("/dev/ttyUSB0", 2000);  //Warning: Dereferencing possible null pointer
        }catch(PortInUseException e){
            System.out.println(e);
        }
        

        this.serialPort = (SerialPort) commPort; //Warning: Accessing static field serialPort

        //Set the baud rate and other parameters of the serial port
        try{
            this.serialPort.setSerialPortParams(115200, 8, 1, 0);  //Warning: Accessing static field serialPort

        }catch(UnsupportedCommOperationException e){
            System.out.println(e);
        }
}

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args){
        // TODO code application logic here
        UsbStick beschreiben = new UsbStick();
  
        beschreiben.verbinden();
    
        
        InputStream in = null; //Warning: The assigned value is never used
        String zeile = null;   //Warning: The assigned value is never used
        BufferedReader reader = null;  //Warning: The assigned value is never used
        while(true){
  
                in = this.serialPort.getInputStream(); //Error: non static variable this cannot be referenced from a static context
                if (in != null){
                    try{
                        reader = new BufferedReader(new InputStreamReader(in));
                        zeile = reader.readLine();
                        System.out.println(zeile);
                        beschreiben.dateiBeschreiben(zeile);
                    }catch(IOException e){
                        System.out.println("IOException");
                    }          
                }
        }        
    }  
}

[Udo S. (urschmitt)]

Habe mir nur den ersten angesehen:

Javanier schrieb:
> //Warning: Dereferencing possible null pointer

Du holst dir "portIdentifier", aber was machst du wenn das nicht 
funktioniert?
Nichts du druckst die Exception (besser wäre hier e.printStackTrace();) 
und willst dann fröhlich weitermachen obwohl durch den fehler 
"portIdentifier" null ist.
Richtig wäre hier die Exception (oder eine eigene weiter nach "oben" zu 
werfen. Oder aber zumindest die Funktion mit einem Fehlercode 
abzubrechen, den du dann auch auswerten müsstest.

Javanier schrieb:
> try{
>             portIdentifier =
> CommPortIdentifier.getPortIdentifier("/dev/ttyUSB0");
>         }catch(NoSuchPortException e){
>             System.out.println(e);
>         }

[Wühlhase (Gast)]

Ich würde dir raten, mal ein Buch über Entwurfsmuster zu lesen. 
Vielleicht "Entwurfsmuster von Kopf bis Fuß", das arbeitet mit Java und 
ich finde es sehr gut.

Danach wirst du einige Anhaltspunkte haben, was du an deinem Code ändern 
solltest.

[oerks (Gast)]

Warnungen aller Art einfach nach /dev/null umleiten.
Problem geloest!

[Javanier (Gast)]

@Udo S.
Danke für den guten Tipp.

Hat sonst noch jemand Tipps wie man die Programmierung von der 
Verbindung verbessern kann!

import java.io.*;

import gnu.io.CommPort;

import gnu.io.CommPortIdentifier;

import gnu.io.SerialPort;

import gnu.io.NoSuchPortException;

import gnu.io.PortInUseException;

import gnu.io.UnsupportedCommOperationException;

/**

 *

 * @author

 */

public class UsbStick {

    

   private static SerialPort serialPort = null;

    

    

    public void dateiBeschreiben(String zeile){

     ...

    }

    

    public void verbinden(){

        CommPortIdentifier portIdentifier = null;

        CommPort commPort = null;

        

        try{

            portIdentifier = CommPortIdentifier.getPortIdentifier("/dev/ttyUSB0");

        }catch(NoSuchPortException e){

            e.printStackTrace();

        }

        

        //Open the port and give the port name and a timeout 

        try{

            commPort = portIdentifier.open("/dev/ttyUSB0", 2000);  //Warning: Dereferencing possible null pointer

        }catch(PortInUseException e){

            e.printStackTrace();

        }

        

        this.serialPort = (SerialPort) commPort; //Warning: Accessing static field serialPort

        //Set the baud rate and other parameters of the serial port

        try{

            this.serialPort.setSerialPortParams(115200, 8, 1, 0);  //Warning: Accessing static field serialPort

        }catch(UnsupportedCommOperationException e){

            e.printStackTrace();

        }

}

    /**

     * @param args the command line arguments

     */

    public static void main(String[] args){

        // TODO code application logic here

        UsbStick beschreiben = new UsbStick();

  

        beschreiben.verbinden();

    

        

        InputStream in = null; //Warning: The assigned value is never used

        String zeile = null;   //Warning: The assigned value is never used

        BufferedReader reader = null;  //Warning: The assigned value is never used

        while(true){

  

                in = this.serialPort.getInputStream(); //Error: non static variable this cannot be referenced from a static context

                if (in != null){

                    try{

                        reader = new BufferedReader(new InputStreamReader(in));

                        zeile = reader.readLine();

                        System.out.println(zeile);

                        beschreiben.dateiBeschreiben(zeile);

                    }catch(IOException e){

                        e.printStackTrace();

                    }          

                }

        }        

    }  

}

[quotendepp (Gast)]

sind die warning und error kommentare fehlermeldungen des compilers?

wenn ja: lösen!

sonst geht da nie was weiter

[Jonas B. (jibi)]

Dann benutz doch auch deine static variable, anstatt eine lokale 
Variable zu benutzen. Die anderen Fehler kannst du ignorieren, die 
kommen weil du die Variablen vor einer Schleife initialisiert und bevor 
du den Wert jemals ausliest einen neuen zuweist.