how to prevent spagetti code | Together Java | Page 1

dry ospreyBOT May 3, 2026, 7:57 AM

#

<@&987246399047479336> please have a look, thanks.

#

Here is an AI assisted attempt to answer your question 🤖. Maybe it helps! In any case, a human is on the way 👍. To continue talking to the AI, you can use </chatgpt:1108714622413963314>.

#

TJ-Bot

I have a big minecraft plugin for my custom minecraft server but the structure is really bad

To prevent spaghetti code, especially in a large Minecraft plugin, focus on clean architecture and code organization:

Modularize Your Code: Break down your plugin into smaller, well-defined classes and packages. Each class should have a single responsibility (Single Responsibility Principle).
Use Design Patterns: Patterns like Command (for handling commands), Listener (for events), and Factory (for object creation) can give structure.
Separate Concerns: Keep your game logic, data management, and command handling in separate layers.
Consistent Naming & Conventions: Follow consistent naming conventions to make your code easier to navigate.
Avoid Large Methods: Refactor long methods into smaller ones that perform one specific task.
Use Interfaces and Abstractions: This helps make your code more flexible and testable.
Documentation and Comments: Maintain clear comments, and consider adding a README or architecture overview.

Remember, refactoring incrementally works best for large existing projects. Analyze the parts that cause the most confusion or bugs first.

Useful links:

waxen gull May 3, 2026, 8:04 AM

#

How can i structure the project better

quiet zenith May 3, 2026, 8:05 AM

#

have you got static variable + methods

#

remove them all

waxen gull May 3, 2026, 8:05 AM

#

not really tbh

#

but i just have a lot of classes and packages in packages

#

cuz the project is getting really big

quiet zenith May 3, 2026, 8:06 AM

#

how big is big?

#

200 file?

waxen gull May 3, 2026, 8:06 AM

#

hmm idk

quiet zenith May 3, 2026, 8:06 AM

#

just don't do more deep than 3 package deep

waxen gull May 3, 2026, 8:07 AM

#

something arround 100 i think

quiet zenith May 3, 2026, 8:08 AM

#

can you say why you think it is spagetti?

#

are you getting lost?

waxen gull May 3, 2026, 8:08 AM

#

yes

#

i gotta use classes from a lot of places for 1 script

quiet zenith May 3, 2026, 8:09 AM

#

make your classes do 1 thing with a small surface interface

waxen gull May 3, 2026, 8:09 AM

#

my main class is a mess

#

300 lines for only initialising stuff

waxen gull May 3, 2026, 8:10 AM

#

quiet zenith make your classes do 1 thing with a small surface interface

yea i dont really use interfaces a lot yet

#

i think i understand what it does but not the use case for it yet

quiet zenith May 3, 2026, 8:11 AM

#

do you have it public?

#

your code?

waxen gull May 3, 2026, 8:11 AM

#

no not really

#

i vibe coded a lot too

#

but i started learning how to code more

#

and i wanna stop

#

the code is old tbh

#

cuz the project i made was old and since then i learned al ot

#

i wanna start all over again

quiet zenith May 3, 2026, 8:12 AM

#

ok, well keep it all for now

#

but start from 0 in another module

waxen gull May 3, 2026, 8:12 AM

#

the code has al ot of bugs too and i can do it a lot better now

quiet zenith May 3, 2026, 8:12 AM

#

spigot plugin?

waxen gull May 3, 2026, 8:12 AM

#

paper plugin ig

#

wait i haven't clicked this so i dont think so

quiet zenith May 3, 2026, 8:12 AM

#

ok

waxen gull May 3, 2026, 8:13 AM

#

but i use some paper import things

quiet zenith May 3, 2026, 8:13 AM

#

what dod you have right now?

waxen gull May 3, 2026, 8:13 AM

#

quiet zenith what dod you have right now?

atm and phone system

#

and some other htings

#

but those are the biggest things ig

waxen gull May 3, 2026, 8:13 AM

#

waxen gull atm and phone system

phone system integrates with simplevoicechat etc

quiet zenith May 3, 2026, 8:13 AM

#

your main plugin should look like this

@NullMarked
public class MainPlugin extends JavaPlugin {
  public void registerCommands(ReloadableRegistrarEvent<Commands> commands) {
    commands.registrar().register(FlySpeedCommand.build(this));
    commands.registrar().register(BuildCommand.build(this));
  }

  @Override
  public void onEnable() {
    var manager = Bukkit.getPluginManager();
    for (var event : eventsToRegister()) {
      manager.registerEvents(event, this);
    }
    this.getLifecycleManager().registerEventHandler(LifecycleEvents.COMMANDS, this::registerCommands);
  }

