I made this mod to try to help the servers a little. This mod contains commands and a worldscript block.
Download: https://github.com/WilLiam12-teddy/lamb ... master.zip
Lamby
-
- New member
- Posts: 6
- Joined: Thu Jun 18, 2020 21:33
- GitHub: WilLiam12-teddy
- IRC: WilLiam12
- In-game: WilLiam12
- Noodlemire
- Member
- Posts: 61
- Joined: Sun May 27, 2018 00:07
- GitHub: Noodlemire
- In-game: Noodlemire
Re: Lamby
I really think you should add a little more documentation, mainly what each command does. People shouldn't have to dig through the code itself to understand anything about what your mod actually provides.
Also, your die command is weird. It looks like something that you were seriously overthinking. It should be as simple as setting the user's hp to 0. Doing that automatically kills players, no questions asked. The if statement spaghetti is unnecessary, and assumes too much about the currently played game. Players can have max hp values other than 20. Death can have consequences other than clearing the inventory and being sent specifically to spawn. Actually, a lot of the time, death is less harsh because the standard is to dump the inventory into a bone block that the player can retrieve after respawning and returning to their place of death. Furthermore, players can have more inventory lists than "main" and "craft". For example, in any server with unified_inventory, items in the player's bags will be safe, so the die command would quite likely become a "restore 20 hp and escape all danger at will" command.
Also, your die command is weird. It looks like something that you were seriously overthinking. It should be as simple as setting the user's hp to 0. Doing that automatically kills players, no questions asked. The if statement spaghetti is unnecessary, and assumes too much about the currently played game. Players can have max hp values other than 20. Death can have consequences other than clearing the inventory and being sent specifically to spawn. Actually, a lot of the time, death is less harsh because the standard is to dump the inventory into a bone block that the player can retrieve after respawning and returning to their place of death. Furthermore, players can have more inventory lists than "main" and "craft". For example, in any server with unified_inventory, items in the player's bags will be safe, so the die command would quite likely become a "restore 20 hp and escape all danger at will" command.
- Pyrollo
- Developer
- Posts: 385
- Joined: Mon Jan 08, 2018 15:14
- GitHub: pyrollo
- In-game: Naj
- Location: Paris
Re: Lamby
Hi WilLiam12,
You can follow these guidelines to set up your post : viewtopic.php?f=11&t=1271
They will make it much more readable and it will be required when you'll want the topic to be moved to Mods release.
You can follow these guidelines to set up your post : viewtopic.php?f=11&t=1271
They will make it much more readable and it will be required when you'll want the topic to be moved to Mods release.
[ Display Modpack ] - [ Digiterms ] - [ Crater MG ] - [ LATE ]
Who is online
Users browsing this forum: No registered users and 31 guests