←back to thread

Go is still not good

(blog.habets.se)
644 points ustad | 6 comments | | HN request time: 0.784s | source | bottom
Show context
813ac4312b25c ◴[] No.44983544[source]
> Probably [hello NIGHTMARE !]. Who wants that? Nobody wants that.

I don't really care if you want that. Everyone should know that that's just the way slices work. Nothing more nothing less.

I really don't give a damn about that, i just know how slices behave, because I learned the language. That's what you should do when you are programming with it (professionally)

replies(4): >>44983610 #>>44983714 #>>44983885 #>>44985349 #
terminalbraid ◴[] No.44983610[source]
If you're fine with that then you should be upset by the subsequent example, because by your own definition "that's just not the way slices work".
replies(1): >>44983646 #
1. 813ac4312b25c ◴[] No.44983646[source]
I am fine with the subsequent example, too. If you read up about slices, then that's how they are defined and how they work. I am not judging, I am just using the language as it is presented to me.

For anyone interested, this article explains the fundamentals very well, imo: https://go.dev/blog/slices-intro

replies(1): >>44983686 #
2. terminalbraid ◴[] No.44983686[source]
Then you seem to be fine with inconsistent ownership and a behavioral dependence on the underlying data rather than the structure.

You really don't see why people would point a definition that changes underneath you out as a bad definition? They're not arguing the documentation is wrong.

replies(3): >>44983824 #>>45008203 #>>45008256 #
3. assbuttbuttass ◴[] No.44983824[source]
The definition is perfectly consistent. append is in-place if there's enough capacity (and the programmer can check this directly with cap() if they want), and otherwise it allocates a new backing array.
replies(1): >>44983869 #
4. terminalbraid ◴[] No.44983869{3}[source]
Yes, it's consistent and complicated and non-intuitive.

"Consistent" is necessary but not sufficient for "good".

5. ◴[] No.45008203[source]
6. rustandmore ◴[] No.45008256[source]
Ownership is perfectly consistent.

`append` always returns a new slice value

`func append(slice []Type, elems ...Type) []Type`

The only correct way to use append is something like `sl = append(sl, 1, 2, 3)`

`sl` is now a new slice value, as `append` always returns a new slice value. You must now pass the new slice value back to the user, and the user must use the new slice value. The user must not use the old slice value.

It's trivial to fix the "bug" in the article, once you actually understand what a slice value is: https://go.dev/play/p/JRMV_IuXcZ6

A slice is not the underlying array, and the underlying array is not the slice. A slice is just a box with 3 numbers.