Jump to content

[1.11.2] Wierd compilation error


MisterPeModder

Recommended Posts

Hello!

I currently working on a mod called TooManyOres mod that depends on a library mod called HexianCore.

The mod worked properly and i could compile it without any problem but recently, I encountered some pretty strange compilation error sthat only occurs when I'm trying to build the project (works fine in dev environnement).

It says that I didn't implemented the method GuiContainer::drawGuiContainerBackgroundLayer even though it is implemented the the GuiConainerBase class.

So I almost tried everthing and I could not find a solution to this problem, so if someone knows what i did wrong, I would be very grateful ;)!

 

GuiContainerBase:

Spoiler

package misterpemodder.hc.main.client.gui;
import java.awt.Dimension;
import java.awt.Point;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;
import org.apache.commons.lang3.tuple.MutablePair;
import org.apache.commons.lang3.tuple.Pair;
import misterpemodder.hc.main.client.gui.tabs.TabArmorInventory;
import misterpemodder.hc.main.client.gui.tabs.TabBase;
import misterpemodder.hc.main.client.gui.tabs.TabBase.TabPos;
import misterpemodder.hc.main.client.gui.tabs.TabBase.TabTexture;
import misterpemodder.hc.main.client.gui.tabs.TabMain;
import misterpemodder.hc.main.client.gui.tabs.TabPlayerInventory;
import misterpemodder.hc.main.compat.craftingtweaks.CraftingTweaksCompat;
import misterpemodder.hc.main.compat.jei.JEICompat;
import misterpemodder.hc.main.compat.jei.JEIPluginHC;
import misterpemodder.hc.main.inventory.ContainerBase;
import misterpemodder.hc.main.tileentity.TileEntityContainerBase;
import misterpemodder.hc.main.utils.StringUtils;
import net.minecraft.client.Minecraft;
import net.minecraft.client.gui.FontRenderer;
import net.minecraft.client.gui.Gui;
import net.minecraft.client.gui.GuiButton;
import net.minecraft.client.gui.inventory.GuiContainer;
import net.minecraft.client.renderer.GlStateManager;
import net.minecraft.client.renderer.RenderHelper;
import net.minecraft.init.SoundEvents;
import net.minecraft.item.ItemStack;
import net.minecraft.util.text.TextFormatting;
import net.minecraftforge.fml.client.config.GuiUtils;
public abstract class GuiContainerBase<C extends ContainerBase<TE>, TE extends TileEntityContainerBase> extends GuiContainer {
    
    public static final int TAB_OFFSET = 4;
    private static List<String> hoveringText;
    private static int hoveringTextMaxWidth;
    public C container;
    
    protected MutablePair<TabBase<C, TE>, TabBase<C, TE>> selectedTabs;
    protected List<TabBase<C, TE>> tabs;
    
    protected List<GuiButton> selectedButtonsLeft;
    protected List<GuiButton> selectedButtonsRight;
    
    private short ctButtonsState = -1;
    private List<GuiButton> ctButtons = new ArrayList<>();
    
    
    public GuiContainerBase(C container) {
        super(container);
        this.container = container;
        this.tabs = registerTabs();
        
        if(tabs.size() < 2) throw new IllegalArgumentException("There must be at least 2 tabs!");
        initTabs();
        
        Pair<TabBase<C, TE>, TabBase<C, TE>> p = getDefaultPair();
        this.selectedTabs = MutablePair.of(p.getLeft(), p.getRight());
        
        Dimension dl = selectedTabs.left.getTexture().dim;
        Dimension dr = selectedTabs.right.getTexture().dim;
        this.xSize = (Math.max(dl.width, dr.width) + TabBase.WIDTH);
          this.ySize = dl.height + dr.height;
          
          this.selectedButtonsLeft = new ArrayList<>();
          this.selectedButtonsRight = new ArrayList<>();
          
          if(CraftingTweaksCompat.isModLoaded() && CraftingTweaksCompat.guiTweakButtonClass != null) {
              ctButtonsState = 0;
          }
    }
    
    private void initTabs() {
        for(TabBase<C, TE> tab : tabs) {
            tab.setGuiContainer(this);
        }
    }
    
    @Override
    public void initGui() {
        super.initGui();
        for(TabBase<C, TE> tab : tabs) {
            tab.initButtons(this.guiLeft, this.guiTop);
        }
    }
    
    @Override
    public void setWorldAndResolution(Minecraft mc, int width, int height) {
        tabs.forEach(t -> t.getButtonsList().clear());
        super.setWorldAndResolution(mc, width, height);
    }
    
