Skocz do zawartości

Rekomendowane odpowiedzi

Ogólnie pomysł bardzo fajny i wykonanie nienajgorsze tylko nieoptymalne
Drobne uwagi:

  • Ładowanie wszystkiego do yml to raczej nie jest za dobry pomysł, zwłaszcza ciągłe otwieranie i zapisywanie
  • Itemy w yml, mogłeś to na normalnych zmiennych zrobić, byłoby to wydajniejsze i lepsze rozwiązanie (nawet jeden dodatek by ci odpadł)
  • Mógłbyś też zamiast ciągle tworzyć nowe inventory każdemu graczowi, zrobic jedno i je otwierać (przy aktualizacji automatycznie każdemu graczowi by się zmieniło)
Spoiler

Możesz też niektóre rzeczy zrobić prościej np

set {_li} to size of (yaml node keys "list" from "rynek.data")
set slot numbers between 0 and 53 of loop-value-1's current inventory to air

Propozycje

  • Jakiś config (do tego użyj yml a nie itemów xD)
  • Schowek na zakupione przedmioty
Spoiler
Godzinę temu, wisienak napisał:

- wygląd prezentacji skryptu lekko inspirowany xAxee uIPLnsu.jpg

:kappaseems:

 

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324037
Udostępnij na innych stronach

13 godzin temu, xAxee napisał:
  • Ładowanie wszystkiego do yml to raczej nie jest za dobry pomysł, zwłaszcza ciągłe otwieranie i zapisywanie
  • Itemy w yml, mogłeś to na normalnych zmiennych zrobić, byłoby to wydajniejsze i lepsze rozwiązanie (nawet jeden dodatek by ci odpadł)

Główny powód dlaczego wykorzystałem YAML do zapisywania przedmiotów to brak pomysłu na zapisywanie reszty danych typu: cena, sprzedawca w zmiennej. Gdy próbowałem wszystko zapisać w jednej zmiennej jako string i odseparować to znakami, praser nie czytał jakiejś tam wartości (już nie pamiętam czy to była nazwa czy lore) z stringu, dlatego zdecydowałem się na zapis w yamlu. Pierwotnie dane zapisywałem w oddzielnych "nodach?" i to się sprawdzało, ale po dłuższym testowaniu zauważyłem że enchantowane książki nie zapisują się, a jedynym rozwiązaniem są nbt tagi, więc zdecydowałem się przenieść wszystkie wartości do nbt.

 

13 godzin temu, xAxee napisał:

Mógłbyś też zamiast ciągle tworzyć nowe inventory każdemu graczowi, zrobic jedno i je otwierać (przy aktualizacji automatycznie każdemu graczowi by się zmieniło)

Fakt jest to skuteczniejsze rozwiązanie, popracuję nad ewentualną zmianą.

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324060
Udostępnij na innych stronach

Takie pytanie, nie ma mnie w domu więc sam nie sprawdzę

Co jeżeli przedmiot ma opis? W inventory click sprawdzasz linijkę przedmiotu by zdobyć ID przedmiotu. - czy wtedy przypadkiem nie będzie to inna linia? Jeżeli gracz ustawi tą linię w opisie przedmiotu, to będzie mógł nieźle się pobawić z rynkiem 🙄

O jeszcze, linia 90 i 91 do wywalenia chyba, nie widzę byś używał tych zmiennych 🤨

Edytowane przez W1tt
Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324142
Udostępnij na innych stronach

W dniu 20.07.2022 o 13:19, wisienak napisał:

Główny powód dlaczego wykorzystałem YAML do zapisywania przedmiotów to brak pomysłu na zapisywanie reszty danych typu: cena, sprzedawca w zmiennej. Gdy próbowałem wszystko zapisać w jednej zmiennej jako string i odseparować to znakami, praser nie czytał jakiejś tam wartości (już nie pamiętam czy to była nazwa czy lore) z stringu, dlatego zdecydowałem się na zapis w yamlu. Pierwotnie dane zapisywałem w oddzielnych "nodach?" i to się sprawdzało, ale po dłuższym testowaniu zauważyłem że enchantowane książki nie zapisują się, a jedynym rozwiązaniem są nbt tagi, więc zdecydowałem się przenieść wszystkie wartości do nbt.

mogłeś po prostu zrobic o osobnych zmiennych np

{cos::%{_id}%::item}
{cos::%{_id}%::cena}
{cos::%{_id}%::opis}

 

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324144
Udostępnij na innych stronach

W dniu 21.07.2022 o 21:47, W1tt napisał:

