Computer program cleanup #647
Conversation
…ore message and button texts separately
…Data(), replacing ButtonListProgram.onButtonPressed()
…d SburbServerData
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -56,14 +56,14 @@ public ComputerBlockEntity(BlockPos pos, BlockState state) | |||
| /** | |||
| * 0 = client, 1 = server, 2 = disk burner, 3 = settings | |||
| */ | |||
| public HashSet<Integer> installedPrograms = new HashSet<>(); | |||
| private final Set<Integer> installedPrograms = new HashSet<>(); | |||
There was a problem hiding this comment.
Variable 'installedPrograms' must be private and have accessor methods.
| public ComputerScreen gui; | ||
| @Nullable | ||
| public PlayerIdentifier owner; | ||
| //client side only | ||
| public int ownerId; | ||
| public Hashtable<Integer, String> latestmessage = new Hashtable<>(); | ||
| public CompoundTag programData = new CompoundTag(); | ||
| private SburbClientData sburbClientProgramData = new SburbClientData(this::markDirtyAndResend); |
There was a problem hiding this comment.
Variable 'latestmessage' must be private and have accessor methods.
| public Hashtable<Integer, String> latestmessage = new Hashtable<>(); | ||
| public CompoundTag programData = new CompoundTag(); | ||
| private SburbClientData sburbClientProgramData = new SburbClientData(this::markDirtyAndResend); | ||
| private SburbServerData sburbServerProgramData = new SburbServerData(this::markDirtyAndResend); |
There was a problem hiding this comment.
Variable 'programData' must be private and have accessor methods.
| */ | ||
| public HashSet<Integer> installedPrograms = new HashSet<>(); | ||
| public ComputerScreen gui; | ||
| private final Set<ProgramType> installedPrograms = new HashSet<>(); |
There was a problem hiding this comment.
Variable 'installedPrograms' must be private and have accessor methods.
| @Nullable | ||
| public PlayerIdentifier owner; | ||
| private Runnable guiCallback = null; | ||
| @Nullable |
There was a problem hiding this comment.
Variable 'owner' must be private and have accessor methods.
| public Hashtable<Integer, String> latestmessage = new Hashtable<>(); | ||
| public CompoundTag programData = new CompoundTag(); | ||
| public int programSelected = -1; | ||
| private SburbClientData sburbClientProgramData = new SburbClientData(this::markDirtyAndResend); |
There was a problem hiding this comment.
Variable 'programData' must be private and have accessor methods.
| public CompoundTag programData = new CompoundTag(); | ||
| public int programSelected = -1; | ||
| private SburbClientData sburbClientProgramData = new SburbClientData(this::markDirtyAndResend); | ||
| private SburbServerData sburbServerProgramData = new SburbServerData(this::markDirtyAndResend); |
There was a problem hiding this comment.
Variable 'programSelected' must be private and have accessor methods.
| private SburbClientData sburbClientProgramData = new SburbClientData(this::markDirtyAndResend); | ||
| private SburbServerData sburbServerProgramData = new SburbServerData(this::markDirtyAndResend); | ||
| @Nullable | ||
| public ProgramType programSelected = null; |
There was a problem hiding this comment.
Variable 'hieroglyphsStored' must be private and have accessor methods.
No description provided.