Hänger in Anwendung (war: An KonradN)

[Juelin]

Hallo Konrad,
ich habe ja de SLF4J Logger bei mir im Programm
und habe jetzt mal mit geloggt
In jeder Routine habe ich am Anfang und am Ende einen Logeintrag ausgegeben..
Logfile als PDF anbei.
Ich betätigte las letztes den Knopf1.
Wenn du dir das Logfile ansiehst ist das betätigen des Knopfes drin (Blau). (Start) (End) fehlt ja.
In der Routine wird dann der Knopf auf Visible = False gesetzt und dann die Y-position vom Tableview gesetzt. (Rot)
Nun müsste eigentlich die Abfrage mit dem Alert kommen, tut sie aber nicht.

    void knopf1Click(ActionEvent event)
        {
        int a;
        int c;
        int d;
        String x;
        if (Vital.mitLoggen)
            {
            logger1.info("################# <Vital.java><knopf1Click><Start>");  
            }
        if (Vital.datenbank.dbOpen)
            {
            if (stringgridRowNum == -1)
                {
                setAnzeigeText(2, "Erst Daten in Tabelle anklicken");
                }
            else
                {
                stringgridClick = Boolean.FALSE;  
                cursor_ = Cursor.cursor("WAIT");
                VitalController.scene4.setCursor(cursor_);
                Vital.datenbank.formatter = DateTimeFormatter.ofPattern("dd.MM.yyyy", Locale.GERMAN);
                Vital.datenbank.dd1 = Vital.datenbank.formatter.format(Vital.datenbank.dgeburtsdatum);
                Vital.datenbank.formatter = DateTimeFormatter.ofPattern("dd.MM.yyyy", Locale.GERMAN);
                Vital.datenbank.dd2 = Vital.datenbank.formatter.format(Vital.datenbank.ddatum);
                Vital.datenbank.formatter = DateTimeFormatter.ofPattern("HH:mm:ss", Locale.GERMAN);
                Vital.datenbank.dt = Vital.datenbank.formatter.format(Vital.datenbank.duhrzeit);
                stringgridRowNum = -1;
                setKnopfVisible(1, Boolean.FALSE);
                Alert alert1 = new Alert(Alert.AlertType.CONFIRMATION);
                alert1.setTitle("Abfrage");
                alert1.setHeaderText("Eintrag:\n"+Vital.datenbank.dvorname+" "+Vital.datenbank.dnachname+"\n"+
                                     "Geburtsdatum: "+Vital.datenbank.dd1+"\n"+
                                     "Datum: "+Vital.datenbank.dd2+" Uhrzeit: "+Vital.datenbank.dt+"\n"+
                                     "wirklich Meditindaten löschen?");
                alert1.setContentText("");
                ButtonType buttonTypeYes = new ButtonType("Ja");
                ButtonType buttonTypeNo = new ButtonType("Nein");
                alert1.getButtonTypes().setAll(buttonTypeYes, buttonTypeNo);
                Optional<ButtonType> result = alert1.showAndWait();
                if (result.get() == buttonTypeYes)
                    {
                    Vital.datenbank.formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd", Locale.ENGLISH);
                    Vital.datenbank.dd1 = Vital.datenbank.formatter.format(Vital.datenbank.dgeburtsdatum);
                    Vital.datenbank.formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd", Locale.ENGLISH);
                    Vital.datenbank.dd2 = Vital.datenbank.formatter.format(Vital.datenbank.ddatum);
                    Vital.datenbank.formatter = DateTimeFormatter.ofPattern("HH:mm:ss", Locale.ENGLISH);
                    Vital.datenbank.dt = Vital.datenbank.formatter.format(Vital.datenbank.duhrzeit);
                    Vital.datenbank.dbTabelle = "vital";
                    Vital.datenbank.dbBefehl = "DELETE FROM "+Vital.datenbank.dbTabelle+" WHERE Vorname = '"+Vital.datenbank.dvorname+"' AND Nachname = '"+Vital.datenbank.dnachname+"' AND Geburtsdatum = '"+Vital.datenbank.dd1+"' AND Datum = '"+Vital.datenbank.dd2+"' AND Uhrzeit = '"+Vital.datenbank.dt+"'";
                    Vital.datenbank.dbTabellenDelete();
                    if (Vital.datenbank.dbRet != 0)
                        {
                        setAnzeigeText(2, datenbank.FehlerAus());
                        }
                    }
                if (Vital.datenbank.dbRet == 0)
                    {
                    stringgridAnz = 1;
                    stringgridYPos = stringgridAnz * 35;
                    stringgridSetYPos();
                    stringgrid.getItems().clear();
                    VitalShowController.anzeintraege = 0;
                    setAnzeigeText(2, "");
                    VitalShowController.dLesArt = 2;
                    Vital.datenbank.dbBefehl = "SELECT * FROM "+Vital.datenbank.dbTabelle+" ORDER BY Nachname, Vorname, Geburtsdatum, Datum DESC, Uhrzeit DESC";
                    Platform.runLater(() -> lesenDatenbank(Vital.datenbank.dbBefehl));
                    }
                Thread bspThread = new Thread(new BspThread());
                bspThread.setPriority(1);
                bspThread.start();
                }
            stringgridClick = Boolean.TRUE;
            }
        if (Vital.mitLoggen)
            {
            logger1.info("################# <Vital.java><knopf1Click><End>");  
            }
        }

