Jump to content

How to get "/title actionbar" or GuiIngame.overlayMessage text?


olari

Recommended Posts

I'm trying to get the text from the /title actionbar command to get some information that a server presents there.

From the Minecraft decompilation I've gathered that the text itself is kept and used in net.minecraft.client.gui.GuiIngame, but I don't have access to it.

public class GuiIngame {
	protected String overlayMessage; // text itself
	public void setOverlayMessage(...); // setter
	// no getter
}


The only way I can come up with to get this information is to catch the SPacketTitle itself when it arrives.
This seems like a pretty daunting task so I thought I'd ask here first for alternatives.

If networking is the only option, I would like some guidance on how to approach it.

javaw_2018-07-29_16-31-23.png

Link to comment
Share on other sites

20 minutes ago, olari said:

The only way I can come up with to get this information is to catch the SPacketTitle itself when it arrives.

You could always use reflection. Make sure to use ReflectionHelper.getPrivateValue and provide both the deobf name of the field as well as the SRG name.

  • Thanks 1
Link to comment
Share on other sites

1 minute ago, V0idWa1k3r said:

You could always use reflection. Make sure to use ReflectionHelper.getPrivateValue and provide both the deobf name of the field as well as the SRG name.

Thank you very much for the hasty reply. I'll look into using reflection.

Link to comment
Share on other sites

13 minutes ago, olari said:

Thank you very much for the hasty reply. I'll look into using reflection.

Got it working!

 

String title = ReflectionHelper.getPrivateValue(GuiIngame.class, mc.ingameGUI, 8);

 

Edited by olari
Link to comment
Share on other sites

