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



×
×
  • Create New...

Important Information

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