Keine Ahnung wieso!!!!!!

Hast Du da noch eine Idee?

Zip-File Projekt anbei.

Ein schönes Wochenende
Jürgen

 

[Marinek]

Hi,

ich finde das Posting geht in eine falsche Richtung. Das hier ist ja kein Konrad Forum, wo man öffentlich chatten kann... Sonst kannst du ihm eine Mail schreiben. Ich persönlich finde auch, dass ein Log immer eine txt Datei sein sollte und kein PDF. Gerade hier im Forum downloade ich grundsätzlich keine Dateien, so dass ich da nicht reinschauen kann. Ein Log kann man hier auch posten, wenn es mehr als 100 Zeilen hat, dann bringt das auch nix.

Wie dem auch sei: Offensichtlich wird eine bestimmte Methode nicht aufgerufen. Hier gibt es zwei Vorgehensweisen:

1. Logging nutzen
2. Debugger nutzen

Zu 1) Genau so wie in allen anderen Programmiersprachen (e.g. Delphie) ist es so, dass wenn ich an der falschen Stelle eine ein Logging setze, dass mir nix bringt, ich auch das Logging weglassen kann. Hier ist so ein Fall. Du hast das Logging an einer Stelle, die dich kaum weiterbringt.

Hier schlage ich vor das Log so zu verfeinern, dass man daraus erkennen kann, bis zu welcher Zeile oder welchem kleineren Block man kommt. Hier sollte man auch Zwischenergebnisse sinnvoll loggen.

Zu 2) Ich weiß nicht, welche IDE du benutzt, aber ich schlage vor sich hier passend zur IDE zu erkundigen, wie ein Debugger genutzt werden kann. Hier kann man sich das Log sparen und man kann sehen, bis wohin die Ausführung geht und warum sie abbricht. Dies ist gerade für so komplexe Programme, wie das hier vorliegende, unbedingt notwendig.

Hier noch eine Doku zu der Klasse Alert: https://docs.oracle.com/javase/8/javafx/api/javafx/scene/control/Alert.html

Gerade deine Überprüfung der Results sieht komisch aus, daher erwähne ich das hier.
Gruß,
Martin

 

[M.L.]

welche IDE

der TE nutzt (mindestens) NetBeans (und handelt sich (u.a. durch Java-Programmierung im Delphi-Stil) unnötige Probleme ein)

 

[KonradN]

Ja, die Posts direkt im Forum sind etwas unschön. Ich hatte es zuvor dann schnell löschen können, aber das ist nicht wirklich, wie das Forum genutzt werden sollte.

Das Problem ist, dass Du im Gespräch das Projekt nicht anhängen kannst? Wie wäre es, wenn ich Dir eine OneDrive Freigabe gebe, auf der Du Dateien hoch laden kannst? Dann würde man dieses Problem umgehen können.

Prinzipiell könnte man auch ein git Repository anlegen - falls Du Interesse hast, Dich da auch noch etwas einzuarbeiten? Wenn GitHub und Co Dir nicht zusagen (Da kannst auch ein privates Repository erstellen) könnte ich Dir auch auf meiner eigenen gitea Instanz einen Account einrichten.

