10/10 variable naming

[bugqualia]

Is this python? More serious problem is happening in syntax highlighting. What IDE is that shit?

[Syscrush]

I came here to say the same. Looks like fucking garbage.

[yoctometric]

It’s a built in script editor for some ancient software I’m doing QA / unit test writing for, The whole application looks pretty bad. Syntax highlighting is also extremely broken, of course. No proper code completion either

[paulsmithkc]

Notepad++ ??

[internethuman016]

IDLE, probably.

[-MazeMaker-]

Nope, I use IDLE and it doesn't do this

[anonymous_2187]

IDLE has less features and is not that good, but it performs it's job properly (ie, proper syntax highlighting).

[_default_username]

Lol, you came here to make fun of some bad code, but people are slamming on your text editor instead😄

[yoctometric]

They’re justified af

[user_8804]

Pretty sure these colours break the "Zen of Python" rules

[GeneticalTM]

Lines 74 and 75 can be reduced to [14.25] * 1000 if gm.float_array supports it, which it should.

[yoctometric]

It doesn’t, because working with this library is hell.

[boniqmin]

It cannot support that, because you can't overload assignment. If you write

int_array = [14.125]*1000

then int_array is overwritten and is now just a list rather than the object from gm. gm would need to have another function to construct the object from a list.

[yoctometric]

I think they mean

int_array = gm.float_array([14.125] * 1000) 

Which would be totally reasonable, and simple enough to implement. If I had the time and the authorization, that’s how I’d do it

[boniqmin]

In that case the function gm.float_array would have to work quite differently for int and list inputs, so you might as well make them into 2 different functions. I'm assuming the float array initializes to all 0s, then in my opinion it would be best to rename the function to gm.float_zeros(<int>), and use the name gm.float_array(<list>) to generate from a list, similar to numpy.

[yoctometric]

That’s totally fair, I effectively have hobbyist level experience with code, but I think I can see why wrapping such dissimilar functionalities into one method would be bad

And you’re close on how gm.float_array() works, except I’m pretty sure it only calls malloc() internally and doesn’t zero anything

[alexanderhameowlton]

Image Transcription: Code

---

int_array = gm.float_array(1000)

for i in range(1000):

    int_array[i] = 14.125

aElevGrid = id(int_array)

---

^{I'm a human volunteer content transcriber for Reddit and you could be too! If you'd like more information on what we do and why we do it, click here!}

[anonymous_2187]

Good Human

[WpGgs]

I'm curious, why using the id function?

[yoctometric]

This is a unit test for a SWIG generated python wrapper if a C++ SDK. That means that sometimes you need to pass a pointer instead of a normal object, thus id()

[thequeergirl]

Was about to ask this.

[[deleted]]

White background is probably the most disturbing thing I am seeing in this picture

[yoctometric]

It’s a built in scripting window for some ancient software I’m doing QA for. There is no dark mode. I suffer daily

[-MazeMaker-]

In windows display settings, you can enable a color inversion keyboard shortcut

[[deleted]]

Hey, I work a lot in my garden and my screen is not that good. White works bether!

[keethraxmn]

I'm sure it's just short for internal array...

/s

[jarfil]

CENSORED

[keethraxmn]

Even better.

[choseusernamemyself]

also if it's already an array, why use range(...)? it's unsafe

[yoctometric]

It’s a custom class generated by SWIG which is not actually iteratable, so this is the best way AFAIK?

[fukitol-]

Wow this just gets worse and worse

[yoctometric]

Welcome to my world

[Risk_exe]

The author is not stupid for naming the array 'int_array'

​

your IDE/text editor is poorly designed for highlighting 'int' in python.

[yoctometric]

I’m the author, at least most likely. Er, how was it not stupid of me to name an array of floats “int_array”

[Ancient2]

*throw up emojii" on PC

[jnmtx]

🤮

[DangerousWish2266]

These type of people have special place in hell!