mpHub - Plugin do wyboru serwera za pomocą przedmiotu OpenSource

mpHub

... to rozbudowywany plugin wprowadzający przedmiot, dzięki któremu można zmienić serwer. Plugin posiada wiele rozmaitych możliwości. Wspiera wersje spigot 1.7.x-1.12.x

Pobierz:  KLIK

Błędy Propozycje: KLIK

Kod źródłowy: KLIK

Funkcje:

• Odświeżanie danych 
• Statusy serwerów
• Obsługuje maksymalnie 5 serwerów
• [..]

Uprawnienia:

• Brak

Prezentacja:

Gdzie link do pobrania? https://github.com/Inder00/mpHub/releases tu nic nie ma????? 

Teraz, MandMLove napisał:

Gdzie link do pobrania? https://github.com/Inder00/mpHub/releases tu nic nie ma????? 

Jest

a po co ten limit 5 serverów? a tak to ja pier... ten kod to zlepka kopiownych fragmentów, w połowie kodu taby, w drugiej połowie spacje xD do tego główna klasa z małej listery i kompletnie bez sensowne i nie mające prawa działać:

if(inst == null) return new mpHub();

.replace("&", "§")

a od tego to jest ChatColor.translate...

W klasie Server nie wiem czemu jest lista a nie np hashmap, no ale serverów jest mało to różnicy w wydajności nie będzie

wiadomości hardcoded więc nie da się zmienić ;/

  public static Config getInst()
  {
    if (inst == null) {
      return new Config();
    }
    return inst;
  }
  
  public Config()
  {
    inst = this;
  }

niezbyt to obiektowe podejście... a do tego konstruktor publiczny pomimo że to singleton, a jednocześnie nadpisuje instancje, kompletnie bez sensu, tak się nie robi.
No i masz publiczne pola, to też nie jest poprawne obiektowe podejście w javie.

    catch (IOException e)
    {
      Bukkit.getConsoleSender().sendMessage("�4Nie mozna polaczyc sie do serwera " + target);
    }

po 1, od tego jest logger, a po 2... nie wyświetlasz nigdzie błędu ani nic z nim nie robisz, więc admin nie dowie się co spowodowało ten błąd, bo jest wiele możliwości zawsze.

2 minuty temu, GotoFinal napisał:

a po co ten limit 5 serverów? a tak to ja pier... ten kod to zlepka kopiownych fragmentów, w połowie kodu taby, w drugiej połowie spacje xD do tego główna klasa z małej listery i kompletnie bez sensowne i nie mające prawa działać:

if(inst == null) return new mpHub();

.replace("&", "§")

a od tego to jest ChatColor.translate...

 

W klasie Server nie wiem czemu jest lista a nie np hashmap, no ale serverów jest mało to różnicy w wydajności nie będzie

wiadomości hardcoded więc nie da się zmienić ;/

 

  public static Config getInst()
  {
    if (inst == null) {
      return new Config();
    }
    return inst;
  }
  
  public Config()
  {
    inst = this;
  }

niezbyt to obiektowe podejście... a do tego konstruktor publiczny pomimo że to singleton, a jednocześnie nadpisuje instancje, kompletnie bez sensu, tak się nie robi.
No i masz publiczne pola, to też nie jest poprawne obiektowe podejście w javie.

 

    catch (IOException e)
    {
      Bukkit.getConsoleSender().sendMessage("�4Nie mozna polaczyc sie do serwera " + target);
    }

po 1, od tego jest logger, a po 2... nie wyświetlasz nigdzie błędu ani nic z nim nie robisz, więc admin nie dowie się co spowodowało ten błąd, bo jest wiele możliwości zawsze.

Dzięki za wychwycenie moich błędów i za poświęcony czas

13 godzin temu, GotoFinal napisał:

a po co ten limit 5 serverów?

On chciał się pochwalić że umie ustawiać itemy w hoperze poprzez plugin  

https://github.com/Inder00/mpHub/blob/master/src/pl/inder00/minepixel/hub/channels/BungeeCord.java

kod z tutorialów, "google -> bungeecord messaging channel"

zbędny kod, czasem sobie nawet myślę "jakim cudem to może działać".

no i po co ten limit 5 serwerów? XD

jeszcze te InventoryType.HOPPER...