Помогите Потоки

  • Автор темы Автор темы AlexMor
  • Дата начала Дата начала

AlexMor

Активный участник
Пользователь
Баллы
76
Здравствуйте, как можно создавать новый поток (в потоке: цикл повторяющийся каждую минуту, и проверку, если игрок онлайн то вызывать другой метод) для каждого зашедшего игрока, причем по 1 потоку на игрока.

Цель: каждую минуту забирать у игрока определенное кол-во денег если он онлайн.
 
AlexMor сказал(а):
Здравствуйте, как можно создавать новый поток (в потоке: цикл повторяющийся каждую минуту, и проверку, если игрок онлайн то вызывать другой метод) для каждого зашедшего игрока, причем по 1 потоку на игрока.

Цель: каждую минуту забирать у игрока определенное кол-во денег если он онлайн.
Нажмите, чтобы раскрыть...
Как то вроде через HashMap можно... Но как
 
AlexMor сказал(а):
Как то вроде через HashMap можно... Но как
Нажмите, чтобы раскрыть...
Код: 
private HashMap<Player, Thread> players = new HashMap<Player, Thread>();

@EventHandler
public void onJoin(PlayerJoinEvent e) {
   Thread t = new Thread() {
      public void run() {
         while(true) {
            minusMoney(e.getPlayer());
            Thread.sleep(5000);
         }
      }
   };
   players.put(e.getPlayer(), t);
   t.start();
}

@EventHandler
public void onLeave(PlayerLeaveEvent e) {
   players.get(e.getPlayer()).stop();
   players.remove(e.getPlayer());
}
Как-то так, писал с планша, поэтому могут быть ошибки где-то.
 
Qmaks сказал(а):
Код: 
private HashMap<Player, Thread> players = new HashMap<Player, Thread>();

@EventHandler
public void onJoin(PlayerJoinEvent e) {
   Thread t = new Thread() {
      public void run() {
         while(true) {
            minusMoney(e.getPlayer());
            Thread.sleep(5000);
         }
      }
   };
   players.put(e.getPlayer(), t);
   t.start();
}

@EventHandler
public void onLeave(PlayerLeaveEvent e) {
   players.get(e.getPlayer()).stop();
   players.remove(e.getPlayer());
}
Как-то так, писал с планша, поэтому могут быть ошибки где-то.
Нажмите, чтобы раскрыть...
Сяпки. поржал. :D
 
Для таких вещей потоки не нужны. Достаточно использовать шедулер. Это такая штука, которая в одном единственном потоке выполняет все задачи - либо единоразово, либо каждый раз повторяет с определенной задержкой. Тебе нужно добавить как раз такую задачу. Подробнее о шедулерах можно почитать в других темах или в интернете.
Qmaks сказал(а):
Код: 
private HashMap<Player, Thread> players = new HashMap<Player, Thread>();

@EventHandler
public void onJoin(PlayerJoinEvent e) {
   Thread t = new Thread() {
      public void run() {
         while(true) {
            minusMoney(e.getPlayer());
            Thread.sleep(5000);
         }
      }
   };
   players.put(e.getPlayer(), t);
   t.start();
}

@EventHandler
public void onLeave(PlayerLeaveEvent e) {
   players.get(e.getPlayer()).stop();
   players.remove(e.getPlayer());
}
Как-то так, писал с планша, поэтому могут быть ошибки где-то.
Нажмите, чтобы раскрыть...
Сяпки, поржал [х2]
На каждого игрока по потоку... Жирно очень. От десяти заполненных серверов умрёт любой хост.
 
Dymeth сказал(а):
Для таких вещей потоки не нужны. Достаточно использовать шедулер. Это такая штука, которая в одном единственном потоке выполняет все задачи - либо единоразово, либо каждый раз повторяет с определенной задержкой. Тебе нужно добавить как раз такую задачу. Подробнее о шедулерах можно почитать в других темах или в интернете.
Сяпки, поржал [х2]
На каждого игрока по потоку... Жирно очень. От десяти заполненных серверов умрёт любой хост.
Нажмите, чтобы раскрыть...
Он попросил - я сделал. В чем проблема?
Все, что ты написал - я и так знаю, но это его требования.
Dymeth сказал(а):
Для таких вещей потоки не нужны. Достаточно использовать шедулер. Это такая штука, которая в одном единственном потоке выполняет все задачи - либо единоразово, либо каждый раз повторяет с определенной задержкой. Тебе нужно добавить как раз такую задачу. Подробнее о шедулерах можно почитать в других темах или в интернете.
Сяпки, поржал [х2]
На каждого игрока по потоку... Жирно очень. От десяти заполненных серверов умрёт любой хост.
Нажмите, чтобы раскрыть...
Шулдерами, кстати, я не пользовался. Может стоит апи как-то почитать...
 
