code kürzen

  • Themenstarter Gelöschtes Mitglied 62813
  • Beginndatum
G

Gelöschtes Mitglied 62813

Gast
gibt es eine möglichkeit den code zu kürzen oder zu vereinfachen?

Java:
public void readAndStoreTags(String path) {
        Map<String, Object> tag_map = TagReader.readTags(path);
        for (String key : tag_map.keySet()) {
            if (key == "title") {
                if (tag_map.get(key).toString() != "" || tag_map.get(key).toString() != null) {
                    title = tag_map.get(key).toString().trim();
                }
            } else if (key == "author") {
                if (tag_map.get(key).toString() != "" || tag_map.get(key).toString() != null) {
                    author = tag_map.get(key).toString().trim();
                }  
            } else if (key == "album") {
                if (tag_map.get(key).toString() != "" || tag_map.get(key).toString() != null) {
                    album = tag_map.get(key).toString().trim();
                }
            } else if (key == "duration") {
                if ((long) tag_map.get(key) > 0) {
                    duration = (long)tag_map.get(key);  
                }
            }
        }
    }
 
Zuletzt bearbeitet von einem Moderator:

LimDul

Top Contributor
Sinnvoll wäre es im ersten Schritt die Fehler zu beseitigen :) Wenn du ihn in CODE-Tags setzt, wird er lesbar.
Strings vergleicht man nicht mit == sondern mit equals. Bei duration wird dir eine Nullpointer Exception fliegen, wenn der Wert in der map null ist.

Du kannst allerdings Strings in einem switch-case verwenden, dann sparst du dir die if-else cascade und kannst mit case-Statements arbeiten, das ist dann übersichtlicher. Die Abfrage ob der Value in der Map null ist, kannst du unter Umständen auch am Anfang machen und bei null einfach continue aufrufen. Oder man könnte - sofern das fachlich ok wäre - default Werte setzen, wenn der Wert null bzw. ein Leer String ist.
 

mrBrown

Super-Moderator
Mitarbeiter
Die Schleife ist überflüssig, da du ja die Tags kennst, der Vergleich mit == klappt nicht und der Check auf null ist in der Form sinnlos, du würdest einfach ein NPE bekommen, wenn irgendwas null ist.


Java:
public void readAndStoreTags(String path) {
        Map<String, Object> tag_map = TagReader.readTags(path);
                if (tag_map.get("title")) != null && !tag_map.get("title").toString().trim().isEmpty()) {
                    title = tag_map.get("title").toString().trim();
                }
               if (tag_map.get("author")) != null && !tag_map.get("author").toString().trim().isEmpty()) {
                    title = tag_map.get("author").toString().trim();
                }
               if (tag_map.get("album")) != null && !tag_map.get("album").toString().trim().isEmpty()) {
                    title = tag_map.get("album").toString().trim();
                }
               if (tag_map.get("duration")) != null) {
                    title = (long)tag_map.get("duration");  
                }
        }
    }
 

MoxxiManagarm

Top Contributor
Ich hätte es in etwa so probiert:
Java:
public void readAndStoreTags(String path) {
    Map<String, Object> tag_map = TagReader.readTags(path);

    title = tag_map.getOrDefault("title", "").toString().trim();
    author = tag_map.getOrDefault("author", "").toString().trim();
    album = tag_map.getOrDefault("album", "").toString().trim();
    duration = (long)tag_map.getOrDefault("duration", 0);
}
 

White_Fox

Top Contributor
Wenn ich mich recht entsinne, kann man Strings (im Gegensatz zu anderen Objekten) auch als Case-Separator in einem Switch-Statement verwenden.
 

mrBrown

Super-Moderator
Mitarbeiter
Ich hätte es in etwa so probiert:
Java:
public void readAndStoreTags(String path) {
    Map<String, Object> tag_map = TagReader.readTags(path);

    title = tag_map.getOrDefault("title", "").toString().trim();
    author = tag_map.getOrDefault("author", "").toString().trim();
    album = tag_map.getOrDefault("album", "").toString().trim();
    duration = (long)tag_map.getOrDefault("duration", 0);
}
Wobei die Werte dann im Zweifel auf 0/"" gesetzt werden, Je nachdem wie das benutzt wird kann das ein Problem sein
 
G

Gelöschtes Mitglied 62813

