Jump to content

[1.12] Get world age in days


Jacky2611

Recommended Posts

Quick question, is there any way to get a worlds age in days?

I know that I can use getTotalWorldTime() to get the real time the world has existed, but that totally ignores sleeping and cheating admins.

Right now I am thinking about using the world tick event to check all 100 ticks if the world time is smaller than it was the last time I checked, and if yes increase the worlds day counter by one.

 

Is there any better solution? Andwould the overworlds time keep running if all my players are in a different dimension?

Edited by Jacky2611

Here could be your advertisement!

Link to comment
Share on other sites

World#getTotalWorldTime() / 24000

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

Are you sure @Draco18s? I just tried that in a world and whenever I changed the time of day it had absolutely no effect on getTotalWorldTime.

 

 


[05:24:06] [Server thread/INFO]: [STDOUT]: total world time is 103200
[05:24:11] [Server thread/INFO]: [STDOUT]: total world time is 103300
[05:24:16] [Server thread/INFO]: [Player357: Added 300 to the time]
[05:24:16] [main/INFO]: [CHAT] Added 300 to the time
[05:24:16] [Server thread/INFO]: [STDOUT]: total world time is 103400

Here could be your advertisement!

Link to comment
Share on other sites

Sorry, try World.# getWorldTime()

I get the two confused sometimes.

 

And yes, it accounts for sleeping.

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

19 minutes ago, shultzy said:

Actucally, according to the source code this refers to WorldInfo#worldTime which is in the range of 0-23999 (in ticks) or a full day.

Well, that's the funny thing about it. I also thought that it would be in the range of 0-23999 ticks, but apparently it keeps counting up in most cases. I have to get the long back into the range on my own to make my own code work.

 

here is what I wrote to get a proper day count for my worlds. I call updateCurrentTime all 200 ticks from inside a world tick event with the worlds current time.

 

public class DayCounterWorldSavedData extends WorldSavedData {
	private static final String DATA_NAME = DSMain.MODID + "_ExampleData";
	  
	private int ageInDays = 0;
	long lastTime=0;
	
	// Required constructors
	public DayCounterWorldSavedData() {
		super(DATA_NAME);
	}

	public DayCounterWorldSavedData(String s) {
		super(s);
	}
	  
	  
	@Override
	public void readFromNBT(NBTTagCompound nbt) {
		this.ageInDays=nbt.getInteger("ageInDays");
		this.lastTime=nbt.getLong("lastTime");

		
	}
	
	@Override
	public NBTTagCompound writeToNBT(NBTTagCompound compound) {
		NBTTagCompound nbt = new NBTTagCompound();
		
		nbt.setInteger("ageInDays", this.ageInDays);
		nbt.setLong("lastTime", this.lastTime);

		
		return nbt;
	}

	
	public static DayCounterWorldSavedData get(World world) {
		  MapStorage storage = world.getPerWorldStorage();
		  DayCounterWorldSavedData instance = (DayCounterWorldSavedData) storage.getOrLoadData(DayCounterWorldSavedData.class, DATA_NAME);

		  if (instance == null) {
		    instance = new DayCounterWorldSavedData();
		    storage.setData(DATA_NAME, instance);
		  }
		  
		  return instance;
	}
	
	public int getAgeInDays() {
		return this.ageInDays;
	}
	
	public void updateCurrentTime(long currentTime) {
		
		System.out.println("last time: " + this.lastTime);
		System.out.println("current time: " + currentTime);
		
		//we get values above 23999, TO-DO: get it back in range here
		currentTime=currentTime%24000;
		
		if(currentTime<this.lastTime)
    		this.addDay();
    			
		this.lastTime=currentTime;
	}
	
	public void addDay() {
		this.addDays(1);
	}

	public void addDays(int i) {
		this.ageInDays+=1;
		this.markDirty();
		
		System.out.println("The worlds age is "+this.getAgeInDays() +" day(s).");
	}
}

 

Edited by Jacky2611

Here could be your advertisement!

Link to comment
Share on other sites

48 minutes ago, shultzy said:

Actucally, according to the source code this refers to WorldInfo#worldTime which is in the range of 0-23999 (in ticks) or a full day.

