Unnecessary Redundant Comment:

// this get data
public void getData(Data data)
{
.......
}

Why Just Why?

He just wanted to document how to call this.getData()

Why does the comment not say it is public?!? I am totally confused now and would not be able to use that method properly.

Why the getter method doesn't return data?

@Gxost I don't know... He used void instead

@Gxost reference passed in parameter, I suppose.

Never have I ever seen a getter return void.

@Ranchonyx exactly

@Gxost it clearly says in the content that this will get the data, not the caller

@johnmelodyme and you've actually, really, discovered this in some Codebase?

Well, it does get data.

@Ranchonyx agree that void isn't common, but in some languages heavy constructs are better built in place of the given parameter than returning by value 🤓

@Ranchonyx Yea

I love that Data type. Isn’t everything basically data?

Useless as f*ck comment
A void getter
A Data type

3 problems in 2 lines

I'd be more concerned not about a comment someone probably forgot to remove but about the fact that a method that's supposed to return data is passed that data as an argument...

Given it returns `void` and accepts a `Data` object as a parameter, I would expect this to set the data... 🤷‍♂️

What if the function gets the data from an api using some parameters inside the Data variable ?

@kiki that reminds me of
https://javascript.info/ninja-code
where it says:

The ideal name for a variable is data. Use it everywhere you can. Indeed, every variable holds data, right?

…But what to do if data is already taken? Try value, it’s also universal. After all, a variable eventually gets a value.