[1.14.3] send message to chat

[MrNoodles75]

So what I would like to do is make it so when you touch a block (click on it)

then the block will say something like "what are you doing!" or "hello"

in the chat

but I seem to be getting stuck at the printing it to the chat.

I looked around and I found

    protected Minecraft minecraft;

@Override
    public boolean onBlockActivated(BlockState state, World world, BlockPos pos, PlayerEntity player, Hand hand, BlockRayTraceResult result) {
        if (!world.isRemote) {
            TileEntity tileEntity = world.getTileEntity(pos);
            if (tileEntity instanceof INamedContainerProvider) {
                NetworkHooks.openGui((ServerPlayerEntity) player, (INamedContainerProvider) tileEntity, tileEntity.getPos());
            } else {
                throw new IllegalStateException("Our named container provider is missing!");
            }
            return true;
        }

      //THIS SENDS THE MESSAGE
        this.minecraft.player.sendChatMessage("hello I am a COMPUTER");
      //THAT SENDS THE MESSAGE

        return super.onBlockActivated(state, world, pos, player, hand, result);
    }

But that does not work, instead, the game crashes.

[MrNoodles75]

@diesieben07

thanks, now I am doing

player.sendMessage(new TranslationTextComponent("computerTalk.hello"));

but it now prints it twice and throughs an error:

[16:40:24.467] [Server thread/FATAL] [minecraft/ThreadTaskExecutor]: Error executing task on Server
java.lang.IllegalStateException: Our named container provider is missing!
	at com.runningmanstudios.harrysmod.blocks.Computer.onBlockActivated(Computer.java:62) ~[?:?] {}
	at net.minecraft.block.BlockState.onBlockActivated(BlockState.java:296) ~[?:?] {}
	at net.minecraft.server.management.PlayerInteractionManager.func_219441_a(PlayerInteractionManager.java:366) ~[?:?] {}
	at net.minecraft.network.play.ServerPlayNetHandler.processTryUseItemOnBlock(ServerPlayNetHandler.java:896) ~[?:?] {}
	at net.minecraft.network.play.client.CPlayerTryUseItemOnBlockPacket.processPacket(CPlayerTryUseItemOnBlockPacket.java:42) ~[?:?] {}
	at net.minecraft.network.play.client.CPlayerTryUseItemOnBlockPacket.processPacket(CPlayerTryUseItemOnBlockPacket.java:12) ~[?:?] {}
	at net.minecraft.network.PacketThreadUtil.lambda$checkThreadAndEnqueue$0(PacketThreadUtil.java:14) ~[?:?] {}
	at net.minecraft.util.concurrent.TickDelayedTask.run(TickDelayedTask.java:20) ~[?:?] {}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.run(ThreadTaskExecutor.java:131) [?:?] {pl:accesstransformer:B}
	at net.minecraft.util.concurrent.RecursiveEventLoop.run(RecursiveEventLoop.java:22) [?:?] {}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.driveOne(ThreadTaskExecutor.java:106) [?:?] {pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.func_213205_aW(MinecraftServer.java:713) [?:?] {pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.driveOne(MinecraftServer.java:707) [?:?] {pl:accesstransformer:B}
	at net.minecraft.util.concurrent.ThreadTaskExecutor.drainTasks(ThreadTaskExecutor.java:93) [?:?] {pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.runScheduledTasks(MinecraftServer.java:692) [?:?] {pl:accesstransformer:B}
	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:637) [?:?] {pl:accesstransformer:B}
	at java.lang.Thread.run(Thread.java:748) [?:1.8.0_221] {}

 

[MrNoodles75]

 

 

0

 Advanced issue found

▲

 

 

1 minute ago, diesieben07 said:

Yes, because you are doing it on server and client.

@diesieben07

is there a way to make it go only on the client or the server, so it only prints once

[MrNoodles75]

@diesieben07

that is correct but my excuse is that I am new to this whole modding thing and there was only one tutorial I could find on youtube.

but...

I did make it work, and although it is probably inefficient it does work.

 

so what I found was that when clicking the block it would run the method twice(before you ask I couldn't find a reason) so I coded it to only say something every other time the method is run.

    
    protected int conversation = 1;
    protected boolean isTalking = false;
    protected boolean canTalk = true;

@Override
    public boolean onBlockActivated(BlockState state, World world, BlockPos pos, PlayerEntity player, Hand hand, BlockRayTraceResult result) {
        if (canTalk) {
            isTalking = true;
            canTalk = false;
        } else {
            canTalk = true;
        }

        if (isTalking) {
            if (conversation == 1) {
                player.sendMessage(new TranslationTextComponent("computerTalk.hello"));
                isTalking = false;
                conversation++;
            } else if (conversation == 2) {
                player.sendMessage(new TranslationTextComponent("computerTalk.whatdoyouneed"));
                isTalking = false;
                conversation++;
            } else if (conversation == 3) {
                player.sendMessage(new TranslationTextComponent("computerTalk.goodbye"));
                isTalking = false;
                conversation = 1;
            }
        }

        return true;
    }

witch works perfectly.

[poopoodice]

31 minutes ago, MrNoodles75 said:

@diesieben07

that is correct but my excuse is that I am new to this whole modding thing and there was only one tutorial I could find on youtube.

but...

I did make it work, and although it is probably inefficient it does work.

 

so what I found was that when clicking the block it would run the method twice(before you ask I couldn't find a reason) so I coded it to only say something every other time the method is run.

    
    protected int conversation = 1;
    protected boolean isTalking = false;
    protected boolean canTalk = true;

@Override
    public boolean onBlockActivated(BlockState state, World world, BlockPos pos, PlayerEntity player, Hand hand, BlockRayTraceResult result) {
        if (canTalk) {
            isTalking = true;
            canTalk = false;
        } else {
            canTalk = true;
        }

        if (isTalking) {
            if (conversation == 1) {
                player.sendMessage(new TranslationTextComponent("computerTalk.hello"));
                isTalking = false;
                conversation++;
            } else if (conversation == 2) {
                player.sendMessage(new TranslationTextComponent("computerTalk.whatdoyouneed"));
                isTalking = false;
                conversation++;
            } else if (conversation == 3) {
                player.sendMessage(new TranslationTextComponent("computerTalk.goodbye"));
                isTalking = false;
                conversation = 1;
            }
        }

        return true;
    }

witch works perfectly.

I think if you put this in your block class, every instance of this block will share the same values.

[MrNoodles75]

it still works

[MrNoodles75]

7 minutes ago, diesieben07 said:

No.

well it does

 

[DaemonUmbra]

Judging from your package naming you are following a HarryTalks tutorial, throw the whole thing out and start with a tutorial that's not garbage

[MrNoodles75]

@DaemonUmbra

I actually followed Jorrit Tyberghein

[DavidM]

40 minutes ago, MrNoodles75 said:

well it does

 

No, it does not.

Your block will break once there are multiple instance of your blocks in your world, as all of them use the same value stored in a singleton of your block.

Use TileEntity instead.

Edited December 7, 2019 by DavidM

[Draco18s]

3 hours ago, MrNoodles75 said:

@DaemonUmbra

I actually followed Jorrit Tyberghein

Not the worst stuff I've seen, but I didn't watch all of it.

[RaphGamingz]

31 minutes ago, Draco18s said:

Not the worst stuff I've seen, but I didn't watch all of it.

I've watched most of it and most of it is OK, I had to make my own changes to the code. 

[MrNoodles75]

21 hours ago, Draco18s said:

Not the worst stuff I've seen, but I didn't watch all of it.

@Draco18s

I have been figuring everything out since then, by going through the default code and getting information on how to do something