Refactoring

Refactoring

Doppelter Code

In der Klasse SadMinecraftCommandArgumentParser wird in der Methode extractStateType ein String zu einem int konvertiert. Dasselbe wurde in der Methode extractBlockIdentifier mehrfach getan und daher bereits in eine Methode parseInt ausgelagert. extractStateType verwendet diese nicht und hat in der Dopplung dieses Codes auch noch einen Fehler eingebaut, indem die falsche Exception abgefangen wird um auf nicht konvertierbare Strings zu prüfen.

Zusammenführen des Codes

extractStateType wurde so umgeschrieben, dass es die Methode parseInt verwendet. Da das verwenden eines OptionalInt in extractStateType als sinnvoller als ein Optional<Integer> erachtet wurde, wurde zusätzlich die Methode parseInt und deren Aufrufe in extractBlockIdentifier entsprechend geändert.

Zu lange Methode

In der Klasse SadMinecraftCommandArgumentParser ist die Methode extractBlockIdentifier mit einem 40 Zeilen langen Body (ohne Leerzeilen) relativ lang und unübersichtlich.