    protected Pair<TabBase<C, TE>, TabBase<C, TE>> getDefaultPair() {
        MutablePair<TabBase<C, TE>, TabBase<C, TE>> p = new MutablePair<>();
        for(TabBase<C, TE> tab : tabs) {
            if(tab instanceof TabMain) {
                p.setLeft(tab);
            } else if(tab instanceof TabPlayerInventory) {
                p.setRight(tab);
            }
        }
        if(p.getLeft() == null || p.getRight() == null) {
            p.setLeft(tabs.get(0));
            p.setRight(tabs.get(1));
        }
        return p;
    }
    
    public abstract List<TabBase<C, TE>> registerTabs();
    
    @Override
    public void drawScreen(int mouseX, int mouseY, float partialTicks) {
        super.drawScreen(mouseX, mouseY, partialTicks);
        this.selectedTabs.left.drawScreen(mouseX, mouseY, partialTicks);
        this.selectedTabs.right.drawScreen(mouseX, mouseY, partialTicks);
        addButtons();
    }
    
    @Override
    protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY) {
        //Disabled Tabs
        GlStateManager.pushMatrix();
        GlStateManager.color(1.0f, 1.0f, 1.0f, 1.0f);
        this.drawDisabledTabs();
        
        //Top Part
        TabTexture lt = selectedTabs.getLeft().getTexture();
        this.mc.getTextureManager().bindTexture(lt.screenTexture);
        Gui.drawModalRectWithCustomSizedTexture(this.guiLeft, this.guiTop, 0, 0, lt.dim.width, lt.dim.height, lt.textureSize.width, lt.textureSize.height);
        
        //Bottom part
        TabTexture rt = selectedTabs.getRight().getTexture();
        this.mc.getTextureManager().bindTexture(rt.screenTexture);
        Gui.drawModalRectWithCustomSizedTexture(this.guiLeft, getBottomPartPos(), 0, 0, rt.dim.width, rt.dim.height, rt.textureSize.width, rt.textureSize.height);
        
        //Enabled Tabs
        this.drawTab(selectedTabs.getLeft(), true);
        selectedTabs.getLeft().drawGuiContainerBackgroundLayer(partialTicks, mouseX, mouseY);
        this.drawTab(selectedTabs.getRight(), true);
        selectedTabs.getRight().drawGuiContainerBackgroundLayer(partialTicks, mouseX, mouseY);
        
        GlStateManager.popMatrix();
    }
    
    protected void drawTab(TabBase<C, TE> tab, boolean enabled) {
        TabTexture texture = tab.getTexture();
        Point uv = enabled? texture.enabledCoords : texture.disabledCoords;
        Point coords = tab.getPos();
        this.mc.getTextureManager().bindTexture(tab.getTexture().tabTexture);
        Gui.drawModalRectWithCustomSizedTexture(this.getGuiLeft()+coords.x , this.getGuiTop()+coords.y, uv.x, uv.y, TabBase.WIDTH, TabBase.HEIGHT, 128F, 128F);
    }
    
    @Override
    protected void drawGuiContainerForegroundLayer(int mouseX, int mouseY) {
        
        selectedTabs.getLeft().updateButtons();
        selectedTabs.getRight().updateButtons();
        
        selectedTabs.getLeft().drawGuiContainerForegroundLayer(mouseX, mouseY);
        selectedTabs.getRight().drawGuiContainerForegroundLayer(mouseX, mouseY);
        
        boolean flag = true;
        for(TabBase<C, TE> tab : tabs) {
            RenderHelper.enableGUIStandardItemLighting();
            ItemStack stack = tab.getItemStack();
            Point tabPos = tab.getPos();
            this.itemRender.renderItemIntoGUI(stack, tabPos.x + 8, tabPos.y + 6);
            this.itemRender.renderItemOverlays(this.fontRendererObj, stack, tabPos.x + 8, tabPos.y + 6);
            RenderHelper.disableStandardItemLighting();
            if(isPointInRegion(tab.getPos().x+1, tab.getPos().y, TabBase.WIDTH, TabBase.HEIGHT-1, mouseX, mouseY)) {
                flag = false;
                addHoveringText(TextFormatting.GRAY + "" + TextFormatting.ITALIC + StringUtils.translate(tab.getUnlocalizedName()));
            }
        }
        
        
        
        if(flag && JEICompat.isModLoaded()) {
            List<RecipeClickableAreaHC> list = getRecipeClickableAreas();
            for(RecipeClickableAreaHC r : list) {
                if(r.checkHover(mouseX, mouseY)) {
                    addHoveringText(StringUtils.translate("jei.tooltip.show.recipes"));
                }
            }
        }
        
        if(hoveringText != null && !hoveringText.isEmpty()) {
            GuiUtils.drawHoveringText(hoveringText, mouseX-this.getGuiLeft(), mouseY-this.getGuiTop(), this.width, this.height, hoveringTextMaxWidth, fontRendererObj);
            hoveringText = null;
        }
        
    }
    
    private List<RecipeClickableAreaHC> getRecipeClickableAreas() {
        List<RecipeClickableAreaHC> l = new ArrayList<>();
        if(selectedTabs.getLeft().hasRecipeClickableAreas()) {
            l.addAll(Arrays.asList(selectedTabs.getLeft().getRecipeClickableAreas()));
        }
        if(selectedTabs.getRight().hasRecipeClickableAreas()) {
            l.addAll(Arrays.asList(selectedTabs.getRight().getRecipeClickableAreas()));
        }
        return l;
    }
    
    @Override
    protected void mouseClicked(int mouseX, int mouseY, int mouseButton) throws IOException {
        
        this.selectedTabs.left.mouseClicked(mouseX, mouseY, mouseButton);
        this.selectedTabs.right.mouseClicked(mouseX, mouseY, mouseButton);
        
        for(TabBase<C, TE> tab : tabs) {
            Point pos = tab.getPos();
            if(isPointInRegion(pos.x+1, pos.y, TabBase.WIDTH, TabBase.HEIGHT-1, mouseX, mouseY)) {
                if(tab.isEnabled() && selectedTabs.getLeft() != tab && selectedTabs.getRight() != tab) {
                    this.mc.player.playSound(SoundEvents.UI_BUTTON_CLICK, 1.0F, 1.0F);
                    TabPos tabPos = tab.getTabPos();
                    if(tabPos == TabPos.TOP_LEFT || tabPos == TabPos.TOP_RIGHT) {
                        selectedTabs.setLeft(tab);
                    } else {
                        selectedTabs.setRight(tab);
                    }
                    selectedTabs = tab.forceTabConfig();
                    setContainerTabs();
                    this.container.hideSlots();
                }
            } else if(JEIPluginHC.hasJEIRuntime()) {
                List<RecipeClickableAreaHC> list = getRecipeClickableAreas();
                for(RecipeClickableAreaHC r : list) {
                    if(r.checkHover(mouseX, mouseY)) {
                        JEIPluginHC.getJEIRuntime().getRecipesGui().showCategories(r.getRecipeCategoryUids());
                        this.onGuiClosed();
                    }
                }
            }
        }
        super.mouseClicked(mouseX, mouseY, mouseButton);
    }
    
    @SuppressWarnings({ "unchecked", "rawtypes" })
    public void setContainerTabs() {
        this.container.setSelectedTabs((Pair)selectedTabs);
    }
    
    public void updateScreen() {
        super.updateScreen();
        TE te = this.container.getTileEntity();
        if(te == null || !te.hasWorld() || te.getWorld().getBlockState(te.getPos()).getBlock() != te.getBlockType()) {
            this.onGuiClosed();
            this.mc.player.closeScreen();
        }
    }
    
    public boolean isPointInTheRegion(int rectX, int rectY, int rectWidth, int rectHeight, int pointX, int pointY) {
        return this.isPointInRegion(rectX, rectY, rectWidth, rectHeight, pointX, pointY);
    }
    
    @Override
    public void onGuiClosed() {
        this.selectedTabs.left.onGuiClosed();
        this.selectedTabs.right.onGuiClosed();
        
        tabs.forEach(t -> t.getButtonsList().clear());
        super.onGuiClosed();
    }
    
    protected void keyTyped(char typedChar, int keyCode) throws IOException {
        if(!selectedTabs.left.keyTyped(typedChar, keyCode) && !selectedTabs.right.keyTyped(typedChar, keyCode)) {
            super.keyTyped(typedChar, keyCode);
        }
    }
    
    private void addButtons() {
        List<GuiButton> leftButtons = this.selectedTabs.left.getButtonsList();
        List<GuiButton> rightButtons = this.selectedTabs.right.getButtonsList();
        
        if(leftButtons == null || leftButtons.isEmpty()) {
            selectedButtonsLeft.clear();
        }
        else if(!selectedButtonsLeft.containsAll(leftButtons)) {
            selectedButtonsLeft.clear();
            selectedButtonsLeft.addAll(leftButtons);
        }
        
        if(rightButtons == null || rightButtons.isEmpty()) {
            selectedButtonsRight.clear();
        }
        else if(!selectedButtonsRight.containsAll(rightButtons)) {
            selectedButtonsRight.clear();
            selectedButtonsRight.addAll(rightButtons);
        }
        
        
        
        if(ctButtonsState == 0 && !buttonList.isEmpty()) {
            for(GuiButton b : buttonList) {
                if(CraftingTweaksCompat.guiTweakButtonClass.isAssignableFrom(b.getClass())) {
                    this.ctButtonsState = 1;
                    this.ctButtons.add(b);
                }
            }
        }
        
        try {
        if(ctButtonsState == 1 && !buttonList.isEmpty() && !ctButtons.isEmpty()) {
            List<GuiButton> blist = new ArrayList<>(buttonList);
            for(GuiButton b1 : blist) {
                for(GuiButton b2 : ctButtons) {
                    if(b1 == b2) {
                        buttonList.remove(b1);
                    }
                }
            }
        }
        } catch(Exception e) {
            e.printStackTrace();
        }
        
        this.buttonList.clear();
        this.buttonList.addAll(selectedButtonsLeft);
        this.buttonList.addAll(selectedButtonsRight);
        if(this.selectedTabs.right instanceof TabArmorInventory && !ctButtons.isEmpty()) {
            buttonList.addAll(ctButtons);
        }
        
    }
    
    @Override
    protected void actionPerformed(GuiButton button) throws IOException {
        
        if(selectedButtonsLeft.contains(button)) {
            this.selectedTabs.left.onButtonClicked(button);
        }
        else if(selectedButtonsRight.contains(button)) {
            this.selectedTabs.right.onButtonClicked(button);
        }
    }
    
    public static void addHoveringText(String str) {
        addHoveringText(Arrays.asList(str), -1);
    }
    
    public static void addHoveringText(List<String> strs, int maxWidth) {
        if(strs != null && !strs.isEmpty()) {
            hoveringText = strs;
            hoveringTextMaxWidth = maxWidth;
        }
    }
    
    public FontRenderer getFontRenderer() {
        return this.fontRendererObj;
    }
    
    public int getBottomPartPos() {
        return this.getGuiTop() + selectedTabs.getLeft().getTexture().dim.height;
    }
    
    protected void drawDisabledTabs() {
        for(TabBase<C, TE> tab : tabs) {
            if(tab != selectedTabs.getLeft() && tab != selectedTabs.getRight()) {
                this.drawTab(tab, false);
            }
        }
    }
    
    @Override
    public boolean isPointInRegion(int rectX, int rectY, int rectWidth, int rectHeight, int pointX, int pointY) {
        return super.isPointInRegion(rectX, rectY, rectWidth, rectHeight, pointX, pointY);
    }
    
    public MutablePair<TabBase<C, TE>, TabBase<C, TE>> getSelectedTabs() {
        return this.selectedTabs;
    }
    public List<TabBase<C, TE>> getRegisteredTabs() {
        return this.tabs;
    }
    
    public Minecraft getMinecraft() {
        return this.mc;
    }
    
}

 

 

