Updated Libraries, fixed Bugs, Contextmenu for Struktogramme, Print a…

[FairyTail2000]

…nd save to self choosen location

[codemanyak]

@FairyTail2000 Just a recommendation: In general, it would certainly help the revision if you first generated an issue (or as many issues as to be fixed) with some detailed description / specification of what you intend. This would also help Kevin formulate the release notes and update the documentation, I guess. At least you ought to give some details in the pull request comment. Even if you think that code itself is the best documentation, it's always better to have an opportunity to compare intention with implementation. Just my two cents...

[FairyTail2000]

I did everything which Mister Arndt (former Teacher of Kevin) suggested, I will look up what he requested and make some Notes

[kekru]

Hi Rafael,

thank you for your huge contribution :)
I added a few comments, but all in all it looks good!

I don't know a Mister Arndt. My Teacher was Mister L.
Are you pupil of AMG Beckum, too?

[FairyTail2000]

Hi Rafael,

thank you for your huge contribution :) I added a few comments, but all in all it looks good!

I don't know a Mister Arndt. My Teacher was Mister L. Are you pupil of AMG Beckum, too?

I live in Osnabrück in Germany, Lower Sachsony

Mister Matthias Arndt told me he knows you, he said he will contact you for this pull request, He works in BBS Schölerberg as a Teacher. He also suggested all of the patches I've made I've read all of your comments and made the necessary patches to my own code which I'll pull later

[kekru]

Thanks for your update. I'll try to review your changes at the weekend

Now I remember: I wrote some e-mails with Mr. Arndt in 2018, but he was not my teacher in school ;)

[FairyTail2000]

Take your time, feel free to request any changes, I will make the necessary changes

[kekru]

It added few more comments. Please either change code or comment again ;)

And always look on the "changed files" tab, so that you chan check if the requested changes can be reviewed easily.

e.g. your formatter does crazy things:
It reformats very much code, so that the diff becomes very large and it is very difficult to see, what really changed.
Better just turn it off

All in all you have 2,253 new lines and 1,903 lines that were removed (but most of them only formatter), thats very much

[kekru]

Lass uns auf deutsch weiter schreiben, ich glaube das ist einfacher :)

Ich finde es leider immer noch sehr schwierig nachzuvollziehen, was du geändert hast und wo du nur die Code-Formatierung angepasst hast.
Klick mal auf changed files. Dann siehst du, was ich meine. Es gibt extrem viele geänderte Zeilen, aber eigentlich wurde nur die Formatierung geändert.
Ich habe die Versionen auch mal mit WinMerge verglichen, aber auch dort ist es sehr schwierig.

Also nochmal die Bitte:
Besonders die Dateien "Controlling.java", "Struktogramm.java" und "Fallauswahl.java" sind sehr unübersichtlich.

• Deaktiviere den Formatter in deinem eclipse
• Kopiere die drei Dateien aus meinem Repo in dein eclipse, sodass du wieder meinen alten Stand hast
• Füge deine Änderungen nochmal ein, aber ohne geänderte Formatierung
• Commit + Push und dann schau unter changed files, dass man bei den Dateien nur die wirklich relevanten Zeilen sieht

Prüfe auch nochmal unter changed files, ob du alle meine Kommentare erwischt hast. z.B. CustomJButton.java ist immer noch im Code, obwohl du doch den TabbedPaneClosingButton angelegt hast

[kekru]

Und ich bekomme hier noch eine Nullpointer-Exception.
Ich hab ein Struktogramm erstellt und abgespeichert, wobei folgende Datei heraus kam:

<?xml version="1.0" encoding="UTF-8"?>
<struktogramm fontfamily="83;101;114;105;102;" fontstyle="0" fontsize="15" caption="-1;">
  <strelem typ="2" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
    <text>70;97;108;108;97;117;115;119;97;104;108;</text>
    <fall fallname="49;">
      <strelem typ="3" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
        <text>48;32;60;32;105;32;60;32;97;110;122;97;104;108;</text>
        <schleifeninhalt>
          <strelem typ="5" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
            <text>68;111;45;87;104;105;108;101;32;83;99;104;108;101;105;102;101;</text>
          </strelem>
          <strelem typ="8" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
            <text>65;117;102;114;117;102;</text>
          </strelem>
        </schleifeninhalt>
      </strelem>
    </fall>
    <fall fallname="50;">
      <strelem typ="0" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
        <text>65;110;119;101;105;115;117;110;103;</text>
      </strelem>
    </fall>
    <fall fallname="51;">
      <strelem typ="9" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
        <text>248;</text>
      </strelem>
    </fall>
    <fall fallname="83;111;110;115;116;">
      <strelem typ="0" zx="0" zy="0" textcolor="-16777216" bgcolor="-1">
        <text>65;110;119;101;105;115;117;110;103;</text>
      </strelem>
    </fall>
  </strelem>
