Archief - [JAVA] vraagje over een if-structuur

godf@ther zei:

Maar ik snap niet goed waarom hij die fout zou geven als alle landen niet gelijk zijn

Verstuurd vanaf mijn Joyz met Tapatalk 4

Klik om te vergroten...

de variabele i itereert over alle landen van continent j:
En een beetje verder op, gebruik je i om een land op te roepen voor continenten 0, 1, 3, 4 en 5:
Niets garandeert dat die continenten een i-de land hebben. Als een van die contenten minder dan i landen heeft, krijg je dus een IndexOutOfBoundsException.

Ik zou het properder doen zoals al is voorgesteld hierboven


of wat dan ook voor die laatste regel als je het enkel voor de current player ofzo wilt checken.

in die ownedByPlayer check je dan in 1 regel of het symbool van dat land overeenkomt met dat van de meegegeven speler.

Passieve verantwoordelijkheden zoals dat moet je altijd zo diep mogelijk duwen.

In de klasse continent heb ik de volgende methode toegevoegd:

public boolean isVeroverdDoor(String naamSpeler)
{
for(Land l : landen)
{
if(!l.getNaamSpeler().equals(naamSpeler))
return false;
}
return true;
}

---------------------------------------

In de klasse Wereldkaart heb ik het volgende gedaan (vorige foute code weggedaan ook trouwens):

if(sp.getOpdracht().contains("Afrika"))
{
if(continenten.get(0).isVeroverdDoor(sp.getNaam()) && continenten.get(1).isVeroverdDoor(sp.getNaam()) &&
(continenten.get(2).isVeroverdDoor(sp.getNaam()) || continenten.get(3).isVeroverdDoor(sp.getNaam())
|| continenten.get(4).isVeroverdDoor(sp.getNaam()) || continenten.get(5).isVeroverdDoor(sp.getNaam())))
{
isGewonnen = true;
}
}

Ik heb er over nagedacht en dit leek me toch de kortste versie om te controleren of de 2 verplichte continenten te controleren of ze veroverd waren en dan nog 1 optionele continent.

Alvast bedankt voor jullie hulp, hetgeen jullie zeggen houdt ook steek voor me, die fout zal ik in de toekomst dus niet meer maken ^^

Dus uw klasse Wereldkaart (brrr) staat vol met dergelijke code? As in, voor elke mogelijke opdracht? Vind je dat zelf niet 'lelijk'? Of anders gezegd: vind je het makkelijk om je code aan te passen of om een bepaalde opdracht terug te vinden in al die lijnen code?

Uw Wereldkaart zou eigenlijk niet veel meer moeten zijn dan een collection van uw continenten.

Mja ik zou nie direct weten hoe ik het nog compacter kan maken? Ik werkte verder op het project van iemand anders (dit moest ik doen) en die had de methode isGewonnen in de klasse Wereldkaart gezet. Daarom had ik al deze code bij in die methode gezet.

Verstuurd vanaf mijn Joyz met Tapatalk 4

Ik snap wel wa ge wilt zeggen, deze code in de klasse Continent zetten, maar maakt het een verschil?

Verstuurd vanaf mijn Joyz met Tapatalk 4

Oftewel een eigen klasse Opdracht maken, en daar dan alle wincondities in verwerken. Maar da zou wel alles omvergooien van mijn gemaakte code, en daar heb ik spijtig genoeg niet meer de tijd voor

Nog een vraag:

Met deze methode:

public boolean isVeroverdDoor(String naamSpeler)
{
boolean isVeroverdDoor = true;
for(Land l : landen)
{
if(!l.getNaamSpeler().equals(naamSpeler))
isVeroverdDoor = false;
}
return isVeroverdDoor;
}

Als ik deze dan oproep met Continenten.get(1).isVeroverdDoor()

Gaat hij dan gwn de landen af die tot dat continent behoren, of gaat hij alle landen af die in het spel zijn?

Ik heb nu de volgende code:

- klasse Wereldkaart http://pastebin.com/b5Rt3FT3
- klasse Continent klasse Continent - Pastebin.com

Maak van uw opdrachten een enum en check de win in een specifieke klasse. Bvb WinConditionFactory.getInstance (sp.getOpdracht());
12 if-else if-else if- stukken zijn niet leesbaar en niet te onderhouden

godf@ther zei:

Oftewel een eigen klasse Opdracht maken, en daar dan alle wincondities in verwerken. Maar da zou wel alles omvergooien van mijn gemaakte code, en daar heb ik spijtig genoeg niet meer de tijd voor

Klik om te vergroten...

Er zijn nog mooiere oplossingen, maar deze is alvast makkelijk implementeerbaar + je hoeft er de interface van Wereldkaart (of Speler) niet voor aan te passen (ok, gewoon een adder toevoegen).

Concreet:
- abstracte klasse AbstractObjective die een referentie naar Wereldkaart heeft (zodat je met het spel kan communiceren), een referentie naar de 'owner' (de speler die deze opdracht heeft) en een abstracte method isAchieved() (oid)
- concrete klasses die elk 1 bepaalde opdracht implementeren (bvb: ConquerAfricaAmericaAsiaObjective) (en overerven van AbstractObjective) en isAchieved() concret implementeren.

- je klasse Wereldkaart bevat dan een lijst van AbstractObjective objecten (met size = aantal spelers), die je in 'isGewonnen()' dan simpelweg kan aflopen en aan elke opdracht kan vragen 'isAchieved()'
- Alternatief voor dat laatste: Player referentie naar naar concrete objective laten bijhouden en Wereldkaard lijst van Players laten aflopen.

Hangt beetje van huidige code af I guess, maar dat alternatief vind ik wel net iets properder.



Maar eerlijk gezegd: denk je dat je het verdient om te slagen (en naar een volgend jaar te gaan?) als je deze vragen moet stellen? Je moet écht wat meer tijd steken in oefenen (zowel qua analyse, modeling als programming zelf)

Mja, mijn herexamen van programmeren is goed gelukt ... Bij dit project weet ik ook het juiste er niet van, omdat ik moest verder werken op iemand anders zijn code ...

Ik zal de volledige klasse wereldkaart eens posten, de methode isGewonnen geeft altijd een true terug, terwijl deze false moet zijn ...

package domein; import java.util.ArrayList; import java.util.Collections; i - Pastebin.com

130 lijnen lelijke code? Sorry, daar begin ik niet aan..

In hoeverre moest ge de bestaande code behouden? En waarom?
Zolang interface bewaard wordt kan je imho alle code aanpassen...

Ik mocht alles aanpassen, maar ik moest ook denken aan de tijd waarin ik het moest doen, ik had enkele herexamens, en daartussen moest ik dan nog dit project uitbreiden. Ik had gekozen om gewoon de opdracht te doen, zonder te letten op mooie code en dergelijke, nu heb ik wat extra tijd om dit te doen.

alles werkt eindelijk, er was nog wel het een en ander mis ... Maar goed. Gasten bedankt voor de hulp!