Unfassbarer Fehler. Brauche Ideen zum Debuggen

[Thallius]

Hallo,

ich habe eine extrem große umd komplexe Applikation in der ein Fehler auftaucht den ich einfach nicht zu fassen bekomme. Schreibe ich eine Test-Applikation mit minimalem Code tritt bei gleichem Code der Fehler nicht auf.

Ich versuche das Ganze so gut es geht zu beschreiben aber es ist nicht einfach und MB von Code würden sicher auch nicht helfen.

Ich öffne ein Panel mit einer JTable. Alles soweit standard Zeug. Unter dem ScrollPane in dem die JTable ist ist ein weiteres Panel welches ein Label enthält das den Table Status anzeigt (Also zum beispiel "15 / 200 Jobs" je nach Filtereinstellungen). Links vom den ScrollPane ist ein weiteres Panel welches die Filterelemente für die Tabelle beinhaltet. Also DropDownMenus, Checkmarks und Searchfields etc.

Dem TableModel gebe ich nun das Status Label und den Filter mit damit dieses entsprechend die Anzeige machen kann.

    private ArrayList<MultiJob> joblist;
    public ArrayList<MultiJob> filterList;

    private JLabel countLabel;

    public JobListTableModel(JobsFilter jobfilter, JLabel countlabel) 
    {
        filter = jobfilter;
        filter.addJobsFilterListener(this);
        countLabel = countlabel;

        joblist = new ArrayList<>();
        filterChanged();
    }

Der eigentliche JobListController öffnet einen Langzeit-Requester mit Cancel Button und lädt die Jobs asynchron ein. Wenn der JobLoader die Jobs geladen hat gibt er dem jobController über seine Listener-Funktionen die geladene Liste.
Der Controller öffnet dann das oben beschriebene Panel und setzt die neu geladene Liste.

    @Override
    public void jobLoaderDidFinish(ArrayList<MultiJob> list)
    {
        if(cancelDialog != null)
            cancelDialog.close();
        cancelDialog = null;

        jobList = list;
        if(jobListPanel == null)
        {
            UserSettings.setSettingsValue("SelectedMainPanelButton", "JobList");
            jobListPanel = new JobListPanel(filter, getLastUpdate());
            jobListPanel.setJobListPanelListener(this);
            mainPanel.openJobListPanel(jobListPanel);
        }
        tableModel.replaceList(jobList);
    }

Das Model setzt dann die Tabelle

    public boolean replaceList(ArrayList<MultiJob>list)
    {
        joblist = list;
        filterChanged();
        return true;
    }

    public void filterChanged() 
    {
        filterList = new ArrayList<>();
        for(MultiJob entry : joblist)
        {
       .... hier kommen nur die Abfrage ob die Filter greifen oder eben nicht, das hat aber nichts mit der Sache zu tun
            filterList.add(entry);
        }
        System.out.println(joblist.size());

        if(countLabel != null)
            countLabel.setText(filterList.size()+" / "+joblist.size()+" "+"Jobs"); 

        fireTableDataChanged();
    }

Jetzt kommt der eigentlich Spaß. Wenn ich in der Zeile countLabel.setText() das joblist.size() weglasse oder die joblist leer ist (also die size() == 0) oder wenn ich die ganze Zeile ausdokkumentiere, dann wird das Ganze Panel nicht mehr angezeigt. Es gibt keinen Fehler, keine Exception, das Panel bleibt einfach leer.

Ich benutzte das countLabel nirgendwo anders. Es wird nur im Panel erstellt, "Jobs" vorinitialisiert und hier verändert. Sonst keine Besonderheit.

Und um es jetzt ganz verrückt zu machen

      int g = joblist.size();
      if(countLabel != null)
            countLabel.setText(filterList.size()+" / "+g" "+"Jobs");

funktioniert.

      int g = joblist.size();
      //if(countLabel != null)
      //      countLabel.setText(filterList.size()+" / "+g" "+"Jobs");

funktioniert nicht.

Lasse ich das panel.add(countLabel) beim Erstellen des Panels weg geht es auch nicht.
Lösche ich das countLabel und alle seine Verweise geht auch nichts.
Ersetze ich das joblist.size() im setText durch eine Konstante oder irgend eine andere Variable geht es auch nicht.

