Jump to content

[Solved] Config enum value text


Kirdow
 Share

Recommended Posts

So I currently have an annotation based config class fully set up and working. Though one of the properties is an enum, and I don't like the way that it shows the enum instance name, I want to have a custom string for each value
I currently have this code, where I added a String property and put it in toString to return the name, but it still use the enum variable text to display in the configuration menu. (note that I use the built in menu automatically generated by Forge upon using @Config ).
 

// This is inside the root class for the config file
public static class Inventory {
	@Config.Name("Rarity Glow")
	public boolean rarityGlow = true;

	@Config.Name("Rarity Type")
	public RarityGlowType rarityGlowType = RarityGlowType.GRADIENT_COLOR;
	
	public enum RarityGlowType {
		SOLID_COLOR(0, "Solid Color"),
		GRADIENT_COLOR(1, "Gradient Color");

		private int id;
		private String name;

		private RarityGlowType(int id, String name) {
			this.id = id;
			this.name = name;
		}

		public int getId() {
			return id;
		}

		public String toString() {
			return name;
		}
	}
}

Basically this makes the config gui show "GRADIENT_COLOR" as the default option while I want it to show "Gradient Color" as returned in RarityGlowType#toString
How would I go ahead and fix this, without creating my own config gui of course?

Edited by Kirdow
Changing title to solved
Link to comment
Share on other sites

30 minutes ago, Kirdow said:

So I currently have an annotation based config class fully set up and working. Though one of the properties is an enum, and I don't like the way that it shows the enum instance name, I want to have a custom string for each value
I currently have this code, where I added a String property and put it in toString to return the name, but it still use the enum variable text to display in the configuration menu. (note that I use the built in menu automatically generated by Forge upon using @Config ).
 


// This is inside the root class for the config file
public static class Inventory {
	@Config.Name("Rarity Glow")
	public boolean rarityGlow = true;

	@Config.Name("Rarity Type")
	public RarityGlowType rarityGlowType = RarityGlowType.GRADIENT_COLOR;
	
	public enum RarityGlowType {
		SOLID_COLOR(0, "Solid Color"),
		GRADIENT_COLOR(1, "Gradient Color");

		private int id;
		private String name;

		private RarityGlowType(int id, String name) {
			this.id = id;
			this.name = name;
		}

		public int getId() {
			return id;
		}

		public String toString() {
			return name;
		}
	}
}

Basically this makes the config gui show "GRADIENT_COLOR" as the default option while I want it to show "Gradient Color" as returned in RarityGlowType#toString
How would I go ahead and fix this, without creating my own config gui of course?

Do some debugging, find out how it generates that text to display and find a way to modify it. If your successful please post your results!

About Me

Spoiler

My Discord - Cadiboo#8887

My WebsiteCadiboo.github.io

My ModsCadiboo.github.io/projects

My TutorialsCadiboo.github.io/tutorials

Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support.

When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible.

Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org

Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)

Link to comment
Share on other sites

22 minutes ago, Cadiboo said:

Do some debugging, find out how it generates that text to display and find a way to modify it. If your successful please post your results!

I managed to fix it.
So while trying to find how it gets the value, I realized something I noticed a few years ago, how enums during compilation gets turned into a class extending Enum. This means my RarityGlowType would instead result in a public static class RarityGlowType extends Enum
Now I looked at my enum's super class to get to Enum, and Enum has 2 variables, "name" which is a String and "ordinal" which is the index. the "name" variable is returned in a method called "name()", though since all fields and methods are final I can't override them.
So I decided to look up if reflection allowed modifying final fields and found this stackoverflow answer . So I took that code, and put it in my enum's constructor resulting in the following code
 

public enum RarityGlowType {
	SOLID_COLOR(0, "Solid Color"),
	GRADIENT_COLOR(1, "Gradient Color");

	private int id;
	private String name;

