Помогите Упростить код прокрутки GUI

Нужно уменьшить код
(Внимание) Опасно для психики, смотреть только профессионалам!
src - click
VT

 

Подпишусь на темку только для того чтобы увидеть, какие дибилы будут скачивать это, ради того чтоб тебе помочь..)

 

Антивирус сказал вирусня. =)[DOUBLEPOST=1486117137,1486117091][/DOUBLEPOST]https://www.microsoft.com/security/...pursint.F!cl&threatid=2147717281&enterprise=0

 

Кееееек



Ясно кто это был[DOUBLEPOST=1486117406,1486117150][/DOUBLEPOST]Список файлов показывает только один класс. кидай сюда

 

Залей на гитхаб, че как лох

 

не думаю что он умеет

 

Пусть в [соde=java] запихнет))

 

хе хе ))0)0))

 

Code:

http://pastebin.com/FiH1DRDN

Rubukkit не дал залить слишком много кода XD

 

шутки про кровь из глаз еще актуальны?

 

втф

 

ну я так понял из кода, что он хочет прокручивать вещи в инвентаре.

 

так и есть,просто не нашел вариант как можно прокручивать и сделал циклом ручной перебор

 

Code:

final static Random random = new Random(); //подключаем наш супирпупир рандом
//не забываем что рандом - это поле (вне метода), а все что ниже  - это где то в методе должно быть
        final Inventory inv = Bukkit.createInventory(null, 27, "name"); //создаем инвентарь
        final List<ItemStack> items = new ArrayList<>();
        items.add(new ItemStack(Material.ACACIA_DOOR));//заполняем массив с вещами
        for (int i=0; i<27; i++){
            if (((i>=0&&i<9)||(i>=18&&i<27))&&(i!=4&&i!=22)){
                inv.setItem(i, ItemUtil.create(35,(short)14, null));
            } else if (i==4||i==22) {inv.setItem(i, new ItemStack(Material.GLASS));} //засовываем в первую и третью строку стекло.
        }
   
        for (int i =0; i<9; i++) {
            ItemStack item = items.get(random.nextInt(items.size()));
            inv.setItem(i+9, item); //засовываем во вторую строку инвентаря вещи, благодоря рандому они берутся случайно из массива с вещами.
        }
        p.openInventory(inv);
        BukkitRunnable cani = new BukkitRunnable() { //таймер
            double i = 1;
            double time = 0;
            ItemStack winitem  = null;
            @Override
            public void run() {
                if (time>i&&time<100){
                    for (int f=9; f<17; f++) {
                        inv.setItem(f, inv.getItem(f+1)); //смещаем все вещи во второй троке на один слот влево.
                    }
                    inv.setItem(17, items.get(random.nextInt(items.size()))); //заполняем слот который оказался пустым, т.к. мы все сместили. вещь снова берем рандомную из того же массива               
                    i=i*1.125; //магия. тут я делаю что бы прокрутка сначала шла быстро, а последние несколько делала "из последних сил"
                }
                time++; //увеличиваем некий аналог "времени"
                if (p.getInventory().getName()!=genS.genStrng(plugin.cases.getString("cases."+casetype+".Name"))) {p.openInventory(inv);} //если игрок дибил и вдруг закрыл инвентарь до того как он прокрутился, то открываем его снова
                if (time==100) {
                    winitem = inv.getItem(13); //получаем из середины инвентаря вещь которую выиграл игрок
                    p.sendMessage("ты дурак, который выиграл вещь {item}".replace("{item}", inv.getItem(13).getItemMeta().getDisplayName())); //поздравляем игрока
                }
           
           
                if (time>100&&time<150&&time%10==0){ //это сделано чтобы вещь которую игрок выиграл еще немного помаргала.
                    if (time%20==0){
                        inv.setItem(13, winitem);
                    }else {
                        inv.setItem(13, null);       
                        }
                }//о да, анимация
                if (time>150){
                    p.getInventory().addItem(winitem); //выдаем игроку его вещь которую он выиграл. Тут же делаем еще кучу всего что связано с выигрышем. поздравлять можно тоже тут, но я не стали ибо "0)0)))0)"
                    p.closeInventory(); //закрываем инвентарь куда подальше
                    cancel(); //останавливаем таймер, ибо нефиг опертивку кушть
                }
            }
        };
        cani.runTaskTimer(plugin, 10, 1);//ну а это запускатор таймера. про сокр. форму не писать ибо мне такая нравится.

Вот мой код прокрутки инвентаря.

 

можешь это в мой гавноплагин вшить ? XD

 

каким куем?

 

Такое количество говнокода я ещё не видел *like*

 

а я видел

 

Писец моим глазам... Да там все переписывать нужно.
http://pastebin.com/qPsCN8vi

 

Нет, ну я понимаю, что люди говнокодят, потому что это быстрее, чем научиться писать нормально. Но наверно это не тот случай. ТС, тебе было не лень это всё писать? Вот серьёзно... 1300 строк? Что-о-о?
Ладно...
На данном этапе могу лишь посоветовать определять однотипные участки кода в отдельные методы. А различающиеся значения передавать в параметрах. Так читабельность кода заметно возрастёт, а вот размер текста уменьшится в разы.
Вот сделаешь это - кидай обновлённый паст в тему, а то в твоём нынешнем варианте вряд ли кто-то будет разбираться.