|
What? do i need to write 4 lines to avoid an If?, I have better things to do...
|
|
|
|
|
SarK0Y wrote: Here, i'd like to discuss how to reduce conditional branches in the code. 1st of
the all, i would like to share some tricks.
I would suggest that you learn what automated profiling is. Then buy or find a tool that does that. Then learn how to use it effectively.
|
|
|
|
|
I'm definitely a beginner but I have a question:
Doesn't the program end up doing an if/then regardless of whether or not you're removing the actual syntax? There still needs to be a comparison done somewhere, or at least I thought.
|
|
|
|
|
The old IBM 360 JCL didn't have any if statements.
In PERL, you'd normally see something like
open ("fred") || die
which translates to
if (not open("fred")) then die
If you look at the assembler, in both cases, it is exactly the same.
|
|
|
|
|
We had such discussion in the eighties when programmers wanted to be artists, writing unreadable code just to be irreplaceable.
If you would be one of my programmers I would fire you immediately for this waste of money and time and delivering error-prone code. I thought those times were gone!
------------------------------
Author of Primary ROleplaying SysTem
How do I take my coffee? Black as midnight on a moonless night.
War doesn't determine who's right. War determines who's left.
|
|
|
|
|
heh, i wasn't here for awhile actually, ye're wrong on that: readable codes are good, only if you have no the needs of deep optimizations. deep optimization means vast asming.
P.S.
for fast sorting of floats, my if-reduced code http://sourceforge.net/projects/fsort--no-if/?source=navbar
|
|
|
|
|
SarK0Y wrote: readable codes are good, only if you have no the needs of deep optimizations. deep optimization means vast asming.
Code can be both: Optimized and readable. It's just depends on the plannings. Cryptic code isn't any faster than good, optimized readable code. It's just harder to understand and correct. That's all.
|
|
|
|
|
Ingo, did you write asm codes ever???
|
|
|
|
|
SarK0Y wrote: Ingo, did you write asm codes ever???
Okay. I talked about higher languages. If you write asm codes, I won't mind if you code another way.
|
|
|
|
|
Without a doubt, that's the worst idea I've seen in a long time. Instead of using a nice, easy to read if , you've turned it into a very-expensive-to-support-and-debug piece of crap that you'd have to throw unit test after unit test at with a crap-ton of edges cases. Congratulations!
Now go back and wipe that smiley face off your post before I smack it off.
|
|
|
|
|
A true gem...
If CurrentContext = "D" Then GoTo c45
GetUserDataRecord aForm
c45:
There were no other ref's to c45, so replaced by...
If CurrentContext <> "D" Then GetUserRecord aForm
Further down in same procedure:
GoSub ResolveTabOrder
Exit Function
ResolveTabOrder:
If uControlNbr < 2 Then Return
GoTo ResolveTabOrder
End Function
modified 7-Feb-13 7:54am.
|
|
|
|
|
Now this is what happens if you Google "coding" in the "Fill in the blanks" forum
|
|
|
|
|
Just came across a [view name].[column name] combination in some SQL code I'm working with that is that many characters long. Actually, if it included the square brackets, it'd be 81 characters.
Here it is (characters changed to protect the guilty):
vw_AaaaAaaaaaaa_BbbbBbbbBBB_vw_CcccCcccCcccc.AaaaAaaaaaaaDdddddDddDddddddDddd
|
|
|
|
|
Quite long. Suppose this is an unique name...
|
|
|
|
|
I'm sure I already read this a few weeks ago... Did you get stuck in a loop? Codebase day...?
It's an OO world.
public class Naerling : Lazy<Person>{
public void DoWork(){ throw new NotImplementedException(); }
}
|
|
|
|
|
You know, you may be right. I did have a feeling of deja vu when I posted it.
|
|
|
|
|
Maybe this is why it seems familiar. That must be the answer.
|
|
|
|
|
Nope, I still remember the Aaaa_Bbbbb.Cccccc_Ddddd kind of notation.
It's an OO world.
public class Naerling : Lazy<Person>{
public void DoWork(){ throw new NotImplementedException(); }
}
|
|
|
|
|
working with Dates in codes and reports on different date formats and zones contributes to some of my coffee breaks.
|
|
|
|
|
Do dates go well with coffee? Have you tried figs as well?
|
|
|
|
|
I would love to take dates for coffee...
|
|
|
|
|
hahahah guess you got the whole point
|
|
|
|
|
Our old customer service system has some brilliantly bad code in it, but I just had to share this one I spotted this morning.
I think the guy got halfway through and forgot exactly what he was even meant to be fixing, let alone how to fix it.
If CurrentUser.WorkingSession.Market.TaxInclusiveInCatPrice Then
Else
End If
Me.lblTotalOutstanding.Text = String.Format("{0:c}", decOutstanding)
I just wish we still had our old issue tracker to see what the mystical ISSUE 2862 is...
Phil
modified 6-Feb-13 5:50am.
|
|
|
|
|
I have seen something like that, but this time the reason being, the contractor didn't get paid, yup so he eloped with the pending code.. thankfully he didnt' leave behind any ISSUE 2862 for us to figure out
|
|
|
|
|