jSignSpy - szpieguj tabliczki!

DOWNLOAD

Hejo, przekonwertowałem skrypt jahumena na plugin

Możecie go ocenić? Dobrze go zrobiłem?

V 1.0
 

V 1.1

TODO:

Zapisywanie do pliku

ChangeLog:

V 1.0: wydanie pluginu
V 1.1:
- Usunięcie klasy SpyPlayer, 
- Dodanie Listy graczy do głównej klasy, 
- Posortowanie klas w pakiety, 
- Dodano zapisywanie tabliczek do pliku (plugins/SignSpy/createdSigns.log)

Łatwiej było zrobić hashmapkę z <Player, boolean>, a tak tworzysz niepotrzebnie nowe metody.

Jak już coś to klasy w których są zawarte metody.

No ale cóż, muszę przyznać, że prościej

public class SpyPlayer
  extends ArrayList<Player>

takie coś nie ma sensu  i nazwa jest myląca, jak by miało reprezentować jednego gracza, a to jest tylko lista.

Tym bardziej takie coś:

  public ArrayList<Player> getList()
  {
    return this;
  }

po co zwracać to samo co jest, nic to nie zmienia.

  public boolean isSet(Player p)
  {
    for (Player in : this) {
      if (in.getName().equalsIgnoreCase(p.getName())) {
        return true;
      }
    }
    return false;
  }

a do takich operacji są lepsze mapki i zwyczajnie sety, trzymasz tam też graczy, więc wystraczy tylko .contains

I no właśnie, trzymasz tam graczy, a to zły pomysł, bo jak gracz wyjdzie z serwera, to dalej zostaje tam - i zajmuje to pamięć.

5 minut temu, knugi napisał:

Jak już coś to klasy w których są zawarte metody.

No ale cóż, muszę przyznać, że prościej

No może i tak, ale to jest jak szukanie czegoś na siłę, skoro wiemy o co chodzi

17 godzin temu, KrejzolekPRO napisał:

No może i tak, ale to jest jak szukanie czegoś na siłę, skoro wiemy o co chodzi

Takie coś ma duże znaczenie, powiedzmy ,że jest to ogromny projekt na githubie, np. sektory, robisz sobie forka i je przerabiasz, i jak wszystko znajdziesz skoro to jest tak ponazywane?

Co do tematu:

Ja nie rozumiem tego:

                    this.main.getClass();
                    s.sendMessage(String.valueOf("§9jSignSpy §8> §r ") + "§cNie posiadasz uprawnien signspy.use!");

nie rozumiem, po co to getClass()? (Nie wspomne już nawet o String.valueOf)

No i też nazywaj dobrze klasy i je układaj w dobrych package.

Wszystko masz pod jednym package, przy wiekszym projekcie sie zaczniesz gubić, tak samo z nazywaniem klas.

final Player player4 = p;

na co to?

                final Player p = Bukkit.getPlayer(args[0]);
                if (p == null) {

lepiej jest uzywac getPlayerExact (troche za duzo sie czepiam)

No i z tym obiektem SpyPlayer troche przesadziles.

Wystarczyła by jakaś lista/mapka w głownej klasie.

53 minuty temu, yooniks napisał:

Ja nie rozumiem tego:

                    this.main.getClass();
                    s.sendMessage(String.valueOf("§9jSignSpy §8> §r ") + "§cNie posiadasz uprawnien signspy.use!");

nie rozumiem, po co to getClass()? (Nie wspomne już nawet o String.valueOf)

54 minuty temu, yooniks napisał:

final Player player4 = p;

na co to?

                final Player p = Bukkit.getPlayer(args[0]);
                if (p == null) {

 

Nie wiem skąd to wziąłeś ale tego nie mam w swoim kodzie.

Co do drugiego cytatu to u mnie nie ma final, a o metodzie getPlayerExact nawet nie wiedziałem

A co do klasy SpyPlayer w ogóle miałem inny zamiar, lecz po paru sekundach postanowiłem zrobić to na ArrayList i nie zmieniłem nazwy klasy. Oczywiście w update w ogóle ją usunę i w klasie Main dam HashMapę.

5 minut temu, knugi napisał:

 

Nie wiem skąd to wziąłeś ale tego nie mam w swoim kodzie.

Co do drugiego cytatu to u mnie nie ma final, a o metodzie getPlayerExact nawet nie wiedziałem

 

A co do klasy SpyPlayer w ogóle miałem inny zamiar, lecz po paru sekundach postanowiłem zrobić to na ArrayList i nie zmieniłem nazwy klasy. Oczywiście w update w ogóle ją usunę i w klasie Main dam HashMapę.

 

Uroki dekompilatora.

4 godziny temu, KrejzolekPRO napisał:

Uroki dekompilatora.

Racja

Zaktualizowano plugin do wersji 1.1. ChangeLog w temacie