km Życia z KM!

Witam chciałbym zaprezentować mój skrypt na życia z KM jestem początkującym twórcą wiem że kod jest taki sobie ale przynajmniej jest

POBIERZ!
DOWNLOAD V1: https://code.skript.pl/hSwgPQbd

DOWNLOAD V2: https://code.skript.pl/fuhtMcPn
DOWNLOAD V3: https://code.skript.pl/LiJNmaGV

CHANGELOG

- Małe poprawki (dziękuje @LjayPL789)
- Dodano polskie znaki! (dziękuje @KuzynProcesor)

Ten temat został przeniesiony.

Cytat

kod jest taki sobie

jak to taki sobie? kod jest spoko, mi się podoba. po co pisać coś skomplikowanego, skoro tego się nie ma sensu pisać tego w jakiś 'ultra-zaawansowany' sposób. polecam już na tym etapie wchodzić w temat funkcji, przyda Ci się ta wiedza do pisania wielu przydatnych skryptów - pięknym i przejrzystym kodem.

Dziękuje

kilka uwag "logicznych"

1. w komendzie /wskrzes sugeruję sprawdzać czy gracz ma bana (czy ma 0 żyć), żeby gracz nie został oscamowany
2. proponuję używać SCSS (to chyba tak się nazywało)
3. w komendzie /zycie kup można zrobić to w "jednej" linijce

   add arg-1 to {zycia.%player%}
   send "&4&lZakupiles %arg-1% zycie" to player
   set {_x} to 20 * arg-1
   execute console command "eco take %player% %{_x}%"

nie wiem czy dobrze napisałem ten kod, bo już dawno nie pisałem skryptów, ale no...
a tak to spoko

dziękuje  poprawie i dam nową wersję

6 godzin temu, boleknowak napisał:

kilka uwag "logicznych"

1. w komendzie /wskrzes sugeruję sprawdzać czy gracz ma bana (czy ma 0 żyć), żeby gracz nie został oscamowany
2. proponuję używać SCSS (to chyba tak się nazywało)
3. w komendzie /zycie kup można zrobić to w "jednej" linijce

   
   add arg-1 to {zycia.%player%}
   send "&4&lZakupiles %arg-1% zycie" to player
   set {_x} to 20 * arg-1
   execute console command "eco take %player% %{_x}%"

nie wiem czy dobrze napisałem ten kod, bo już dawno nie pisałem skryptów, ale no...
a tak to spoko

troche nie działa ponieważ ja robie to na wersji 1.14.4

Cześć, dam Ci kilka rad, aby Twoje skrypty były bardziej przejrzyste i optymalne.

• Kropki w zmiennych - {zycia.%player%} zamień na {<nazwa_skryptu>::<nazwa_zmiennej>::%player%}
• command /zycia: - poprawna wersja command /zycia [<text>]:
• on death: - jak zginie mob, to ten event i tak się zaktywuje - musisz użyc on death of player:
• execute console command "ban %player% &4&lskonczyly ci sie zycia!" - w skripcie istnieje opcja banowania - ban player due to "powod"
• execute console command "pardon %arg 1%" - poprawna wersja - unban player
• send "&4&lGracz %player% wskrzesil gracza %arg 1%" - tu raczej powinno być send "&4&lGracz %player% wskrzesil gracza %arg 1%" to all players
•  Nie rób tego w taki sposób:

•         if player have 20 diamonds:
              execute console command "pardon %arg 1%"
              remove 20 diamond from player
              send "&4&lGracz %player% wskrzesil gracza %arg 1%"
          else:
              send "nie posiadasz 20 diamentow"

  Tylko w taki:

          if player don't have 20 diamonds:
              send "nie posiadasz 20 diamentow"
              stop
           execute console command "pardon %arg 1%"
           remove 20 diamond from player
           send "&4&lGracz %player% wskrzesil gracza %arg 1%"
           stop

   

