31
The Yandere Dev/Pirate Software dilemma (media.piefed.social)
submitted 5 months ago by to c/godot@programming.dev
12 comments fedilink hide all child comments
 

UPDATE: The consensus seems to be overwhelmingly in favour of the match variant. And not to worry, I have replaced the magic numbers with an enum. Will try to remember to merge the branch tomorrow

Does an if-statement block or a switch statement fit better here? For context (and advertisement), this is part of my all-purpose utility plugin ( Codeberg link)

The code:

		# Method 1 (Yandere Dev Technique)  
		if self.throw_errors and status==MpupTest.TESTSTATUS.ERROR:  
			push_error(result)  
		if self.throw_warnings and status==MpupTest.TESTSTATUS.WARNING:  
			push_warning(result)  
		
		# Method 2 (Pirate Software Technique)  
		match status:  
			MpupTest.TESTSTATUS.ERROR:  
				if self.throw_errors:  
					push_error(result)  
			MpupTest.TESTSTATUS.WARNING:  
				if self.throw_warnings:  
					push_warning(result)
a code screenshot (code in description)
you are viewing a single comment's thread
view the rest of the comments
[–] 5 points 5 months ago*

Both these devs are far too full of themselves to think of using an enum table or anything clever with datatypes themselves.

Because that would mean the way they've been doing things was potentially grossly inefficient at runtime... and they've been doing that for years...

But a narcissist never makes a mistake, so instead, here's a bunch of reasons why actually writing optimized code is stupid, first among them being 'well it didn't occur to me, therefore doing it this way would reduce code readability for this code base that only I and no else needs to read'.

I am guessing the concept of a sentinel value or an overloadable function would make their minds implode.

But anyway, 'switch' is 'match' in gdscript:

https://docs.godotengine.org/en/latest/tutorials/scripting/gdscript/gdscript_basics.html#match

match <test value>:
	<pattern(s)>:
		<block>
	<pattern(s)> when <pattern guard>:
		<block>
	<...>
  • permalink
  • fedilink
  • source
  • parent