	private RarityGlowType(int id, String name) {
		this.id = id;
		this.name = name;
	
		try {
			Field field = Enum.class.getDeclaredField("name");
			field.setAccessible(true);

			Field modifiersField = Field.class.getDeclaredField("modifiers");
			modifiersField.setAccessible(true);
			modifiersField.setInt(field, field.getModifiers() & ~Modifier.FINAL);

			field.set(this, name);

			System.out.println("Successfully changed RarityGlowType name!");
		} catch (Throwable throwable) {
			throwable.printStackTrace();
		}
	}

	public int getId() {
		return id;
	}

	public String toString() {
		return name;
	}
}

And while running this in IntelliJ it successfully showed up as "Solid Color" and "Gradient Color" like I wanted. Just note, as the stackoverflow answer states, if the environment has some sort of SecurityManager blocking changing the modifiers, this would break, so hopefully this wont break when the version is released to the public.

Link to comment
Share on other sites

Cough.

EnumHelper, supplied by Forge.

Cough.

 

Also, if this is your own class, why the hell do you need reflection to do this?

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

8 minutes ago, Draco18s said:

Cough.

EnumHelper, supplied by Forge.

Cough.

Oh I forgot about this one, thanks, I modified the enum's constructor to use this instead.
Just to help out future visitors, this is the code I put in the constructor for it to work.

private RarityGlowType(int id, String name) {
	this.id = id;
	this.name = name;

	try {
		EnumHelper.setFailsafeFieldValue(ReflectionHelper.findField(Enum.class, "name"), this, name);

		System.out.println("Successfully changed RarityGlowType name!");
	} catch (Throwable throwable) {
		throwable.printStackTrace();
	}
}

 

Link to comment
Share on other sites

38 minutes ago, diesieben07 said:

This is a terrible idea...

I am aware of this. I'm sure as long as you don't do too specific tasks with this it should be fine, I'm currently only going to use it in one if statement so I'm sure it will be fine. But then again it's Java so anything could happen :P I'll probably revert back if this starts causing any problems though.

Link to comment
Share on other sites

  • 2 months later...

Here's a more stable way, that requires only the EnumHelper:

static
{
	for (RarityGlowTypeNames value : RarityGlowTypeNames.values())
	{
		String name = "";
		for (String word : value.name().split("_"))
			name += word.substring(0, 1) + word.substring(1).toLowerCase() + " ";

		EnumHelper.addEnum(RarityGlowType.class, name.substring(0, name.length() - 1), new Class<?>[0]);
	}
}

// This is inside the root class for the config file
public static class Inventory
{
	@Config.Name("Rarity Type")
	public RarityGlowType rarityGlowType = RarityGlowType.values()[0];
}

public enum RarityGlowType {}

public enum RarityGlowTypeNames
{
	SOLID_COLOR, GRADIENT_COLOR;
}

 

Or here's a variant of that without dynamic name generation:

static
{
	for (RarityGlowTypeNames name : RarityGlowTypeNames.values())
		EnumHelper.addEnum(RarityGlowType.class, name.toString(), new Class<?>[0]);
}

// This is inside the root class for the config file
public static class Inventory
{
	@Config.Name("Rarity Type")
	public RarityGlowType rarityGlowType = RarityGlowType.values()[0];
}

public enum RarityGlowType {}

public enum RarityGlowTypeNames
{
	SOLID_COLOR("Solid Color"),
	GRADIENT_COLOR("Gradient Color");

	private String name;

	private RarityGlowTypeNames(String name)
	{
		this.name = name;
	}

	@Override
	public String toString()
	{
		return name;
	}
}

 

An example of checking the value could be:

ConfigMod.inventory.rarityGlowType.ordinal() == RarityGlowTypeNames.GRADIENT_COLOR.ordinal()

 

Link to comment
Share on other sites

2 minutes ago, Phylogeny said:

An enum is what tells Forge to generate a cycling button config element. Do you have a constructive suggestions as to how to tell Forge to create one with formatted text (with or without an enum)? It helps to give input on what should be done, as well a what shouldn't.

Make a pull request to Forge.

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
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.

 Share



×
×
  • Create New...

Important Information

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