• command /zyciareset: - to samo co wyżej
• execute console command "eco take %player% 20" - poprawna forma - remove 20 from player's balance
• Jeśli w argumentach używasz cyfr to argument nie może być textem, tylko numberem lub integerem,

Mam nadzieję, że opublikujesz poprawiony kod
Pozdrawiam

Godzinę temu, LjayPL789 napisał:

Cześć, dam Ci kilka rad, aby Twoje skrypty były bardziej przejrzyste i optymalne.

• Kropki w zmiennych - {zycia.%player%} zamień na {<nazwa_skryptu>::<nazwa_zmiennej>::%player%}
• command /zycia: - poprawna wersja command /zycia [<text>]:
• on death: - jak zginie mob, to ten event i tak się zaktywuje - musisz użyc on death of player:
• execute console command "ban %player% &4&lskonczyly ci sie zycia!" - w skripcie istnieje opcja banowania - ban player due to "powod"
• execute console command "pardon %arg 1%" - poprawna wersja - unban player
• send "&4&lGracz %player% wskrzesil gracza %arg 1%" - tu raczej powinno być send "&4&lGracz %player% wskrzesil gracza %arg 1%" to all players
•  Nie rób tego w taki sposób:

• 
          if player have 20 diamonds:
              execute console command "pardon %arg 1%"
              remove 20 diamond from player
              send "&4&lGracz %player% wskrzesil gracza %arg 1%"
          else:
              send "nie posiadasz 20 diamentow"

  Tylko w taki:

  
          if player don't have 20 diamonds:
              send "nie posiadasz 20 diamentow"
              stop
           execute console command "pardon %arg 1%"
           remove 20 diamond from player
           send "&4&lGracz %player% wskrzesil gracza %arg 1%"
           stop

   

• command /zyciareset: - to samo co wyżej
• execute console command "eco take %player% 20" - poprawna forma - remove 20 from player's balance
• Jeśli w argumentach używasz cyfr to argument nie może być textem, tylko numberem lub integerem,

Mam nadzieję, że opublikujesz poprawiony kod
Pozdrawiam

 

Postaram się opublikować nowy kod   i dziękuje również pozdrawiam

Mam jeszcze kilka uwag językowych :

1. Od wersji 1.13 są wspierane polskie znaki, więc mógłbyś je dodać  
2. 

skonczyly ci sie zycia

nie posiadasz 20 diamentow

Zdania i zwroty bezpośrednie do danej osoby piszemy z dużej (litery).  
3. 

Graczu o nazwie %player% skonczyly sie zycia! i zostal zbanowany

Popraw na "Gracz" i przesuń wykrzyknik (po "zycia") na koniec  

Teraz kilka sugestii na temat samego skryptu :
1. Mógłbyś dodać konfigurację YML (albo opcje) - dla mniejszego skryptu nie stanowiłoby to problemu  
2. Te komendy od żyć (zyciadodaj, zycie kup, zyciasprawdz) mógłbyś upchnąć w jedną komendę, np. /zycia
3. Popraw całość skryptu pod SSCC
4. Dodaj możliwość ustawienia własnej komendy na banowanie i odbanowywanie, ewentualnie zrób to za pomocą Skripta:
 

ban %player% due to "123"
unban %player%

Skrypt bardzo mi się podoba, czegoś takiego jeszcze (chyba) nie było!
Tak jak @TakiJakTy liczę, że opublikujesz wersję z zasugerowanymi przeze mnie poprawkami!
Pozdrawiam i życzę coraz lepszych skryptów!  

W dniu 15.04.2020 o 10:50, KuzynProcesor napisał:

Mam jeszcze kilka uwag językowych :

1. Od wersji 1.13 są wspierane polskie znaki, więc mógłbyś je dodać  
2. 

skonczyly ci sie zycia

nie posiadasz 20 diamentow

Zdania i zwroty bezpośrednie do danej osoby piszemy z dużej (litery).  
3. 

Graczu o nazwie %player% skonczyly sie zycia! i zostal zbanowany