The javadoc on that method is wrong. If you follow where it gets its value from you'll find that nothing modulates it down below 24,000.

 

46 minutes ago, Jacky2611 said:

Interesting. It looks like sleepingDoesntResetTheTimeCounter.

But as soon as someone does use a command we are back to 0. Looks like I have to write my own solution.


You mean /time set 0? Well...yeah...

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

OK, I just realized that days don't have to be 24000 ticks long in all dimensions. (I am way too tired for this) Which means that my code above is mostly useless should I ever want to use it somewhere else since there is no way to figure out how many ticks are in another dimension sun cycle.

 

What I could do instead is to trigger a new day whenever the isDaytime boolean for a world changes. If I combine that with my current approach (in order to notice should someone just skip an entire day/night) my system should be (mostly) foolproof.

 

EDIT:

Even /time set day sets it back down to 0. And users don't even have to use commands. As soon as another mod starts messing around with time I am screwed. And as I already said above, I can't rely on the tick count because not all dimension have to have the same day cycle.

 

What I did now is to rely on the world#isDaytime boolean to catch normal day changes while also triggering a change should someone mess around with the server time. This is what my sleep deprived mind came up with.

	public void updateCurrentTime(long currentTime, boolean currentDay) {
		
		//check if someone used a command to reset time or if the night/day switched to day
		if((currentTime<this.lastTime) || ((this.isDay!=currentDay))&&currentDay)
    		    this.addDay();
    			
		this.lastTime=currentTime;
		this.isDay = currentDay;
	}

 

And did I write my reply above in camel case o.O?

Edited by Jacky2611