  private List<Listener> eventsToRegister() {
    List<Listener> events = new ArrayList<>(7);
    events.add(new PlayerJoined(this));
    events.add(new ServerListPing(this));
    events.add(AsyncPlayerChat.configured(this));
    events.add(new PlayerDeathAndRespawn(this));
    events.add(new PlayerFish(this));
    events.add(new PlayerAutoRegen(this));
    events.add(new RemoveCreeper(this));
    return events;
  }
}

#

just a simple place to link it all

lyric spade May 3, 2026, 8:14 AM

#

how to prevent spagetti code
experience

#

theres no magic formula to it

#

its what differentiates the junior from the senior

#

its pure experience

waxen gull May 3, 2026, 8:15 AM

#

lyric spade its what differentiates the junior from the senior

okay thx

waxen gull May 3, 2026, 8:15 AM

#

quiet zenith your main plugin should look like this ```java @NullMarked public class MainPlug...

yea i have different functions to initialise certain stuff

#

for example listeners

#

etc

lyric spade May 3, 2026, 8:15 AM

#

the more u code, the more u know how to code something in a particular situation in the best way, so to speak

quiet zenith May 3, 2026, 8:15 AM

#

public record ServerListPing(JavaPlugin plugin) implements Listener {
  public ServerListPing {
    plugin.getLogger().info("ServerListPing");
  }

  @EventHandler
  public void onEvent(PaperServerListPingEvent event) {
    int numPlayer = Math.max(event.getNumPlayers(), 0);
    event.setNumPlayers(numPlayer);
    event.setMaxPlayers(100);
    event.motd(Component.text("§aWelcome to the server!"));
  }
}

example listener

waxen gull May 3, 2026, 8:18 AM

#

quiet zenith your main plugin should look like this ```java @NullMarked public class MainPlug...

why do you add the listeners to an array?

#

this is a small part of my main class 😭

📎 message.txt

dry ospreyBOT May 3, 2026, 8:19 AM

#

waxen gull this is a small part of my main class 😭

I uploaded your attachments as Gist. This makes them more accessible, for example to mobile users.

waxen gull May 3, 2026, 8:19 AM

#

but i got a lot more experience now

quiet zenith May 3, 2026, 8:20 AM

#

waxen gull why do you add the listeners to an array?

because it allow me later on to have subpackage with a group of listener that work together
then I would just include it all later with addall

#

many feature require multiple listeners to make work

waxen gull May 3, 2026, 8:21 AM

#

quiet zenith many feature require multiple listeners to make work

true

waxen gull May 3, 2026, 8:21 AM

#

quiet zenith because it allow me later on to have subpackage with a group of listener that wo...

do you have an example?

#

would you recommend to make a lot of small plugins or 1 big one

#

cuz the code is seperated then but at the same time i need to manage multiple build.gradles

#

and i can't access the classes as easy

quiet zenith May 3, 2026, 8:23 AM

#

I just make module with each feature

#

then make a module in which I include all feature I need

waxen gull May 3, 2026, 8:24 AM

#

i dont really know what a module is yet

#

let me search it up

waxen gull May 3, 2026, 8:25 AM

#

quiet zenith many feature require multiple listeners to make work

i had a system where the phone should always be in the left slot you weren't allowed to drop it put it in your offhand etc

#

i had multiple listeners for that too

quiet zenith May 3, 2026, 8:26 AM

#

waxen gull i had multiple listeners for that too

yes, so I would make a package

#

different class for each part of the system

#

then just plug them all in the register

waxen gull May 3, 2026, 8:28 AM

#

okay i had a gui system in my old plugin and a lot of classes needed it cuz i could make really ez guis with it i should make a package for that too right?

quiet zenith May 3, 2026, 8:28 AM

#

events.addAll(Atm.listeners(this));

#

something like this

#

Then I would just add it all in it

quiet zenith May 3, 2026, 8:29 AM

#

waxen gull okay i had a gui system in my old plugin and a lot of classes needed it cuz i co...

yes

#

group it

#

and make it 1 thing

#

then make small method that take max 3-4 parameters and give you a gui all built

#

some factory methods

#

the goal is to only use these factory methods elsewhere in other package

waxen gull May 3, 2026, 8:31 AM

#

huh i see a really weird system now too my plugin extended the gui class 😭

quiet zenith May 3, 2026, 8:31 AM

#

it become your public api in the rest of the code

waxen gull May 3, 2026, 8:32 AM

#

waxen gull huh i see a really weird system now too my plugin extended the gui class 😭

and i had a lot of gui's that isn't good right?

quiet zenith May 3, 2026, 8:32 AM

#

I dunno, you are talking about minecraft internal gui?

#

the one with buttons