Ansonsten ein paar Anmerkungen zum Logging:
a) In jeder Klasse hast Du ja eine Klassenvariable erstellt:
private static final Logger logger1 = LoggerFactory.getLogger(Vital.class);
Hier wäre aber das Vital.class immer anzupassen. In VitalDeleteController sollte es also VitalDeleteController.class sein. Denn beim Logging wird die Klasse immer mit angegeben aber bei Dir ist das derzeit immer com.juelin.projekt10.Vital

b) Du kommst bis zum Ende von stringgridSetYPos() und er kommt nicht mehr ins setAnzeigeText. Daher muss es bei diesen Zeilen sein:

                    stringgrid.getItems().clear();
                    VitalShowController.anzeintraege = 0;

Letzteres ist eine einfache Zuweisung - da passiert nichts. Also bleibt das getItems().clear().

Du hattest da sehr viele Einträge drin, oder? Da überlege ich, ob es evtl. einfach nur sehr lange dauert, alle Elemente zu löschen zumal es eine ObservableList ist und die TableView sich da ja anpassen will. Ich habe den Code nicht geprüft, aber das wäre ggf. eine Möglichkeit.

Statt dem clear() könntest Du einmal versuchen, einfach eine neue List zu setzen. Etwas wie:

ObservableList<Eintrag> neueListe = FXCollections.observableArrayList(new ArrayList<Eintrag>());
stringgrid.setItems(neueListe);

Wenn Du dann die Datenbankeinträge neu geladen hast, dann wäre die Liste natürlich wieder zu setzen.

Das ist aber nur ein Workaround. Besser wäre es, einfach nur die Einträge aus der Liste zu entfernen, die aus der Datenbank gelöscht wurden. Das müsste den geringsten Aufwand bedeuten.

 

[Manul]

Ein paar Anmerkungen zum Logging:

public class VitalDeleteController
    {
    ...
    private static final Logger logger1 = LoggerFactory.getLogger(Vital.class);
    ...
        if (Vital.mitLoggen)
            {
            logger1.info("################# <Vital.java><knopf1Click><Start>");  
            }
        ...
        if (Vital.mitLoggen)
            {
            logger1.info("################# <Vital.java><stringgridClicked><Start> >"+String.valueOf(stringgridClick)+"<");   
            }

-->

1. Man sollte möglichst für jede Klasse eine eigene (passende) Loggerinstanz haben, also

   ... LoggerFactory.getLogger(VitalDeleteController.class);

   usw. Sonst landet alles in einem Logfile und wenn dann auch noch die Meldungen uneindeutig/falsch sind viel Spaß beim Zuordnen.
2. Laut Java-Konvention sollten "static final"-Felder möglichst (es gibt Ausnahmen, aber hier seh ich die nicht) Uppercase sein, also z.B.

   private static final Logger LOG = LoggerFactory.getLogger(VitalDeleteController.class);

3. Was du da machst ist klassisches Call-Tracing. Kann man machen wenns was bringt. Dafür gibt es aber eigentlich TRACE. Ins INFO-Log sollte sowas nicht.
4. Das Log über ein statisches Boolean-Feld im Javacode zu steuern ist eher unschön. Normalerweise würde man die Logs immer mit dem für den jeweiligen Logeintrag passenden Loglevel schreiben und bei Bedarf das Log-Level für die Applikation bzw. Klasse ändern.
5. Wenn du 1. beherzigst kannst du dir auch sparen jedesmal die Klasse nochmal ins Log zu schreiben. Die ist schon Teil des Log-Entries. Außerdem zeigt schon das obige Beispiel wie schnell sich dadurch Fehler einschleichen (Falsche Kategorie: Vital.java statt VitalDeleteController.java).
6. Alle modernen Logger-Implementierungen unterstützen Platzhalter. Auch .toString() bzw. String.valueOf(...) muss man beim Logging effektiv nie aufrufen. Also einfach z.B.

   LOG.trace("... stringgridClick: {}", stringgridClick);

7. Und zu guter Letzt: Log-Enträge sollten stets sinnvolle Meldungen und einen Mehrwert für die Fehlersuche enthalten.

 

[Manul]

Ergänzend zu 3 (und da es auch schon erwähnt wurde): Ich würde hier auch eher empfehlen, sich mal mit dem Debugger vertraut zu machen, statt die kognitive Komplexität dieses schon jetzt eigentlich kaum noch wartbaren Codes mit zahlreichen Tracing-Log-Ausgaben noch weiter zu erhöhen.