GuiContainerDestabilizer:

Spoiler

package misterpemodder.tmo.main.client.gui;
import java.util.ArrayList;
import java.util.List;
import org.objectweb.asm.commons.Method;
import misterpemodder.hc.main.client.gui.GuiContainerBase;
import misterpemodder.hc.main.client.gui.tabs.TabBase;
import misterpemodder.hc.main.client.gui.tabs.TabPlayerInventory;
import misterpemodder.tmo.main.client.gui.tabs.TabArmorInventoryTMO;
import misterpemodder.tmo.main.client.gui.tabs.TabIO;
import misterpemodder.tmo.main.client.gui.tabs.TabInfo;
import misterpemodder.tmo.main.client.gui.tabs.TabMainDestabilizer;
import misterpemodder.tmo.main.inventory.ContainerDestabilizer;
import misterpemodder.tmo.main.tileentity.TileEntityDestabilizer;
import net.minecraft.entity.player.InventoryPlayer;
import net.minecraftforge.fml.relauncher.ReflectionHelper;
public class GuiContainerDestabilizer extends GuiContainerBase<ContainerDestabilizer, TileEntityDestabilizer> {
    public GuiContainerDestabilizer(InventoryPlayer playerInv, TileEntityDestabilizer te) {
        super(new ContainerDestabilizer(te, playerInv));
    }
    @Override
    public List<TabBase<ContainerDestabilizer, TileEntityDestabilizer>> registerTabs() {
        List<TabBase<ContainerDestabilizer, TileEntityDestabilizer>> list = new ArrayList<>();
        list.add(new TabMainDestabilizer());
        list.add(new TabInfo<ContainerDestabilizer, TileEntityDestabilizer>(false));
        list.add(new TabPlayerInventory<ContainerDestabilizer, TileEntityDestabilizer>());
        list.add(new TabArmorInventoryTMO<ContainerDestabilizer, TileEntityDestabilizer>());
        list.add(new TabIO<>(container.getTileEntity().getIoConfigHandler()));
        return list;
    }
    
}

 

