Hilfe bei Countdown! :/

[NinjaGamers]

Hallo leute,

ich versuche schon seit mehreren Tagen einen funktionierenden Countdown zu machen für ein kleines Server Spiel. Das Problem ist das der Countdown nur für einen Spieler funktioniert und nicht für alle die ins Spiel rein gegangen sind. Hier mein Code:

	public static int Countdown;
	static int count = Integer.parseInt(b.getConfig().getString("Config.Countdown"));
	public static boolean countdownRunning = false;

	public static void startCountdown(final Bounce b, final Player p) {
		countdownRunning = false;
		Countdown = b.getServer().getScheduler().scheduleSyncRepeatingTask(b, new Runnable() {
			
			public void run() {
				
				countdownRunning = true;
				
				if(count != -1) {
					if(count != 0){
						for(int i = 0; i < b.playersInGame.size(); i++) {
							p.sendMessage(b.prefix_p + ChatColor.GREEN + "Spiel startet in " + ChatColor.GOLD + count + ChatColor.GREEN +  " Sekunden!");
						    count--;
						}
					
				} else {
					for(int i = 0; i < b.playersInGame.size(); i++) {
					    
					    startGame(b, p);
						Bukkit.getScheduler().cancelTask(Countdown);
					    countdownRunning = false;
						setCount();
					}
					}
				}
				
			}
			
			public void setCount(){
					 count = 30;
			}
			
		},0L, 20L);	
	}

Das ist der Countdown zum Spiel starten und meine startGame() Methode ist hier:

        public static boolean gameRunning  = false;
	public static int Game;
	public static int gameTime = Integer.parseInt(b.getConfig().getString("Config.GameTime"));
	public static void startGame(final Bounce b, final Player p) {
		gameRunning = false;
		
		World gameWorld = b.getServer().getWorld(b.getConfig().getString("Config.Game.Arena1.world"));
		Location game = new Location(gameWorld, b.getConfig().getDouble("Config.Game.Arena1.x"), b.getConfig().getDouble("Config.Game.Arena1.y"), b.getConfig().getDouble("Config.Game.Arena1.z"));
		p.teleport(game);
		
		p.sendMessage(b.prefix_p + ChatColor.DARK_PURPLE + "The game has started! Good luck! :)");
		
		Game = b.getServer().getScheduler().scheduleSyncRepeatingTask(b, new Runnable() {
			
			public void run() {
				
				gameRunning = true;
				
				if(gameTime != -1) {
					if(gameTime == 10){
						
						for(int i = 0; i < b.playersInGame.size(); i++) {
							p.sendMessage(b.prefix_p + ChatColor.GREEN + "The game ends in 10 seconds!");
						}
						
				if(gameTime != 0) {
						gameTime--;
						if(gameTime == 10){
							
							for(int i = 0; i < b.playersInGame.size(); i++) {
								p.sendMessage(b.prefix_p + ChatColor.GREEN + "The game ends in 10 seconds!");
							}
					}
				} else {
					for(int i = 0; i < b.playersInGame.size(); i++) {
					stopGame(b, p);
					Bukkit.getScheduler().cancelTask(Game);
					gameRunning = false;
					}
				}
					}
				}
			}
		}, 0L, 20L);
		
	}

Bitte helft mir und sagt mir was ich falsch gemacht habe

MfG Fisell

 

[NinjaGamers]

Habs mit nem Timer gemacht #Closed !

 

[abbix]

Vielleicht haben noch andere Leute das selbe Problem - Würdest du denen zeigen, wie du es hinbekommen hast?

 

[Chrisliebaer]

Normalerweise würde ich dir ja zustimmen, aber die Antwort zu der Frage war bereits zu Anfang klar:
http://docs.oracle.com/javase/7/docs/api/java/util/Timer.html

Das ist quasi immer die Lösung, wenn du einen Timer brauchst, wie auch bereits der Name vermuten lässt

 

[NinjaGamers]

ich hab einfach einen neuen timer in der methode startCountdown() gemacht also einfach:

public int countdown = 30;
	public void startCountdown(final PLUGINNAME a, final int arena){
		final Timer t = new Timer();
		sendGameMessage("§bEs sind genügend Spieler online! §dDas Spiel startet in 30 Sekunden!", b, arena);
        final TimerTask Countdown = new TimerTask() {
            public void run() {
	               if(countdown == 0){
						startGame(a, arena);
						t.cancel();
	               }
	               else{
		               countdown--;
		               for(Player p : a.arenaleute.get(arena)){
		            	   p.setLevel(countdown);
		               }
		               if(countdown == 20){
		            	   sendGameMessage(a.prefix_p + "§2Das Spiel startet in " + countdown + " Sekunden", a, arena);
		               }
		               if(countdown <= 10){
		            	   sendGameMessage(b.prefix_p + "§2Das Spiel startet in " + countdown + " Sekunden!", a, arena);
		               }
	               }
	               if(countdown <= 5){
	            	     playSoundCountdown(a, arena);
	               }
            }
        };
        t.scheduleAtFixedRate(Countdown, 1000, 1000);
	}

 

[Chrisliebaer]

Hast du den Code denn schon getestet? Du rufst da Player.setLevel() innerhalb des Timer Threads auf. Wenn du Pech hast, kann das bis zu einem Servercrash führen. Dazu kommen dann noch eine ganze Reihe weitere Aufrufe außerhalb des Main-Threads. Vor allem startGame() sieht sehr nach einem potentiellen Crash aus.

Je nach dem, wie a die Spieler ermittelt kannst du auch hier in Threadingprobleme kommen. Es ist zwar aufgrund der kurzen Ausführungszeit des Timers extrem unwahrscheinlich, aber sicher ist es nicht.

 

[NinjaGamers]

hab alles schon getestet und es funktioniert einwandfrei aber danke für die information ^^

 

[delta242]

Für bessere Performance würde ich evtl. den Bukkit Scheduler hernehmen: http://wiki.bukkit.org/Scheduler_Programming/de

 

[Jorre222]

Beispielcode

Spoiler: Der Countdown an sich

Integer i = 30;
Integer i2 = 0;

i2 = Bukkit.getScheduler.scheduleSyncDelayedTask(this, new Runnable() {
    if(i == 0) {
        i = 30;
        Bukkit.getScheduler.cancelTask(i2);
    }    
    i = i - 1;
    player.sendMessage("Noch '" + i + "' Sekunden!");
}, 0L, 20L);

Spoiler: Nachricht nur an die Spieler die in dem Spiel sind

ArrayList<String> gameplayers = new ArrayList<String>();

public void SpielJoin() {
if(gameplayers.contains(player.getName())) {
player.sendMessage("Du bist schon im Spiel");
} else {
gameplayers.add(p.getName());
}
}

public void SpielQuit() {
if(gameplayers.contains(player.getName())) {
gameplayers.remove(player.getName());
} else {
player.sendMessage("Du bist in keinem Spiel!");
}
}

Spoiler: Die FORschleife!

for (Player rs : Bukkit.getOnlinePlayers()) {
if(gameplayers.contains(player.getName) {
rs.sendMessage("Noch ....");
}
}

Aus dem Kopf geschrieben! Daher könnte es 1 / 2 Fehler geben!