|
Just a bit too verbose
private void AddYaxisChannelForMathChannelInFramesBasedTimeline(PlaybackChannelMathChannelItemViewModel viewmodel)
{
.
.
.
}
If it's not broken, fix it until it is.
Everything makes sense in someone's mind.
Ya can't fix stupid.
|
|
|
|
|
What do you get when you cross a joke with a rhetorical question?
The metaphorical solid rear-end expulsions have impacted the metaphorical motorized bladed rotating air movement mechanism.
Do questions with multiple question marks annoy you???
|
|
|
|
|
Scandalous!
The 'a' in 'Yaxis' should be a capital letter!
|
|
|
|
|
Snap![^]
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Well, it was an obvious mistake!
|
|
|
|
|
Thats a wonderful observation !!!
|
|
|
|
|
I love that it's a private function.
That's solidly self-commenting code though.
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|
|
Never touch anyone else's private parts, unless you really really really trust each other
Kitty at my foot and I waAAAant to touch it...
|
|
|
|
|
That's terrible - the method name should obviously be:
AddYAxisChannelForMathChannelInFramesBasedTimeline
It's "Y Axis", not "Yaxis".
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Verbose, yes, but far better than:
private void DoSomething(MyType data)
{
.
.
.
}
Just because the code works, it doesn't mean that it is good code.
|
|
|
|
|
My week was made today while working on one of our old production applications. Trying to hunt down a particularly obtuse relation, among the piles of tables, I found this gem:
x.dbo.tbl50
number float notnull
SELECT * FROM [x].dbo.tbl50
number
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|
|
Nathan Minier wrote: tbl50 At least it is descriptive.
There are only 10 types of people in the world, those who understand binary and those who don't.
|
|
|
|
|
I think that's the wonderful part.
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|
|
That's just crazy! They should have used an int!
"Go forth into the source" - Neal Morse
|
|
|
|
|
There's a reason I included the schema
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|
|
Charlie Gilley
<italic>Stuck in a dysfunctional matrix from which I must escape...
"Where liberty dwells, there is my country." B. Franklin, 1783
“They who can give up essential liberty to obtain a little temporary safety deserve neither liberty nor safety.” BF, 1759
|
|
|
|
|
|
Thank you.
The fact that this is actually a thing made me a little sad, and afraid for the world.
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|
|
It's a thing because the alternatives sucks.
Tally tables are a well established trick to gain performance, and we're talking huge differences here.
It's all about not using procedural code. Databases works best with sets. A tally table is a set.
Now I want you to read the conclusion in Graus's article, because it's good.
|
|
|
|
|
Wow, learn something new every day. Thanks.
When you are dead, you won't even know that you are dead. It's a pain only felt by others.
Same thing when you are stupid.
modified 19-Nov-21 21:01pm.
|
|
|
|
|
Once you stop learning, you may as well be dead.
#SupportHeForShe
Government can give you nothing but what it takes from somebody else. A government big enough to give you everything you want is big enough to take everything you've got, including your freedom.-Ezra Taft Benson
You must accept 1 of 2 basic premises: Either we are alone in the universe or we are not alone. Either way, the implications are staggering!-Wernher von Braun
|
|
|
|
|
Honestly, for anyone that works regularly with LINQ "thinking in sets" is the norm. It's just that those sets are considered after a data layer call has completed, not before. I guess it's all about the abstraction layer that we live in.
On your suggestion I re-read Graus's article and, with inspiration from the article, I found that the table did indeed act as a tally for 2 SPs, which are executed annually, on about 1000 rows between them.
So ultimately tbl50 does indeed have a purpose that would have been better served by an incremental (in this specific case). I appreciate the guidance!
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|
|
Well, I wouldn't dirty down the database with a tally table if performance isn't of any importance for something that's used twice a year.
The right tools for the right job is a good rule.
|
|
|
|
|
So now go and rename the table "TallyTable" or add an Description property to the table or something to help the next pilgrim who emerges from the brambles.
cheers
Chris Maunder
|
|
|
|
|
It's a nice thought CM, but honestly I'm gutting the thing for useful logic. SPs will be moved into application code so that I can properly abstract this thing, and that table will have no place in the new version.
"There are three kinds of lies: lies, damned lies and statistics."
- Benjamin Disraeli
|
|
|
|