Création d'un mod client-side qui notifie son utilisateur quand son inventaire est full

robin4002

L’event est toujours appelé à chaque tick. 300 tick c’est 12 secondes, tu es sûr que le message ne s’affiche pas toutes les 12 secondes ? (au passage tes return ne servent absolument à rien).

Gess1t

@robin4002
je l’ai modifié, le message s’envois toutes les 12 secondes quand j’ai un inventaire plein, mais il faut attendre 12s quand je vais ingame pour que le premiers message apparaît, pareil quand je recupère un item au sol pour remplir l’inventaire, 12s après avoir recup l’item, le message est envoyé

robin4002

Vu comment ton code est fait c’est normal.

Il faudrait utiliser un boolean et mettre ceci sur false dès que le message est affiché et si elle est sur false lancer le compteur puis remettre sur true à la fin du compteur.

Gess1t

@robin4002

ya une fonction pour detecter un message client side?
si c’est pas le cas au pire j’ai juste à utiliser une variable right?

robin4002

Pourquoi tu veux une fonction pour ça ?
Tu as justement besoin d’une variable en effet.

Gess1t

@robin4002 a dit dans Création d'un mod client-side qui notifie son utilisateur quand son inventaire est full :

Vu comment ton code est fait c’est normal.

Il faudrait utiliser un boolean et mettre ceci sur false dès que le message est affiché et si elle est sur false lancer le compteur puis remettre sur true à la fin du compteur.

donc je créer une bolean timer = true; au début de ma method chkInv, après mon message je met timer = !timer;
mais je suis sensé lancer le check comment si je ne peux pas lancer le compteur en dehors de la boucle if?
je suis sensé run if(Minecraft.getMinecraft().world.getTotalWorldTime() % 240 == 3L ) comme première commande? comme genre :

EDIT: le message n’est plus déclenché donc j’ai ya un truc qui va pas, j’me relis 2s pour voir.
RE: je vois pas ce qui empêche le message de s’envoyer.
—> bon bas ma method n’a plus l’air de fonctionner dutout, pas d’errors, impossible de savoir d’où ça vient. même avec un manual undo

EDIT2: les backups, c’est la vie. ducoup ma class de backup marche, juste perdu la boolean.
quand j’ai supprimé la bolean et que j’ai remis la fonction worldgetTotalWorldTime() manuellement à sa place, le problème étais toujours là, donc aucune idée de pourquoi j’ai eu le problème d’avant.

@SubscribeEvent
	public static void chkInv(TickEvent.ClientTickEvent event) throws InterruptedException {
		
		boolean timer = false;
		if(Minecraft.getMinecraft().player != null ) {
			if(Minecraft.getMinecraft().world.getTotalWorldTime() % 200 == 3L && timer = false)
				int s = 0;
				for(int i=0; i < 37; i++) {
					if(!Minecraft.getMinecraft().player.inventory.getStackInSlot(i).isEmpty() && Minecraft.getMinecraft()
						.player.inventory.getStackInSlot(i).getCount() == Minecraft.getMinecraft().player.inventory.getStackInSlot(i).getMaxStackSize()) {
						s++;
						if(s > 35) {
							if(event.phase == TickEvent.Phase.END && timer == false) {
								Minecraft.getMinecraft().player.sendMessage(new TextComponentString("Your inventory is full"));
								timer = !timer;
								return;
							}
						}
					return;
					}
				
				}
			
				
			}
			
			{
			return;

		}
		
	}

Gess1t

