[Plugin] Particle Capes

[IdkMan]

Mój pierwszy plugin na spigocie: Particle Capes

Co tu dużo gadać? Wszystko jest pięknie opisane na spigocie

Klikać w link, myszki nie żałować! :]

[SPIGOT LINK

 

[Nocny]

no to tak, plugin całkiem spoko, ale jest kilka rzeczy, które warto poprawić:

1. klasa Packets mogłaby być statyczna, bo to tylko wysyłanie pakietów i tworzenie, więc nie ma sensu tego trzymać jako obiekt.
2. plugin jest mało abstrakcyjny, przez co rozbudowa w późniejszym czasie będzie utrudniona
3. klasa ServerVersion nie jest w ogóle potrzebna, nadmiar kodu który jest useless
4. do zmiennych używa się camelCase, a nie "_" :<
5. package scope leży i kwiczy, trzeba poprawić
6. zmienna configVersion nie jest do niczego potrzebna, więc nie wiem po co istnieje
7. no i ostatnie, plugin jest mało rozbudowany! dodaj funkcje, feature itd :<

[IdkMan]

7 godzin temu, Nocny napisał:

no to tak, plugin całkiem spoko, ale jest kilka rzeczy, które warto poprawić:

1. klasa Packets mogłaby być statyczna, bo to tylko wysyłanie pakietów i tworzenie, więc nie ma sensu tego trzymać jako obiekt.
2. plugin jest mało abstrakcyjny, przez co rozbudowa w późniejszym czasie będzie utrudniona
3. klasa ServerVersion nie jest w ogóle potrzebna, nadmiar kodu który jest useless
4. do zmiennych używa się camelCase, a nie "_" :<
5. package scope leży i kwiczy, trzeba poprawić
6. zmienna configVersion nie jest do niczego potrzebna, więc nie wiem po co istnieje
7. no i ostatnie, plugin jest mało rozbudowany! dodaj funkcje, feature itd :<

1. Poprawię

2. Nie sądzę, aczkolwiek postaram się poprawić

3. Tak, racja, dam to do innej klasy po prostu, choć uwierz mi, że nie powinno być różnicy czy jest w tej czy oddzielnej klasie oprócz 0,001kb dodatkowego kodu

4. Nie wiem co to camelCase, wygoogluję potem

5. Nie mam pojęcia co to package scope, jeśli chodzi o ułożenie klas w pakietach, to po prostu tak jest wygodnie

6. configVersion jest na przyszłość, specjalnie zaprojektowana wcześniej

7. Oczywiście, że dodam i to najszybciej jak to możliwe, to dopiero pierwsza wersja w trakcie budowy.

Dzięki za porady.

[IdkMan]

Bump update 0.2, check it out

https://www.spigotmc.org/resources/particle-capes-1-9-1-10-1-11-1-12.42381/updates

[BoeingC135]

Fajny pomysl na plugin

[KamilMinecraftBalloon]

Słabe , tak jak ten twój iberion , SebiX  i IdkMan to nooby

[IdkMan]

@bump

Aktualizacja 0.3 z masą usprawnień i nowości

[SPIGOT LINK]

 

[Blackovskyy]

Ciekawe, użyję

[WeliCraft]

10/10

[Gigoss07]

Fajne, fajne