Null-Check trotz try-catch-Block (Java, Programmierstil)?

Habe eine Frage zum Programmierstil. Meine IDE weist mich an mehreren Stellen auf mögliche NullPointerExceptions hin, nachdem ich mir Objekte von einer API bzw. DB gezogen habe. Ich habe das aber sowieso mit try-catch umgeben, sodass die NPE gefangen werden würde und nichts "schlimmeres" passiert. Ich frage mich jetzt, ob man das so lässt und die Warnung einfach ignoriert, oder soll ich jetzt an einigen Stellen immer prüfen, ob etwas null oder empty ist, obwohl ich dann sowieso selbst nichts mache als in den finally-Block zu gehen und die Fehlermeldung zu schmeißen?

Beispiel (Java):

try
{
  Auto bmw = api.get(...);
  //Mache ich hier Null-Checks?
  //if(bmw != null)
  //if(bmw.getFarbe() != null)
  String lackierung = bmw.getFarbe().toString();
}
catch(Exception e)
{
  //...
}
finally
{
  //...
}

Das ist eigentlich unnötiger Code. Finde es aber auch merkwürdig, die Warnings und mögliche NullPointer zu ignorieren. Wie macht man sowas?

Das Ergebnis basiert auf 5 Abstimmungen

Ignorieren 60%

Null-Checks 40%

6 Antworten

Vom Fragesteller als hilfreich ausgezeichnet

Destranix

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Computer, IT, programmieren

25.07.2022, 13:08

Ich frage mich jetzt, ob man das so lässt und die Warnung einfach ignoriert, oder soll ich jetzt an einigen Stellen immer prüfen, ob etwas null oder empty ist, obwohl ich dann sowieso selbst nichts mache als in den finally-Block zu gehen und die Fehlermeldung zu schmeißen?

Vor allem solltest du nicht einfach alle Exceptons einfangen, sondern jeweils die erwartbaren aufschlüsseln.

Aber auch ansonsten ist es gut, explizit auf null zu prüfen, statt einfach danach die Exception aufzufangen, außer das würde sich gut einbauen (da sowieso ein Haufen Exceptions zu erwarten wären), dann sollte das aber mit einem Kommentar angemerkt werden.

Das Prüfen auf != null kannst du oft in Asserts packen.

schnfz

25.07.2022, 13:15

Das Prüfen auf != null kannst du oft in Asserts packen.

Die in produktivem Code eher nichts verloren haben und ohne -enableassertions auch von der JVM ignoriert werden.

Ein benutzerfreundliches API sollte - wenn es überhaupt nulls zurückgeben muss, weil Exceptions unpassend sind - Optionals zurückgeben, dann kann man vernünftig damit umgehen.

Destranix

25.07.2022, 13:19

@schnfz

Die in produktivem Code eher nichts verloren haben und ohne -enableassertions auch von der JVM ignoriert werden.

Im Debugmodus werden die ausgeführt, in der fertigen version nicht. Und doch klar packt man soetwas auch in produktiven Code!

Ein benutzerfreundliches API sollte - wenn es überhaupt nulls zurückgeben muss, weil Exceptions unpassend sind - Optionals zurückgeben, dann kann man vernünftig damit umgehen.

Wäre womöglich toll. Ich kann mich allerdings nicht erinnern das jemals bei einem API gesehen zu haben.

schnfz

25.07.2022, 13:22

@Destranix

Stimmt,da habe ich mich schlecht ausgedrückt - sie sind im Sourcecode, aber in Produktion zur Laufzeit eben wirkungslos. Wenn ich mich dann aber darauf verlasse, erlebe ich eine Überraschung…

Destranix

25.07.2022, 13:24

@schnfz

Korrekt! Wenn man zur Laufzeit einen Null-Check braucht, dann sollte man diesen entsprechend auch implementieren.

paulfragtgf

Fragesteller

25.07.2022, 13:26

@Destranix

Danke euch beiden!

paulfragtgf

Fragesteller

25.07.2022, 13:28

Kannst du mir vielleicht auch noch hierbei helfen? Wie verpackt man die URL für einer REST-Abfrage richtig?

