Skocz do zawartości
  • 👋 Witaj na MPCForum!

    Przeglądasz forum jako gość, co oznacza, że wiele świetnych funkcji jest jeszcze przed Tobą! 😎

    • Pełny dostęp do działów i ukrytych treści
    • Możliwość pisania i odpowiadania w tematach
    • System prywatnych wiadomości
    • Zbieranie reputacji i rozwijanie swojego profilu
    • Członkostwo w jednej z największych społeczności graczy

    👉 Dołączenie zajmie Ci mniej niż minutę – a zyskasz znacznie więcej!

    Zarejestruj się teraz

[DL] Autorski AntiCobbleKcr


Gość KamiloPL1456

Rekomendowane odpowiedzi

Opublikowano

Witam was, to już mój drugi plugin

 Jak sam nazwa wskazuje po tym że jak wpiszemy cmd /cobble to nam nie drop cobel do eq

                                          Następny pl będzie na generator stone

 DL: http://www.speedyshare.com/CSz5g/AntiCobbleKcr.jar

 Skan: https://www.virustotal.com/pl/file/1db4ffd77aee8267a174554540e550256af080098f786fc21e6bf5cdd9d62120/analysis/

Opublikowano

 else if (blok.getType() == Material.STONE)
{
p.getInventory().addItem(new ItemStack[] { kamien });
e.setCancelled(true);
blok.setType(Material.AIR);
}

Bez sensu.

ArrayList<String> cobble = new ArrayList();

Czemu robisz to w array, zamiast w mapce?

 

Oraz mogłeś napisać w 3 class'ach, zamiast w jednej.

+

 

public void onPlayerMove(BlockBreakEvent e)

Czemu nazwałeś to onPlayerMove? :D

733761405517115815990.png

"To, że milczę, nie znaczy, że nie mam nic do powiedzenia" ~Jonathan Carroll

Opublikowano




 else if (blok.getType() == Material.STONE)
{
p.getInventory().addItem(new ItemStack[] { kamien });
e.setCancelled(true);
blok.setType(Material.AIR);
}

Bez sensu.





ArrayList<String> cobble = new ArrayList();

Czemu robisz to w array, zamiast w mapce?

 

Oraz mogłeś napisać w 3 class'ach, zamiast w jednej.

+

 





public void onPlayerMove(BlockBreakEvent e)

Czemu nazwałeś to onPlayerMove? :D

 

On tego nie pisał sam, to plugin ściągnięty z neta zmienił tylko nazwe package. Zobacz jego poprzedni "autorski" plugin.

CPU: i5 6500 / RAM: 8GB / GPU: GTX 1070 / MOBO: MSI H110M / HDD: 1TB / SSD: Iridium PRO 

BLACKPRISON.HOSTMC.PL - najlepszy serwer prison w Polsce

Opublikowano

On tego nie pisał sam, to plugin ściągnięty z neta zmienił tylko nazwe package. Zobacz jego poprzedni "autorski" plugin.

wes sie zamknij

Opublikowano

Witam was, to już mój drugi plugin

 Jak sam nazwa wskazuje po tym że jak wpiszemy cmd /cobble to nam nie drop cobel do eq

                                          Następny pl będzie na generator stone

 DL: http://www.speedyshare.com/CSz5g/AntiCobbleKcr.jar

 Skan: https://www.virustotal.com/pl/file/1db4ffd77aee8267a174554540e550256af080098f786fc21e6bf5cdd9d62120/analysis/

Czyli generator stone napisz na podstawie moich stoniarek, które są open source? xDDDD

 

 

Ps. Zbieram warny! :D

 

Opublikowano

Czyli generator stone napisz na podstawie moich stoniarek, które są open source? xDDDD

 

 

Ps. Zbieram warny! :D

 

Nie?

Opublikowano

Plugin to czysty plagiat.
Z resztą, tak jak twój poprzednik. A wiemy, co mpc robi z plagiatami.
Pozdrawiam.

733761405517115815990.png

"To, że milczę, nie znaczy, że nie mam nic do powiedzenia" ~Jonathan Carroll

Opublikowano

Plugin to najprawdopowniej plagiat... można napisać lepsze i krótsze w 3 minuty...
 
Dobra, przejdźmy do kodu...
 
Upchane jak nie wiem, dobra mniej ważne, niż to co napiszę zaraz...
 

ArrayList<String> cobble = new ArrayList();

Hm... nie wiem po co tutaj ArrayList, można tu zastąpić ArrayList normalną listą.
 

    if (this.cobble.contains(p.getName()))
    {
      if (blok.getType() == Material.STONE)
      {
        e.setCancelled(true);
        blok.setType(Material.AIR);
      }
    }
    else if (blok.getType() == Material.STONE)
    {
      p.getInventory().addItem(new ItemStack[] { kamien });
      e.setCancelled(true);
      blok.setType(Material.AIR);
    }

 


9419314034671541012418.jpeg