j’étais en train de créer mon fichier de config puis j’ai tenté de démarrer le jeu mais ça a crash et la ligne “if(cfg.haschanged()) {” en serait la cause
mon fichier config

public static void readConfig() {
		Configuration cfg = CommonProxy.config;
		try {
            cfg.load();
            initGeneralConfig(cfg);
        } finally {
            if (cfg.hasChanged()) {
                cfg.save();
            }
        }
    }

le crash report :

net.minecraftforge.fml.common.LoaderExceptionModCrash: Caught exception from Full Inventory Checker (fullinvchecker)
Caused by: java.lang.NullPointerException
	at Gess.mod.config.Config.readConfig(Config.java:21)
	at Gess.mod.Main.preInit(Main.java:41)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at net.minecraftforge.fml.common.FMLModContainer.handleModStateEvent(FMLModContainer.java:611)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:91)
	at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:150)
	at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:76)
	at com.google.common.util.concurrent.MoreExecutors$DirectExecutor.execute(MoreExecutors.java:399)
	at com.google.common.eventbus.Subscriber.dispatchEvent(Subscriber.java:71)
	at com.google.common.eventbus.Dispatcher$PerThreadQueuedDispatcher.dispatch(Dispatcher.java:116)
	at com.google.common.eventbus.EventBus.post(EventBus.java:217)
	at net.minecraftforge.fml.common.LoadController.sendEventToModContainer(LoadController.java:253)
	at net.minecraftforge.fml.common.LoadController.propogateStateMessage(LoadController.java:231)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at com.google.common.eventbus.Subscriber.invokeSubscriberMethod(Subscriber.java:91)
	at com.google.common.eventbus.Subscriber$SynchronizedSubscriber.invokeSubscriberMethod(Subscriber.java:150)
	at com.google.common.eventbus.Subscriber$1.run(Subscriber.java:76)
	at com.google.common.util.concurrent.MoreExecutors$DirectExecutor.execute(MoreExecutors.java:399)
	at com.google.common.eventbus.Subscriber.dispatchEvent(Subscriber.java:71)
	at com.google.common.eventbus.Dispatcher$PerThreadQueuedDispatcher.dispatch(Dispatcher.java:116)
	at com.google.common.eventbus.EventBus.post(EventBus.java:217)
	at net.minecraftforge.fml.common.LoadController.distributeStateMessage(LoadController.java:148)
	at net.minecraftforge.fml.common.Loader.preinitializeMods(Loader.java:604)
	at net.minecraftforge.fml.client.FMLClientHandler.beginMinecraftLoading(FMLClientHandler.java:266)
	at net.minecraft.client.Minecraft.init(Minecraft.java:508)
	at net.minecraft.client.Minecraft.run(Minecraft.java:416)
	at net.minecraft.client.main.Main.main(Main.java:118)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
	at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.lang.reflect.Method.invoke(Unknown Source)
	at net.minecraftforge.gradle.GradleStartCommon.launch(GradleStartCommon.java:97)
	at GradleStart.main(GradleStart.java:25)

si jamais tu veux voir ma method preInit :
line 41 = Config.readConfig();

@EventHandler
	public void preInit(FMLPreInitializationEvent event){
		File directory = event.getModConfigurationDirectory();
        config = new Configuration(new File(directory.getPath(), "FullInventoryChecker.cfg"));
        Config.readConfig();
	}

j’ai aussi éssayer de faire jouer un son du côté client avec Minecraft.getMinecraft().getSoundHandler().playSound(); mais ya une erreur dans mon code, quelque soit le code, j’ai éssayé de voir dans des exemples ou d’autres mods open source, mais ya rien qui marche

pour finir, je me demandais si dans

if(!Minecraft.getMinecraft().player.inventory.getStackInSlot(i).isEmpty() && Minecraft.getMinecraft()
				.player.inventory.getStackInSlot(i).getCount() == Minecraft.getMinecraft().player.inventory.getStackInSlot(i).getMaxStackSize())

je pouvais ajouter l’ID d’un item comme exception car sur certain serveurs comme hypixel skyblock ou wynncraft, ya une limite custom de stacks par items .