Popraw na "Gracz" i przesuń wykrzyknik (po "zycia") na koniec  

Teraz kilka sugestii na temat samego skryptu :
1. Mógłbyś dodać konfigurację YML (albo opcje) - dla mniejszego skryptu nie stanowiłoby to problemu  
2. Te komendy od żyć (zyciadodaj, zycie kup, zyciasprawdz) mógłbyś upchnąć w jedną komendę, np. /zycia
3. Popraw całość skryptu pod SSCC
4. Dodaj możliwość ustawienia własnej komendy na banowanie i odbanowywanie, ewentualnie zrób to za pomocą Skripta:
 

ban %player% due to "123"
unban %player%

 

Skrypt bardzo mi się podoba, czegoś takiego jeszcze (chyba) nie było!
Tak jak @TakiJakTy liczę, że opublikujesz wersję z zasugerowanymi przeze mnie poprawkami!
Pozdrawiam i życzę coraz lepszych skryptów!  

Dziękuje poprawie! Nie wiedziałem że od 1.13 już są polskie znaki O.o!

Uwaga! W kupowaniu żyć nie sprawdza czy gracza na to stać!
Troszkę do poprawy jeszcze polskie znaki:

skonczyly sie zycia

Usuń "o nazwie" - będzie bardziej poprawnie  

Graczowi o nazwie

Dziękuję za opublikowanie wersji z polskimi znakami! Liczę, że jeszcze poprawisz skrypt!
Pozdrawiam  

czy zna ktoś placeholder czy coś zeby pod tab pokazywało kto ile ma żyć?

1 minutę temu, _Piotrek_ napisał:

czy zna ktoś placeholder czy coś zeby pod tab pokazywało kto ile ma żyć?

Musiałbyś sobie zarejestrować samemu placeholder. Możesz to w prosty sposób zrobić z pomocą dodatku skript-placeholders.

Rejestrowanie masz opisane na wiki dodatku. Link do niej znajdziesz na stronie dodatku.

Teraz, Kormic napisał:

Musiałbyś sobie zarejestrować samemu placeholder. Możesz to w prosty sposób zrobić z pomocą dodatku skript-placeholders.

Rejestrowanie masz opisane na wiki dodatku. Link do niej znajdziesz na stronie dodatku.

czy to nie było by to ? %{zycia.%player%}% lub %{skript_zycia.%player%}%   ?

1 minutę temu, _Piotrek_ napisał:

czy to nie było by to ? %{zycia.%player%}% lub %{skript_zycia.%player%}%   ?

Owszem, możesz.

Wpisz komendę...

/papi ecloud download skript

...a następnie możesz użyć placeholderu.

%skript_{zycia.%player_name%}%

Powinno pomóc.

1 minutę temu, Kormic napisał:

Owszem, możesz.

Wpisz komendę...

/papi ecloud download skript

...a następnie możesz użyć placeholderu.

%skript_{zycia.%player_name%}%

Powinno pomóc.

ten skrypt nie działa

Teraz, _Piotrek_ napisał:

ten skrypt nie działa

znaczy placeholder

Teraz, _Piotrek_ napisał:

znaczy placeholder

W takim razie albo coś źle robisz, albo nie da rady tego zrobić w ten sposób.

Zarejestruj więc sobie ten placeholder i po problemie. Trudno.

1 minutę temu, Kormic napisał:

W takim razie albo coś źle robisz, albo nie da rady tego zrobić w ten sposób.

Zarejestruj więc sobie ten placeholder i po problemie. Trudno.

nie wiek co to znaczy w tym przypadku zarejestruj ale wpisalem to 

/papi ecloud download skript

Teraz, _Piotrek_ napisał:

nie wiek co to znaczy w tym przypadku zarejestruj ale wpisalem to 

/papi ecloud download skript

Najpierw miałeś wpisać komendę, a następnie użyć placeholderu...

Co do rejestrowania, napisałem ci wyżej o skript-placeholders.