waxen gull May 3, 2026, 8:33 AM

#

quiet zenith I dunno, you are talking about minecraft internal gui?

no the container gui

#

it is basically a chest

#

wait i can send a screen

quiet zenith May 3, 2026, 8:33 AM

#

wow, this look complicated a lot xD

#

https://docs.papermc.io/paper/dev/dialogs/

Dialog API | PaperMC Docs

A guide to the dialog API introduced in 1.21.7.

#

you know dialogs exist?

waxen gull May 3, 2026, 8:34 AM

#

no im not using dialog

quiet zenith May 3, 2026, 8:34 AM

#

ok

waxen gull May 3, 2026, 8:34 AM

#

#

i showed this without the resourecpack

#

so u know what i mean now

quiet zenith May 3, 2026, 8:35 AM

#

yeah I get it

waxen gull May 3, 2026, 8:35 AM

#

i extended safegui (the system)

safegui removes your inv when you open it

#

and restores it when you close it

quiet zenith May 3, 2026, 8:36 AM

#

so it use a plugin?

waxen gull May 3, 2026, 8:36 AM

#

it uses my own

#

and itemsadder

quiet zenith May 3, 2026, 8:36 AM

#

ok

waxen gull May 3, 2026, 8:36 AM

#

this is with the resourcepack

#

prob gonna change the textures etc

#

😭

quiet zenith May 3, 2026, 8:36 AM

#

nice

waxen gull May 3, 2026, 8:37 AM

#

looks a bit fat now

waxen gull May 3, 2026, 8:37 AM

#

quiet zenith nice

thank u 😄

#

so the system is good but isn't it weird that it extends a class? to make the gui

#

i prob should use parameters right?

#

or not idk

#

😭

#

im confused

quiet zenith May 3, 2026, 8:37 AM

#

peepo_shrug not really

waxen gull May 3, 2026, 8:37 AM

#

it is a bit early too first thing i went to do is made this qusetion

quiet zenith May 3, 2026, 8:37 AM

#

it is a choice

waxen gull May 3, 2026, 8:37 AM

#

okay

#

yea okay i realise minecraft extends Screen too for screens (this is mc modding tho)

#

so that isn't smth weird to do

#

okay i prob just gotta go start with it

#

gotta go eat first

#

okay im done eating

#

one small question now 😭

#

i first wanna make the atm

#

should i just make a package called GUI then

quiet zenith May 3, 2026, 8:53 AM

#

sure

waxen gull May 3, 2026, 8:53 AM

#

and put there the gui class in?

#

and do you recommend a lot of small plugins and then an api

#

or 1 big one

quiet zenith May 3, 2026, 8:55 AM

#

1 big one that you setup as you want if the plugin is just for your server

#

if it is for others, lots of small one with configuration

waxen gull May 3, 2026, 8:56 AM

#

it is 1 big one

#

but im planning on that it's gonna be a really big server

#

with a big plugin

quiet zenith May 3, 2026, 8:56 AM

#

but just for you?

waxen gull May 3, 2026, 8:56 AM

#

quiet zenith but just for you?

no

#

other people will join

quiet zenith May 3, 2026, 8:56 AM

#

I mean just for 1 server

#

yours

waxen gull May 3, 2026, 8:56 AM

#

yea

quiet zenith May 3, 2026, 8:57 AM

#

1 big is good

waxen gull May 3, 2026, 8:57 AM

#

alr 😄

#

are a few static variables bad?

quiet zenith May 3, 2026, 9:01 AM

#

yes

waxen gull May 3, 2026, 9:01 AM

#

for the main class too?

#

cuz there is only 1 main class

quiet zenith May 3, 2026, 9:01 AM

#

it point to spaghetti

waxen gull May 3, 2026, 9:01 AM

#

im not really gonna turn it into a object

quiet zenith May 3, 2026, 9:01 AM

#

yes

waxen gull May 3, 2026, 9:01 AM

#

quiet zenith it point to spaghetti

thx

quiet zenith May 3, 2026, 9:01 AM

#

you never need them

waxen gull May 3, 2026, 9:03 AM

#

true

quiet zenith May 3, 2026, 9:04 AM

#

the problem with static attributes is that it is like a cheat code when you start programming.
but it is a lie.
it is like glue
it make it impossible later to change anything.

waxen gull May 3, 2026, 9:04 AM

#

quiet zenith the problem with static attributes is that it is like a cheat code when you star...

and making a object isn't that hard 💀

#

but it isn't always bad right

#

otherwise it wouldn't be a thing 😭

quiet zenith May 3, 2026, 9:05 AM

#

in my mind it is an error xD

#

it take memory when not needed

