mAdminChat

hej to mój pierwszy skrypt od ~2 lat. wiem że jest bardzo prosty ale po 1 nie miałem pomysłu a po 2. nie umiałem :((

planuje kiedyś dodać kilka rzeczy takich jak: 

• logi 
• lepszy wygląd
• force disable/enable dla osob ze specjalną permisją

pozdrawiam :DD

1.  A co tu to robi?

               else:
                   stop

   Usuń to, bo jest niepotrzebne.

2. Po co robisz coś takiego?
    

           loop all players:
               if loop-player has permission {@permisja}:
                   send "%player% %coloured message%" to loop-player

   Nie lepiej tak?

   on chat:
   	if {adminchat::%player%} is true:
   		loop all players where [input doesn't have permission {@permisja}]:
   			remove loop-player from chat recipients

    

wprowadziłem funkcję force-adminchat <nick> która pozwala włączyć komuś adminchat. Zmienił się system sprawdzania permisji adminczatu, teraz nie trzeba miec permisji do czytania adminczatu, tylko trzeba miec go włączonego. 

update (caly czas ten sam link): 

@Michix nie wiedziałem że tak sie da ;D

1. Permisję w komendzie (tak samo 'permission command') nie trzeba podawać w cudzysłowie.

2. Skrypt nie zadziała poprawnie, gdyż w pętli 'loop all players' zatrzymujesz skrypt gdy zapętlany gracz nie posiada ustawionej zmiennej. Tak więc usuń dwie ostatnie linijki z tej pętli i wszystko będzie dobrze.

3. W komendzie /force-adminchat powinieneś dodać parametr 'executable by: players' lub zamienić w wiadomości %player% na %sender%, aby nie było "<none>" w przypadku gdyby konsola była senderem. Możesz również dodać w kodzie sprawdzanie czy sender to konsola, czy też gracz i w zależności od tego dać różne wysyłane wiadomości.

Teraz, Kormic napisał:

1. Permisję w komendzie (tak samo 'permission command') nie trzeba podawać w cudzysłowie.

2. Skrypt nie zadziała poprawnie, gdyż w pętli 'loop all players' zatrzymujesz skrypt gdy zapętlany gracz nie posiada ustawionej zmiennej. Tak więc usuń dwie ostatnie linijki z tej pętli i wszystko będzie dobrze.

3. W komendzie /force-adminchat powinieneś dodać parametr 'executable by: players' lub zamienić w wiadomości %player% na %sender%, aby nie było "<none>" w przypadku gdyby konsola była senderem. Możesz również dodać w kodzie sprawdzanie czy sender to konsola, czy też gracz i w zależności od tego dać różne wysyłane wiadomości.

Dzięki! Jutro poprawie i wrzuce poprawke ;D

Jak na skrypt pisany po przerwie - jest całkiem ok.

Mam pewną propozycję:

send "admin %player% wlaczyl ci admin czat" to arg 1

Ta oto linijka kodu przedstawiona powyżej wysyła brzydki biały napis
Może by tak trochę tchnąć życie w twój skrypt i dodać trochę kolorków?
No nie wiem, chociażby coś takiego:

send "&7Admin &a%player% &7wlaczyl ci admin czat!" to arg 1

No tutaj koledzy wyżej się wypowiedzieli, lecz ja bym chciał zaproponować coś, co chyba tu jest rzadkie w takich skryptach:

Pisanie na AdminChat nie z komendy, ale z napisania na chacie wiadomości przed którą jest znak, np "!". 

No to czyli na przykład wygląda to tak:

!Ej, ten xXkoks_IgorXx jest jakiś podejrzany

No skoro dopiero zaczynasz, może to być trudne, bo ja sam ostatnio robiłem taki skrypt i nawet ja nie miałem pomysłu na naprawienie jednego błędu Ewentualnie mógłbym Ci z tym lekko pomóc.

13 minut temu, JkodGames napisał:

No tutaj koledzy wyżej się wypowiedzieli, lecz ja bym chciał zaproponować coś, co chyba tu jest rzadkie w takich skryptach:

Pisanie na AdminChat nie z komendy, ale z napisania na chacie wiadomości przed którą jest znak, np "!". 

No to czyli na przykład wygląda to tak:

!Ej, ten xXkoks_IgorXx jest jakiś podejrzany

No skoro dopiero zaczynasz, może to być trudne, bo ja sam ostatnio robiłem taki skrypt i nawet ja nie miałem pomysłu na naprawienie jednego błędu Ewentualnie mógłbym Ci z tym lekko pomóc.

 

Dobrze, a co w przypadku gdy chcesz rozpocząć wiadomość od wykrzyknika? Tu już się pojawia problem.

W mojej ocenie komendy mimo wszystko są lepsze, nie trzeba się martwić takimi pierdółkami.

10 minut temu, Kormic napisał:

Dobrze, a co w przypadku gdy chcesz rozpocząć wiadomość od wykrzyknika? Tu już się pojawia problem.

W mojej ocenie komendy mimo wszystko są lepsze, nie trzeba się martwić takimi pierdółkami.

Właśnie to był mój problem, który rozwiązałem. 

1 minutę temu, JkodGames napisał:

Właśnie to był mój problem, który rozwiązałem. 

Ah, chyba chodzi o to że chcesz zwykła wiadomość. Ale kto używa wykrzyknika jako pierwszy znak? 

3 minuty temu, JkodGames napisał:

Właśnie to był mój problem, który rozwiązałem. 

 

Ah, chyba chodzi o to że chcesz zwykła wiadomość. Ale kto używa wykrzyknika jako pierwszy znak? 

W takich sytuacjach nie należy racjonalnie myśleć "A kto tego używa?", a trzeba mieć świadomość, że ktoś może spróbować napisać taką wiadomość.

Ale jeżeli już chcemy myśleć racjonalnie w tym przypadku...

"!!! SPRZEDAJE DIAMENTOWY MIECZ !!!" # Jakieś ogłoszenia
"!!COŚ TY ZROBIŁ!!!!!" # W skrócie jakiś młody gracz Minecrafta w stanie amoku

Jeszcze na spokojnie znalazłoby się parę przykładów.

no ale jakby sie sprawdzało czy np {adminchat::%player%} jest na true i dopiero jeżeli jest na true to wykrywa wiadomości po ! ?

25 minut temu, xMuffionex napisał:

no ale jakby sie sprawdzało czy np {adminchat::%player%} jest na true i dopiero jeżeli jest na true to wykrywa wiadomości po ! ?

Oczywiście, dałoby się

no ja wiem że by sie dało, tylko to była moja propozycja

1 minutę temu, xMuffionex napisał:

no ja wiem że by sie dało, tylko to była moja propozycja

Ah, no to możesz tak zrobić

====== 0.3 ======
Dodałem informacje przy starcie, kiedy jest dostępny update
 

1 minutę temu, xMuffionex napisał:

====== 0.3 ======
Dodałem informacje przy starcie, kiedy jest dostępny update
 

 

To ja się odniosę do update'u.

Dobrze, że zrobiłeś coś takiego jak powiadomienie o aktualizacji, ale jedna rzecz jest tutaj niepotrzebna, natomiast druga rzecz jest cholernie irytująca.

1. Niepotrzebną rzeczą jest wrzucanie wartości typu 'number' w tekst tylko po to, aby ją z powrotem zparse'ować na typ 'number'. Mam tutaj na myśli {@version}.

2. Na twoim miejscu zrobiłbym coś z tą pętlą wysyłająca powiadomienia o dostępnej aktualizacji. Jakbym miał mieć wysłane 15 linijek dot. aktualizacji na czacie, to bym się wnerwił na coś takiego.

Jeszcze mam propozycję co do powiadomienia o możliwej aktualizacji - możesz wykorzystać JSONa ze Skripta, aby zrobić klikalną wiadomość, która przenosi na stronę ze skryptem. Jeżeli nie wiesz jak to zrobić, odsyłam do poradnika Pana Marudy (JSON w Skript czy jakoś tak to się nazywało).

====== 0.4 ======
Co usunięto?
Pętle z notyfikacją o updatecie

Co zmieniono?
Wygląd wiadomości na bardziej schludny 

Co dodano?
opcje prefix która pozwala nam ustawić prefix skryptu :DD


@Kormic probowalem zrobic ten json ale cos mi nie wychodzi

2 minuty temu, xMuffionex napisał:

probowalem zrobic ten json ale cos mi nie wychodzi

No to zobacz sobie jeszcze raz ten poradnik. Próbuj do skutku.

39 minut temu, xMuffionex napisał:

====== 0.4 ======
Co usunięto?
Pętle z notyfikacją o updatecie

Co zmieniono?
Wygląd wiadomości na bardziej schludny 

Co dodano?
opcje prefix która pozwala nam ustawić prefix skryptu :DD


@Kormic probowalem zrobic ten json ale cos mi nie wychodzi

A gdzie link?

Teraz, maxcom1 napisał:

A gdzie link?

link ciągle ten sam (╯°□°）╯︵ ┻━┻