Fazit:
Es muss dieses Label auf dem Panel sein und es muss ein setText(joblist.size()) darauf gemacht werden. Sonst wird das gesammte Panel (Table, Filter, Status) nicht dargestellt.

Ich habe schon den Namen von countLabel händisch geändert (Also ohne Refactoring) um sicher zu gehen das es nicht doch noch eine anderer Referenz darauf gibt aber ist nicht.

Das ist alles komplett unlogisch ...

Ich habe echt keine Idee was ich noch debuggen soll. Ich kann in das Label schreiben was ich will, solange das joblist.size() drin bleibt.

Gruß

Claus

 

[mrBrown]

Ganz spontan würde ich auf ein irgendwo fehlendes repaint(/revalidate/whatever) irgendwo tippen.

Kannst mal versuchsweise 'nen Breakpoint in die setText setzen, und gucken was da passiert.

 

[Joose]

countLabel = countlabel;

Über diese Zeile (im Konstruktor) bin ich gerade gestolpert, dachte du hast dich verschrieben und ein "this." vergessen. Aber die beiden Variablennamen sind ja doch anders nur sehr schwer erkennbar

In welchem Thread wird denn der Text vom Label gesetzt? Funktioniert die GUI weiterhin (also der Rest) oder ist es dann komplett vorbei?

 

[Thallius]

Über diese Zeile (im Konstruktor) bin ich gerade gestolpert, dachte du hast dich verschrieben und ein "this." vergessen. Aber die beiden Variablennamen sind ja doch anders nur sehr schwer erkennbar

In welchem Thread wird denn der Text vom Label gesetzt? Funktioniert die GUI weiterhin (also der Rest) oder ist es dann komplett vorbei?

Ja ist eine Marotte von mir das ich Übergabe-Parameter immer im lowercase und Instanz-Variablen dann halt im camelCase schreibe. So kann ich die für mich gut unterscheiden und weis doch das sie eigentlich das gleiche beinhalten und zusammen gehören (zumindest in dem Moment

Also ich kann mit dem Programm ganz normal weiter arbeiten. Auf andere Panels schalten etc. Nur eben dieses Panel bleibt leer. Es wird nichtmal eine Background Color gesetzt oder sowas. Das Dingen ist einfach weg. Wenn ich im Debug Mode starte und dann die App anhalte steht sie auch sauber im Waiting. Also da ist nichts was abstürzt oder so.

Gruß

Claus

 

[Thallius]

Ganz spontan würde ich auf ein irgendwo fehlendes repaint(/revalidate/whatever) irgendwo tippen.

Kannst mal versuchsweise 'nen Breakpoint in die setText setzen, und gucken was da passiert.

Aber warum sollte das repaint nur aufgerufen werden wenn das joblist.size() in dem setText() steht? Dann müßte es ja bei jedem Text getriggert werden.

Gruß

Claus

 

[Thallius]

Ok, der Hint war nicht schlecht. Wenn ich statt des setText() ein statusLabel.revalidate() mache dann geht es auch. Was das jetzt damit zu tun hat welchen Text ich in setText() benutze ist mir zwar trotzdem nicht klar aber immerhin ein Ansatz zum weiter suchen.

Gruß

Claus

 

[Thallius]

Ok, es fehlte tatsächlich ein revalidate() nach dem Einfügen des neuen Panels. Erklärt immer noch nicht warum der Text im setText() von Bedeutung war aber ist mir jetzt ehrlich gesagt auch sch.... egal.

Danke für alle die sich die Zeit genommen haben zu helfen

Claus

 

[mrBrown]

Das revalidate wird ausgeführt, wenn sich der Text des Labels ändert.
Wenn der restliche Text in dem Label bis auf die Listenlänger immer gleich ist, führt der nicht zu einem Revalidate, sondern nur die variable Länge.
Vermutlich würde das auch mit einem Math.random als Text klappen und mit der Liste hat das nicht viel zu tun

 

[JuKu]

Ein Label ohne Text wird glaube nicht angezeigt.
Und ein leeres Panel wird vermutlich auch nicht gerendert.
Vllt. hat das was mit deinem Problem zu tun?

 

[mrBrown]

Ein Label ohne Text wird glaube nicht angezeigt.
Und ein leeres Panel wird vermutlich auch nicht gerendert.
Vllt. hat das was mit deinem Problem zu tun?

In diesem Fall ist weder Panel noch Label leer und das Problem ist wohl auch schon gelöst