help-development

[lost matrix]

Because setStorm fires the WeatherChangeEvent again
resulting in an infinite loop

[wet breach]

Maybe it is just a language you dont quite grasp like some others.

Nothing wrong with that

[frank kettle]

Wrong way of doing it

[tall dragon]

u just need to cancel the wheather change

it implements cancellable

[tawny pine]

Wait are you using path tracing?

[wet breach]

Unless your complaint is with boiler plate code with c++

In which case you can tailor your ide to handle most of that lmao

[young knoll]

Can’t you just disable weather with a gamerule now

[wet breach]

Yes

[lost matrix]

Ive learned everything to C17 in uni for 2 years. Ive written games, a database, website backends
and machine learning applications with it. I think i have a pretty decent understanding of the
language and i can confidently say that writing C++ is the worst experience i had with any
programming language so far. And dont get me started with CMake. Absolutely abysmal.

[wet breach]

Lol

[lost matrix]

Rust is literally a superior language

[wet breach]

No, rust doesnt run hardware or os's

[young knoll]

Can’t wait for them to release C—

[tardy delta]

i cant really get used to the structs and impl blocks lol

impl blocks everywhere

[wet breach]

Cant call something superior when it doesnt do what c++ does. I agree that it might be better when you dont quite need a language like c++ but to say it is superior is not quite right either

[lost matrix]

Thats not an argument. Most operating systems (At least the kernel modules and most drivers) are not written in C++ either

[quaint mantle]

anyone got some example source code for custom block drops i cant find any

[lost matrix]

And writing an OS with a language doesnt make it good

[wet breach]

Not sure what drivers you are talking about, but it is a mixture of C and C++

C++ however can load c libs but c cant load c++

[lost matrix]

You could simply listen to the BlockDropItemEvent

[quaint mantle]

well i just so happend to fnd this

[chrome beacon]

Or you can use a datapack

[wet breach]

Certainly doesnt help its case for the opposite. Java had an OS at one point.

[quaint mantle]

i wanna know how to use trhat

[lost matrix]

And even if C++ was pleasent to write, CMake catapults it right back

[tardy delta]

oh now i know whats waiting for me

[lost matrix]

Sounds like a bad idea...

[wet breach]

Java os was great for anything java

And you dont have to use cmake

[lost matrix]

There arent any good build systems for C++
Only Cmake and some abominations nobody uses or
which havent been updated since 1996

[wet breach]

There is alternatives like gnu make, apache buildr, apache ant, automake

All of those are maintained that i listed

Maybe not apache buildr would have to check when i get home, but the others are.

[lost matrix]

By all means, write C++ if you want to. But its dead to me. I would rather
write in any other language unless im writing some ultra low-level applications
or embedded code. Well even for embedded id use Rust.

[brisk estuary]

It's a dumb question, but I should use equals() to check if two inventories are the same, right?

[lost matrix]

For what purpose? This would trigger every tick.

[young knoll]

/gamerule doDaylightCycle false

[tardy delta]

yes

[tall dragon]

if ur looking for a time "skip" https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/event/world/TimeSkipEvent.html

[brisk estuary]

thanks

[tall dragon]

only gets triggered for stuff like all players tho

when it jumps to day

[lost matrix]

Just keep in mind that this does not account for the normal daycycle

[tawny pine]

in bukkit how do I set a parameter to null? do i just leave it blank or write "null" or what?

[young knoll]

If you want always day just use the gamerule

[wet breach]

You just put null

[lost matrix]

Never do that in java.

[tawny pine]

ah ok thanks 😉

[lost matrix]

If you have to do that then you are doing something wrong

[tawny pine]

wrong emoji

🙂

😄

[tardy delta]

😏

[wet breach]

Unfortunately some things in bukkit require that

So cant say it is wrong when the implementation is the one that is wrong

[young knoll]

Bukkit makes it clear what is nullable

[lost matrix]

There should never be something on your heap that needs to be set to null...

[wet breach]

Sure i agree, doesnt change the implementation expectations

[languid radish]

How to enable fly on spigot server

[lost matrix]

#help-server

[languid radish]

Ok

[lost matrix]

Wait we arent speaking about fields... Just parameters. My bad, i thought he wanted to manually set a field to null.

[wet breach]

Lol

[tardy delta]

i thought are we getting into stack and heap now

[nocturne steppe]

@eternal oxide help since i did your thing i can't look at bukkit/spigot's file without using intellij's decompiler
ping me when you respond ❤️

[tall dragon]

@lost matrix how do u handle stuff that can be empty tho, using Optional<> ?

[wet breach]

Very carefully

[lost matrix]

Depends. Sometimes i allow null values and sometimes i use optionals.

[tall dragon]

i usually allow null as well tbh

and handle it appropiately

[tardy delta]

the billion dollar mistake 🙏

[lost matrix]

Yeah i thought we where talking about variables in general.
And you dont want to have live objects on the heap (like players, chunks, entities etc)

[young knoll]

If my code was worth a billion dollars I wouldn’t have to worry about null ever again

[tardy delta]

where woiuld they be otherwise?

[lost matrix]

stack

[tawny pine]