To elaborate, reflection is relatively slow/inefficient. (It's also generally a bad idea, but Minecraft's source code gives us no other options in some cases, as you've discovered.) Therefore, it should be used only when necessary, and its returned objects cached whenever possible.

Whatever Minecraft needs, it is most likely not yet another tool tier.

Link to comment
Share on other sites

2 hours ago, IceMetalPunk said:

reflection is relatively slow/inefficient.

If you do it correctly, it's pretty much as fast as normal field access.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

 

1.12 -> 1.13 primer by williewillus.

 

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

 

http://www.howoldisminecraft1710.today/

Link to comment
Share on other sites

5 minutes ago, IceMetalPunk said:

Define "correctly"?

Store the Field instance in a static field somewhere. Calling Field#get will pretty much be the same as normal field access performance wise.

Don't PM me with questions. They will be ignored! Make a thread on the appropriate board for support.

 

1.12 -> 1.13 primer by williewillus.

 

1.7.10 and older versions of Minecraft are no longer supported due to it's age! Update to the latest version for support.

 

http://www.howoldisminecraft1710.today/

Link to comment
Share on other sites

Just now, larsgerrits said:

Store the Field instance in a static field somewhere. Calling Field#get will pretty much be the same as normal field access performance wise.

So... cache it, like I said? XD The actual reflective call is only happening once in that case, which is why it's fast. But if you were to call the reflection access methods as often as you'd otherwise call a field name, the performance drops significantly.

Whatever Minecraft needs, it is most likely not yet another tool tier.

Link to comment
Share on other sites

Here's the final code:
 

private Field overlayMessageField = null;

// Called at initialization
private void findOverlayTextField() {
    this.overlayMessageField = ReflectionHelper.findField(GuiIngame.class, "overlayMessage");
}

public String getActionbar() {
    try {
        return (String)this.overlayMessageField.get(mc.ingameGUI);
    } catch (IllegalAccessException e) {
        e.printStackTrace();
    }

    return null;
}

 

Link to comment
Share on other sites

Return an empty string instead of null.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

16 minutes ago, diesieben07 said:

No. What even...

Would I rather check validate that the string returned by this function is null, or display an empty string (or the string "null")?

I think I'd rather have a non-null string.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

8 hours ago, diesieben07 said:

Neither should ever happen in this case.

Oh I agree it should never happen.

But if there's one thing I know about programming. "Should never happen" is not as absolute as one would think.

Edited by Draco18s

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

21 minutes ago, diesieben07 said:

True. But if it does happen, just carrying on as if nothing was wrong is not the way to go.

This is what crash reports are for.

In which case, I would do something like this:

public String getActionbar() {
	try {
	        return (String)overlayMessageField.get(Minecraft.getMinecraft().ingameGUI);
	    } catch (IllegalAccessException e) {
	    	e.printStackTrace();
	        throw new RuntimeException("Get Message Overlay failed!");
	    }
}

If it should crash, then let it crash. Don't handle the exception, then return null, allowing other code to null-check and not crash.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Another Minecraft Launcher https://multimc.org/#Download or https://atlauncher.com/downloads
    • Remove these mods and combine OptiFine HD U I6 with Forge 47.2.18 Start with Optifine - without any other mods - if this works, add the mods one by one or in groups until you find the mod, conflicting with Optifine
    • Ok yeah I think this might be where the problem starts. For starters, I think you can get rid of your RecipeType impl and make it a static constant instead. I'm not 100% sure this will fix it but it's posssible. RuneInscribingRecipe public class RuneInscribingRecipe implements Recipe<SimpleContainer> { public static final RecipeType<RuneInscribingRecipe> RECIPE_TYPE = new RecipeType<>(){}; // .... @Override public RecipeType<?> getType() { return RECIPE_TYPE; } // .... } RuneInscriberMenu private Optional<RecipeHolder<RuneInscribingRecipe>> getCurrentRecipe() { SimpleContainer inventory = new SimpleContainer(2); inventory.setItem(BASE_INPUT_SLOT, this.inventory.getStackInSlot(BASE_INPUT_SLOT)); inventory.setItem(TEMPLATE_INPUT_SLOT, this.inventory.getStackInSlot(TEMPLATE_INPUT_SLOT)); List<RecipeHolder<RuneInscribingRecipe>> list = this.blockEntity.getLevel().getRecipeManager().getRecipesFor(RuneInscribingRecipe.RECIPE_TYPE, inventory, this.blockEntity.getLevel()); if (list.isEmpty()) { return Optional.empty(); } return Optional.of(list.get(0)); }   Also, this here is a little suspicious where you're getting the `this.inventory.getStackInSlot` calls. I'd try and make sure those are what you expect they are with either a Debug line in IDE or print statement SimpleContainer inventory = new SimpleContainer(2); inventory.setItem(BASE_INPUT_SLOT, this.inventory.getStackInSlot(BASE_INPUT_SLOT)); System.out.printf("CHECKING BASE INPUT OF RECIPE: %s\n", this.inventory.getStackInSlot(BASE_INPUT_SLOT)); inventory.setItem(TEMPLATE_INPUT_SLOT, this.inventory.getStackInSlot(TEMPLATE_INPUT_SLOT)); System.out.printf("CHECKING TEMPLATE INPUT OF RECIPE: %s\n", this.inventory.getStackInSlot(TEMPLATE_INPUT_SLOT)); ^ See what the result of those print statements are, make sure they're right
    • Upon testing again after adding them, the game still doesn't crash however the resource packs are still unfortunately broken.
    • Hi! Can someone please point me to the correct FAQ or place? I'm not able do DuckDuckGo myself out of this one. As you can hopefully see, chunks are not rendered. I run a forge server, fresh install on a linux VM. Curseforge on the Win11 client with Embeddium, Oculus and Unbound shader. No mods. Forge 47.2.23 on server & client.  No errors, no crashes, no apparent CPU or RAM issues with the VM. This screenshot was captured while creative, but the same happens when flying in survival. Waiting (floating) for a few minutes sooner or (often) later, lets the missing chunks appear. Usually takes 5-10 minutes.  When on a local game, the chunks appear slowly, but they do appear without pause.   This must be something simple, obvious, but I can't find the correct "search words" to find my way out. "Chunks not loading / rendering" didn't help. Browsing several forums neither. 
  • Topics

×
×
  • Create New...

Important Information

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