Nie działający skrypt na topki

Jeżeli robisz sobie to ręcznie to ta druga funkcja jest zbędna.

loop {_topki::*}:
	set {_list::*} to loop-value split at " ---> "

Ogólnie co do błędu to masz tylko jednego gracza na serwerze ze zmienną więc jest tylko jeden wynik. Dalsze wyniki topki będą wynosić null (none). Powtarzający się wynik '91970' związany jest ze zmienną, która nie jest czyszczona w kodzie funkcji, postaram się to poprawić w najbliższym czasie i opublikować kolejną wersję skryptu.

W twoim przypadku wystarczy sprawdzać czy dany wynik topki istnieje i jeżeli nie to wyświetlać jakiś tekst o braku:

on load:
    set {symbol} to " ---> "	
function createTop(t: string, i: integer) :: strings:
    set {_offlinePlayers::*} to all offline players
    loop {_offlinePlayers::*}:
        if {%{_t}%::%loop-value%} is set:
            add {%{_t}%::%loop-value%} to {_top::*}
            add loop-value to {_players::*}
    loop {_i} times:
        set {_max} to max({_top::*})
        loop {_top::*}:
            if loop-value-2 = {_max}:
                set {_index} to loop-index
                delete {_top::%{_index}%}
        add "%{_players::%{_index}%}%%{symbol}%%{_max}%" to {_return::*}
        delete {_players::%{_index}%}
    return {_return::*}

on inventory click:
    if "%inventory name of player's current inventory%" is "&8[ &6Topki Serwerowe &8]":
        cancel event		

options:
	format: &7{TOP}. &f{NICK} &8» &6{VALUE}
command /top [<text>]:
	trigger:
		open chest with 1 rows named "&8[ &6Topki Serwerowe &8]" to player
		set {_topki::*} to createTop("coins", 8)
		set {_i} to diamond named "&eTop posiadanych coinsow"
		loop {_topki::*}:
			set {_list::*} to loop-value split at " ---> "
			set {_f} to "{@format}"
			replace all "{TOP}" with "%loop-index%" in {_f}
  			if {_list::1} is set: 
				replace all "{NICK}" with "%{_list::1}%" in {_f}
				replace all "{VALUE}" with "%{_list::2}%" in {_f}
			else:
				replace all "{NICK}" with "Brak" in {_f}
				replace all "{VALUE}" with "Wyników" in {_f}
  			delete {_list::*} #Tak na wszelki wypadek
			set line (loop-index parsed as integer) of lore of {_i} to colored {_f} #numer pętli oszczędzi ci dodatkowych zmiennych i liczenia linijek
		set player's current inventory's slot 0 to {_i}

To powinno zadziałać.

Nie dziala

http://prntscr.com/oj273s

12 godzin temu, LuckyPotato napisał:

Jeżeli robisz sobie to ręcznie to ta druga funkcja jest zbędna.

loop {_topki::*}:
	set {_list::*} to loop-value split at " ---> "

 

Ogólnie co do błędu to masz tylko jednego gracza na serwerze ze zmienną więc jest tylko jeden wynik. Dalsze wyniki topki będą wynosić null (none). Powtarzający się wynik '91970' związany jest ze zmienną, która nie jest czyszczona w kodzie funkcji, postaram się to poprawić w najbliższym czasie i opublikować kolejną wersję skryptu.

W twoim przypadku wystarczy sprawdzać czy dany wynik topki istnieje i jeżeli nie to wyświetlać jakiś tekst o braku:

on load:
    set {symbol} to " ---> "	
function createTop(t: string, i: integer) :: strings:
    set {_offlinePlayers::*} to all offline players
    loop {_offlinePlayers::*}:
        if {%{_t}%::%loop-value%} is set:
            add {%{_t}%::%loop-value%} to {_top::*}
            add loop-value to {_players::*}
    loop {_i} times:
        set {_max} to max({_top::*})
        loop {_top::*}:
            if loop-value-2 = {_max}:
                set {_index} to loop-index
                delete {_top::%{_index}%}
        add "%{_players::%{_index}%}%%{symbol}%%{_max}%" to {_return::*}
        delete {_players::%{_index}%}
    return {_return::*}

on inventory click:
    if "%inventory name of player's current inventory%" is "&8[ &6Topki Serwerowe &8]":
        cancel event		

options:
	format: &7{TOP}. &f{NICK} &8» &6{VALUE}
command /top [<text>]:
	trigger:
		open chest with 1 rows named "&8[ &6Topki Serwerowe &8]" to player
		set {_topki::*} to createTop("coins", 8)
		set {_i} to diamond named "&eTop posiadanych coinsow"
		loop {_topki::*}:
			set {_list::*} to loop-value split at " ---> "
			set {_f} to "{@format}"
			replace all "{TOP}" with "%loop-index%" in {_f}
  			if {_list::1} is set: 
				replace all "{NICK}" with "%{_list::1}%" in {_f}
				replace all "{VALUE}" with "%{_list::2}%" in {_f}
			else:
				replace all "{NICK}" with "Brak" in {_f}
				replace all "{VALUE}" with "Wyników" in {_f}
  			delete {_list::*} #Tak na wszelki wypadek
			set line (loop-index parsed as integer) of lore of {_i} to colored {_f} #numer pętli oszczędzi ci dodatkowych zmiennych i liczenia linijek
		set player's current inventory's slot 0 to {_i}

To powinno zadziałać.

tak samo jest