Open AurumTheEnd opened 3 years ago
Currently, a lot of frontend classes are too complicated:
[ohrdlicka@ohrdlicka frontend]$ sloc --details --format cli-table --keys total,source,empty,comment . ┌────────────────────────────────────────────────────────┬──────────┬────────┬───────┬─────────┐ │ Path │ Physical │ Source │ Empty │ Comment │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ utility/AgentApiGenerator.java │ 379 │ 339 │ 37 │ 3 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ utility/CommonUtils.java │ 169 │ 147 │ 19 │ 3 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ utility/ImageButtonFactory.java │ 182 │ 142 │ 39 │ 3 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ utility/ScreenFinder.java │ 129 │ 76 │ 14 │ 39 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ utility/TeeOutputStream.java │ 44 │ 26 │ 10 │ 8 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/about/AboutView.java │ 53 │ 44 │ 9 │ 1 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/filesystem/NewFsVmView.java │ 189 │ 162 │ 27 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/filesystem/NewFsVmController.java │ 81 │ 62 │ 19 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/license/LicenseView.java │ 51 │ 42 │ 9 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/overwrite/FileToClassValidator.java │ 111 │ 91 │ 15 │ 5 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/overwrite/LatestPaths.java │ 67 │ 53 │ 14 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/overwrite/OverwriteClassDialog.java │ 828 │ 742 │ 83 │ 3 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/remote/NewConnectionController.java │ 51 │ 37 │ 9 │ 5 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/remote/NewConnectionView.java │ 136 │ 117 │ 19 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/settings/SettingsView.java │ 413 │ 336 │ 76 │ 3 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/ConfigPanel.java │ 66 │ 54 │ 11 │ 1 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/FileSelectorArrayAddRow.java │ 34 │ 24 │ 10 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/FileSelectorArrayPanel.java │ 78 │ 62 │ 14 │ 2 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/FileSelectorArrayRow.java │ 108 │ 89 │ 19 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/FileSelectorPanel.java │ 72 │ 57 │ 15 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/MessagePanel.java │ 19 │ 14 │ 5 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/OkCancelPanel.java │ 59 │ 47 │ 9 │ 3 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/PluginConfigurationEditorController.java │ 318 │ 273 │ 45 │ 1 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/PluginConfigurationEditorView.java │ 90 │ 70 │ 15 │ 5 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/PluginListPanel.java │ 32 │ 24 │ 8 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/PluginTopOptionPanel.java │ 60 │ 49 │ 11 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/plugins/TextInputPanel.java │ 36 │ 28 │ 8 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/BytecodeDecompilerView.java │ 948 │ 783 │ 143 │ 30 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/DecompilationController.java │ 611 │ 529 │ 64 │ 21 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/GlobalConsole.java │ 169 │ 151 │ 16 │ 2 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/LoadingDialog.java │ 51 │ 41 │ 10 │ 0 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/MainFrameView.java │ 482 │ 379 │ 65 │ 41 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/OverridesManager.java │ 147 │ 131 │ 15 │ 1 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/popup/ClassListPopupMenu.java │ 67 │ 51 │ 15 │ 1 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/renderer/ClassListRenderer.java │ 87 │ 64 │ 23 │ 1 │ ├────────────────────────────────────────────────────────┼──────────┼────────┼───────┼─────────┤ │ frame/main/renderer/VmListRenderer.java │ 106 │ 90 │ 16 │ 1 │ └────────────────────────────────────────────────────────┴──────────┴────────┴───────┴─────────┘
- contain a lot of inner classes (deduct 1 for the top-level class) ```sh [ohrdlicka@ohrdlicka frontend]$ find $(pwd) -name "*.java" -exec echo {} \; -exec grep -c -E \\sclass\\s[A-Z] {} \; -exec echo \; /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/main/BytecodeDecompilerView.java 4 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/main/DecompilationController.java 4 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/main/MainFrameView.java 2 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/remote/NewConnectionView.java 2 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/settings/SettingsView.java 5 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/overwrite/OverwriteClassDialog.java 6 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/overwrite/FileToClassValidator.java 2 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/filesystem/NewFsVmController.java 2 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/frame/filesystem/NewFsVmView.java 2 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/utility/CommonUtils.java 2 /home/ohrdlicka/Projects/java-runtime-decompiler/runtime-decompiler/src/main/java/org/jrd/frontend/utility/AgentApiGenerator.java 4
Notable outliers are of cource MainFrameView, BytecodeDecompilerView, and OverwriteClassDialog.
thats the spirirt!
judovana
commented
3 years ago judovana
commented
3 years ago
Currently, a lot of frontend classes are too complicated:
Notable outliers are of cource MainFrameView, BytecodeDecompilerView, and OverwriteClassDialog.