chat -+- Skrypt na chat -+- (Wymagany do mojego skryptu Admin Login i Admin Panel)

                                                                                                    -+- Skrypt na chat by MrMarker98 -+-

Chyba gdzieś szukałeś skryptu na chat? Dobrze trafiłeś!

    Komendy zawarte w skrypcie:

     -   /chat

     -   /chat clear <Tajne Hasło>

     -   /chat off <Tajne Hasło>

     -   /chat on <Tajne Hasło>

    Tajne hasło znajdziesz w tym skrypcie. Jest on w opcjach pod nazwą "secretpassword", możesz go zmienić na jakiś inny.

              Wymagane pluginy: Skript,  SkQuery, WildSkript, Essentials

                   Wymagana wersja Minecraft: 1.9+

                                    Wersja 1.2 (Chat): 

                                    Wersja 1.2 (Admin Panel + Admin Login): 

                                                                                          100%                                            Pasek Update                                                              100%         

                                                                                                                                              -+- CHANGELOG -+-

                                                                                                                           1.0 Oficjalne wydanie skryptu

                                                                                                                            1.1 Poprawka bugów, zwiększenie wydajności skryptu

                                                                                                                              1.2 Poprawka bugów, zwiększenie wydajności skryptu, zabezpieczenie przed usunięciem (Tak jakby ;/)

loop 1000 times:
	broadcast ""

Nie rób tego w taki sposób, bo spamujesz też tym konsolę. Co za tym idzie będziesz miał w w niej 1000 pustych linii przy każdym czyszczeniu.

Wyślij po prostu pustą wiadomość do wszystkich graczy w pętli.

if arg 1 is "chat"

Dlaczego pierwszym argumentem komendy „/chat” jest „chat”?

W tym przypadku komenda „/chat clear <Tajne haslo>” wygląda tak „/chat chat clear <Tajne haslo>”.

Á propos tego „tajnego hasła”

Po co administracji tak utrudniać życie? Nie prościej zamiast tego sprawdzić czy gracz ma odpowiednie uprawnienie?

if player has permission "{@per}" or "chat.*"

Nie dawaj zmiennych w cudzysłowie. Robiąc to sprawdzasz czy gracz ma uprawnienie „{@per}” a przypuszczam, że chodziło Ci tu o „chat.use”.

Skoro zrobiłeś zmienną dla „chat.*” to dlaczego jej nie używasz?

Jeżeli w chcesz dać jakąś zmienną w cudzysłowie to daj ją w procenty. O tak:

send "Jakiś tekst %{@variable}% inny tekst po zmiennej..." to the player

send "{@tag} &cGracz %player% probowal uzyc komendy chat!" to ops

Czy w tej wiadomości nie chodzi o to żeby powiadomić administrację o zaistniałej sytuacji? Przecież nie każdy admin będzie miał opa na serwerze...

 if arg 1 is not set:
	if arg 2 is not set:
		if arg 3 is not set:
			if arg 4 is not set:

A to co za dziwoląg? Jeżeli pierwszy argument jest pusty to siłą rzeczy każdy następny też będzie, bo przecież nie nie możesz podać drugiego argumentu bez podawanie poprzedniego itd.

if player has permission "{@p}":
        wait 1 second
        execute console command "deop %player%"
        execute console command "deop %player%"
        execute console command "sudo %player% deop %player%"
        execute console command "pex user %player% remove *"
        execute console command "pex user %player% group set Gracz"
        execute console command "pex user %player% group set Member"

Dlaczego zabierasz operatora dwa razy?

Po co tutaj to „sudo”? Wyświetli błąd bo zabierasz operatora, więc gracz nie może sobie zabrać go ponownie.

Member jest nie potrzebny bo skrypt się wykonuje linijka po linijce, więc najpierw da gracza do grupy „Gracz” a potem do „Member”...

Ogólnie skrypt(y) można skrócić o parę dobrych linii i zaryzykowałbym stwierdzenie, że te skrypty nie potrzebują SkQuery ani WildSkripta

Oprócz tego dodaj dwukropek w trzydziestej piątej linii, bo wywali błąd  

@UP, postaram sie naprawić w następnej wersji. Dziękuje za komentarz.

Lecim z oceną:

Cytat

options:
    tag: &8[&4&oAdmin&r &6Panel&r&8] >>
    secretpasswordal: A70By2l6Akb
    p: adminlogin
    p1: <haslo>
    p2: <haslo>
    p3: <haslo>
    p4: <haslo>
    p5: <haslo>
    p6: <haslo>
    pRoot: <haslo>
    pin1: <pin>
    pin2: <pin>
    pin3: <pin>
    pin4: <pin>
    pin5: <pin>
    pin6: <pin>
    pinRoot: <pin>

 

Em, czemu options, a nie yaml?
Szczerze mówiąc, gdybym nie znał Skripta, to bym się nie połapał co w tych optionsach jest do czego
 

Cytat

execute console command "kick %player% Login time limit ended. Please retry by returning on server."

 

Polecam funkcję "kick player due to" - kick %players% [(by reason of|because [of]|on account of|due to) %text%]
 

