Skip to main content
Commonmark migration
Origen Enlace

###Pero esto tiene un bug

Pero esto tiene un bug

###Sobre agregar dos racers.

Sobre agregar dos racers.

###Pero esto tiene un bug

###Sobre agregar dos racers.

Pero esto tiene un bug

Sobre agregar dos racers.

Origen Enlace
rnrneverdies
  • 16.8k
  • 4
  • 53
  • 81

Al parecer es una carrera de Threads y no importa que halla un solo Racer pues el contador distancia es propio de cada thread al estar definido en la función race. En términos generales no hay ninguna razón para no usar el mismo Runnable en dos threads, pero debes tener cuidado en la implementación pues los miembros de la clase serán compartidos entre ambos threads. En la mayoría de los casos prácticos, utilizarías diferentes instancias.

###Pero esto tiene un bug

El método is isRaceWon, así como esta, puede ser accesado por ambos hilos al mismo tiempo, por lo que existe una condición de carreta (o race condition ref 1).

Nunca sucederá que ambos ganen, pues hay una sola variable para guardar el ganador. Sin embargo es un error de implemetación y es una buena oportunidad para que lo aprendas pues esta situación es bastante común.

Java tiene una forma muy simple de solucionar esta condicion de carrera (cuantas carreras, parece un examen). Se llamada synchronized methods ref 2 y agregando el modificador synchornized a la declaración de la función, impide que ambos hilos accedan al metodo al mismo tiempo (pues hay un solo Runnable). Esta sentencia se resume en algo así: aquí solo entra un hilo a la vez.

Seria.. private synchornized boolean isRaceWon(int totalDistanceCovered)

Esto evita que tengas un falso ganador. O mejor dicho que el segundo, logre establecer su nombre como el ganador, a pesar de haber llegado segundo.

###Sobre agregar dos racers.

Si quieres duplicar los racers, solo para probarlo, tienes que sincronizarlo de otro modo, pues el recurso a proteger es Racer.winner y al ser dos instancias, el mecanismo anterior no sirve. Pero Java tiene otro mecanismo para este tipo de problemas, llamado synchronized statements ref 3.

Seria así:

private boolean isRaceWon(int totalDistanceCovered) {
    boolean isRaceWon =  false;
    // este es el bloque protegido que no pueden ingresar dos threads a la vez.
    synchronized(Racer.winner) {
        // desde aqui
        if((Racer.winner==null )&&(totalDistanceCovered==100)){
            String winnerName = Thread.currentThread().getName();
            Racer.winner = winnerName;
            // hasta aqui, es la zona critica
            System.out.println("Winner is :"+Racer.winner);
            isRaceWon = true;
        }else if(Racer.winner==null){
            isRaceWon = false;
        }else if(Racer.winner!=null){
            isRaceWon = true;
        }
    } // fin del bloque protegido     
    return isRaceWon;
}

Luego puede instanciar dos racers y pasarle uno a cada thread, para ver que sucede!