Nexus' TeamService v1.1

I’ve created a Module that makes teams a bit easier to handle, instead of having to do many lines of code to put tools and things in the player’s backpack when they join that team, this module does that for you,

Here’s how the module looks:
billede

As you can see, you can set if the team change should remove the backpack the player already has, or you could also set it so players won’t get tools from the team they’re about to join.

I’m planning to make a V2 in the future, so please give some feedback on how this could be improved in the future, have fun! :smiley:

Link to module:
https://www.roblox.com/library/7083537293/TeamService-Module
Update V1.1:
List of features added in v1.1:

  • Added ClearAllChildren to ClearPlayerBackpack instead of going through every children of the backpack.
  • Added functionality for mass teaming via tables
  • Separated AddTools and ClearPlayerBackpack to their own functions for less messy code
1 Like

This module lacks functionality & is pretty poorly written.

  1. :ClearAllChildren() exists, you don’t have to do a :GetChildren() loop to clear every child.
  2. Could have at least separated the remove & add tools functionality. If you’re gonna add more to this module then doing the same thing you’re currently doing is gonna make it messy very quickly.
  3. Why do Teams[Team] when you can literally just directly reference a Team and send it as a argument?

aka

local Team = game.Teams.Team1
TeamService:SetTeam(Player,Team)
  1. _,x and a,b do not contribute to the module’s readability. At least make it more readable by doing _,Tool.
  2. At least have a tools folder as an argument. Don’t make people put tools into places where they don’t belong without modifiying the module.
  3. As mentioned earlier, this module lacks some needed functionality & is pretty insignificant. Improve by adding more functions like mass-teaming/unteaming.

But he is not deleting everything, only tools

3 Likes

There would only be tools in the player’s backpack, so it doesn’t matter if I use in pairs or clearallchildren.

There could be local scripts and more

2 Likes

Why would there be any kind of script in backpack at all?

3 Likes

New scripters do that a lot. Like all the time because they follow bad or outdated tutorials

1 Like

That’s their problem then. They’ll learn to not do it if they ever meet 1 to 1 with this issue.

2 Likes

Then it’s their fault that they followed an outdated tutorial and can’t use this module because of it.

Nothing is stopping them from learning how to use modules and later on write their own modules.

3 Likes

(post withdrawn by author, will be automatically deleted in 1 hour unless flagged)

Slow and useless? That’s pretty rude bruv :pensive:

4 Likes

Any type of game can use this, not only “real good games”

Please at least show proof to back up your claim. Text is not enough.

What do you mean by “good” like a actually well made proper game or just a simple yet playable game?

A lot of games use teams, such as jailbreak.

2 Likes

Soo, jailbreak? [Thanks polarisprog.]

And many many tycoon games that are sometimes on the front page use teams for easier things, for example look at 2 player tycoons, they depend on teams for it to work properly.

Viruses do exist yes, but not in the good free models, free models with viruses aren’t on top of the toolbox, they’re way down and can easily be rid of viruses if you know how to do so.

3 Likes

Why would you use StringValue over a Team instance? You’re just making things unnecessary complicated.

1 Like