Nie wiem, po co coś takiego... else if nie potrzebny w ogóle, i po co wepchnąłeś this w this.cobble.contains(p.getName()) ? I dlaczego nazwałeś ten void onPlayerMove? :v
Oto przykład bardziej wydajnego i krótszego kodu(bez jakiegoś setowania, po co to?!):

List<String> cobble;
@EventHandler
public void blockbreak(BlockBreakEvent e)
{
    if(e.getBlock().getType() == Material.STONE)
    {
	if(cobble.contains(e.getPlayer().getName()))
        {
	    e.getBlock().getDrops().clear();
	}
    }
}

I zgaduję, że teraz użyje mojego kodu xP

Reszta kodu nawet ok, choć w onCommand() jest kilka błędów.

 

Następnym razem nie udostępniaj beznadziejnych pluginów, które mają po jedną klasę... takie pluginy  są do napisania w 3 minuty...

Staraj się pisać coś rozbudowanego, bo takie 66 linijek to strata zasobów.

 

Pozdrawiam :)

f

Opublikowano
ArrayList<String> cobble = new ArrayList();

Hm... nie wiem po co tutaj ArrayList, można tu zastąpić ArrayList normalną listą.

A po co stowować listę, jeśli lepiej stosować w tym przypadku Mapy?

Tak to musi sprawdzać całą listę, przypuszczając że wejdzie z 200 graczy i użyje tego, to lepiej jest użyć indywidualnej zmiennej dla gracza.

(przynajmniej tak mi się wydaje)

733761405517115815990.png

"To, że milczę, nie znaczy, że nie mam nic do powiedzenia" ~Jonathan Carroll

Opublikowano

Plugin to najprawdopowniej plagiat... można napisać lepsze i krótsze w 3 minuty...

 

Dobra, przejdźmy do kodu...

 

Upchane jak nie wiem, dobra mniej ważne, niż to co napiszę zaraz...

 

ArrayList<String> cobble = new ArrayList();

Hm... nie wiem po co tutaj ArrayList, można tu zastąpić ArrayList normalną listą.

 

    if (this.cobble.contains(p.getName()))
    {
      if (blok.getType() == Material.STONE)
      {
        e.setCancelled(true);
        blok.setType(Material.AIR);
      }
    }
    else if (blok.getType() == Material.STONE)
    {
      p.getInventory().addItem(new ItemStack[] { kamien });
      e.setCancelled(true);
      blok.setType(Material.AIR);
    }

 

 

9419314034671541012418.jpeg

 

 

Nie wiem, po co coś takiego... else if nie potrzebny w ogóle, i po co wepchnąłeś this w this.cobble.contains(p.getName()) ? I dlaczego nazwałeś ten void onPlayerMove? :v

Oto przykład bardziej wydajnego i krótszego kodu(bez jakiegoś setowania, po co to?!):

List<String> cobble;
@EventHandler
public void blockbreak(BlockBreakEvent e)
{
    if(e.getBlock().getType() == Material.STONE)
    {
	if(cobble.contains(e.getPlayer().getName()))
        {
	    e.getBlock().getDrops().clear();
	}
    }
}

I zgaduję, że teraz użyje mojego kodu xP

 

Reszta kodu nawet ok, choć w onCommand() jest kilka błędów.

 

Następnym razem nie udostępniaj beznadziejnych pluginów, które mają po jedną klasę... takie pluginy  są do napisania w 3 minuty...

Staraj się pisać coś rozbudowanego, bo takie 66 linijek to strata zasobów.

 

Pozdrawiam :)

"Hm... nie wiem po co tutaj ArrayList, można tu zastąpić ArrayList normalną listą."

A co według ciebie jest normalną listą? :D

Bo pierdzielisz od rzeczy

 

 

Plugin oczywiście lipny... ale i tak rozwalił mnie pan wyżej @MikolajGra od "Normalnej listy" :D

 

 

A po co stowować listę, jeśli lepiej stosować w tym przypadku Mapy?

Tak to musi sprawdzać całą listę, przypuszczając że wejdzie z 200 graczy i użyje tego, to lepiej jest użyć indywidualnej zmiennej dla gracza.

(przynajmniej tak mi się wydaje)

Mapę z String i boolean? możesz mi powiedzieć czym to się ma niby różnić? :D nawet to nie ma sensu robić mapki dla booleana.

 

 

A ogólnie jak już używać jakiś list/mapek i innych kolekcji itd, to powinien użyć HashSet.

Widzę że nalot expertów

To już jest koniec smerfa:


http://www.mpcforum.pl/topic/1323530-info-znikam/


GG: 48522543


PS: Na innych forach i stronach znajdziesz mnie pod nickiem: 


BukkitSmerf

Zarchiwizowany

Ten temat przebywa obecnie w archiwum. Dodawanie nowych odpowiedzi zostało zablokowane.

×
×
  • Dodaj nową pozycję...