je vais passer par un else et ajouter les ID manuellement, et si j’arrive à faire fonctionner ce fichier de config, alors j’ajouterais un filtre d’exception. je n’arrive pas à trouver comment comparer le stack trouvé avec getStackInSlot avec un item genre mineccraft:head (je supose je dois utiliser getItem() mais ya aucun moyen de savoir comment elle marche)

Gess1t

vu que j’avais plus de réponses sur ce forum, j’ai utilisé le forum officiel, puis j’ai fais une pause, car perdre son temps pour rien et pas spécialement mon truc, j’éssaye de faire marcher “e.entity” pour trouver un moyen de faire fonctionner le mod sur un serveur, le problème étant que “e.” ne marche pas, donc je me demandais à quoi ça correspondait. e.entity.PNG

je viens de faire un texte et genre event.getEntityPlayer() et tous les autres arguments pour minecraft ne fonctionnent plus et remplacé par ça :
Capture d’écran (587).png

event.[autre argument en rapport avec mc] ne marche plus sur tous mes projets.

robin4002

C’est quoi le contexte complet dans lequel se trouve ta variable event ?

Gess1t

dans une methode test avec comme genre

@SubscribeEvent
public void test(TickEvent.ClientTickEvent event) {
if(event.
}

et ya rien qui s’affiche alors que c’était le cas avant

robin4002

TickEvent.ClientTickEvent est bien reconnu ?

Gess1t

yep, sur mes autres methodes aussi

Edit: pour ma méthode qui détecte client-side quand mon inventaire change (je cherche toujouts à la faire marcher):

	private ItemStack[] previous;
	private ItemStack[] now;
	
	@SubscribeEvent
	public void PckUpItm(TickEvent.ClientTickEvent event) {
		if(Minecraft.getMinecraft().player != null) {
			EntityPlayer player = Minecraft.getMinecraft().player;
			int g = 0;
		for(int e=0; e < 36; e++) {
			System.out.println("checking slot " + e);
			ItemStack stack = player.inventory.getStackInSlot(e);
            if(previous[e] != stack) {
					if(!stack.isEmpty() && stack.getCount() == stack.getMaxStackSize()) {
						g++;
						System.out.println("g = " + g);
						previous[e] = stack;
						System.out.println("Itemstack " + e + "copied");
						if(g > 34) {
						Minecraft.getMinecraft().player.sendMessage(new TextComponentString("Your inventory is full"));
						return;
						
					}
				
                               }
				
			}
				
		}
					
	}				

}

la méthode ne trigger même pas ingame, elle est sesné être déclenchée tous les ticks pourtant non?

robin4002

Étrange, essaies de refaire un setup de la workspace ?

Et pour l’event, vérifies que la classe d’event est bien enregistré.

Gess1t

j’ai refais un workspace, j’ai utilisé le build recommandé de forge 1.12.2 cette fois, j’ai remis le code, tous semble fonctionner sauf ma fonction de detection, lorsque j’éssaie de debug, je rejoinds un serveur et là, le jeu crash, à cause de ma ligne

 if(previous[e] != stack) {

pourquoi cette ligne par contre ?

j’avais vu que d’utiliser [e] permetrais de stocker plusieurs valeur dans une même variable, comme une genre de liste de supose, mais ça a pas l’air de fonctionner donc je me demande si je devrais pas stocker chaque item dans chaque slot 1 / 1. à moins qu’il y ai un moyen de fix ça.
car c’est sensé pouvoir stocker chaque item en fonction de la valeur de e, c’est à dire des slots de l’inventaire (de 0 à 35)

EDIT: j’ai edit quelques truc, maintenant ça crash pas quand je vais sur le serveur, mais ça me timeout .5s après avoir rejoinds le serveur, et la méthode ne marcge toujours pas en solo

robin4002

Ton array previous n’est jamais initialisé, donc normal que ça plante …

Il faudrait les logs pour savoir pourquoi tu timeout.

Gess1t

@robin4002

[20:43:41] [main/INFO] [minecraft/GuiConnecting]: Connecting to localhost, 25565
[20:43:41] [Netty Client IO #4/INFO] [FML]: Aborting client handshake "VANILLA"
[20:43:41] [Netty Client IO #4/INFO] [FML]: [Netty Client IO #4] Client side vanilla connection established
[20:44:06] [Netty Client IO #4/ERROR] [FML]: NetworkDispatcher exception
java.io.IOException: Une connexion existante a dû être fermée par l’hôte distant // timeout
    at sun.nio.ch.SocketDispatcher.read0(Native Method) ~[?:1.8.0_212]
    at sun.nio.ch.SocketDispatcher.read(Unknown Source) ~[?:1.8.0_212]
    at sun.nio.ch.IOUtil.readIntoNativeBuffer(Unknown Source) ~[?:1.8.0_212]
    at sun.nio.ch.IOUtil.read(Unknown Source) ~[?:1.8.0_212]
    at sun.nio.ch.SocketChannelImpl.read(Unknown Source) ~[?:1.8.0_212]
    at io.netty.buffer.PooledUnsafeDirectByteBuf.setBytes(PooledUnsafeDirectByteBuf.java:288) ~[PooledUnsafeDirectByteBuf.class:4.1.9.Final]
    at io.netty.buffer.AbstractByteBuf.writeBytes(AbstractByteBuf.java:1100) ~[AbstractByteBuf.class:4.1.9.Final]
    at io.netty.channel.socket.nio.NioSocketChannel.doReadBytes(NioSocketChannel.java:372) ~[NioSocketChannel.class:4.1.9.Final]
    at io.netty.channel.nio.AbstractNioByteChannel$NioByteUnsafe.read(AbstractNioByteChannel.java:123) [AbstractNioByteChannel$NioByteUnsafe.class:4.1.9.Final]
    at io.netty.channel.nio.NioEventLoop.processSelectedKey(NioEventLoop.java:624) [NioEventLoop.class:4.1.9.Final]
    at io.netty.channel.nio.NioEventLoop.processSelectedKeysOptimized(NioEventLoop.java:559) [NioEventLoop.class:4.1.9.Final]
    at io.netty.channel.nio.NioEventLoop.processSelectedKeys(NioEventLoop.java:476) [NioEventLoop.class:4.1.9.Final]
    at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:438) [NioEventLoop.class:4.1.9.Final]
    at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:858) [SingleThreadEventExecutor$5.class:4.1.9.Final]
    at java.lang.Thread.run(Unknown Source) [?:1.8.0_212]
[20:44:07] [main/INFO] [FML]: Applying holder lookups
[20:44:07] [main/INFO] [FML]: Holder lookups applied

Les logs pour le timeout
Un autre mod a fais la même chose et ça marche

robin4002

Ce sont les logs du client ou du serveur ça ?
Car si c’est le client qui timeout, ça veut dire que le serveur ne répond plus, donc l’erreur est probablement sur le serveur.

Gess1t

@robin4002 nan, le serveur marche parfaitement, je peux me connecter dessus avec mon compte mc sans problème, ce sont des logs client btw, car ya aucune info côté serveur sur la raison du timeout, il dit juste que tel client a timeout. Le serveur run en local.

robin4002

Je ne sais pas du tout d’où cela peut venir dans ce cas

Gess1t

@robin4002 il ont aussi l’air de galérer du côté du forum officiel ahahah, moi ça me dépasse complêtement, j’ai enviee de le terminer, mais forge veux pas =/

j’étais tester de initialize previous avec null, les crash ont arrêté, la method ne fonctionne toujours pas donc je supose que ya un problème avec ma variable previous

forcer le jeu à faire marcher la method via un call fais crash le jeu avec comme erreur ma ligne pour comparer previous au stack actuel e

je pensse que je vais devoir hard code tous mes slots pour stock tous mes items vu que ça marche pas avec ce que forge me donne