</struktogramm>

Wenn ich versuche die Datei zu öffnen, bekomme ich diese Fehlermeldung

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
        at de.kekru.struktogrammeditor.control.XMLLeser.listenelementeErstellen(XMLLeser.java:96)
        at de.kekru.struktogrammeditor.control.XMLLeser.erstelleElementeRek(XMLLeser.java:66)
        at de.kekru.struktogrammeditor.control.XMLLeser.listenelementeErstellen(XMLLeser.java:109)
        at de.kekru.struktogrammeditor.control.XMLLeser.erstelleElementeRek(XMLLeser.java:66)
        at de.kekru.struktogrammeditor.control.XMLLeser.listenelementeErstellen(XMLLeser.java:109)
        at de.kekru.struktogrammeditor.control.XMLLeser.erstelleElementeRek(XMLLeser.java:77)
        at de.kekru.struktogrammeditor.control.XMLLeser.listenelementeErstellen(XMLLeser.java:109)
        at de.kekru.struktogrammeditor.control.XMLLeser.ladeXML(XMLLeser.java:171)
        at de.kekru.struktogrammeditor.control.XMLLeser.ladeXLM(XMLLeser.java:36)
        at de.kekru.struktogrammeditor.control.Struktogramm.laden(Struktogramm.java:868)
        at de.kekru.struktogrammeditor.control.Controlling.openStruktogramm(Controlling.java:172)
        at de.kekru.struktogrammeditor.control.Controlling.laden(Controlling.java:165)
        at de.kekru.struktogrammeditor.control.Controlling.actionPerformed(Controlling.java:216)
        at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
        at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
        at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
        at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
        at javax.swing.AbstractButton.doClick(Unknown Source)
        at javax.swing.plaf.basic.BasicMenuItemUI.doClick(Unknown Source)
        at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(Unknown Source)
        at java.awt.Component.processMouseEvent(Unknown Source)
        at javax.swing.JComponent.processMouseEvent(Unknown Source)
        at java.awt.Component.processEvent(Unknown Source)
        at java.awt.Container.processEvent(Unknown Source)
        at java.awt.Component.dispatchEventImpl(Unknown Source)
        at java.awt.Container.dispatchEventImpl(Unknown Source)
        at java.awt.Component.dispatchEvent(Unknown Source)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
        at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
        at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
        at java.awt.Container.dispatchEventImpl(Unknown Source)
        at java.awt.Window.dispatchEventImpl(Unknown Source)
        at java.awt.Component.dispatchEvent(Unknown Source)
        at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
        at java.awt.EventQueue.access$500(Unknown Source)
        at java.awt.EventQueue$3.run(Unknown Source)
        at java.awt.EventQueue$3.run(Unknown Source)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
        at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
        at java.awt.EventQueue$4.run(Unknown Source)
        at java.awt.EventQueue$4.run(Unknown Source)
        at java.security.AccessController.doPrivileged(Native Method)
        at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
        at java.awt.EventQueue.dispatchEvent(Unknown Source)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
        at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
        at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
        at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
        at java.awt.EventDispatchThread.run(Unknown Source)

[SchoolGuy]

@FairyTail2000 Ping! Könntest du weiter machen damit? Ich würde wenn ihr den PR hier durch habt versuchen das Ding mal nach openSUSE Factory zu bekommen. Das hieße man könnte mit wenig Aufwand das ganze dann auch nach Fedora bekommen... :)

[FairyTail2000]

@FairyTail2000 Ping! Könntest du weiter machen damit? Ich würde wenn ihr den PR hier durch habt versuchen das Ding mal nach openSUSE Factory zu bekommen. Das hieße man könnte mit wenig Aufwand das ganze dann auch nach Fedora bekommen... :)

Ah sorry, ich hatte bisher noch keine Zeit weiter zu machen, wenn du es nach openSUSE factory verschiebst, kann ich danach noch committen?

[SchoolGuy]

@FairyTail2000 Das wird nicht verschoben. Du machst hier weiter ohne von mir zu wissen. Das Packaging hat nichts mit der Repository hier direkt zu tun. :)