Page 3 of 3 First 123
  1. #31
    blackrabbit's Avatar
    Registered
    25/07/02
    Location
    Brussel
    Posts
    1,001
    iTrader
    5 (100%)
    Mentioned
    0 Post(s)
    Reputation
    2/2
    Quote Originally Posted by godf@ther View Post
    This quote is hidden because you are ignoring this member. Show
    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
    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)
    no votes  

  2. #32
    godf@ther's Avatar
    Registered
    27/05/10
    Location
    /
    Posts
    2,234
    iTrader
    0
    Mentioned
    0 Post(s)
    Reputation
    0/17
    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
    no votes  

  3. #33
    blackrabbit's Avatar
    Registered
    25/07/02
    Location
    Brussel
    Posts
    1,001
    iTrader
    5 (100%)
    Mentioned
    0 Post(s)
    Reputation
    2/2
    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...
    no votes  

  4. #34
    godf@ther's Avatar
    Registered
    27/05/10
    Location
    /
    Posts
    2,234
    iTrader
    0
    Mentioned
    0 Post(s)
    Reputation
    0/17
    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.
    no votes  

  5. #35
    godf@ther's Avatar
    Registered
    27/05/10
    Location
    /
    Posts
    2,234
    iTrader
    0
    Mentioned
    0 Post(s)
    Reputation
    0/17
    alles werkt eindelijk, er was nog wel het een en ander mis ... Maar goed. Gasten bedankt voor de hulp!
    no votes  

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  

Log in

Log in