[Plugin] CHATIX - Kolejny darmowy chat dla twojego serwera

[Fennec]

­Hi

napisałem sobie taki plugin na chat, jakby ktoś chciał, na prosty chat.

 

Komendy:

/chatix - info

/chatix reload - reload configu (nie działa lol)

/chatix on - włączenie chatu

/chatix off - wyłączenie chatu

/chatix toggle - Włącz/Wyłącz czat

 

Uprawnienia:

- chatix.bypass - pozwala pisać gdy chat wyłączony

- chatix.admin - komendy on/off/toggle/reload

 

Config:

tag: 'twój tag przed wiadomościami'

W configu można używać tych znaczków & - nie trzeba paragrafów. 

 

To-do:

- slowmode

- cenzura

- automsg

- konfiguracja wiadomości

 

DL:

https://mega.nz/#!71Rk3QLb!bYp60MhppllFjUcnk9x_gLEV9_1EN_1sx3l1Qlx7m9U

Skan:

https://www.virustotal.com/pl/file/c6fc7afa9e2d828df24e1db82a7946c3fbc36999953e486964c22d4a7b7b003c/analysis/1464021602/

Github:

https://github.com/draconite/Chatix

[AdamGrzegorz]

1. Popracuj nad nazewnictwem, nazw klas nie rozpoczynamy małą literą (takie zasady Oracle, a zasady są po to żeby ich nie łamać) i ogólnie metody są ponazywane strasznie c*****o.

2. 

if(plugin.chat_status==true){ 

Chyba nie muszę tłumaczyć?

3. 

 if(p instanceof Player){

a jak nie? To co? Nic nie zrobi?

4. Lepiej zrobić switcha /chat, zamiast /chat on/off

5. 

         if (plugin.chat_status == true) {
                            turnOffChat(p);
                        } else {
                            turnOnChat(p);
                        } 

nie lepiej po prostu

plugin.chat_status ? turnOffChat(p); : turnOnChat(p);

6. To jest dziwny sposób loadowania configu, radzę zrobić to tak:

saveDefaultConfig();
reloadConfig() albo loadConfig() nie pamietam 

7. Po co to istnieje?

    @Override
    public void onDisable(){}

8. Po co to istnieje (j.w)

private main plugin; 

9. Po co to istnieje (j.w x2)

    public void onPlayerJoin(PlayerJoinEvent e){
        Player p = e.getPlayer();

    }

10. 

                        for (int i = 0; i < 100; i++) {
                            Bukkit.broadcastMessage("");
                        } 

Błagam cię, i za każdym czyszczeniem konsola będzie miała dosłownie ROZJEBANE miejsce pustymi linijkami a w logach gówno? Testował to ktoś?

Ogólnie to kod do poprawy.

 

#ps

Nie będę już wspominał o tym, że takich pluginów było masa... i to jeszcze na 1.7.10. Dlaczego nie 1.9? Dlaczego? Powie mi to ktoś k***a? ragequit

­

[.D3verSon.]

Kolejny raczej nie potrzebny plugin na chat. Gościu tego jest pełno na internetach (bukkit, spigot), wymyśl coś kreatywniejszego, sprawdź, czy czasem nie ma takich plg wielu i wtedy udostępniaj.

[Fennec]

@AdamGrzegorz trochę naprawiłem, do konsoli nie pluje już 100 lini.­ Jest na gicie. Niestety ten twój solution nie działa :/ (short if)

 

//Działa już ;p

[MrFretko]

Co do pluginu nic szczegolnego i kod mi troszke podchodzi kodem od kamilkime
@topic
Dodaj własne fortmatowanie czatu z opcją wyłączenia i właczenia tego

[AdamGrzegorz]

@AdamGrzegorz trochę naprawiłem, do konsoli nie pluje już 100 lini.­ Jest na gicie. Niestety ten twój solution nie działa :/ (short if)

 

//Działa już ;p

Będę na githubie ci pisał pod różnymi linijkami bo nie chce mi się tutaj tego kopiować i pisać. ;p

 

­

[bartzz]

@AdamGrzegorz, 5. -> to ma być toggle? Nie lepiej setChatStatus(!getChatStatus) czyli przeciwieństwo?

 

@topic

Fajny plug-in

[AdamGrzegorz]

@AdamGrzegorz, 5. -> to ma być toggle? Nie lepiej setChatStatus(!getChatStatus) czyli przeciwieństwo?

 

@topic

Fajny plug-in

Ale o co ci chodzi? Przecież można

public boolean getChat() {
   return !jakaszmiennabool;
}

i się z tym zgadzam przecież, zmieniłem tylko kod żeby się nauczył że czasami robienie multum ifów jest gorsze niż jedna linijka operatora warunkowego. A to że to jest dziwnie zrobione to inna sprawa, zresztą na githubie już mu napisałem jak tego toggla zrobić ;p

https://github.com/dracconi/Chatix/commit/cb9a12d5de4cb1bfa34f2a8593cfe3219443532c#diff-3be38ac4284242337fdf1cb034237a1cR26

­