ElisaCastelli/ing-sw-2020-cappella-castelli-corti

mixing logic and network

Closed this issue · 5 comments

at:
@OverRide
public void run(){
System.out.println("Sono in ascolto");
//se sono il primo ad essermi connesso mando un messaggio al client per sapere il numero di giocatori
if(user.getIndexArrayDiClient()==0){
sendUpdate(new AskNPlayerEvent());

You are adding logic in IO/network.
pls revise you design.
network hs to know NOTHING about logic.

Provvederemo subito a cambiarlo, grazie mille. Le posso chiedere se è solo questa la parte che non va bene o se c'è altro in quella classe? Inoltre, ha per caso guardato il nostro visitor per i messaggi di rete? è ancora in fase di sviluppo e sappiamo che può essere snellito e migliorato, ma volevamo solo sapere se eravamo sulla strada giusta o avevamo sbagliato completamente l'approccio.

  1. we check usual "hot" parts (networks and so on..) searching for common mistakes
  2. I don ot check visitor. if any doubt pls email us identify your points.

Ok, va bene la ringrazio. Ne discuto con le mie colleghe e, se fosse il caso, le faremo sapere. Grazie per l'aiuto

seems fixed. pls write some text about fix

Con il rifacimento della rete abbiamo spostato la logica di connessione nel game e la gestione nella virtualview, così da rendere la parte della connessione server/client indipendente