waxen gull May 3, 2026, 9:05 AM

#

i made a voicechanger mod for simplevoicechat

#

i used static there

#

it was fine ig

#

package dev.griekseei.simplevoicechanger.config;

import com.google.gson.Gson;
import com.google.gson.GsonBuilder;
import net.fabricmc.loader.api.FabricLoader;

import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;

public class Config {
    public static final Config CONFIG = new Config();
    private static final Gson GSON = new GsonBuilder().setPrettyPrinting().create();
    public static final Path CONFIG_PATH = FabricLoader.getInstance().getConfigDir().resolve("simplevoicechanger.json");

    public boolean enabled = true;
    public float pitchAmount = 1.0f;

    public static void load() {
        if (Files.exists(Config.CONFIG_PATH)) {
            try {
                String json = Files.readString(CONFIG_PATH);
                Config loaded = GSON.fromJson(json, Config.class);

                CONFIG.enabled = loaded.enabled;
                CONFIG.pitchAmount = loaded.pitchAmount;
            } catch (IOException e) {
                e.printStackTrace();
            }

        }
    }

    public static void save() {
        try {
            String json = GSON.toJson(CONFIG);
            Files.writeString(CONFIG_PATH, json);
        } catch (IOException e) {
            e.printStackTrace();
        }


    }
}

dry ospreyBOT May 3, 2026, 9:06 AM

#

waxen gull ``` package dev.griekseei.simplevoicechanger.config; import com.google.gson.Gso...

Detected code, here are some useful tools:

Formatted code

package dev.griekseei.simplevoicechanger.config;

import com.google.gson.Gson;
import com.google.gson.GsonBuilder;
import net.fabricmc.loader.api.FabricLoader;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;

public class Config {
  public static final Config CONFIG = new Config();
  private static final Gson GSON = new GsonBuilder().setPrettyPrinting().create();
  public static final Path CONFIG_PATH = FabricLoader.getInstance().getConfigDir().resolve("simplevoicechanger.json");
  public boolean enabled = true;
  public float pitchAmount = 1.0f;
  public static void load() {
    if (Files.exists(Config.CONFIG_PATH)) {
      try {
        String json = Files.readString(CONFIG_PATH);
        Config loaded = GSON.fromJson(json, Config.class );
        CONFIG.enabled = loaded.enabled;
        CONFIG.pitchAmount = loaded.pitchAmount;
      } catch (IOException e) {
        e.printStackTrace();
      }
    }
  }
  public static void save() {
    try {
      String json = GSON.toJson(CONFIG);
      Files.writeString(CONFIG_PATH, json);
    } catch (IOException e) {
      e.printStackTrace();
    }
  }
}

quiet zenith May 3, 2026, 9:06 AM

#

it never get cleaned

#

do you put GSON like this in all class?

waxen gull May 3, 2026, 9:07 AM

#

quiet zenith do you put GSON like this in all class?

no 😭

#

was a really small mod

#

#

but getting the values was really ez Config.CONFIG.pitchAmount

quiet zenith May 3, 2026, 9:07 AM

#

just create it when needed.

quiet zenith May 3, 2026, 9:08 AM

#

waxen gull but getting the values was really ez Config.CONFIG.pitchAmount

yes, easy was a lie

waxen gull May 3, 2026, 9:08 AM

#

quiet zenith just create it when needed.

but there is only 1 config

#

so why would i need multiple objects

quiet zenith May 3, 2026, 9:08 AM

#

it make the rest of the code structure not flow

#

as you used a magic portal

waxen gull May 3, 2026, 9:08 AM

#

so yall never use static?

quiet zenith May 3, 2026, 9:08 AM

#

waxen gull so why would i need multiple objects

just use 1 object

waxen gull May 3, 2026, 9:09 AM

#

quiet zenith just use 1 object

how can i access that 1 object in other classes

#

then

#

if it isn't static

quiet zenith May 3, 2026, 9:09 AM

#

well give them?

#

or make the structure public

#

save(Gson gson)

#

or make the save not in the config class

waxen gull May 3, 2026, 9:10 AM

#

quiet zenith well give them?

how would i give it then 😭

waxen gull May 3, 2026, 9:10 AM

#

quiet zenith or make the save not in the config class

isn't it logical to put it into the config class?

quiet zenith May 3, 2026, 9:10 AM

#

waxen gull isn't it logical to put it into the config class?

not really

waxen gull May 3, 2026, 9:10 AM

#

🙁

quiet zenith May 3, 2026, 9:11 AM

#

you could just have another class that save/load config class/other class

waxen gull May 3, 2026, 9:11 AM

#

yea ig it could be seperate but it wasn't relaly an issue tbh

#how to prevent spagetti code