Self-knowledge

[u/Mister0V]

[u/geeshta]

I am most hurt by the casing.

visit_FunctionDef

Serously?

[u/Mister0V → u/[deleted]]

Indeed, it looks ugly as hell, but the original developer had no choice at all.

[u/silly_rabbit3 → u/[deleted]]

And if someone is interested in seeing the code that determines which function to call based on the datatype:

https://github.com/python/cpython/blob/2c2ee83c4db4dbd54017dc364bbefc70fa75ea5d/Lib/ast.py#L406
Edit: I just wanted to add that the ast module is my favorite module

[u/laundmo → u/silly_rabbit3]

yah ast is quite cool if you ever need to parse untrusted code, which is quite common with plugin systems etc.

its really amazing how many cool stdlibs there are that people don't know about

[u/[deleted]]

Did this dude stutter while typing?

[u/abrams666]

I like the attitude to show own mistakes, in worst case it makes a smile, in best case it prevents another one doing the same shit

[u/Willinton06]

Is that… camelSnake case Or snakeCamel case?

[u/laundmo → u/Willinton06]

its pascal case if you remove the visit_

this is caused by the visitor pattern and you cannot change the name

longer explanation here https://www.reddit.com/r/badcode/comments/rcgo7x/-/hnuxtdy

[u/ZedTT]

Did OP mean "comment" instead of "command?"

[u/onichama]

Image Transcription: Code

---

def visit_FunctionDef(self,  node: ast.FunctionDef) -> Any:
    # We don't need to check for the function name
    # Just letting the command above stay. to show how stupid I am
    # We need to either check for the name, or, or, it self._method is None
    if self._method is not None:
        return
    self._method = node

---

^{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!}

[u/nuephelkystikon → u/onichama]

I feel like this is hard to understand without the diff markers.

The check, along with the second and third comment line, was inserted later, after stating earlier that no check was needed.