Gast
Ich hätte es in etwa so probiert:
Java:
public void readAndStoreTags(String path) {
    Map<String, Object> tag_map = TagReader.readTags(path);

    title = tag_map.getOrDefault("title", "").toString().trim();
    author = tag_map.getOrDefault("author", "").toString().trim();
    album = tag_map.getOrDefault("album", "").toString().trim();
    duration = (long)tag_map.getOrDefault("duration", 0);
}

ich habe das mal in mein programm eingefügt und eine testdatei ausgeführt. in dem geänderten programmteil läuft alles gut, aber jetzt bekomme ich an einer anderen stelle einen fehler, da mir dein code aber besser gefällt würde ich deins gerne übernehmen. der test meldet in folgendem abschnitt einen fehler, hast du eine idee wie das zusammenhängt?

Java:
/**
     * Convert string
     *
     * @return string
     */
public String toString() {
        String auxiliaryString = super.toString();
        String auxiliaryDuration = getFormattedDuration();
        
        if (album != "" && album != null) {
            return (auxiliaryString + " - " + album + " - " + auxiliaryDuration);
        } else {
            return (auxiliaryString + " - " + auxiliaryDuration);
        }
    }
 

LimDul

Top Contributor
Strings vergleicht man nicht mit == bzw !=. Das ist fehleranfällig. Und der Vergleich auf != null gehört an die erste Stelle in der If Bedingung.