Co jeżeli przedmiot ma opis? W inventory click sprawdzasz linijkę przedmiotu by zdobyć ID przedmiotu. - czy wtedy przypadkiem nie będzie to inna linia? Jeżeli gracz ustawi tą linię w opisie przedmiotu, to będzie mógł nieźle się pobawić z rynkiem 🙄

w gui rynku nie pokazuje lore przedmiotu tylko nadpisuje wartościami o sprzedawcy, cenie i id, ale item po zakupie/po usunięciu z rynku otrzymuje przedmiot z lore, #edit; też do poprawy, faktycznie gracz może bawić się 
 

 

W dniu 21.07.2022 o 21:47, W1tt napisał:

O jeszcze, linia 90 i 91 do wywalenia chyba, nie widzę byś używał tych zmiennych 🤨

zapomniałem usunąć linijki z czytania enchantów gdy wszystko przenosiłem do NBT

 

dzięki za uwagi 🙏

 

W dniu 21.07.2022 o 22:05, xAxee napisał:

mogłeś po prostu zrobic o osobnych zmiennych np

that's facts, ale wykorzystanie tylu zmiennych będzie bardziej optymalne niż czytanie wszystkie z pliku YAML?

nie jestem zorientowany w tej kwestii, ale korzystałem z tego skryptu na serwerze gdy było trochę itemków i nie było problemów.

fajnie byłoby zorientować się co jest najwydajniejszą opcją? korzystanie z YAML i zapisywanie wszystkich wartości do pliku i czytanie go/edytowanie, czy zabawa zmiennymi?

chodzi mi o skrypty które właśnie zawierają więcej informacji tak jak ten rynek

Edytowane przez wisienak
Szifter

Łączę posty

Notatka dodana przez Szifter

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324145
Udostępnij na innych stronach

1 godzinę temu, wisienak napisał:

that's facts, ale wykorzystanie tylu zmiennych będzie bardziej optymalne niż czytanie wszystkie z pliku YAML?

nie jestem zorientowany w tej kwestii, ale korzystałem z tego skryptu na serwerze gdy było trochę itemków i nie było problemów.

fajnie byłoby zorientować się co jest najwydajniejszą opcją? korzystanie z YAML i zapisywanie wszystkich wartości do pliku i czytanie go/edytowanie, czy zabawa zmiennymi?

chodzi mi o skrypty które właśnie zawierają więcej informacji tak jak ten rynek

oczywiście że zmienne i to nawet kilkadziesiąt razy bardziej optymalne niż yaml, przecież po to są żeby z nich korzystać ;)
 

Spoiler

 

kerOHnx.pngs.png

 

 

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324156
Udostępnij na innych stronach

Wróciłem do domu :D

Błędy:

1. Możliwość wystawienia powietrza

2. Do gui można włożyć item z swojego ekwipunku, klikając na niego trzymając shift - Po kliknięciu na ten item pojawia się na czacie informacja o zakupie <none> za <none>

3. Możliwość wystawienia przedmiotów za minusową i zerową cenę

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324370
Udostępnij na innych stronach

4 godziny temu, W1tt napisał:

Wróciłem do domu :D

Błędy:

1. Możliwość wystawienia powietrza

2. Do gui można włożyć item z swojego ekwipunku, klikając na niego trzymając shift - Po kliknięciu na ten item pojawia się na czacie informacja o zakupie <none> za <none>

3. Możliwość wystawienia przedmiotów za minusową i zerową cenę

Wszystkie błędy tutaj wymienione zostaną naprawione i sugestie podane przez xAxee co do optymalizacji zostaną wprowadzone w krótce, dzięki za przetestowanie :D 

Odnośnik do komentarza
https://skript.pl/temat/53139--/#findComment-324380
Udostępnij na innych stronach

Dołącz do dyskusji

Możesz dodać zawartość już teraz a zarejestrować się później. Jeśli posiadasz już konto, zaloguj się aby dodać zawartość za jego pomocą.

Nieaktywny
Dodaj odpowiedź do tematu...

×   Wklejono zawartość z formatowaniem.   Usuń formatowanie

  Dozwolonych jest tylko 75 emoji.

×   Odnośnik został automatycznie osadzony.   Przywróć wyświetlanie jako odnośnik

×   Przywrócono poprzednią zawartość.   Wyczyść edytor

×   Nie możesz bezpośrednio wkleić grafiki. Dodaj lub załącz grafiki z adresu URL.

  • Ostatnio przeglądający   0 użytkowników

    • Brak zarejestrowanych użytkowników przeglądających tę stronę.
×
×
  • Dodaj nową pozycję...