Gradle log:

Spoiler

This mapping 'snapshot_20161220' was designed for MC 1.11! Use at your own peril.This mapping 'snapshot_20161220' was designed for MC 1.11! Use at your own peril.

#################################################
         ForgeGradle 2.2-SNAPSHOT-d13a6c8        
  https://github.com/MinecraftForge/ForgeGradle  
#################################################
               Powered by MCP unknown               
            http://modcoderpack.com             
         by: Searge, ProfMobius, Fesh0r,         
         R4wk, ZeuX, IngisKahn, bspkrs           
#################################################
:extractMcpMappings SKIPPED
:deobfCompileDeobfDepTask0
:deobfCompileDeobfDepTask1 SKIPPED
:deobfCompileRemapDepSourcesTask2 SKIPPED
:deobfCompileRemapDepSourcesTask3 SKIPPED
:deobfCompileDummyTask
:deobfProvidedDummyTask
:sourceApiJava UP-TO-DATE
:compileApiJava
:processApiResources UP-TO-DATE
:apiClasses
:sourceMainJava
:compileJavaC:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\tileentity\TileEntityTitaniumChest.java:5: error: TileEntityTitaniumChest is not abstract and does not override abstract method update() in ITickable
public class TileEntityTitaniumChest extends TileEntityCustomChest {
       ^
C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\tileentity\TileEntityStrongPiston.java:70: error: no suitable method found for findMethod(Class<CAP#1>,<null>,String[],Class<?>[])
            return ReflectionHelper.findMethod(clazz, null, methodNames, methodTypes);
                                   ^
    method ReflectionHelper.<E>findMethod(Class<? super E>,E,String[],Class<?>...) is not applicable
      (cannot infer type-variable(s) E
        (argument mismatch; Class<CAP#1> cannot be converted to Class<? super E>))
    method ReflectionHelper.findMethod(Class<?>,String,String,Class<?>...) is not applicable
      (argument mismatch; String[] cannot be converted to String)
  where E is a type-variable:
    E extends Object declared in method <E>findMethod(Class<? super E>,E,String[],Class<?>...)
  where CAP#1 is a fresh type-variable:
    CAP#1 extends Object from capture of ?
C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerDestabilizer.java:20: error: GuiContainerDestabilizer is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer
public class GuiContainerDestabilizer extends GuiContainerBase<ContainerDestabilizer, TileEntityDestabilizer> {
       ^
C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerInjector.java:17: error: GuiContainerInjector is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer
public class GuiContainerInjector extends GuiContainerBase<ContainerInjector, TileEntityInjector> {
       ^
C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerThermoelectricGenerator.java:17: error: GuiContainerThermoelectricGenerator is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer
public class GuiContainerThermoelectricGenerator extends GuiContainerBase<ContainerThermoelectricGenerator, TileEntityThemoelectricGenerator> {
       ^
C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerTitaniumAnvil.java:16: error: GuiContainerTitaniumAnvil is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer
public class GuiContainerTitaniumAnvil extends GuiContainerBase<ContainerTitaniumAnvil, TileEntityTitaniumAnvil>{
       ^
C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerTitaniumChest.java:18: error: GuiContainerTitaniumChest is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer
public class GuiContainerTitaniumChest extends GuiContainerBase<ContainerTitaniumChest, TileEntityTitaniumChest>{
       ^
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\items\TMOItemVariants.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
7 errors
 FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':compileJava'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 1 mins 44.976 secs
 

 

 

PS: sorry for my bad english :)

Edited by MisterPeModder
PS: sorry for my bad english :)
Link to comment
Share on other sites

1) Your reflection is broken you need to add the srg names as well. The findMethod you are using is deprecated and switching to the other one will make it compile. I believe that the problem is the null instance

2) All the other errors are that you are not implementing things in the classes. You need to implement the missing methods.

 

Once these things are fixed it compiles

Edited by CoderAtParadise
grammer

Did you really need to know?

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.