Jump to content

New TvT for aCis


Recommended Posts

6 hours ago, Williams said:

 

which would be better to list the players?

 

private Map<Integer, Player> _registered = new ConcurrentHashMap<>();

 

or 

 

    private Set<Player> _registered = ConcurrentHashMap.newKeySet();

Depends on your needs . If you need key+ value then use it. Else if you need only the player use the set. Both of them are synchronized

Link to comment
Share on other sites

11 hours ago, melron said:

+	public static final <T> T get(Set<T> set)
+	{
+		return get(new ArrayList<>(set));
+	}

:happyforever:

 

Still, I think it's still preferable to Collections.shuffle ONCE rather than doing 15 or 20 (let's say, up to 200/300 for specific servers) Rnd.get.

 

And I don't like the fact to hide something. It basically means you didn't use the correct container for the correct job.

 

But yes, it's the Rnd.get(_registered) solution. I don't think it's really appropriate for the location. You would generate one new ArrayList PER player team attribution. So, if you got 300 ppl to register - 300 arrayLists, 3000 players =3000 arrayLists.

Edited by Tryskell
Link to comment
Share on other sites

9 hours ago, Williams said:

 

which would be better to list the players?

 

private Map<Integer, Player> _registered = new ConcurrentHashMap<>();

 

or 

 

    private Set<Player> _registered = ConcurrentHashMap.newKeySet();

 

The  advantage of ChMap is if you want to pick a particular objectId/Player pair, otherwise it's not needed.

Link to comment
Share on other sites

I am working hard to improve ThreadPool and remove the setOriginalCoordinates method, updated topic with v2 I will improve the main topic with more information.

 

 

Link to comment
Share on other sites

  • 1 year later...

Does anyone still have this full TvT mod and can make it available I appreciate it, I added this one in the Diffs Acisma but it gave many errors, if anyone can make it available for L2jAcis 398,399 I will be grateful.

Link to comment
Share on other sites

10 hours ago, jhonata123 said:

Does anyone still have this full TvT mod and can make it available I appreciate it, I added this one in the Diffs Acisma but it gave many errors, if anyone can make it available for L2jAcis 398,399 I will be grateful.

https://www.google.com/search?client=firefox-b-d&q=pastebin+tvt+l2jacis

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.




×
×
  • Create New...