Guys I'm using the code:
meta.hasChargedProjectiles().equals){
but equals "cannot be resolved".
I'm trying to get it to be .equals(true).
Where did I go wrong and how should I fix this?

[tardy delta]

i clearly havent learnt good enough

[tender shard]

if(meta.hasChargedProjectils())

or if(meta.hasChargedProjectiles() == true)

it returns a boolean (primitive)

?learnjava

[undone axle]

Here are some links to get you started on learning Java:

• https://www.codecademy.com/learn/learn-java
• https://www.sololearn.com/learning/1068
• https://www.learnjavaonline.org/
• https://programmingbydoing.com/
• https://docs.oracle.com/javase/tutorial/java/index.html

The last one is the only official one, however some of those concepts assume that you already know a bit about programming.

[tawny pine]

ah ok ty 🙂

[young knoll]

Cursed

[mighty pier]

wtf

[tardy delta]

my classmates code like that

[tardy delta]

only refs to those are on the stack right?

[tall dragon]

if ((meta.hasChargedProjectiles() == true) ? true : false) better <

[tardy delta]

im confused lol

[young knoll]

Needs more Boolean.valueOf

[lost matrix]

This reminds me of the terrible plugin

[tardy delta]

objects are on the heap but objects consists of primitives which are on the stack again 🤔

i have no idea what im saying

[mighty pier]

give me the link to it cause i forgot

[young knoll]

https://preview.redd.it/vhzz4u9akjf51.jpg?auto=webp&s=499bd5a67c203ad063b85a51171b51fe130d9400

[tall dragon]

oh god yes send me the github

id lost it

i need some laughing in my life rn

[lost matrix]

https://github.com/Jaimss/terrible-plugin

[tardy delta]

just too late

[lost matrix]

Im even a proud contributor of this.
Laughed so hard when pushing my PR

XDDD

[tardy delta]

uh oh

[tall dragon]

the best thing are the comments actually trying to justify everything

so fking funny

[mighty pier]

its actually a style of programming, please do not harass the author

[tender shard]

i added the abstractmotdfactory stuff and the Logic class

[tardy delta]

erm

best random

[tall dragon]

lmfaoooo

[tardy delta]

lmao

[lost matrix]

I love the comments

[tender shard]

try {
  throw new RuntimeException("Something went not really wrong");
} catch (RuntimeException ex) {
  if(ex.getMessage().equals("Something went not really wrong")) {
    // All good
  } else {
    throw ex;
  }
}

[tall dragon]

yea the comments are so good

[tardy delta]

reminds me of someone who can barely speak english but when he does it sounds funny

[tardy delta]

realizing all plugins can do that 💀

[dense geyser]

what does HandlerList#bake do? the javadoc says it bakes a handlerlist which is not the most helpful of things

[young knoll]

I do like the sql storage system they have

[lost matrix]

Why doesnt spigot use this method? It looks better than the current getOnlinePlayers() implementation

  public Stack<Player> getEveryPlayerWhichIsCurrentlyOnlineOnTheServer() {

    Stack<Player> allPlayers_which_arOnline = new Stack<>();

    for (OfflinePlayer player_that_is_not_online : Arrays.stream(Bukkit.getOfflinePlayers()).collect(Collectors.toList())) {
      if (player_that_is_not_online.isOnline()) {
        Player player_that_might_be_online = player_that_is_not_online.getPlayer();
        if (isThisThingReallyOnTheServer((Entity) player_that_might_be_online)) {
          allPlayers_which_arOnline.push(player_that_might_be_online);
        }
      }
    }

    return allPlayers_which_arOnline;
  }

[tall dragon]

yea same, mostly the fact thats randomly made in kotlin xd

[tardy delta]

😂

[lost matrix]

You should not call this method. It gets called whenever the handlerlist changes (events getting (un)registered)

[granite owl]

got a question not directly related to spigot, anyhow: do i understand it correctly that i can implement actual new mobs with only a data pack and resource pack?

[dense geyser]

ah alright, thank you

[tall dragon]

any1 tried actually running this plugin?

pretty sure it would nuke ur server

[young knoll]

What counts as actual new mobs

[tardy delta]

you can create new mobs with nms, still gotta have a texturepack for their texture

zombies in disguise

[granite owl]

yea but if i inherit say the zombie class i can not update its skin without overriding the vanilla one

[young knoll]

You can do a lot of stuff with display entities

Or armorstands if you’re feeling 2022

[lost matrix]

Custom AI logic cant be done with datapacks

And we can also not add new entity models to the game using resourcepacks.
You will always need some sort of magic to make that happen.

[granite owl]

maybe thats a better approach than nms tho because nms is well we all know its fragile, obfuscated(yes ik ik maven) and its not persistent across server restarts

[lost matrix]

Im just going to let this sentence as is...

[tender shard]

We should stop calling it nms and start calling it slughorn

[tardy delta]

nms not persistent accross restarts uh oh

[young knoll]

What

[tender shard]

„How can i spawn an npc using slughorn“

[tardy delta]

uh no my server is gone

[young knoll]

Where did you come up with slughorn

[tender shard]

„Ugh why are you using slughorn to send action bar messages“

[tender shard]

Idk just came to my mind

[tardy delta]

is that some german thing?

[tender shard]

No its from harry potter 6

[tardy delta]

💀

[granite owl]

i mean if i inherit the zombie class to modify its behavior incl pathfinder goals, and then spawn one it behaves as desired, but once i shutdown the server, save the entity and restart the server its going back to the defaulting zombie class

maybe i did do it wrong idk

[lost matrix]

You should not mix datapacks and plugins. There is nothing you can do with a datapack that
you couldnt do with a plugin. Also datapacks are interpreted and quite a bit slower in many cases.

[granite owl]

but from my exp its not persistent by default

[tardy delta]

i have no idea actually

havent done much with nms

[young knoll]

Good luck adding new biomes and structures with a plugin

[lost matrix]

Nothing is persistent by default

[young knoll]

Well you can

But it’s NMS

[tardy delta]

minestom is way funnier

[lost matrix]

🤷

[granite owl]

thats not what i meant, i mean its defaulting back to the vanilla mob after a restart and tbh

i havent figured out how to inject my custom mob behavior into the maps that in the end store entity states

[young knoll]

Datapacks are fine for most static features

[lost matrix]

Since we have mojang mappings i find writing nms to be decent

[young knoll]

It’s only functions that start to impact performance

[granite owl]

hm

well tbh its been a while

i think last time i did custom mob behavior in nms was

hmm, 1.15? xD

maybe the api changed idk really

[paper venture]

I'm working on lobby and minigames system, I'm making it from scratch. On other servers when you select some lobby or a game you will be put in a new world and in tab you see only players that are currently in the same world. I know that I can do it by managing tab system, but it seems to me it's incorrect... I guess that these servers use bungeecord to separate players, but I don't know exactly...

[young knoll]

Probably

[tall dragon]

probably?

[granite owl]

i think most larger servers do use bungeecord although im not familiar with how it works exactly😅. but multiple smaller dedicated servers have the advantage to be able to run on less resources since say u have a minigame with 20ppl per server instead of a megaserver with 4k players

[tall dragon]

pretty sure every single self respecting minigame server does it that way

[tardy delta]

isnt that how its supposed to work?

[granite owl]

and since its a minigame, these 20ish players playing their game have no business with the rest 3.9k anyways

[wise mesa]

You compiled the header

That’s why it’s annoyed

You don’t compile headers just cpp or c files

[tardy delta]

ah

[granite owl]

headers contain declarations

[wise mesa]

Should I avoid using the Bukkit class

[tardy delta]

makes sense lmao

[granite owl]

and source files definitions

[wise mesa]

Like the static class that has the same functions as Server

[granite owl]

@tardy delta ud rather do namespace...

extern constexpr int max_class_size;
``` in ur header file

and then define it in the same named .cpp file

[tardy delta]

that was the exercice i got

and tbh i dont understand what half of those keywords does

[granite owl]

c/c++ is completly different from managed languahes

[tardy delta]

to me it feels like they all do the same

[granite owl]

wdym

xD

did u #pragma once in ur .cpp file?

@tardy delta ^?

[hazy parrot]

You shouldnt have pragma once in cpp

As you are not including it

[granite owl]

i think an explanation what it does is more helpful

[tardy delta]

no just in header

[hazy parrot]

Pragma one only include file once by the compiler if its included by user multiple times

[tardy delta]

ik

[granite owl]

#pragma once is a guard clausel to tell the compiler to not include the same header multiple times as this could cause an infinity loop if 2 headers include each other

right

[hazy parrot]

Also why not just use Class and static fields for constants, I means it's nearly same as namespace but I guess it would be easier to you as its close to java

[tardy flame]

Is there any way for displaying transparent blocks? Or even a schematic from world edit?

[granite owl]

theres so many ways xD.

macros, typedefs, constants

[tardy delta]

imma miss java switch

[granite owl]

why

cause u cant pass hashed strings?

🤣

in a switch statement

[tardy delta]

no need for return

or break whatever

[granite owl]

huh

i always use break and returns in java for switch statements

[hazy parrot]

Ig he is talking about enhanced switch

From java 11 or what

[tardy delta]

ig

yes

[granite owl]

if i may ask why r u learning c++?

[tardy delta]

want to learn smth new

[granite owl]

i mean its my personal fav lang xD but

[granite owl]

id suggest learning C then

[tardy delta]

mmmh

that has no classes

[granite owl]

C++ is the same as C but more advanced with classes

well yes but the only reason to learn C/C++ is to learn an unmanaged language

[hazy parrot]

No macros in cpp

[granite owl]

and c++ has so many abstractions that u can basically use it like java

[granite owl]

exactly c is more barebone, dont get me wrong i prefer c++ over c by far, but to learn the fundamentals C is better from a didactic pov

[quaint mantle]

i cant export my maven shit it always says acsess denied no matter where

[granite owl]

wdym macros are great

esp for conditional compiling

[hazy parrot]

No reason to use them in cpp as utility or whatever is it used for in c

[granite owl]

one should understand as many different methods as possible

to have a broad view

[hazy parrot]

It's not strongly typed, you have inlined functions for same purpose

[hazy parrot]

True

[granite owl]

i do use inline members for my own project

[hazy parrot]

Cpp is somewhere between functional and object oriented

That it's why you can do many weird things

[granite owl]

just because something is bad practice in a particular circumstance

it doesnt mean its not worth to be known

i barely use macros too, but if u really need em its good to be able to

and conditional compiling is one such instance in for example ive written cross plattform code thats able to compile from windows and unix plattforms using the same code base by replacing the plattform specific functions and includes with macros @hazy parrot

[granite owl]

access denied to what from what

[hazy parrot]

Tbh never knew about that usage of macros

Good to know ig

[granite owl]

conditional compiling?

#ifdef whatever
...
#elif something else
...
#else
...
#endif
``` then u can for example check for the target OS to define macros with whatever u need to fit that plattform

if u want i can dm u an example cause i dont want to spam here any more xD

[smoky adder]

@last temple what you explained to me today worked, now it works without making the variable static thanks a lot

[hazy parrot]

ik about that, but never saw someone using macros there

[true perch]

I wrote a static hashmap within the main class which stores some Custom ItemData based on an ID system. Is this good design? It will get referenced throughout the codebase.

/**
 * A HashMap of every item's ItemData.
 *
 * Integer - The number used to represent this item's unique identification.
 * ItemData - All the data associated with 7 Realm's Custom Item system.
 */
public static final HashMap<Integer, ItemData> ITEM_DATA = new HashMap<>();```

[lost matrix]

1. Dont make it static
2. Dont make it public

[true perch]

Oka oka. So what's a good way to reference it? Add a getter and reference the main class object?

That'd mean I would need access to the main class object throughout the code though, right?

[compact haven]

you’d have a class responsible for sharing that data with proper methods

[lost matrix]

public class ItemDataManager {
  
  // Never expose the map directly
  private final Map<Integer, ItemData> itemDataMap;
  
  public ItemDataManager() {
    this.itemDataMap = new HashMap<>();
  }
  
  public void addData(ItemData itemData) {
    this.itemDataMap.put(itemData.getId(), itemData);
  }
  
  public ItemData getData(int id) {
    return this.itemDataMap.get(id);
  }
  
  public void removeData(int id) {
    this.itemDataMap.remove(id);
  }
  
}

Something like this. Create one single instance of this class and pass it around.

[compact haven]

^ exposing a hash map is shit api

[tardy delta]

tell that my teacher

ill invite you to have a talk to her

[granite owl]

doesnt it only make sense to write a proper wrapper layer if thats gonna be an actual api and not only used by one plugin only

[true perch]

So can I make the instance of that class static?

[tardy delta]

it looks shit to call SomeClass.someMap.get() anywyas

[lost matrix]

No. Never expose your collections. There is no exception to this rule.

[compact haven]

to be honest I prefer designing the API first and using the API in the implementation, not the other way around

[lost matrix]

Hm, yeah you could do that. But you will learn to write more robust code when you
inject it as a dependency (pass it around through the constructor of other classes)

[compact haven]

it does create problems where you can’t do internally unsafe things that you don’t want in the API, but it leads to a better API in the end imo

[granite owl]

the problem is that you first have to write the apis and most ppl dont have that patience

[true perch]

It seems very over engineered to pass it through every class that needs access to the ItemData hashmap. That's why I'm asking if I could just make it static. Makes it much easier to use and extremely accessible without needing to pass the object everywhere

[compact haven]

my fucking Java code is horrendously beautiful though, I constructor DI each manager that I use instead of the plug-in instance (unless that instance is also needed, then I DI both)

[granite owl]

like i wanted an ini parser i can use in every language, and by now im writing it for 6 months

even though it was supposed to be a 2 week project

[compact haven]

it’s easier to make it static, it’s proper to pass it as a constructor parameter

[true perch]

Even if I end up having several types of hashmaps with various data that needs to be referenced?

[lost matrix]

Its not over engineered. Thats simple dependency injection.
Using a static instance is the singleton pattern. Both approaches
are equally valid with DI being more robust to problems like cyclic dependencies.

[compact haven]

it’s called having a manager class for related data and passing that

once again, you don’t expose collections it just isn’t proper

[granite owl]

im not that versed in java, you mean pass it trough constructors to store a copy of reference to that map? since java is call by reference

[compact haven]

correct, that’s dependency injection

[granite owl]

🙏

[true perch]

Alright noted. I feel I've learned a lot about what's proper now thanks guys 😄

Haven't learned why though

[lost matrix]

Technically Java is call by value. Always.

[granite owl]

as in the reference is copied and as such a value?

basically a copy of a pointer?

[tardy delta]

the value of the reference

so just an address

[compact haven]

the reference isn’t duplicated though, the reference is the same for the original object and injected object

[lost matrix]

Kind of. Yeah.

[granite owl]

man i prefer the freedom to choose though as to what is passed xD

[compact haven]

Unless there’s two references that point to the same memory address

but that doesn’t sound right

[lost matrix]

No the reference is a different reference. It gets copied.

[tardy delta]

a reference is just a fancy name for a pointer

[granite owl]

isnt it like

class foo = new foo();//heap allocation

class foo2 = foo;// pointing to the same unmanaged address?

and then java wraps it

so we dont have direct access to it

[lost matrix]

Dont say that to your C++ teacher. Because thats not quite true.

[cunning pendant]

should I learn java with spigot or should I learn java before spigot

[compact haven]

new foo() has a reference, and both foo and foo2 have the same reference

[lost matrix]

Learn at least the basics of java first

[tardy delta]

maybe its better that i dont have a c++ teacher then

[eternal oxide]

learn java basics

[granite owl]

to be fair most is just fancy wording for things that happen in the background where you dont have access to

[tardy delta]

hmm

[compact haven]

Java is pass by value, but the value is the reference unless it’s a primitive

[cunning pendant]

ok ty

[granite owl]

like a java reference is not a pointer but it behaves the same

[ivory sleet]

its pass by value always by definition

[compact haven]

there’s no 2 references that point to the same object @lost matrix

right?

[tardy delta]

depends what you consider being two same references 🤔

[ivory sleet]

pass by reference is when the language has semantics that allows you to pass semantic variables/references as values

[lost matrix]

Pass by value will always copy the passed value into a new variable.
So the actual reference will be the same but there will be a new variable
containing this reference. So yeah the reference stays the same.

[compact haven]

there’s a memory allocations for each field holding the same reference when you do this, right?

same reference just in 2 spots

[wet breach]

it is backed by a pointer, hence why you can do a reference comparison to check if both references point to the same memory address

[wise mesa]

Why does java gotta call everything enhanced like

For-each loops are enhanced for loops

[granite owl]

no only for the first foo from my understanding.

[wise mesa]

Pattern matching is enhanced switch

[wet breach]

because there is a normal for loops, and you need a way to distinguish the two

[granite owl]

only the new keyword actually allocates heap memory in java

[compact haven]

so yes I’m right 7smile

[eternal oxide]

Should we go with switch 2.0 then?

[compact haven]

this

[wise mesa]

Right but they could just call it for each

Like everyone else

[lost matrix]

"slightly less bad for loops"

[compact haven]

the field is still separate, which is why I can reassign foo2 without foo changing at all

[ivory sleet]

semantically yes

[granite owl]

although thats not entirely true

[compact haven]

but the reference is the same

[ivory sleet]

but u have stuff like the foreign memory api and unsafe

[lost matrix]

enhanced for loop and foreach are two different things in java

[granite owl]

if u for example increase the size of a map by injecting an entry

theres also a heap alloc

hence the trim_to_size member

[wet breach]

enhanced for loop is backed by for each

[ivory sleet]

tim well lets say ur map entry push causes the map to resize the backing array

[wet breach]

and normal for loops are not bad either

if you misuse them sure

[ivory sleet]

then there's a new keyword somewhere to ensure thats happening

[compact haven]

enhanced for: for (X e : z)
for: for (int i=0;i<5;i++)

[granite owl]

yea thats what i mean i just wanted to say, you the user using the new keyword isnt the only way to have heap allocs

[compact haven]

it’s so fucking hard to type code in a phone with autocorrect

[granite owl]

basicslly just corrected myself cause ik absoloute expressions are rarely valued xd

[green prism]

    public static ItemStack getItemStack(Material material, Component displayName, List<Component> lore, boolean glow) {
        ItemStack itemStack = new ItemStack(material);
        ItemMeta meta = itemStack.getItemMeta();

        meta.setDisplayName(Translate.translate(displayName));
        meta.setLore(Translate.translate(lore));

        if(glow) {
            itemStack.addUnsafeEnchantment(Enchantment.WATER_WORKER, 1);
            meta.addItemFlags(ItemFlag.HIDE_ENCHANTS);
        }


        itemStack.setItemMeta(meta);
        return itemStack;
    }

Glow is not working is 1.19.4.
Could you please help me?

[wet breach]

while this is true, it isn't true to say that using the new keyword means it will go on the heap

the JIT can decide that it should go on the stack memory instead

[granite owl]

yea

[ivory sleet]

well implicitly the bytecode instruction new anewarray multinewarray or newarray is called somewhere @granite owl

which is compiled from new

[ivory sleet]

anyway this tho

[compact haven]

for example if I do Sand.test(new Random()), then the instance of Random will go on the stack instead

[granite owl]

and then theres native code

[green prism]

Just need a quick help, please

[granite owl]

where the .dll / .so whatever can return pointers to heap allocated objects

which i for example used

hence im really annoyed tha java doesnt have actual destructors only autocloseables

[wet breach]

also in more recent Java versions, the new keyword doesn't guarantee any allocation of memory on the heap or stack and the JIT can sometimes optimize it away

[ivory sleet]

na

thats still on heap

[granite owl]

like string literals in the byte code?

[ivory sleet]

or no?

[compact haven]

Wait seriously? If I do new Integer(5) it’ll return the pre-made one?

[ivory sleet]

Integer.valueOf maybe

but like afaik, by specification all objects go on heap

[compact haven]

well Integer has the first few hundred integers cached

[wet breach]

unless the JIT can optimize it away

[ivory sleet]

ye

[wet breach]

then no allocation is made

[compact haven]

what’s considered optimizable though

[granite owl]

it should go on the heap as new Random() returns a reference to the newly created random and since we dont know how many new randoms it should be the heap

[ivory sleet]

well there's whats called an escape analysis

but ye apart from that

[tardy delta]

but arent objects at lowest level just a wrapper around a bunch of primitives?

so arent those on the stack then

what have we started lol

[ivory sleet]

primitives live on the stack

[granite owl]

nah, at the lowest lvl objects are just bytes appended

[compact haven]

oh well yeah it would go on the heap

[granite owl]

in a consequtive array of bytes

[compact haven]

the reference to the object from the heap is on the stack

but that’s just how methods are called

[granite owl]

in memory

[tardy delta]

but if the object contains primitives, those are on the stack?

[granite owl]

why do i deserve the clown

[wet breach]

unless the object is too large to go on the stack then it gets placed outside of the stack and heap

[quaint mantle]

how do i create blocks with blockdata that i can later use in a event

[wet breach]

when that happens, the objects is slower to work with

hence why it is best to optimize for stack size 🙂

[compact haven]

StackOverflowError 😏

[granite owl]

doesnt that happen with a recursive infinity loop?

[tardy delta]

hey hey

[granite owl]

🐒

[compact haven]

yes because the stack is too large

[wet breach]

it also happens if you exceed the limit of the amount of objects a single class can handle

[compact haven]

^ you can only have Short.MAX_VALUE methods iirc

[tawny remnant]

Any idea how the Midas Staff on skyblock was created?

[wet breach]

think you mean to say, too small?

[compact haven]

well it’s intending to allocate past the size

[granite owl]

i like how java is supposed to manage everything for u and in the end to be an effective developer u still need to have a good understanding of how a computer works on hardware level🤣

[compact haven]

yep too small, my bad for the wording

[dawn hazel]

With a computer probably

[tawny remnant]

burh

[compact haven]

it was meant to be too large how I was wording it, but I worded it oddly

[tardy delta]

large guy

[compact haven]

it’s “too large” for the size that’s already allocated

[wet breach]

This is why software engineers exist

[tawny pine]

guys how can i declare variables within an event handler?

[granite owl]

i once tried to set a mobs hp to Integer.MAX_VALUE and it basicslly phased out, couldnt kill it, AI froze, unresponsive to external forces like tnt etc

[compact haven]

having good knowledge of the internals of whatever language you use is most helpful anywhere

[wet breach]

software engineer is the person that should be telling you where something needs to be optimized or needs to be done differently, so that your developers don't need to actually know that stuff

[wet breach]

I agree

[compact haven]

even though you don’t technically need to know how the JVM works to use Java, it definitely helps a ton 🤷‍♂️

[granite owl]

what about arrays of primitives 😛

[wet breach]

yep especially in the those unique cases where you need to know some specific thing

[granite owl]

ofc

[wet breach]

for example the endianness of Java. Java stores objects/data in Big Endian format or network order as it is referred to sometimes

[tardy delta]

doesnt the reference to the array lives on the stack but the elements either on stack or heap?

[granite owl]

id say learning C for the basics is a really good thing but most ppl whom i suggest that to tell me they dont have time for that

[tardy delta]

lol

but they have time to be on discord

[tawny pine]

guys how to declare variable in event handelr?

handler*

[granite owl]

ya

[tawny pine]

🙂

[tardy delta]

declare a variable bruh

[green prism]

bump

[tawny pine]

i'm doing what it says in the docs but it says cannot resolve symbol

[tardy delta]

show code

[wet breach]

probably should have researched what it means when it says cannot resolve symbol

[tawny pine]

wait dw i think i got it

[tawny pine]

yeah i did

[granite owl]

iirc a reference can live on both the stack and the heap but for the garbage collector to not swallow ur object at least 1 reference must be active on the stack to keep the reference count above 0

[mighty pier]

what exactly "isnt working"?

[tardy delta]

so the reference is just the address?

[granite owl]

basically but wrapped

in javas fancy i do what i want

[tardy delta]

in java everything is fancy for smth low level

Object impl hehe https://github.com/openjdk/jdk18/blob/0f2113cee79b9645105b4753c7d7eacb83b872c2/src/hotspot/share/oops/oop.hpp#L4

thats the hpp file actually

💀

[granite owl]

is that a jni header?

for type declaration

[tardy delta]

is it?

[granite owl]

must be a rather new iteration since inline members are a c++ 17 standard

[true perch]

Is it good practice to access static methods in this fashion? java public class ItemManager { public static void giveItem(Player player, int id) { ItemData itemData = Realms.ITEM_DATA.get(id); player.getInventory().addItem(itemData.getITEM_STACK()); } }

[granite owl]

well it declares javas primitives in c++

ik this type of definition from the jni.h

[true perch]

or should I pass an instance of the class to use this method?

[tardy delta]

hmm

is ITEM_DATA a collection?

then dont

[granite owl]

but i really dont wanna read trough the whole repo to figure the exact purpose

[true perch]

it is a collection

[tardy delta]

dont expose it

pass in the stuff you need

[wet breach]

but it would be fun

[tardy delta]

or create some itemmanager thing

bruh thats the itemmanager

whys that map in the realms class and is that the main class?

[true perch]

I'm going to rewrite it don't worry

[green prism]

It's not glowing

[true perch]

Yes, I'm rewriting it sorry xD

[ivory sleet]

objects

[granite owl]

btw serious question, if i load a native library and store 64bit pointers to functions so to speak function pointers in a long, is it bad practice to do so in private static fields?

[ivory sleet]

but as frostalf said

[true perch]

Was hoping we would ignore that fact since I am planning to rewrite it lol

[ivory sleet]

jit might do an escape analysis i believe

[wet breach]

static can't be GC'ed

[tardy delta]

my brain is even more fucked up after reading this conversation

[wet breach]

in most cases yes

[granite owl]

basically load library, on first instance of my wrapper class, store function pointers and then use them across all instances since its hardly good practice to load a library once per instance

[ivory sleet]

it can sometimes

like class unloading might allow it

but thats too arbitrary to rely on

[wet breach]

this would be the only way to remove statics lol

[wise mesa]

Set everything to null?

[wet breach]

but you would have to some how unload the class fully and there really isn't a way to check if it happened or if it like half unloaded

[granite owl]

ofc not, thats the point. since its pointers to functions inside the native lib

[ivory sleet]

i feel like maybe unsafe could hack it also

[granite owl]

ill post what i mean here once ive written it

rn im at the c# importer

xD

after finishing c, and c++ for both static and dynamic libs

[wet breach]

possibly but either one I wouldn't really consider it a solution to something that is self created lmao

[ivory sleet]

indeed

[tawny pine]

Guys I'm trying to update the lore on an item.
I'm trying to get the first word on the first line of it's lore.
How would I do this?

[green prism]

substring

[tardy delta]

getLore().get(0)?

[granite owl]

by seperating the string into an array based on the seperator which is a space?

[granite owl]

doesnt that return the first char?

[tawny pine]

Wouldn't that get only the first line?

[tardy delta]

isnt lore a list of strings?

[tawny pine]

Yeah

I'll try using the substring thingy then

[tardy delta]

line.substring(0, line.indexOf(' '))

[granite owl]

🤣

[granite owl]

i guess this is why ill never be a good java developer

[tardy delta]

dont laugh at me

[granite owl]

no i laugh at my low level approach

[tardy delta]

no need to split whole line

performance my friend

[granite owl]

yea ik

[tardy delta]

think about the nanoseconds

[granite owl]

i was more thinking of a ranged for loop iterate each char individually and then memcpy them over

xD

[tardy delta]

wtf

[tawny pine]

what does the indexof part of that do?

[twilit roost]

How can I make Wrappers?
I tried like so:

public abstract class PlayerWrapper implements Player {
    private Menu lastOpenedMenu = null;

    public Crew getCrew(){
        return CrewLedger.getCrew(this);
    }
    public Menu lastOpenedMenu(){
        return lastOpenedMenu;
    }
    public SlotLock getSlotLock(){
        return SlotLock.slotLock(this.getUniqueId());
    }

    public PlayerWrapper lastOpenedMenu(Menu lastOpenedMenu) {
        this.lastOpenedMenu = lastOpenedMenu;
        return this;
    }
}

resulted in error:

Caused by: java.lang.ClassCastException: class org.bukkit.craftbukkit.v1_19_R1.entity.CraftPlayer cannot be cast to class io.tomko.bonestreasures.wrappers.PlayerWrapper (org.bukkit.craftbukkit.v1_19_R1.entity.CraftPlayer is in unnamed module of loader java.net.URLClassLoader @45fe3ee3; io.tomko.bonestreasures.wrappers.PlayerWrapper is in unnamed module of loader 'BaT-0.0.1-shaded.jar' @41c2c70)

[tardy delta]

it gets the index of a space

[granite owl]

why wtf

[tardy delta]

probably want to look into thr string javadocs

[tawny pine]

what if i wanted to get everything up to a "/" character?

[tardy delta]

indexof('/')?

or lastIndexOf whatever u need

[granite owl]

i can see the ulterior problem here

🤔

[tawny pine]

Ah ok.

[tardy delta]

the what

[granite owl]

i dont wanna be that guy

🤣

[tardy delta]

hmm

[twilit roost]

bump

[round finch]

what about string . split

😄

[rotund ravine]

Look into delegates

[tardy delta]

you cannot implement Player

[granite owl]

ranged for loop?

[tawny pine]

I'm using meta.setLore(Collections.singletonList(meta.getLore().split)); but split keeps saying "Cannot resolve symbol"

[rotund ravine]

He can

[tardy delta]

as the server doesnt give a fuck about your impl

well you can

but the server will happily keep constructing craftplayers

[rotund ravine]

He just needs to delegate properly

[tardy delta]

yah

[rotund ravine]

Obv he can’t cast it

[round finch]

lore is list

[twilit roost]

how do they work?

[tawny pine]

wdym?

[round finch]

1 sec

[tardy delta]

just a wrapper

[tawny pine]

ok

[round finch]

i mean list

[tawny pine]

ohhhh dw i needed to put .tostring first

got it

🙂

[granite owl]

is Player a private class?

[rotund ravine]

Lombok https://projectlombok.org/features/experimental/Delegate

[tardy delta]

always fun looking at internals , what is this crap bruh

[granite owl]

wait

its an interface

🐒

[granite owl]

md_5's code?

[tawny pine]

I'm now using meta.setLore((List<String>) Collections.singletonList(meta.getLore().toString().split("/")));
But where should I go from there to get the first half of the string?

[granite owl]

xD

[round finch]

stop this code is awful
so messy

[compact haven]

always fun when you see an assignment used as an expression

[tardy delta]

brr

[tardy delta]

probably

maybe

[compact haven]

I’ve never seen that actually be used in random Java code before

[granite owl]

why exactly this.lore = null and in the next one this.lore.clear() xD

[tawny pine]

should i do like string[1] after that?

[tardy delta]

who will ever say

[twilit roost]

aight thx
how shall I get my wrapper class from Player?

[tardy delta]

you forgot the best part bruh

[granite owl]

i mean after reading it 5x it does make sense, but its pretty redundant

[compact haven]

because an empty list is not a null list

it’s make sense it’s just odd

[tardy delta]

smth like this https://github.com/FourteenBrush/xKingdoms/blob/master/src/main/java/me/fourteendoggo/xkingdoms/player/KingdomPlayer.java

or lombok, but delegating every single method hmm

[granite owl]

still redundant conditioning. if lore == null then lore = Collections.EmptyList();

and then

this.lore.set

please ignore the pseduo before the code

[eternal oxide]

pretty sure EmptyList() is immutable

[granite owl]

🤷‍♀️

[twilit roost]

soo constructing new Wrapper?

[tardy delta]

basically

[tawny pine]

guys i'm using getlore but how can i specify a specific line of lore?

[tardy delta]

its a list

find a method to get a specific item of a list

[desert tinsel]

How can you send emotes like these with spigot?

[rotund ravine]

You’d pass player through the constructor

[granite owl]

i got ptsd from bukkits exception throwing

[rotund ravine]

Store it locally

[tardy delta]

resourcepack probably

[desert tinsel]

nope, its normal chat

[rotund ravine]

Then either delegate each function or make a getPlayer method @twilit roost

[tawny pine]

so the list is
item 1 = line 1
etc?

[tardy delta]

idk if minecraft font supports those characters?

[granite owl]

its like 1000 useless lines of msg and 1 line containing the actual error rest is just where it occured which is so large that it doesnt fit the console

[desert tinsel]

yes, I saw many of them

[regal scaffold]

How to register alias commands in bukkit command system

[tardy delta]

the list is

index 0; first line
index 1; second line
...

[tardy delta]

throught plugin.yml?

[granite owl]

under the command there a param aliases: []

[regal scaffold]

Oh yeah i just saw it

[granite owl]

in the plugin.yml yes

or

u inject it directly into the map at runtime

[desert tinsel]

and there are many "emotes" plugins

[granite owl]

i did

its a pain in my ass

🤣

[desert tinsel]

but I need just to send an emote through a message

[tardy delta]

bruh my brother is cleaning his laptop internals with the vacuum cleaner

[granite owl]

static discharge be like

[pseudo hazel]

pressurized air ftw

[granite owl]

hmm somehow i need a new lappy

[tardy delta]

quick ways to fuck things up

[weak kayak]

ah yes big brain

[granite owl]

remember my attempt from a year ago to manipulate the command list at runtime by directly accessing the private maps with reflection? yea it worked

its nasty but i managed to completly delete vanilla commands at runtime

🤣

[tardy delta]

why wouldnt it

[granite owl]

yea but its a nasty hassle

[tardy delta]

i use reflections when i really shouldnt be using it

instead of making my field not final, i used reflections to change the internals of some lib i used 💀

[granite owl]

i tried to inherit a zombie class

and change its size

[tardy delta]

💀

[granite owl]

which is a private field of like 10 child classes

good news is i did manage to change it

bad news is server didnt like it and crashed

[tardy delta]

💀

[granite owl]

like actual crash not throwing an exception

🤣

[granite owl]

xD yea reflection is wyld

but its a rigid approach and if they change something in their code ur code just goes full nuclear karen

[twilit roost]

What would be the most resource friendly way to check if player holds item and then display action bar if he does

[rotund ravine]

There’s a event for player hotbar switch or smth

[pseudo hazel]

yes but you need to send an action bar message every so often anyways

[twilit roost]

and then timer as long as he holds it

[granite owl]

bukkit runnable?

[tardy delta]

isnt java 18+ supposed to use var- and methodhandles for reflection? cant seem to find anything?

[twilit roost]

yep

[pseudo hazel]

you would probably always have a timer

and then just either show the held item or nothing

on each timer update

[rotund ravine]

Have a set (weak) of players that has the item

One task that runs over the set

Double checks they’re holding the item then display it.

On that event add and remove from the set

[twilit roost]

yee just thought of this
imma implement it
thx

[granite owl]

have a recursive bukkit runable

[pseudo hazel]

why double check

[granite owl]

jk

[rotund ravine]

Dropping items

[pseudo hazel]

you dont even need the event

[rotund ravine]

He asked for a resource friendly way

[pseudo hazel]

yeah, having a single timer is most resource friendly is it not

[granite owl]

isnt it action based = events and state based = ticks?

to accurately check for behavior?

[pseudo hazel]

what behaviour

[granite owl]

like if the player takes the item off their hand

its an event

and to measure the time they helt it

its based on ticks

[pseudo hazel]

well yes

it would be based of a tasktimer

[rotund ravine]

He has a single timer with my idea?

[pseudo hazel]

yes but also this set with players

[rotund ravine]

He’s just not looping all players, but all players from the event.

[granite owl]

a single timer gets more load the more players are online

[rotund ravine]

No

Not the way i suggested.

[granite owl]

id prob have a timer per player and only when they hold it for as long as they hold it

[granite owl]

lemme backread again

[pseudo hazel]

no

[pseudo hazel]

this would be inefficient

[rotund ravine]

It’s mostly useless optimizing, but whatever he wants he gets.

[granite owl]

as in the runnable holds a map with players who hold the item

[rotund ravine]

Not a map.

[granite owl]

which is determined by the event

[rotund ravine]

A simple weak set

[granite owl]

that must be thread safe then

doesnt it?

have a thread safe interface

[pseudo hazel]

for what reason is it not

[granite owl]

isnt a runable a dedicated thread? xD

[rotund ravine]

A weak set does not lock the player interface.

He’s using a bukkit tasktimer

[granite owl]

uh

[rotund ravine]

Or should be atleast

[granite owl]

just read quickly trough it

a single threaded event called at fixed ticks?

[granite owl]

this^

[rotund ravine]

Sigh. Yes, it’s a mai. Thread task

main*

[regal scaffold]

Is the event called if someone is holding X item and switches off of it

Is it InventoryMoveItemEvent?

?jd-s

[undone axle]

https://hub.spigotmc.org/javadocs/spigot

[regal scaffold]

https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/event/player/PlayerItemHeldEvent.html

:/

[pseudo hazel]

use PlayerItemHeldEvent

[rotund ravine]

👍

[regal scaffold]

I assume the event gets called after the Item in hand is changed. so if I wanted to do something if a specific item was being held BEFORE it was changed, I don't even need to check the current item, just the event.getPreviousSlot()

[rotund ravine]

👍

[jovial shadow]

i need help

[regal scaffold]

?ask

[undone axle]

If you have a question, please just ask it. Don't look for staff or topic experts. Don't ask to ask or ask if people are awake or available. Just ask the question to the channel straight out, and wait patiently for a reply. Make sure you use the right channel regarding the topic of your question. Create a thread in case the channel is already in use!

[jovial shadow]

oh thx

[granite owl]

this is how HR thinks problems are solved xD

[jovial shadow]

im trying to make my own server using spigot but the run.bat file wont open, instead cmd prompt opens saying this: "Error: Unable to access jarfile spigot.jar"

[granite owl]

erm

doesnt the spigot server have a version number behind its name

u need to use the explicit file name

[jovial shadow]

oh

ohhhhh

thanks

[rotund ravine]

#help-server @jovial shadow

[jovial shadow]

oh thanks

[granite owl]

and its correct that the command prompt pops up

as u run the JRE from the prompt

[tender shard]

How can people literally not even read the error message lol

[compact haven]

you’re asking for a lot

[rotund ravine]

This is spigot sir

The average iq is 5

[river oracle]

?whereami

[undone axle]

https://cdn.discordapp.com/attachments/864180468680360027/944415049462714469/whereami.mp4

[tender shard]

Sorry

You‘re right

[compact haven]

forgiven

[regal scaffold]

How you been alex

How you feeling?

[granite owl]

astounding how 4 ppl bash on someone who clearly has never run a server before

👍

[rotund ravine]

I bash on all of spigot

[regal scaffold]

1. You're in the wrong channel
2. You can search for the exact 5 words you said in google and first link will give you answer

[tender shard]

^

[granite owl]

3rd i didnt ask the question but answered it kekw

[regal scaffold]

mb wrong tag, not that deep

As a matter fact, the first 4 links

Actually, the first 4 pages

[granite owl]

are you telling me how to google

[regal scaffold]

I'm telling the guy that asked the question how to google

mb wrong tag, not that deep

[granite owl]

yes he couldve done better, no i do not agree with the SO mentality

[tender shard]

idk I think when people are smart enough to blindly copy/paste some random bat file without knowing what it does nor trying to understand what it does, then they should at least copy/paste the error message they get into google too, instead of heading into a plugin development channel to throw it in there without any further context

[river oracle]

you should never paste code you don't know what it is / or what it does

that's how people get hacked

[true perch]

My main class stores a static singleton. This instance of the ItemManager will be heavily used to reference a HashMap found in the singleton. Is this acceptable?

public static final ItemManager ITEM_MANAGER = new ItemManager();```

[river oracle]

generally personally I'd stick with di

?di

[undone axle]

Guide to dependency injection: https://www.spigotmc.org/wiki/using-dependency-injection/

[river oracle]

however, this could be acceptable

[regal scaffold]

I also like DI more but it's fine

[river oracle]

I'd just switch from a static field to a static method

[true perch]

oh okay. So I'll make the singleton non static and make a static method used to reference the singleton?

[tender shard]

Thats basically the same thing

[regal scaffold]

Lately I've just been having a public ItemManager itemManager;
field and initialize in onEnable then just with DI use plugin.itemManager

[tender shard]

Yeah thats the better solution

[regal scaffold]

I like it, it looks clean and does everything correctly

[true perch]

and just pass the itemManager instance into every class that needs it?

[regal scaffold]

    public PhoneManager phoneManager;

    @Override
    public void onEnable() {
            phoneManager = new PhoneManager(this);
    }

Yup

[granite owl]

ik all im saying is that bashing on someone new doesnt really make em feel welcome like:

A: i have a stupid question.
B: ?ask
A: <asks stupid question>
BCDE: why are you so stupid.

this is like eh. remember when u just started did u know what is good and poor practice at the very beginning? xD

[true perch]

Oke, you guys are the pros. I'll do that 😄

[regal scaffold]

Then inside the "manager class"

[tender shard]

Ofc i didnt, but i used google for obvious error messages

[true perch]

public class ItemManager {

    /**
     * A HashMap of every item's ItemData.
     *
     * Integer - The number used to represent this item's unique identification.
     * ItemData - All the data associated with 7 Realm's Custom Item system.
     */
    private final HashMap<Integer, ItemData> itemDataMap;

    public ItemManager() {
        this.itemDataMap = new HashMap<>();
    }

    /**
     * Give a Player a Custom Item.
     */
    // TODO Create a command to execute this method
    public void giveItem(Player player, int id) {
        ItemData itemData = itemDataMap.get(id);
        player.getInventory().addItem(itemData.getITEM_STACK());
    }

    /**
     * Get item data based on unique ID.
     */
    public ItemData getData(int id) {
        return this.itemDataMap.get(id);
    }
}```is this acceptable?

[regal scaffold]

public class PhoneManager {

    private final MainPlugin plugin;

    public PhoneManager(MainPlugin plugin) {
        this.plugin = plugin;
    }

If you do not need a instance of the main class yes

[true perch]

What would be a good way to access the singleton when a player is executing a command though? for example.

[regal scaffold]

We just said it no?

[true perch]

Just not sure how to pass it in without a constructor

[granite owl]

btw intellij throws alot of warnings because i did not create an instance of my main class(obv), do i need some spigot plugin or so to have the IDE interpret the syntax correctly?

[regal scaffold]

Show me what you're tring to do

[regal scaffold]

What type of errors? I don't get any

[true perch]

I haven't done it yet, but I'll get started on the idea give me several minutes

[tender shard]

https://blog.jeff-media.com/getting-your-main-classes-instance-in-another-class/

[regal scaffold]

Use method 2

[granite owl]

not error, warnings that my in my plugin.yml defined main class has no instances

like im getting a warning that i never use it

obviously

[tender shard]

no, just add @SuppressWarnings("unused") to your main class

[regal scaffold]

Interesting

^

[granite owl]

xD

[tender shard]

or add that class as entry point

in the inspections window

[regal scaffold]

I don't get those errors tho

I wonder why

[tender shard]

or just ignore the "unused declaration" things

[regal scaffold]

Are you using maven?

[granite owl]

ya

[tender shard]

probably because of mc dev plugin

[granite owl]

i mean is there another way but maven now

[regal scaffold]

I don't use it

[granite owl]

that they remapped the server jar

[regal scaffold]

I think

Oh wait yeah it's installed

Install the plugin teminator

[granite owl]

xD

aight

[regal scaffold]

https://plugins.jetbrains.com/plugin/8327-minecraft-development

[granite owl]

also i do admit that using maven has its advantages, like u dont need to download a copy of the repo as maven does that for u

[regal scaffold]

Make sure you still make new projects using maven but it's nice to have it in the backgrouhnd

tbh if you're not using maven/gradle I would reconsider

[granite owl]

tbh i hated to start using it cause its just overly complex

but it does have benefits

[regal scaffold]

The days of adding dependency libraries using the jar file are dead

[granite owl]

yea ik

[regal scaffold]

If you want a quicker maven start for new plugin, use alex's architecture

[granite owl]

i already set it up

[regal scaffold]

Then you're ready to go

[granite owl]

after like 2 hours of testing i also finally managed to install the jdk and jre 20 on both my windows machine and my WSL distro

[paper venture]

I want to initiate a enum value with ItemStack. But I need this ItemStack to be modified (add lorelist, set title, etc.), this can't be done in one line. What should I do then?

[granite owl]

and correctly load it everywhere

🤣

[regal scaffold]

Show some code please,

[remote swallow]

item builders is the simple answer

[river oracle]

ItemBuilder

[regal scaffold]

They have arrived

[remote swallow]

or you'd need to use some lambda or consumer/callable for it

[granite owl]

though id still like to know how to download the remapped nms repo

[regal scaffold]

?nms

[undone axle]

https://blog.jeff-media.com/nms-use-mojang-mappings-for-your-spigot-plugins/

[tender shard]

Use the static init block

[granite owl]

i heared there would be one

[remote swallow]

nms isnt on a repo, you have to build buildtools

[paper venture]

Something like this, lambda is just for example...

[regal scaffold]

BuildTools is just better

[granite owl]

i did build it

[true perch]

public class Gi implements CommandExecutor {

    private final Realms main;

    public Gi(Realms main) {
        this.main = main;
    }
    
    @Override
    public boolean onCommand(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, @NotNull String[] args) {
        if (!(sender instanceof Player)) {
            sender.sendMessage("Must be executed by a player.");
            return false;
        }
        main.ITEM_MANAGER.giveItem((Player) sender,1);
        return false;
    }
}```If this is what I'm supposed to do, how should I store the instance of the `Gi` class?

[paper venture]

Lorelist and title depends on enum

[granite owl]

where can i find the nms dependencies

[remote swallow]

it should now be on your local maven rpeo

[granite owl]

kk

[regal scaffold]

Why do you need a instance of GI class

[true perch]

good point lmao

[regal scaffold]

Let it float

Into oblivion

It'll be happy there

[remote swallow]

$USER_HOME/.m2/repository/org/spigotmc/spigot/ver/spigot.version-remapped.jar

[true perch]

😂 okay cool haha

this works

[regal scaffold]

Wandering around your memory

[granite owl]

and i presume ill include that instead of the repo url?

that path

[regal scaffold]

No

Maven will always search local repo first

So you don't need a repo

[rotund ravine]

Consumer<ItemStack>

[true perch]

Are static imports frowned upon or acceptable? I'm interested in using them for easier readability

[regal scaffold]

Just fill the dependency section

[granite owl]

kk

[regal scaffold]

They are not but people tend to abuse static

[granite owl]

will do so once im home haha

🙏

[regal scaffold]

So teaching to not use it when not needed is better at the start

[sacred mountain]

i am so done, ive spent the past half hour figuring out why my listener doesnt work, now realising that i didnt register it

ffs

[regal scaffold]

xd

[true perch]

When is an okay time to use a static import?

[regal scaffold]

Happens to the best

[river oracle]

they tend to be frowned upon unless you are working with a class with a super long name and require static methods from it frequently

[regal scaffold]

y2k can prob answer that better

[sacred mountain]

NOW IT WORKS

AHAHA

[granite owl]

xD

[rotund ravine]

Oh wait nvm @paper venture Function<Void, ItemStack>

[regal scaffold]

but I assume static is mostly just used when you're making a Utility class or something that does not require a instance like a Utility methoid

[remote swallow]

i wonder if i could make an annotation processor to auto register my classes

[true perch]

Noted. I'll avoid them unless the naming gets too unbearably long

[river oracle]

a good example of a decent place to use a static instance would be when working with a builder heavy library such as Brigadier a command library by mojang

[paper venture]

So I just have to pass a functional interface and then an itemstack to my enums?

[rotund ravine]

You can

[rotund ravine]

I just went off what you posted.

[paper venture]

Okay thanks!

[tardy delta]

thats called a Supplier<ItemStack> kek

[sacred mountain]

these are some big scary words imma stick with String

[eternal oxide]

I did that once for auto registering protocol packet classes

not for MC

[river oracle]

you prob could, but like I wouldn't know how to do that on compile time

way to complex for my brain

[tardy delta]

javax annotation processor

[remote swallow]

you know who would know

[granite owl]

i think im going to wrap a function inside a native library in which i write my code in inline assembly. then i load that to my plugin and unlimited power

[nocturne steppe]

hello yesterday i could look at bukkit and spigot's code but now intellij wants me to decompile it to see it

[rotund ravine]

Ah yes that was it thank you

[tardy delta]

still waiting for pritimives support in generics

project valhalla when

[granite owl]

the what

[night verge]

I'm trying to modify the cauldron level but I have failed.
Is there anything wrong with my code?

This is my code. ("block" is the specific block that I want to modify)

Levelled cauldron = (Levelled) block.getBlockData();
cauldron.setLevel(0);
block.setBlockData(cauldron);

[nocturne steppe]

I mean yesterday i could just ctrl+click for example StructureManager to see the class, but now when i do that it asks if i want to decompile it

[granite owl]

not entirely sure what went wrong there but do u have the correct api?

because decompiling sounds as if u have byte code

[nocturne steppe]

i did deactivate filtering in my pom.xml

but apart from that i changed nothing

[granite owl]

idk man but im pretty sure the api is supposed to provide source code

🤣

[nocturne steppe]

ugghhhhh

[vital sandal]

what is the best way to create constantConfig?

[pseudo hazel]

whats that

[tardy delta]

lol

the questions are very weird here sometimes

[eternal oxide]

you also ran mvn clean. I don;t use InteliJ so perhaps you don;t have sources

[vital sandal]

like I want to change the sound of clicking UI which is configurable

[nocturne steppe]

i did have sources like, yesterday. but yeah it dissapeared after I cleaned

[vital sandal]

class/enum/inferface?

[pseudo hazel]

just make normal config file

[river oracle]

a texture pack?

[vital sandal]

nah

[river oracle]

unsure what you are really asking then

[vital sandal]

nvm

[eternal oxide]

no idea how old https://i.stack.imgur.com/15hcl.png

[vital sandal]

like this :l

there are reasons for me to create custom config

[nocturne steppe]

damn i was looking there just when you sent that x) Thanks!