Here could be your advertisement!

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

    • What's the issue your running into? The link for the image you posted is also broken.
    • What does NetworkHooks.openScreen(player, blockEntity, pPos) do?
    • At an attempt at answering my first question I thought I might be able to change the code to this public static final RegistryObject<StatType<?>> FIREWORK_BOOSTS_USED = STATISTICS.register("firework_boosts_used", () -> new StatType<>(ForgeRegistries.STAT_TYPES, Component.literal("firework_boosts"))); or to this public static final RegistryObject<StatType<?>> FIREWORK_BOOSTS_USED = STATISTICS.register("firework_boosts_used", () -> new StatType<>(BuiltInRegistries.STAT_TYPE, Component.literal("firework_boosts"))); but for the first code snippet the StatType constructor requires a Registry not an IForgeRegistry and for the second code snippet we run into the same NPE issue from above.
    • Hi guys,   I'm using Crafty installed on CasaOS and i want to create a BetterMC4 Server. Unfortunatly i got an error message. This error only appear when using my "online" server because when i create a singleplayer adventure, everythings works just fine. https://github.com/OwNuT/Errors/issues/1   Thank you !
    • I am attempting to create a mod that adds custom statistics to Minecraft. I've had some help from LexManos on the Discord server, but posting a lot of what I posted in the #mod-dev-support-1.20 channel here to have a more permanent spot for discussion.   ---   Minecraft creates their stats like this: public class Stats { public static final StatType<Block> BLOCK_MINED = makeRegistryStatType("mined", BuiltInRegistries.BLOCK); public static final StatType<Item> ITEM_CRAFTED = makeRegistryStatType("crafted", BuiltInRegistries.ITEM); public static final StatType<EntityType<?>> ENTITY_KILLED = makeRegistryStatType("killed", BuiltInRegistries.ENTITY_TYPE); public static final StatType<ResourceLocation> CUSTOM = makeRegistryStatType("custom", BuiltInRegistries.CUSTOM_STAT); public static final ResourceLocation LEAVE_GAME = makeCustomStat("leave_game", StatFormatter.DEFAULT); public static final ResourceLocation PLAY_TIME = makeCustomStat("play_time", StatFormatter.TIME); private static ResourceLocation makeCustomStat(String p_13008_, StatFormatter p_13009_) { ResourceLocation resourcelocation = new ResourceLocation(p_13008_); Registry.register(BuiltInRegistries.CUSTOM_STAT, p_13008_, resourcelocation); CUSTOM.get(resourcelocation, p_13009_); return resourcelocation; } private static <T> StatType<T> makeRegistryStatType(String p_13011_, Registry<T> p_13012_) { Component component = Component.translatable("stat_type.minecraft." + p_13011_); return Registry.register(BuiltInRegistries.STAT_TYPE, p_13011_, new StatType<>(p_13012_, component)); } } and some of the registries in the code above are located in BuiltInRegistries.java like this: /** * @deprecated Forge: Use {@link net.minecraftforge.registries.ForgeRegistries#STAT_TYPES} instead. */ @Deprecated public static final Registry<StatType<?>> STAT_TYPE = forge(Registries.STAT_TYPE, (p_259967_) -> { return Stats.ITEM_USED; }); public static final Registry<ResourceLocation> CUSTOM_STAT = registerSimple(Registries.CUSTOM_STAT, (p_259833_) -> { return Stats.JUMP; }); and the ForgeRegistries#STAT_TYPES looks like this: public static final IForgeRegistry<StatType<?>> STAT_TYPES = active(Keys.STAT_TYPES); So, attempting to follow that and the forge documentation on registries I arrived at this: public class SampleModStatisticsRegistry { public static final DeferredRegister<StatType<?>> STATISTICS = DeferredRegister.create(ForgeRegistries.STAT_TYPES, SampleMod.MODID); public static final RegistryObject<StatType<?>> FIREWORK_BOOSTS_USED = STATISTICS.register("firework_boosts_used", () -> makeRegistryStatType("firework_boosts_used", BuiltInRegistries.CUSTOM_STAT)); // code stolen from Minecraft's Stats file public static <T> StatType<T> makeRegistryStatType(String p_13011_, Registry<T> p_13012_) { Component component = Component.translatable("stat_type.minecraft." + p_13011_); return Registry.register(BuiltInRegistries.STAT_TYPE, p_13011_, new StatType<>(p_13012_, component)); } } Clearly, registering a stat twice in a row is the wrong way to do it, but the makeRegistryStatType function was used as it does give the correct return type for the code I had come up with so far.   ---   At this point the game crashes upon trying to use Player#awardStat, because of a NPE that occurs in Stat#locationToKey. java.lang.NullPointerException: Cannot invoke "net.minecraft.resources.ResourceLocation.toString()" because "p_12866_" is null at net.minecraft.stats.Stat.locationToKey(Stat.java:26) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading} at net.minecraft.stats.Stat.buildName(Stat.java:22) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading} at net.minecraft.stats.Stat.<init>(Stat.java:15) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading} at net.minecraft.stats.StatType.lambda$get$0(StatType.java:25) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading} at java.util.Map.computeIfAbsent(Map.java:1054) ~[?:?] {re:mixin} at net.minecraft.stats.StatType.get(StatType.java:24) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading} at net.minecraft.stats.StatType.get(StatType.java:38) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading} at net.minecraft.world.entity.player.Player.awardStat(Player.java:1364) ~[forge-1.20.2-48.1.0_mapped_official_1.20.2-recomp.jar%23199!/:?] {re:classloading,pl:accesstransformer:B} Which happens in the 2nd Stat#locationToKey in the Stat#buildName function. public static <T> String buildName(StatType<T> p_12863_, T p_12864_) { return locationToKey(BuiltInRegistries.STAT_TYPE.getKey(p_12863_)) + ":" + locationToKey(p_12863_.getRegistry().getKey(p_12864_)); } The .getRegistry returns the this.registry class variable with a value of {MappedRegistry@#####} "Registry[ResourceKey[minecraft:root / minecraft:custom_stat] (Stable)]" The value passed into .getKey is "samplemod:fireworks_boosts_used" but the reference found is null. So Lex's conclusion was that "[I'm] passing in null because the registry doesnt have an entry for your custom stat instance. So.. register your custom stat instance in that registry."   ---   So I suppose at this point my questions are 1. How can I create a supplier that returns a Supplier<? extends StatType<?>> to replace the () -> makeRegistryStatType("firework_boosts_used", BuiltInRegistries.CUSTOM_STAT) line of code. 2. Is my registry entry public static final RegistryObject<StatType<?>> FIREWORK_BOOSTS_USED the correct type? 3. Do I need to do anything additional, like create my own StatType<> or my own Stat<> ?
  • Topics

×
×
  • Create New...

Important Information

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