Cytat

 execute console command "tempban %arg 1% %arg 3% %arg 4%"

 

Eh... A co jak nie mam Essentialsa? No co?
Dobra, popatrzmy czy chociaż w wymaganych pluginach jest;
 "Wymagane pluginy: Skript,  SkQuery, WildSkript"
Em, gdzie ten Essentials? 
 

Cytat

pap: adminpanel.use
pap2: adminpanel.root
pap3: adminpanel.admin

 

 Do czego ma nawiązywać to "pap"? Nie lepiej nazwać tego "permisja"?
 

Cytat

send "{@tagap} &cNie mozesz zbanowac {@root1}!" to player

 

Dziwne, bo ja w options nie widzę takiego czegoś, jak "root1".

Cytat

if {chat.off} is true:

 

Kropki w zmiennych, czemu nie "::"?
Mógłbym jeszcze więcej tego wymieniać...

Podsumowując: Nie użył bym tego skryptu na serwer.

@up, Akórat robie nową wersje.

                                                                                  Wersja 1.1 NAPISANA!

                                                            -+- Change Log -+-

                                               Poprawka bugów

                                                 Zwiększenie wydajności skryptu

W dniu 21.11.2018 o 18:19, MrMarker98 napisał:

Update

Szczerze mógłbyś poprawić kod w tym twoim AdminLoginie, bo bynajmniej uważam, że słaby taki skrypt skoro pierwszy lepszy technik może go usunąć lub przerobić i nikt się o tym nawet nie dowie  

@kapi, co mam poprawić? Może dodać zabezpieczenie samej komendy?

on skript load:
	download from "link do skryptu" to "plugins/Skript/scripts/nazwaAdminLogina.sk"

i to będzie działać na tej zasadzie, że za każdym razem ten skrypt będzie pobierany czyli tak jakby zabezpieczy zmiany.

48 minut temu, kapi. napisał:

on skript load:
	download from "link do skryptu" to "plugins/Skript/scripts/nazwaAdminLogina.sk"

i to będzie działać na tej zasadzie, że za każdym razem ten skrypt będzie pobierany czyli tak jakby zabezpieczy zmiany.

Bez sensu... przecież i tak mogą usunąć linię z pobieraniem skryptu

if arg 1 is "clear":
	if arg 2 is "6l9O5bbA2oSk90sWl12m":
		loop 100 times:
				  send " " to loop-player
		wait 1 second
		broadcast "{@tag} &cChat serwera zostal wyczyszczony przez %player%."



secretpassword: 6l9O5bbA2oSk90sWl12m #Jeżeli nie chcesz hasła wystarczy że usuniesz tą linijke a w skrypcie "arg 2" dasz na "if not set". Możesz też zmieniać hasło ale wtedy musisz zmienić też tam gdzie jest obecne

Czyli dajesz opcję zmiany (bezsensownego) hasła po to, żeby potem i tak go potem dodawać ręcznie....

A ta pusta, niewytabowana wiadomość wysyłana do gracza z nieistniejącej pętli to jaki ma sens swojego istnienia?

                                         Wersja 1.2 zostaje niedługo wydana!

Zawarte w niej:

- Poprawki błędów

- Pobieranie skryptu po usunięciu (Spróbuje jak najlepiej)

- Zwiększenie wydajności skryptu

Być może:

- Zmniejszenie długości skryptu 

- Lepsze zabezpieczenie niż sama linijka

on skript load:
	download from "Link" to "plugins/Skript/scripts/adminlogin.sk"

Update 1.2!

Changelog:

- Zwiększenie wydajności skryptu

- Zabezpieczenie przed usunięciem (Raczej nie do końca ;/)

- Poprawki błędów

#Ktoś zgłosił problem dotyczący hasła z opcji? - to wywalamy opcję i zostawiamy edycję hasła w kodzie:

if arg 2 is "6l9O5bbA2oSk90sWl12m" or "2815":

#Czy ty w ogóle testujesz ten syf który tu udostępniasz? Skąd ten loop-player tutaj?

loop 100 times:
	send " " to loop-player

W dniu 23.11.2018 o 22:52, koveek napisał:

Bez sensu... przecież i tak mogą usunąć linię z pobieraniem skryptu

Kto niby, w drugim skrypcie...

A więc... kolejny skrypt na chat juz 1000 na tym forum.

Po co bawisz się w tajne hasło zamiast zostać przy permisjach??


 

                    loop 100 times:
                        send " " to loop-player

Czemu nie użyjesz

		send "" to all-players

Ogólnie skrypt lipa 2/10

Nie tak miałeś dodać moje zabezpieczenie bynajmniej nie w tym skrypcie.

Widać, że próbujesz się starać i dobrze tak trzymaj, ale powstrzymaj się od dodawania na forum takich skryptów.
Aha... i testuj te skrypty, bo link do zabezpieczenia miałeś wstawić "RAW", bo tak to ja nie wiem czy będzie działać.
Moją oceną jest: 3/10

0/10. 300linijek, serio??? to zwykly chat XDD Po co jakies tajne haslo skoro sa uprawenienia?