Мод Как пофиксить дюп с микро блоками?

он есть , НО ОН ПРИВАТНЫЙ. читать умеете или как?
Да и зачем вам 1 аргумент , лень написать лишних пару символов (ItemStack)?...

 

Вы по-моему вообще не в теме, или сами читать не умеете.

 

как раз таки в теме. isItemStackEqual - проверка одинаковые ли 2 айтем стака , вызывать его как ItemStack.isItemStacksEqual(item1, item2). Его надо вызывать когда вы например сравниваите 2 предмета , когда суете их в сундук. Что же делает форестри - он использует другой метод , который без проверки нбт даты.

Так же есть не статик метод , а простой. работает он как item1.isItemStackEqual(item2). Делает тоже самое , точнее статик метод только и делает одну работу - нулл чек и вызов данного метода. Зачем сделали его приватным - я хз. Но в нём нету нул чека.

 

Вы вообще видели, на какой я пост дал ответ? И как мой ответ сформулирован?
Я в курсе, как оно работает, зачем Вы мне это рассказываете?
Для решения дюпа, Вам не поможет isItemStackEqual(par1, par2), если Вы этого не понимаете, а в Forestry используется isItemEqual(par1); => доступа к isItemStackEqual(par1) нет, потом что он private.
Доходит?

 

мне он помог.... У вас руки не с того места чтоли... к тому же - можно в ручную проверять нбт теги , ничего там такого сложного нету.

 

Очень хороший вывод про мои руки конечно, но я не обижусь.
Очень интересно, как же Вам помог тот метод.
Вручную конечно можно проверять, так и нужно.

 

+1 Я тоже не стал ипать мозг и просто вывел nbt тег и прогнал через egual.

 

Код:

    // Чек только нбт
    public static boolean areItemStackTagsEqual(ItemStack par0ItemStack, ItemStack par1ItemStack)

   // Чек нбт + айди / метта
    /**
     * compares ItemStack argument1 with ItemStack argument2; returns true if both ItemStacks are equal
     */
    public static boolean areItemStacksEqual(ItemStack par0ItemStack, ItemStack par1ItemStack)

    // ЧЕК ТОЛЬКО АЙДИ И МЕТТА ( например для крафта , и тд ). Проверяет одинаковый ли ITEM , а не ItemStack !!!
    /**
     * compares ItemStack argument to the instance ItemStack; returns true if the Items contained in both ItemStacks are
     * equal
     */
    public boolean isItemEqual(ItemStack par1ItemStack)

 

Я вовсе через binpatches сделал фикс.

В общем-то это все и так понятно, я просто говорю, что достаточно проверить nbt и что метод, который это делает принимая один аргумент - приватный.


Кстати похоже Sengir сам сделал исправления.

PHP:

@Override
    public int addItem(ItemStack stack, boolean doAdd, ForgeDirection from) {
        ItemStack resource = inventory.getStackInSlot(SLOT_RESOURCE);
        if (resource == null) {
            if (doAdd)
                inventory.setInventorySlotContents(SLOT_RESOURCE, stack.copy());
            return stack.stackSize;
        }

        if (!StackUtils.isIdenticalItem(resource, stack))
            return 0;

        int space = resource.getMaxStackSize() - resource.stackSize;
        if (space <= 0)
            return 0;

        if (doAdd)
            if (stack.stackSize <= space)
                resource.stackSize += stack.stackSize;
            else
                resource.stackSize += space;

        return Math.min(stack.stackSize, space);
    }

 

https://github.com/ForestryMC/ForestryMC/pull/325

 

О, даже так) Ну, я особо не слежу за этим. Хотя что-то в мыслях было такое, чтобы пулл-реквест кинуть им, но потом подумал, что все сразу же узнают о дюпе и ховайся.

 

ну да , он приватный... ну да , ну да...

 

спасибо за скрин. Видем док-ва моих слов. Прям над тем приватным , есть не приватный метод. Он делает нулл чек , как я и говорил , И ВЫЗЫВАЕТ ТОТ ПРИВАТНЫЙ МЕТОД. Опять же , вам лень написать ItemStack. ? Несколько лишних символов .. а уже лень ..

 

Присоединяюсь, isItemStackEqual является приватным.

 

Я же Вам совсем другое пытаюсь объяснить, а Вы в упор стоите на своем

 

ну и что же вы хотие обеснить?

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

 

Очень хорошо, что он есть, но пост, на который я дал ответ, гласил, что необходимо использовать isItemStackEqual, а я написал, что он приватный. Сложно было прочитать, да?)
А Вы мне начали рассказывать, что есть паблик статик методы, которые принимают 2 аргумента, которыми также можно пользоваться и т.п., это все хорошо, это я и без Вас знаю, но суть моего сообщения была вовсе не в этом
Успокойтесь и расслабьтесь. Он то делает все тоже самое, да, но суть была совать в метод один аргумент, вот и завели на этом тему.

 

тфу , спать надо больше. Не заметил что название другое (не заметил разницу между is и are). Так то да , надо было большими буквами написать "смотри на is"