[pseudo hazel]

just create a config file

[regal scaffold]

Static abuser!

[vital sandal]

.

[vital sandal]

why should it not be static?

[regal scaffold]

It can be static, just not recommended, people usually make too much stuff static which is called static abuse when it doesn't have to be static

So learning to not spam static from the start is good

[vital sandal]

It should be static

It is constant value and no need to be like that

and it is editable so not final

[regal scaffold]

Once again, never said it can't be static, just said it's not recommended

[vital sandal]

why?

why not recommended?

[pseudo hazel]

https://www.spigotmc.org/wiki/config-files/

[regal scaffold]

people usually make too much stuff static which is called static abuse when it doesn't have to be static

[pseudo hazel]

read this

having it static is fine

[vital sandal]

I do know this

[pseudo hazel]

hiw else would you be able to use it everywhere

[regal scaffold]

instance and di?

[sacred mountain]

anyone know if it's possible to reduce the size of the initial lingering potion bounding box before it splashes

[regal scaffold]

Which is the principle of oop

[pseudo hazel]

di makes everything messy for bigger plugins

since you have to pass the same thing 50 times in every constructor to just get some config value

[regal scaffold]

Which is why I said

It can be static, just not recommended

[pseudo hazel]

the file only exists once

[vital sandal]

It should be recommeded