BeamerBen

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Computer, programmieren

25.07.2022, 14:19

Null-Checks

catch(Exception e)

ist eher schlechter Stil, du solltest (in der Regel) gezielt die Exception Typen handeln die du erwartest und nicht einfach alle.

Wenn null ein erwarteter Rückgabewert ist, dann sollte man darauf prüfen und nicht das Exception Handling dafür missbrauchen. Exceptions sind für Dinge die eigentlich nicht passieren sollten. Was ich merkwürdig finde ist warum so was wie getFarbe null zurück geben sollte, warum sollte ein Auto keine Farbe haben? Cursed.

Wenn die API eigener Code ist müsstest du dich eventuell entscheiden ob null zurück geben richtig ist oder du lieber direkt dort eine Exception werfen möchtest. Aber meiner Meinung nach wäre eine Exception eben angebracht wenn es auch wirklich ein Fehler ist, egal wo jetzt genau.

So oder so ist null unschön. Ich mache kein Java mehr aber bin generell Fan von so was: https://docs.oracle.com/javase/8/docs/api/java/util/Optional.html

Woher ich das weiß:Berufserfahrung – Software Entwickler / Devops

jort93

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Computer

25.07.2022, 12:53

Muss kein null Check hin. Aber vermutlich kann api.get auch eine exception werfen? Sollte vermutlich auch in einen try Catch block.

paulfragtgf

Fragesteller

25.07.2022, 12:54

Ups, ja, steht in meinem Code auch so. Hab ich dummerweise in der Frage falsch gehabt. Hab ich korrigiert. Danke!

BooWseR

25.07.2022, 12:49

Statt eine allgemeine Exception zu fangen, kannst du auch gezielt Nullpointer catchen und damit umgehen.

catch (NullPointerException npe) {
  ...
}

BooWseR

25.07.2022, 12:50

Oder du prüfst ob bmw überhaupt existiert, bevor du die Funktion "getFarbe()" oder "toString()" ausführst.

paulfragtgf

Fragesteller

25.07.2022, 12:52

@BooWseR

Das ist ja meine Frage. Sollte ich vor dem Zugriff jedes Mal noch prüfen ob null/empty, oder sage ich: Ich spare mir diesen Code und Rechenaufwand, denn wenn eine NullPointerException auftritt, wird sie sowieso gefangen?

Ich stelle mir lediglich die Frage nach dem besseren Stil. Wie geht man damit um?

BooWseR

25.07.2022, 12:55

@paulfragtgf

Also was man machen sollte ist gezielt Exceptions abfangen, wie eben catch (NullPointerException ...)
So kannst du gezielter mit verschiedenen Fehlern umgehen.
Ich würde wohl vorher prüfen ob es eine Farbe gibt, kommt aber ein bisschen auf die Gesamtheit des Codes an.
Ist die Farbe nämlich leer, brauchst du den Rest der Methode auch nicht ausführen, sofern sich die Methode nur um die Farbe dreht.
Also einfach ein Return if not bmw.

Kelec

25.07.2022, 12:56

Das kommt ganz drauf an was Null in dem Sinne hier bedeutet.

Exceptions sind Fehlerbehebungsstrategien sollten aber niemals zum Programmfluss gehören. Also wenn es sein kann, dass bmw und der Returnvalue von GetFarbe Null sein können und das in dem Sinne keinen Fehler sondern nur Fehlende Information darstellt, dann sollte das nicht mit einer Exception abgefangen werden.

Also Exceptions sind für den Fall, dass etwas so schief geht, dass dieser Prozess abgebrochen werden muss, oder wenn irgendwo extern ein Fehler vorliegt, sie sollten aber nicht dafür genutzt werden einfach nur bestimmte Fälle abzufangen welche auch bei einem funktionierenden Programm auftreten können.

Wenn ich beispielsweise sowas habe:

try{

//Anweisungen

//X

//noch mehr Anweisungen

}

catch(Exception A){}

catch(Exception B){}

catch(Exception C){}

und bei meinen Anweisungen eine Exception A auftritt, die durch den ersten catch-Block behandelt wird, wird dann der try-Block an der Stelle, an der die Ausnahme geworfen wurde, nach der Behandlung weiter ausgeführt?

