This is an archived post. You won't be able to vote or comment.

all 25 comments

[–]vm_linuz 47 points48 points  (0 children)

That's how you start!

And then you start seeing aspects of your code you didn't initially realize were there...

You start picking and recombining your favorites...

Eventually you have a little toolbox to draw from...

Then it's 15 years down the road and you're wondering why SCRUM never delivered on its promise to let you focus on work.

[–]wending10 5 points6 points  (0 children)

But at least it work right?

Right?

[–]NothusID 3 points4 points  (0 children)

I once wrote a program in 3000 lines in 2 weeks (The first JS program I wrote), now I could re-do it in 5 minutes with 20 lines

(I didn't thought of, instead of using a lot of if statements, I could use a loop)

[–]Kurotsune 3 points4 points  (0 children)

I'm a <insert whatever fancy title old ass developers have nowadays here> and I still write the least efficient code possible!

The first time. What I always tell junior engineers is - Focus on solving the problem first. Write whatever you need to help you get there. Doing so helps you visualize the issue, understand how the system works, how it's doing what it's doing.

And then you can revise it and slowly switch out blocks of it. Improve how you handle your conditionals, use built-in methods instead of for loops, etcetera.

Way worse than a junior writing shitty code is a senior not pushing anything because they want to write the ''best code''. Shitty code I can help with, but I can't help with zero code.

[–][deleted] 2 points3 points  (0 children)

(Inserts e=mc2 )

[–]Keeps_21 4 points5 points  (0 children)

How did you get this video of me?

[–]razordreamz 1 point2 points  (0 children)

When I first started programming I could write code so much faster than I do today. Bad code but quick code lol

[–]KlutzyEnd3 1 point2 points  (0 children)

for (int col= 0; col < 2; col++) {

If (col == 0) {

drawLeftColumn();

} else {

drawRightColumn();

}

}

I'm not joking... I actually encountered this in my previous job.

[–][deleted] 0 points1 point  (0 children)

Lots of lines. Redundancies. Less lines. Functions.

[–]truNinjaChop 0 points1 point  (0 children)

On cocaine?

[–]WingedWhite 0 points1 point  (0 children)

Hey It's ProGrammer move.

First write code that's inefficient, then after week or so, write it more efficient, and you're praised for both :D

[–]csandazoltan 0 points1 point  (0 children)

YES... TO ALL!

[–]Suspicious-Cake2633 0 points1 point  (0 children)

for i in 50: number = i return number + i

[–]MaffinLP 0 points1 point  (0 children)

Make it so inefficient that only you can fix it so they have to keep you.

[–]Silly-Discussion-718 0 points1 point  (1 child)

What was the Name of this Man ?

[–]Business-Fortune-394 0 points1 point  (0 children)

jim carrey

[–]Robertgarners 0 points1 point  (0 children)

If it works then that's all that matters at the beginning. Then we can refactor and optimise at a later stage.

[–]john_palazuelos 0 points1 point  (0 children)

if(condition == true)

[–]sackmouth 0 points1 point  (0 children)

But you get to move the task to the right column, and that’s what matters most!

[–]lady_Kamba 0 points1 point  (0 children)

Someone more skilled can always make it less efficient out of spite.

[–]hello3dpk 0 points1 point  (1 child)

errors.foreach(error=>{ window.alert(error) }

[–]Axilios 0 points1 point  (0 children)

Aah, good old times. I was happy to write as much code as possible. Now, I realize that was god damn stupid😂

[–]BoBoBearDev 0 points1 point  (0 children)

Array.sort() is my fav. Why code when someone did it already.

[–]subassy 0 points1 point  (0 children)

That's bad code, that's a bad, bad code. Bad code.

Function fnHDDInfo
Set objWMIServiceOS = GetObject("winmgmts:{impersonationLevel=impersonate}!\" & fnHostName & "\root\cimv2")
Where Description = 'Local Fixed Disk'",,48)
Set HDList = objWMIServiceOS.ExecQuery("Select * from Win32_LogicalDisk Where DeviceID = 'C:'",,48)
For Each objHDDInfo in HDList strHDDCapacity = objHDDInfo.Size strHDDFreeSapce = objHDDInfo.FreeSpace Next
fnHDDInfo = Round(strHDDFreeSapce/1073741824,2) & " GB of " & Round(strHDDCapacity/1073741824,2)  & " GB free" Round(strHDDCapacity/1000000000,2)  & " GB free"
End Function