[regal scaffold]

I've explained 3 times why it's not recommended but sure, it's your code, go ahead

[pseudo hazel]

anyways what is the issue?

[regal scaffold]

No idea, I stopped reading when he took advice as a personal attack

[sacred mountain]

another question can you splash a potion forcefully, or does it have to land somewhere

like i want to spawn a splash potion that just splashes instantly

does it do that anyway

[chrome beacon]

Does it have to be a splash potion or are you just trying to apply potion effects?

[sacred mountain]

it's for particle effects

so no there are no actual potion effects

[chrome beacon]

Any reason why you can't just spawn particles without the potion?

[sacred mountain]

it might have functionality later

so i'd like to keep that open

if i want it to do anything in future

[tardy flame]

^

[chrome beacon]

Not really

You can highlight blocks but not make them transparent

[tardy flame]

What do you mean highlight?

[true perch]

What's the acceptable way to reference a Material enum object from a config file?
My first thought was doing something like Material.valueOf("AMETHYST_SHARD"); as an example.

However this seems to produce an error, unsure why

[regal scaffold]

String is the best way

[true perch]

No enum constant org.bukkit.Material.LEGACY_AMETHYST_SHARD

[regal scaffold]

How are you reading it?

Show the config line

In code

[true perch]

public ItemData(int id, String name, String description, String material, int customModelData, ItemRarity rarity, ItemStats stats, ItemAttributes[] attributes, ItemEffects[] effects) {
        this.ID = id;
        this.NAME = name;
        this.DESCRIPTION = org.bukkit.Color.WHITE + description;
        this.MATERIAL = Material.valueOf(material);
        this.CUSTOM_MODEL_DATA = customModelData;
        this.RARITY = rarity;
        this.STATS = stats;
        this.ATTRIBUTES = attributes;
        this.EFFECTS = effects;
        this.ITEM_STACK = generateItemStack(this);
    }```

[regal scaffold]

Before that

[chrome beacon]

You forgot to set the api version in your plugin.yml

[regal scaffold]

You're either doing config.getMaterial or config.getString

[sacred mountain]

is it bad to have empty interfaces just to call instanceof on

for categorizing things

[vital sandal]

hmm singleton is a better way to do It :d

[sacred mountain]

i used DI

[regal scaffold]

Not sure if that's sarcasm or not

[true perch]

I wrote this to simulate reading data from a config filejava public ItemManager() { this.itemDataMap = new HashMap<>(); ItemStats stats = new ItemStats(1, 1, 10, 0, 0, 0, 0, 0, 0, 0); ItemData itemData = new ItemData(1, "Test", "This is a test item.", "AMETHYST_SHARD", -1, ItemRarity.MYTHIC, stats, null, null); this.itemDataMap.put(1, itemData); }

[sacred mountain]

use*

[chrome beacon]

.

[echo basalt]

ew no

[regal scaffold]

Then yes, API version

[true perch]

ohhhh

okay thanks guys

[vital sandal]

Dependency Injection?

[regal scaffold]

?di

[undone axle]

Guide to dependency injection: https://www.spigotmc.org/wiki/using-dependency-injection/

[true perch]

thank you as well @chrome beacon

[vital sandal]

I mean why?