Wenn die Ausnahme also beispielsweise an Stelle X auftritt, werden dann nach der Fehlerbehandlung die "noch mehr Anweisungen" ausgeführt?

lg Kath

...zur Frage

JAVA Exception Handling (Try/Catch und Throw) - Gibt es davon Vor- und Nachteile?

Hallo,

ich befasse mich zur zeit mit Exception Hnadling in JAVA. Genauer gesagt mit Try/Catch und Throw.

Hat Try/Catch Vor- und Nachtteile? Wenn ja, welche?

...zur Frage

Was bringt sich throws im Methodenkopf und welche Funktion hat das (Java)?

Also mir ist unklar was das bringt, weil ich kann auch ohne throws Exception im Methodenkopf im try-catch eine neu Exception werfen.

Danke

...zur Frage

Java: HTTP Header ohne Content?

Hallo,

ich möchte von z.B. 300 Online-Dateien die Größe herausfinden und vorab zusammenaddieren (damit im Nachhinein eine korrekte Prozentzahl berechnet wird).

Ich brauche dafür nur vom HTTP-Header die Content Length.

Wie kann ich vermeiden, dass Java schon den Inhalt herunterlädt?

Bei folgendem Code dauert jede Abfrage ca. eine Sekunde, was jedoch nur für eine Abfrage der Datenlänge zu viel ist.

Kennt ihr eine Lösung?

public static long getFileSize(URL url) {
  URLConnection conn = null;

  try {
    conn = url.openConnection();

    if (conn instanceof HttpURLConnection) {
      ((HttpURLConnection) conn).setRequestMethod("HEAD");
    }

    return conn.getContentLength();
  }
  catch (IOException e) {
    throw new RuntimeException(e);
  }
  finally {
    if (conn instanceof HttpURLConnection) {
      ((HttpURLConnection) conn).disconnect();
    }
  }
}

...zur Frage

Java Assert!= null?

Hallo ich wollte nur kurz fragen was assert ist. Ist

assert permission != null;
town[1] = permission;

das selbe wie

if (permission != null) {
  town[1] = permission;
}

Und dann gild das auch für alles dadrunter?

...zur Frage

Java: Wie kann ich Variablen aus verschiedenen Klassen speichern?

Hallo,

ich würde gern Variablen aus verschiedenen Klassen in einem Textdokument speichern. Doch ich kriege nur die Variablen aus der gleichen Klasse. Vielleicht kann mir da jemand helfen.

Hier mein bisheriger Code:

int Leben = 100;
String LebenS = Integer.toString(Leben);

public void save() throws IOException {
  Config configclass = new Config();
  File Save = new File("saves");
  File config = new File("saves/variabeln.txt");

  if (Save.exists()) {
  }
  else {
    Save.mkdirs();
  }

  if (Save.exists()) {
  }
  else {
    try {
      config.createNewFile();
    }
    catch (IOException e) {
      e.printStackTrace();
    }
  }

  try {
    OutputStream stream = new FileOutputStream(config);

    try {
      stream.write(LebenS.getBytes());
      stream.close();
    }
    catch (IOException e) {
      e.printStackTrace();
    }
  }
  catch (FileNotFoundException e) {
    e.printStackTrace();
  }
}

...zur Frage

Java Exception Handling - Übungsbeispiel?

Sehr geehrte Community,

ich bin gerade dabei über Exceptions in Java zu lernen und bin auf folgende Aufgabe gestoßen, bei der ich mir bei meinem Ansatz unsicher bin.

Aufgabe:

Mein Ansatz:

"Es müsste eine try-catch-Methode eingebaut werden, die den Fehler auffängt und entsprechend wiedergibt."

try {

Anwendung

}

catch(Exception e) {

}

Nun ist mir jedoch unklar wo ich die genannte Methode aus c) einbaue, bzw. wie ich die try - catch Methode damit implementiere.

Vielen Dank schon mal im Voraus.

...zur Frage

Java Zugriff auf datei verweigert

Hallo Leute,

