diff --git a/doc/static_analysis.md b/doc/static_analysis.md index e52f319..6f4acf4 100644 --- a/doc/static_analysis.md +++ b/doc/static_analysis.md @@ -13,12 +13,13 @@ Az elemzés során az előbb említett VSCode extension segítségével végign ### ./Game/Character.java Az esetek túlnyomó részében a hiba itt a következő volt: "Rename this method name to match the regular expression '^[a-z][a-zA-Z0-9]*$'." -A szabály leírása szerint a "naming convention"-ök csapatok együttműködését segítheti elő. A projekt során ezeket sajnos nem konzisztensen követtük mindenhol, így ez adott esetben javításra szorulna olyan szinte, hogy legalább az egyik naming convention-t követni kellene a projekt során. Mivel nem egy több csapat által kezelt projektről van szó, így a SonarLint-es warning által ajánlott convention lecserélhető lenne egy másikra - ha az konzisztensen lenne alkalmazva az egész projektben. +A szabály leírása szerint a "naming convention"-ök csapatok együttműködését segítheti elő. A projekt során ezeket sajnos nem konzisztensen követtük mindenhol, így ez adott esetben javításra szorulna olyan szinte, hogy legalább az egyik naming convention-t követni kellene a projekt során. Mivel nem egy több csapat által kezelt projektről van szó, így a SonarLint-es warning által ajánlott convention lecserélhető lenne egy másikra - ha az konzisztensen lenne alkalmazva az egész projektben. Ez a probléma a projekt egészében előfordul, így innentől kezdve ezt nem emeljük ki. Ezzel szemben a problémák másik része már egy fontosabb hibára világítottak rá: "Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation." Alapjáraton ez problémás lenne. A sonarlint-es warning viszont megoldható azzal, ha egy placeholder kommentet a függvény hasába helyezünk - ezt eddig a függvény felé írtuk ezzel követve a kommentezési szokásokat. ### ./Game/City.java + ### ./Game/Field.java ### ./Game/Game.java ### ./Game/Mechanic.java diff --git a/src/main/java/hu/bomboclat/Game/City.java b/src/main/java/hu/bomboclat/Game/City.java index ddc8675..1c0291e 100644 --- a/src/main/java/hu/bomboclat/Game/City.java +++ b/src/main/java/hu/bomboclat/Game/City.java @@ -43,6 +43,7 @@ public void Step(boolean rnd) { * Visszaadja, hogy hány pontja van a szerelőknek vagyis a consumed értékét. * @return A szerelők pontjai, a consumed tagváltozó értéke. */ + @Override public int getMechanicPoint(){ return consumed; } @@ -52,6 +53,7 @@ public int getMechanicPoint(){ * @param m a csövet felvenni kívánó szerelő játékos. * @return A függvény a felvett cső objektumot adja vissza. */ + @Override public Pipe PickUpPipe(Mechanic m){ Pipe returnPipe = new Pipe(); PipeView pv = new PipeView(returnPipe); @@ -71,9 +73,9 @@ public Pipe PickUpPipe(Mechanic m){ * A városból új pumpákat tudnak felvenni ezzel a függvénnyel. * @return A függvény a felvett pumpa objektumot adja vissza. */ + @Override public Pump PickUpPump() { - Pump p = new Pump(); - return p; + return new Pump(); } diff --git a/target/classes/hu/bomboclat/Game/City.class b/target/classes/hu/bomboclat/Game/City.class index a61857e..d52dc79 100644 Binary files a/target/classes/hu/bomboclat/Game/City.class and b/target/classes/hu/bomboclat/Game/City.class differ