Visualizzazione dei risultati da 1 a 2 su 2

Visualizzazione discussione

  1. #2
    Utente di HTML.it
    Registrato dal
    Oct 2014
    residenza
    Padova
    Messaggi
    361
    codice:
    public void getinizio(Giocatore g){
        g1=newGiocatore(g.getName());
        if(g!=null&&g.getEnergia()>g1.getEnergia()){
            g1=g;
        }else
        g1=g1;
    }
    Ciao, non sono un esperto ma mi pare che intanto questo metodo sia confusionario, cioè tu passi al metodo un Giocatore già instanziato e dentro al metodo invece che fare i controlli e scegliere se assegnarlo o no alla Casella crei un nuovo Giocatore copiando il nome di quello passato (g1=newGiocatore(g.getName());).
    Poi controlli se il giocatore passato al metodo è null (mentre tu dici che devi verificare se la casella contiene già un giocatore quindi dovresti controllare se g1 della casella è null).
    Infine usi due diverse variabili uguali a g1 e nell'ultima riga scrivi g1=g1 (brrrrrrrr).

    Io questo metodo lo riscriverei nel seguente modo:

    codice:
    public void getinizio(Giocatore g)
    {
        if(g1==null)g1=g; // se g1 e' null metto il giocatore nella Casella che è vuota
        else if(g.getEnergia()>g1.getEnergia()) g1=g; // se il giocatore passato ha energia maggiore lo sostituisco,volendo puoi usare un or e unire le due righe
    }
    Poi per quanto riguarda il problema sul numero dei giocatori questo dipende dalla consegna:

    se ogni Casella può averne solo uno allora tieni quello con energia maggiore, se può averne di più ma vuoi che sia "attivo" solo quello con energia maggiore comunque gli altri devono essere memorizzati da qualche parte (Array,ArrayList,Map....)
    Ultima modifica di Ansharja; 17-06-2015 a 16:05

Permessi di invio

  • Non puoi inserire discussioni
  • Non puoi inserire repliche
  • Non puoi inserire allegati
  • Non puoi modificare i tuoi messaggi
  •  
Powered by vBulletin® Version 4.2.1
Copyright © 2025 vBulletin Solutions, Inc. All rights reserved.