Code Conventions
Separate complex Skript expressions from skript-mirror calls
Combining Skript expressions with skript-mirror calls may make your code difficult to read. Use variables to separate these different types of calls.
1
(the player's targeted block).breakNaturally()
Copied!
1
set {_target} to the player's targeted block
2
{_target}.breakNaturally()
Copied!
Keep the target of a skript-mirror call grouped
When calling a method or accessing a field, avoid using spaces when possible.
1
the event.getPlayer()
Copied!
1
event.getPlayer()
Copied!
If the expression is simple (i.e. does not contain other expressions) but requires a space, surround the expression in parentheses.
1
spawned creeper.isPowered()
Copied!
1
(spawned creeper).isPowered()
Copied!
If the target of the expression is not simple (i.e. contains other expressions), extract the expression into a local variable. (rule)
Variables are the exception to this rule and may contain spaces and/or other expressions
1
{my script::%player%::pet}.isDead()
Copied!
Avoid aliasing classes for aesthetic purposes
The purpose of import aliases is to avoid conflicts with other imports and expressions. Do not alias imports in order to make them look like Skript events.
1
import:
2
org.bukkit.event.player.PlayerMoveEvent as move
3
4
on move:
5
# code
Copied!
1
import:
2
org.bukkit.event.player.PlayerMoveEvent
3
4
on PlayerMoveEvent:
5
# code
Copied!
Avoid unnecessary uses of Java reflection
Especially when copying Java code and translating it for skript-mirror, you may run into instances where you need to use reflection to access a private method, field, or constructor. In skript-mirror, private members are visible and accessible by default.
1
set {_mod count field} to {_map}.getClass().getDeclaredField("modCount")
2
{_mod count field}.setAccessible(true)
3
set {_mod count} to {_mod count field}.get({_map})
Copied!
1
set {_mod count} to {_map}.modCount!
Copied!
Last modified 3yr ago