Ich bin gerade dabei ein kleines Programm in Java zu schreiben. darin möchte ich eine Datei kopieren lassen. Dabei wird aber der Zugriff verweigert. habe schon danach gegoogelt aber nichts gefunden. Hier ist der Code, mit dem ich die Datei kopieren möchte.

public class Game {

@SuppressWarnings("resource")
protected Game(File folder) throws IOException{
    File output = new File (folder + "/minecraft.jar");
    System.out.println(output);
    File input = new File("C:/Users/" +MinecraftBackup.user+ "/AppData/Roaming/.minecraft/bin/minecraft.jar");

    if(input.canRead() == true){
        JOptionPane.showMessageDialog(null, "Du hast keine Rechte für diese Datei!");
    }
    else{

        FileChannel inChannel = new FileInputStream(input).getChannel(); 
        FileChannel outChannel = new FileOutputStream(output).getChannel(); 
        try { 
            inChannel.transferTo(0, inChannel.size(), outChannel); 
        } catch (IOException e) { 
            throw e; 
        } finally { 
            if (inChannel != null) 
                inChannel.close(); 
            if (outChannel != null) 
                outChannel.close(); 
        } 
    }
}

}

Allerdings wird, wenn ich den code aufrufe der Zugriff auf die output datei verweigert. hat vielleicht jemand eine Lösung für mich?

Danke in vorraus

TheCubeCrafter

...zur Frage

Powershell Script soll keine Duplikate in der DB erstellen?

Hallo,

ich habe ein Powershell Script welches Daten von API Seiten abfragt und diese in eine Datenbank schreibt. Das funktioniert soweit auch. Leider dupliziert das Script bei jeder Abfrage die Daten in der Datenbank, dabei sollen schon vorhandene Datensätze nicht erneut geschrieben werden sondern nur neue, die auf der abgefragten Seite hinzugekommen sind. Habe verschiedene Lösungswege versucht aber nichts funktionierte. Entweder schrieb das Script trotzdem Duplikate oder es gab andere Fehlermeldungen... Kann mir jemand sagen wie ich das lösen kann?

Das Script:

# Konfiguration der Vote-Server

$voteServer = @(

@("LINKZURAPI1", "1.Server", 3),

@("LINKZURAPI2", "2.Server", 3)

)

# MySQL-Datenbank

$MySQLHost = "localhost"

$MySQLPort = "3306"

$MySQLDatabase = "test"

$MySQLUser = ""

$MySQLPassword = ""

# Funktion zum Überprüfen und Erstellen der Tabelle

function Ensure-DatabaseTable {

param (

[MySql.Data.MySqlClient.MySqlConnection]$connection

)

$tableName = "votings"

$query = @"

CREATE TABLE IF NOT EXISTS $tableName (

id INT PRIMARY KEY AUTO_INCREMENT,

name VARCHAR(100),

voteAt DATE,

server VARCHAR(100),

site VARCHAR(100),

received TINYINT(4) DEFAULT 0,

coins INT DEFAULT 0

);

$command = $connection.CreateCommand()

$command.CommandText = $query

$command.ExecuteNonQuery() | Out-Null

}

# Verbindung zur Datenbank