А зачем д
Qmaks сказал(а):
Он попросил - я сделал. В чем проблема?
Все, что ты написал - я и так знаю, но это его требования.

Шулдерами, кстати, я не пользовался. Может стоит апи как-то почитать...
Нажмите, чтобы раскрыть...
И заодно API ProtocolLib'а :)
 
Qmaks сказал(а):
Ты меня осуждаешь за это или что?
Мне хватало знаний базового языка без лишнего апи, будет надо что-то новое - выучу.
Нажмите, чтобы раскрыть...
Я не осуждаю) Каждый в праве пользоваться чем угодно, вопрос лишь в том, что лучше. Вдруг при сравнении работы твоих утилит с ProtocolLib выяснится, что они работают гораздо быстрее
 
Если уж и создавать на каждого игрока по потоку, то хотя бы с пулом.
 
alexandrage сказал(а):
Да ничего там не случится, ты же там не степень вычисляешь.
Нажмите, чтобы раскрыть...
Это конечно, так.Но если есть возможность оптимизировать, то почему бы ею не воспользоваться?
TS, такой вариант устроит?
Java: 
scheduler.scheduleSyncRepeatingTask(plugin, () -> {
    for (Player player : plugin.getServer().getOnlinePlayers())
        player.decrementMoney(params);
}, 0L, 1200L);
 
AtomicInteger сказал(а):
Это конечно, так.Но если есть возможность оптимизировать, то почему бы ею не воспользоваться?
TS, такой вариант устроит?
Java: 
scheduler.scheduleSyncRepeatingTask(plugin, () -> {
    for (Player player : plugin.getServer().getOnlinePlayers())
        player.decrementMoney(params);
}, 0L, 1200L);
Нажмите, чтобы раскрыть...
Ага збс оптимизировал :D. getOnlinePlayers(). Сяпки - поржал. Так держать!
 
alexandrage сказал(а):
Ага збс оптимизировал :D. getOnlinePlayers(). Сяпки - поржал. Так держать!
Нажмите, чтобы раскрыть...
Да не оптимизировал я там ничего.Я не думаю, что автор пишет для вайма, если захочет уже сам напишет хранилище для игроков, я лишь общий код привёл.
 
alexandrage сказал(а):
getOnlinePlayers()
Нажмите, чтобы раскрыть...
Сегодня задавался вопросом о скорости работы этого метода, но его код так и не смог найти. Вероятно, Вы знаете, как он работает. Подскажите, насколько он быстр и где его использовать не рекомендуется?
 
molor сказал(а):
Сегодня задавался вопросом о скорости работы этого метода, но его код так и не смог найти. Вероятно, Вы знаете, как он работает. Подскажите, насколько он быстр и где его использовать не рекомендуется?
Нажмите, чтобы раскрыть...
Потому что данный метод проходится по всем игрокам, которые когда то были на сервер и проверят isOnline();
По этому причине рекомендуется делать свой собственный метод getOnlinePlayer(name); с помощь эвентов Входа и Выхода игроков заносить\удалять игроков в HashSet<Player>.
 
Exception_Prototype сказал(а):
Потому что данный метод проходится по всем игрокам, которые когда то были на сервер и проверят isOnline();
По этому причине рекомендуется делать свой собственный метод getOnlinePlayer(name); с помощь эвентов Входа и Выхода игроков заносить\удалять игроков в HashSet<Player>.
Нажмите, чтобы раскрыть...
Чего...
Код: 
private final List<CraftPlayer> playerView;

public CraftServer(MinecraftServer console, PlayerList playerList) {
   ...
   this.playerView = Collections.unmodifiableList(Lists.transform(playerList.players, new Function() {
       public CraftPlayer apply(EntityPlayer player) {
           return player.getBukkitEntity();
       }
   }));
   ...
}
Код: 
public final List<EntityPlayer> players = new CopyOnWriteArrayList();

public void onPlayerJoin(EntityPlayer entityplayer, String joinMessage) {
    this.players.add(entityplayer);
    ...
}

public String disconnect(EntityPlayer entityplayer) {
    ...
    this.players.remove(entityplayer);
    ...
}
 
Последнее редактирование:
Разве с Bukkit можно взаимодействовать не с основного потока?
 
Для ответа нужно войти/зарегистрироваться
Назад
Верх