Java:
        if (album != null && !album.equals("")) {
bzw.
Java:
        if (album != null && !"".equals(album)) {
 
G

Gelöschtes Mitglied 62813

Gast
Strings vergleicht man nicht mit == bzw !=. Das ist fehleranfällig. Und der Vergleich auf != null gehört an die erste Stelle in der If Bedingung.

Java:
        if (album != null && !album.equals("")) {
bzw.
Java:
        if (album != null && !"".equals(album)) {

okay :D habs jetzt geändert, funktioniert aber trotzdem nicht :( ich weiß aber auch nicht wo da der zusammenhang zu dem oben ist
 

LimDul

Top Contributor
Inwefern bekommst du den einen Fehler? Compile Fehler? Falsche Ausgabe? Was wäre den richtig?

So ohne Glaskugel ist es schwer :)
 
G

Gelöschtes Mitglied 62813

Gast
ich weiß es auch nicht, wir haben eine testdatei bekommen und die zeigt an dass in der zuletzt eingestellten methode ein fehler ist. ihr habt mir aber schon viel weitergeholfen, wenn ihr jetzt auch keinen fehler mehr seht ists egal. vielen dank für eure antworten und anregungen!
 
G

Gelöschtes Mitglied 62813

Gast
Screenshot_1.png
es gibt quasi keine fehlermeldung, es zeigt nur an in welcher methode noch was schief läuft
 

QU3LLC0D3

Mitglied
Strings vergleicht man nicht mit == bzw !=. Das ist fehleranfällig. Und der Vergleich auf != null gehört an die erste Stelle in der If Bedingung.

Java:
        if (album != null && !album.equals("")) {
bzw.
Java:
        if (album != null && !"".equals(album)) {

Was spricht denn gegen die Verwendung von StringUtils? Da kann man doch gleich alles in einem abfrühstücken...

Code:
if (StringUtils.isNotEmpty(album)) {
  // do something
}

isNotEmpty prüft auf null und Länge = 0, isNotBlank prüft auf null auf Länge = 0 und auf Whitespaces ...

Des Weiteren nutze ich immer gern bei der Zuweisung von Strings die "trimToEmpty()" Methode, ebenfalls aus den StringUtils. Dies sichert mir zu, dass der String niemals null sein kann.
 

mihe7

Top Contributor
Doch, man führt hier Abhängigkeiten zu externen Bibliotheken ein und das für etwas völlig triviales.
Java:
public abstract class Strings {
    public static String isNotBlank(String value) {
        return value != null && !value.isBlank();
    }

    public static String trimToEmpty(String value) {
        return value == null ? "" : value.trim();
    }
}
 
Ähnliche Java Themen
  Titel Forum Antworten Datum
G Code kürzen Java Basics - Anfänger-Themen 5
F Code kürzen Java Basics - Anfänger-Themen 9
J Code kürzen Java Basics - Anfänger-Themen 11
E Interface einfügen und Code kürzen Java Basics - Anfänger-Themen 15
U Code kürzen Java Basics - Anfänger-Themen 4
T Kann mir jemand wörtlich erklären, was in dem Code genau passiert? Java Basics - Anfänger-Themen 1
Ü Dead Code im Programm? Java Basics - Anfänger-Themen 13
I QR code in Java selber generieren Java Basics - Anfänger-Themen 5
terashy VS Code Project run error Java Basics - Anfänger-Themen 10
JaZuDemNo Code Erklärung Java Basics - Anfänger-Themen 3
M Connect-4-Code analysieren Java Basics - Anfänger-Themen 2
N BMI Rechner Was haltet ihr von dem Code habt ihr Verbesserungsvorschläge weil design teschnisch ist das nicht das geilste würde das gerne überarbeiten Java Basics - Anfänger-Themen 12
W In alten Code zurück- und dort wieder zurechtfinden? Java Basics - Anfänger-Themen 17
T code so schreiben das er von sich selber anpasst (code soll die anzahl aller bustaben bestimmen) Java Basics - Anfänger-Themen 16
J Frage zu einem "Taschenrechner" code Java Basics - Anfänger-Themen 9
T Fehlercode bei code der das Alter ausrechnet Java Basics - Anfänger-Themen 2
T Text einlesen code was kommt dahin? Java Basics - Anfänger-Themen 1
jhfjeh Strukturgramm in code Java Basics - Anfänger-Themen 11
D Tipps zum Code Java Basics - Anfänger-Themen 24
W Java-Code mit Array Java Basics - Anfänger-Themen 14
W Java-Code Java Basics - Anfänger-Themen 2
W Java code- TicTac toe Java Basics - Anfänger-Themen 51
W Java-code Java Basics - Anfänger-Themen 8
W Java-code Java Basics - Anfänger-Themen 9
W Java-Code erklären Java Basics - Anfänger-Themen 6
ohneInformatik; For Schleife. Was macht dieser Code?? Java Basics - Anfänger-Themen 5
Say Fehlenden Code finden in einer while-Schleife? Java Basics - Anfänger-Themen 11
Say 2-DIM Array Code lesen und verstehen Java Basics - Anfänger-Themen 5
Say Stelle in Code herausfinden, wie geht man vor? Java Basics - Anfänger-Themen 12
Say do-While Code Ausführung Java Basics - Anfänger-Themen 3
W Rückfrage zur Programmgestaltung (clean code) Java Basics - Anfänger-Themen 12
M intelliJ auf neuem PC, plötzlich kein Code Java Basics - Anfänger-Themen 3
Pinhg Sound in Greenfoot Code einbinden Java Basics - Anfänger-Themen 2
C Java boolean Code läuft nicht Java Basics - Anfänger-Themen 5
I Code für Bezahlsystem (auch bei Offline Aktivität) Java Basics - Anfänger-Themen 7
J Größter gemeinsamer Teiler: mein Code Java Basics - Anfänger-Themen 6
B Den Dateipfad einer Java Datei durch Code in Selbiger finden? Java Basics - Anfänger-Themen 10
A Wie könnte man diesen Code kürzer machen ? Java Basics - Anfänger-Themen 7
J Frage zu meinem Code (OOP) Java Basics - Anfänger-Themen 4
Alen123 Warum funktioniert mein Code nicht? Java Basics - Anfänger-Themen 64
Max246Sch Frage zu Währungsrechner Code Java Basics - Anfänger-Themen 2
S Hilfe bei Umänderung von Java Code Java Basics - Anfänger-Themen 16
I Code wird nicht ausgeführt Java Basics - Anfänger-Themen 2
K Wie kann man diesen Code schnell und effizient interpretieren (Man hat nur 4 Minuten) Java Basics - Anfänger-Themen 3
R ISBN-10-Code überprüfen Java Basics - Anfänger-Themen 7
I Bitte um Hilfe zu unterstehenden Code Java Basics - Anfänger-Themen 6
I Interface von einer EJB Klasse, um Code zu reduzieren Java Basics - Anfänger-Themen 1
I HTML Code säubern Java Basics - Anfänger-Themen 4
B Brauche Hilfe zu einem Code Java Basics - Anfänger-Themen 5
Temsky34 Problem mit dem Code Java Basics - Anfänger-Themen 17
N Fehler im Code (Aufgabe für Anfänger) Java Basics - Anfänger-Themen 11
N Java-Code abwärtskompatibel machen Java Basics - Anfänger-Themen 4
J Erste Schritte Was mache ich in meinem Code falsch. Java Basics - Anfänger-Themen 3
Ameise04 Variablen Inhalt einer Variable im Code verwenden? Java Basics - Anfänger-Themen 9
S Compiler-Fehler Nicht adressierbarer Code ( Non-addressable code ) Java Basics - Anfänger-Themen 5
Aemulit Java Schaltjahr berechnen Code Java Basics - Anfänger-Themen 7
A Code Problem Java Basics - Anfänger-Themen 6
C Fehler im Code Java Basics - Anfänger-Themen 10
A Zu einem bestimmten Ort im Code springen Java Basics - Anfänger-Themen 11
L Ist der Code richtig Java Basics - Anfänger-Themen 3
josfe1234 code vereinfachen Java Basics - Anfänger-Themen 15
nonickatall Ausführbarkeit von Code testen bzw. Remote Debugging Java Basics - Anfänger-Themen 4
F Frage betreff Programm mit dem man C++-Code in JAVA-Code übersetzen lassen kann Java Basics - Anfänger-Themen 2
S Fehler bei Code mit SubStrings für mich nicht auffindbar. Java Basics - Anfänger-Themen 4
G Programm Code Java Basics - Anfänger-Themen 5
C Code zusammenfassen Java Basics - Anfänger-Themen 5
I Erklärung zum Java Code Java Basics - Anfänger-Themen 2
T Programmablaufsplaninterpretation in Code umformen Java Basics - Anfänger-Themen 1
dieter000 Kurze Frage kann mir ejmand kurz diesen Code erklären, bzw wie man die zeilen erklärt und so Java Basics - Anfänger-Themen 1
AlexVo String zu Java Anweisung getString("*** java code ***") Java Basics - Anfänger-Themen 19
M ISBN-Code Java Basics - Anfänger-Themen 26
B Zeitgleiches Arbeiten am Code mit mehreren Personen? Java Basics - Anfänger-Themen 7
S Wie kann ich bei diesem Code erreichen, das als Ergebnis hier 15 herauskommt? Java Basics - Anfänger-Themen 23
N Kann man den Code vereinfachen? Java Basics - Anfänger-Themen 25
marcooooo Code erklären Java Basics - Anfänger-Themen 28
marcooooo Code erklären Java Basics - Anfänger-Themen 4
S Advent of Code Day4 Java Basics - Anfänger-Themen 4
B Nach eingefügtem Code erkennt Compiler keine Instanzvar und meldet SyntaxError Java Basics - Anfänger-Themen 2
Gaudimagspam Caesars Code entziffern in Java Java Basics - Anfänger-Themen 8
Lukasbsc Wie kann ich meinen Code optimieren? Java Basics - Anfänger-Themen 4
NeoLexx equals()-Methode Verständnis Frage anhand Code Beispiel Java Basics - Anfänger-Themen 22
I Input/Output Code wird doppelt ausgeführt Java Basics - Anfänger-Themen 3
T Main startet nicht bei vorgegebenen Code Java Basics - Anfänger-Themen 41
B Frage zum Code verständnis im Resultat Java Basics - Anfänger-Themen 10
J Fehler im Code, aber ich weiß nicht wieso! Java Basics - Anfänger-Themen 6
S Mehrere Probleme im Code Java Basics - Anfänger-Themen 7
M Code nur für Cracks? Crack the Passwort Übung Java Basics - Anfänger-Themen 7
parrot Code entferneJedeZweiteZiffer Java Basics - Anfänger-Themen 6
Bluedaishi Source Code Signieren Java Basics - Anfänger-Themen 22
L Best Practice Code Refactoring für Methoden mit fast gleicher Aufbau Java Basics - Anfänger-Themen 6
L Best Practice Code refactern Java Basics - Anfänger-Themen 30
A Code umschreiben Java Basics - Anfänger-Themen 6
Torsten.E JavaFX mit Visual Studio Code verwenden Java Basics - Anfänger-Themen 1
C Beispiel-Code mit Pair wird nicht compiliert. Java Basics - Anfänger-Themen 8
X Reverse algorithm engineering (Java code) Java Basics - Anfänger-Themen 6
T Bufferedwriter code Nullpointerexception Java Basics - Anfänger-Themen 4
V Switch Methode macht Code kaputt Java Basics - Anfänger-Themen 18
R Was muss ich an meinem Code ändern? Java Basics - Anfänger-Themen 2
S Bewertet meinen Code - Part 1 Java Basics - Anfänger-Themen 8
M Java Code Verständnis Java Basics - Anfänger-Themen 4

Ähnliche Java Themen

Neue Themen


Oben