Wenn ich das Program laufen lasse wirkt es für mich fertig. Leider Time Out bei der Verifizerung
package de.codegym.task.task16.task1627;
import java.util.ArrayList;
import java.util.List;
public class Solution {
public static void main(String[] args) throws InterruptedException {
OnlineGame onlineGame = new OnlineGame();
onlineGame.start();
}
public static class OnlineGame extends Thread {
public static volatile boolean isWinnerFound = false;
public static List<String> actions = new ArrayList<>();
static {
actions.add("Spiel starten");
actions.add("Ressourcen sammeln");
actions.add("Wirtschaft wachsen lassen");
actions.add("Feinde töten");
}
protected Gamer gamer1 = new Gamer("Smith", 3);
protected Gamer gamer2 = new Gamer("Jones", 1);
protected Gamer gamer3 = new Gamer("Gates", 5);
public void run() {
gamer1.start();
gamer2.start();
gamer3.start();
while (!isWinnerFound) {
}
gamer1.interrupt();
gamer2.interrupt();
gamer3.interrupt();
}
}
public static class Gamer extends Thread {
private int rating;
private String name;
boolean gewinner = false;
public Gamer(String name, int rating) {
super(name);
this.rating = rating;
}
@Override
public void run() {
//schreib hier deinen Code
synchronized(this){
int i = 0;
while(!OnlineGame.isWinnerFound){
try{this.sleep(this.rating*1000);}catch(InterruptedException e){}
if(!OnlineGame.isWinnerFound) System.out.println(this.getName()+": "+OnlineGame.actions.get(i));
i++;
if(i==4){OnlineGame.isWinnerFound=true;System.out.println(this.getName()+":gewonnen!");this.gewinner=true;break;}
}
if(OnlineGame.isWinnerFound&&!this.gewinner){System.out.println(this.getName()+":verloren!");Thread.currentThread().stop();}
}
}
}
}
It's much clearer which code blocks belong together, which commands belong where.
The execution flow is much clearer.
By writing your code this way, you make it easier for yourself if you need to fix it later. You also make life easier for your fellow programmers when they need to understand what your code does.
III.) I don't think synchronization is necessary in this task. Unnecessary synchronisation will only slow down the code.
Translated with www.DeepL.com/Translator (free version)