Перезапись метода с помощью Mixin

[AlexCH]

Привет, я в моддинге относительно недавно и вот возникла задача - отменить сброс погоды после сна игрока. Свои попытки разделю на три варианта.

Спойлер: Вариант №1 - нерабочий

Я взял MCP от 1.12, впихнул MCP-SRG и маппинги как у Forge. Нашёл нужный класс, сделал правки, получил пару обфусцированных классов. Вставил их в Jar, игра запускается (благо о нужных аргументах для отключения проверок знаем) но краш при входе в мир. Правка - вырезать к чертям кусок кода (проверка и выполнение метода сброса погоды):

protected void wakeAllPlayers()
    {
        this.allPlayersSleeping = false;

        for (EntityPlayer entityplayer : this.playerEntities.stream().filter(EntityPlayer::isPlayerSleeping).collect(Collectors.toList()))
        {
            entityplayer.wakeUpPlayer(false, false, true);
        }

        if (this.getGameRules().getBoolean("doWeatherCycle"))
        {
            this.resetRainAndThunder();
        }
   
    //Вырезать кусок кода вверху
   
    }

Спойлер: Вариант №2 - рабочий но возможно проблемный

Взять и через @overwrite перезаписать сам метод сброса погоды (resetRainAndThunder) сделав мод с миксином. Формально, такой костыль работает, но не работает команда /weather clean.

Спойлер: Вариант №3 - желательный

Сделать то же, что и в первом варианте, но в моде и через Mixin с перезаписью метода на свой с вырезанным участком сброса погоды. Тогда вопрос, как написать свой вариант метода? В ванили он вызывает другие методы через this и если оставить просто копию метода то среда естественно жалуется:

Весь день курю эту вроде бы небольшую манипуляцию с ванильным кодом.

 

[GlassSpirit]

Доступ к полям класса, в который миксуешься, можно получить с помощью аннотации Shadow. Вынеси их аналогично… Погоди, почему у тебя есть поле с Shadow, но оно называется wakeAllPlayers? Переименуй в worldServer да и все

 

[will0376]

быстро пробежав по теме отмечу:
1. могут ломаться маппинги. их остаётся или чинить или ручками в method указывать ДВА имени метода(нормальное и обф)
2. для каста к таргету используй Object. т.е. -> ((Target)(Object) this)
3. можно сделать Overwrite, но инжектом.
указываешь at = @AT("HEAD"),
пилишь инжект и при любом выходе из твоего метода - CallbackInfo#cancel()

 

[AlexCH]

Object. т.е. -> ((Target)(Object) this)

То-есть, можно оставить

@Overwrite
    protected void wakeAllPlayers() {
        
        this.allPlayersSleeping = false;

        for (EntityPlayer entityplayer : this.playerEntities.stream().filter(EntityPlayer::isPlayerSleeping).collect(Collectors.toList())) {
            
            entityplayer.wakeUpPlayer(false, false, true);
            
        }

и вне метода связать его this с тем, к которому привязан ванильный метод?

PS: задача лёгкая, но мои познания в Mixin начались только сегодня.

 

[AlexCH]

UP: сделал рабочий вариант через каст объекта. И понял, что теперь для соблюдения логики нужно отключить возможность спать при дожде...

 

[will0376]

и вне метода связать его this

?????
Логика миксинов проста. он подмешивает код, который ты пишешь в своём миксин-классе в таргет. изменяя его с учётом аннотаций и т.п.
т.е. (Target)(Object)this будет валидным в момент исполнения кода(и среда его схавает). Но при этом сам миксин класс будет выгружен из памяти после его (бес)успешного микширования

 

[AlexCH]

?????
Логика миксинов проста. он подмешивает код, который ты пишешь в своём миксин-классе в таргет. изменяя его с учётом аннотаций и т.п.
т.е. (Target)(Object)this будет валидным в момент исполнения кода. Но при этом сам миксин класс будет выгружен из памяти после его (бес)успешного микширования

Уже это не важно, я уже понял как это использовать и это уже работает. Осталось отключить возможность спать при дожде/грозе, вот курю ищу нужные методы/классы. Никак не могу найти связь грозы и возможности спать при ней... Перерыл пару десятков классов...

 

[AlexCH]

UP: отключить сброс погоды при сне удалось, осталось понять как запретить спать при грозе а то получаеться... Дюп для пропуска кучи дней?

Неясно где это реализовано, разрешение на сон встречается в десятке классов и сотне методов, может кто знает где копать?

 

[will0376]

где копать?

там, где устанавливается гроза - смотри геттеры.

 

[tox1cozZ]

Зачем перезаписывать весь метод, есть можно просто сделать редирект resetRainAndThunder на пустышку?

 

[TheAndrey]

Вырезать две инструкции - это тривиальная задача для ASM, а вы в свои миксины лезете... Легкие пути не для вас.

 

[will0376]

а вы в свои миксины лезете.

миксинами быстрее получится. нежели лезть изучать АСМ и писать код, который будет удалять то что надо.

 

[AlgorithmLX]

Вырезать две инструкции - это тривиальная задача для ASM, а вы в свои миксины лезете... Легкие пути не для вас.

Mixin проще для изучения и быстрее получается вырезать что-нибудь Inject'ом, нежели ASM, который фиг выучишь... И если вы кроме 1.7.10 нечего не видели, то сочувствую.

 

[will0376]

И если вы кроме 1.7.10, то сочувствую

И на что ты это намекаешь?

 

[AlgorithmLX]

И на что ты это намекаешь?

странно, я редактировал, а почему-то не отредакт