I need help with a random sword generator

[zyria]

For some odd reason my skript will not work and i can't figure it out please help
 

on load:
	clear {_swords::*}
	add "iron sword", "diamond sword" and "stone sword" to {_swords::*}
command sword:
	trigger:
		set {_random} to a random element of {_swords::*}
		give {_random} to player
		send "test"

 

Edited March 2, 2023 by zyria

[lostmatterr]

Whats not working? the command, the random sword giver, etc.

[ApexSplat]

parse the random var as an item type as you are giving the player a string rather than an item type

[Znyx]

When you're adding the items to the list you're adding strings, not items. Also, I belive that the swords variable is local and not global which means you can't access the sword variable from the sword command. I might be wrong on that though, I'm quite new to skript. 

I would belive this works:

on load:
	set {swords::*} to diamond sword, iron sword and stone sword

command /sword:
	trigger:
		set {_random} to a random element of {swords::*}
		give {_random} to player
		send "&cHere's a random sword!" to player

[ApexSplat]

3 hours ago, Znyx said:

When you're adding the items to the list you're adding strings, not items. Also, I belive that the swords variable is local and not global which means you can't access the sword variable from the sword command. I might be wrong on that though, I'm quite new to skript. 

I would belive this works:

on load:
	set {swords::*} to diamond sword, iron sword and stone sword

command /sword:
	trigger:
		set {_random} to a random element of {swords::*}
		give {_random} to player
		send "&cHere's a random sword!" to player

yes this fixes it

[Hitmorningstar]

On 3/2/2023 at 4:50 PM, zyria said:

For some odd reason my skript will not work and i can't figure it out please help
 

on load:
	clear {_swords::*}
	add "iron sword", "diamond sword" and "stone sword" to {_swords::*}
command sword:
	trigger:
		set {_random} to a random element of {_swords::*}
		give {_random} to player
		send "test"

 

on load:

    clear {_swords::*}

    add "iron sword", "diamond sword" and "stone sword" to {_swords::*}

command /sword:

    trigger:

        set {_random} to a random element of {_swords::*}

        give {_random} to player

        send "test"

[devyy]

Bro easy:
 

on load:
	clear {_swords::*}
	add "iron sword", "diamond sword" and "stone sword" to {_swords::*}

command /sword:
	trigger:
		give (random element of {_swords::*} parsed as item type) to player