try {

[System.Reflection.Assembly]::LoadFrom("C:\Program Files (x86)\MySQL\MySQL Connector NET 8.2.0\MySQL.Data.dll") | Out-Null

$constr = "server=$($MySQLHost);port=$($MySQLPort);database=$($MySQLDatabase);user id=$($MySQLUser);password=$($MySQLPassword)"

$con = New-Object MySql.Data.MySqlclient.MySqlConnection($constr)

$maxAttempts = 5

$attempts = 0

while ($attempts -lt $maxAttempts -and $con.State -ne 'Open') {

try {

$con.Open()

} catch {

Write-Output "Fehler beim Verbinden zur Datenbank: $_"

}

$attempts++

Start-Sleep -Seconds 1

}

# Verbindung erfolgreich oder nicht

if ($con.State -ne 'Open') {

Write-Output "Verbindung zur Datenbank konnte nicht hergestellt werden."

Exit

} else {

Write-Output "Datenbankverbindung erfolgreich geöffnet."

}

# Abfrage für die Vote Links

foreach ($server in $voteServer) {

try {

Write-Output "Starte Abfrage für Server $($server[1])"

$r = Invoke-RestMethod -Method Get -Uri $server[0]

if ($r.GetType().FullName -eq "System.String") {

Write-Output "Abruf vom Vote-Server fehlgeschlagen (String erhalten)."

Continue

}

foreach ($voter in $r.data) {

$votedatum = $voter.timestamp

if (![System.DBNull]::Value.Equals($votedatum) -and $votedatum -ne $null -and $votedatum -ne "") {

try {

$date = [datetime]::ParseExact($votedatum, 'yyyy-MM-ddTHH:mm:ssZ', $null)

$cmd = $con.CreateCommand()

$cmd.CommandText = @"

INSERT INTO votings (name, voteAt, server, site, received, coins)

VALUES (@name, @voteAt, @server, @site, @received, @coins)

ON DUPLICATE KEY UPDATE

name = VALUES(name),

voteAt = VALUES(voteAt),

server = VALUES(server),

site = VALUES(site),

received = VALUES(received),

coins = VALUES(coins)

$cmd.Parameters.AddWithValue("@name", $voter.Username) | Out-Null

$cmd.Parameters.AddWithValue("@voteAt", $date.ToString("yyyy-MM-dd")) | Out-Null

$cmd.Parameters.AddWithValue("@server", $server[1]) | Out-Null

$cmd.Parameters.AddWithValue("@site", 'deutsche-arkserver') | Out-Null

$cmd.Parameters.AddWithValue("@received", 0) | Out-Null

$cmd.Parameters.AddWithValue("@coins", 0) | Out-Null

$cmd.ExecuteNonQuery() | Out-Null

Write-Output "Datensatz für $($voter.Username) erfolgreich eingefügt oder aktualisiert."

} catch {

Write-Output "Fehler beim Einfügen oder Aktualisieren in die Datenbank: $_"

}

Write-Output "Abfrage für Server $($server[1]) erfolgreich durchgeführt."

} catch {

Write-Output "Fehler bei Server $($server[1]): $_"

Write-Output $_.Exception.InnerException.StackTrace

}

} catch {

Write-Output "Fehler beim Verbinden zur Datenbank: $_"

} finally {

# Schließe die Datenbankverbindung am Ende der äußeren Schleife

if ($con.State -eq 'Open') {

$con.Close()

Write-Output "Datenbankverbindung erfolgreich geschlossen."

}

...zur Frage

Java mit einem Stream eine .text Datei kopieren?

Ich möchte per Stream die Datei Hallo wie gehts.txt kopieren und eine neue Datei copy.txt erzeugen. Warum funktioniert dies nicht? FileNotFoundException?

Code ist unten angehängt:

package de.thws;                     // Von mir zum testen von Streams

import java.io.*;

public class PerStreamDateiEinlesen {

    public static void main(String[] args) throws FileNotFoundException {

        copyFileWithErrorHandling3Modularized();
    }

    public static void copyFileWithErrorHandling3Modularized()
    {
        // try-with-resource statement
        try (InputStream fis = new FileInputStream("Hallo wie gehts.text");
             OutputStream fos = new FileOutputStream("copy.txt");)
        {
            copyImproved(fis, fos);
        }
        catch (IOException e)
        {
            e.printStackTrace();
        }
    }
    // Hier kopiere ich die Datei
    static void copyImproved(InputStream is, OutputStream os) throws IOException
    {
        byte[] b = new byte[1024];
        int n;
        do
        {
            n = is.read(b); //kann maximal 1024 bytes lesen
            if (n != -1) os.write(b, 0, n);
        }
        while (n != -1);
    }
}

...zur Frage

java intellij if(data == null)?

Der Code hier ist soweit verständlich, nur einen Ausdruck (hier farblich markiert)verstehe ich nicht.

Ich habe eine Methode hasDuplicates2 die als Parameter ein int-Array mit int[ ] data übergeben bekommt.

if(data == null ...

Ein int-Array kann doch gar keine null Werte enthalten?

null bedeutet, dass eine Referenzvariable bislang auf kein Objekt verweist.

Der Standardwert eines int-Arrays ist 0

Bsp. int[ ] arg = new int[3];

Jetzt sind in dem arg 0 0 0 abgelegt, wenn ich das array arg ausgeben würde

Was soll jetzt diese Prüfung if(data == null) ????

...zur Frage

Java If-Bedingung, for-Schleife, boolean Methode?

Ich kenn mich mit Java noch nicht wirklich aus und weiß hier im Moment nicht wie ich das, was ich vorhabe umsetzen kann.
Ich möchte, dass wenn ich die Methode „problemVorhanden“ 5 mal aufgerufen habe, die Konsole „Problem entdeckt“ und ansonsten „Kein Problem entdeckt“ ausgibt.
Ich hab jetzt alles mögliche versucht das irgendwie hinzubekommen, aber es funktioniert nicht.
Was muss ich jetzt in die if-Bedingung schreiben, damit es funktioniert, bzw. kann das so überhaupt funktionieren?

...zur Frage

Binärer Suchbaum in Array (Java)?

Hi, ich studiere im ersten Semester medizinische Informatik und wir sollen einen binären Suchbaum in einem sortierten Array zurückgeben. Kann mir da jemand helfen (wir programmieren in Java)? Das habe ich bis jetzt:

int[] sort() {
    zahlenElemente = new int[this.size()];
    return sort(this, zahlenElemente);
}

int[] sort(IntSearchTree suchbaum, int[]array) { if (suchbaum != null) { if (suchbaum.getLeft() != null){ sort(suchbaum.getLeft(), zahlenElemente); zahlenElemente[index] = suchbaum.getValue(); index++; } if (suchbaum.getRight() != null){ sort(suchbaum.getRight(), zahlenElemente); } for (int i = 0; i < this.size(); i++){ System.out.print(zahlenElemente[i] + " "); } } return zahlenElemente; }

Aus irgendeinem Grund werden nur Nullen ausgegeben und ich verstehe nicht warum...

...zur Frage

Wie kann ich mit Java prüfen, ob ein Satz in einer Textdatei enthalten ist?

else if (event.getSource() == blesen) {
  FileReader fr = null;

  try {
    String text;
    String fileName = "D:/Flo/password/loggin.txt";
    File file = new File(fileName);
    int len = (int)file.length();
    char[] buf = new char[len];

    fr = new FileReader(file);
    fr.read(buf);
    text = new String(buf);

    // ab hier JFrame
    JFrame auslesen = new JFrame("Output Dialog");
    auslesen.setLayout(null);
    auslesen.setVisible(true);
    auslesen.setSize(290, 250);
    auslesen.setResizable(false);
    auslesen.setTitle("Auslesen");

    auslesen.setDefaultCloseOperation(EXIT_ON_CLOSE);
    auslesen.setLocationRelativeTo(null);
    
    // bis hier her JFrame | ab hier dann eingabe feld
    JTextField plattlesen = new JTextField();
    JLabel welcheplatt = new JLabel("&lt;html&gt;Welche Loggin Daten &lt;p /"
      + "&gt;möchtestdu wissen?&lt;/p"
      + "&gt;Plattform:&lt;/html&gt;");
    
    auslesen.add(bfertiglesen);
    bfertiglesen.setBounds(1, 100, 100, 30);
    auslesen.add(plattlesen);
    plattlesen.setBounds(1, 70, 100, 20);
    auslesen.add(welcheplatt);
    welcheplatt.setBounds(1, 1, 200, 50);

    slesen = plattlesen.getText();
    System.out.println(text);
  }
  catch(IOException ex) {
    System.out.println(ex);
  }
  finally {
    try {
      if (fr!= null) fr.close();
    }
    catch(Exception ex) {
    }
  }
}
else if (event.getSource() == bfertiglesen) {
  slesen = plattlesen.getText();

  if (text.indexOf(slesen)) {
    System.out.println(slesen);
  }
}

Ich möchte prüfen, ob ein eingegebener Satz in einem Textdokument enthalten ist.

Wie mache ich das am besten?

...zur Frage

Was möchtest Du wissen?

Deine Frage stellen