|
Marcello Faga wrote: Someone knows how fast this code can kill your application?
It depends on the stack size of your application...
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
|
|
|
|
|
I think everyone here knows that.
|
|
|
|
|
Of course PIEBALD, it was a rethorical question .
The real question is how someone can have coded that one?
****************************
Strong congruence
for strong people;
with a compatible behaviour.
For a semantical way of life.
modified on Wednesday, June 11, 2008 4:54 AM
|
|
|
|
|
It's actually very easy, because recursive properties are perfectly legal. And Refactoring doesn't do anything about it, because you have to invoke it manually for a specific Refactoring case. While this is definitely a subtile bug, it is a pretty common one if you are following the basic .NET Framework Naming convention to use PascalCasing for the public property and camelCasing for the private field. Intellisense can even help you to make it wrong, depending on your exact settings and what you typed before. If you put an underscore in front of the field name, it can't happen (or is at least easy to spot).
|
|
|
|
|
That's one reason I hate camel casing. Pascal casing or Hungarian notation is my thing.
|
|
|
|
|
That problem has been presented here or in Subtle Bugs multiple times.
|
|
|
|
|
Using automatic properties you won't run in such problem.
Giorgi Dalakishvili
#region signature
my articles
#endregion
|
|
|
|
|
Oops...
"The clue train passed his station without stopping." - John Simmons / outlaw programmer
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
|
|
|
|
|
The following code I found some developer used very often. I just wonder might he loves to run getStatus twice to make sure his code executed perfect.
if(GetStatus(id) != "")
string status = GetStatus(id);
private string GetStatus(int recordId)
{
string s = "";
//First, use ADO.NET to get the Status of this recordId
//Then, assign the status to a string.
if(//Check to make sure there is data and not DB null)
s = //The data;
//Finally, return the string
return s;
}
|
|
|
|
|
Possibly he went from the VB realm.
If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.
This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke
|
|
|
|
|
It is an ambitious way to save a memory variable that people succumb to a bad performance of an application by calling a lengthier function twice or more.
Vasudevan Deepak Kumar
Personal Homepage Tech Gossips
All the world's a stage,
And all the men and women merely players.
They have their exits and their entrances;
And one man in his time plays many parts... --William Shakespeare
|
|
|
|
|
If it's worth getting, it's worth getting twice.
|
|
|
|
|
That is a race just waiting to happen!!! In C# I try not to use an 'is' followed by a cast for the same reason. The following code reads so much nicer:
Bar bar = o as Bar;
if(bar != null)
{
}
than this:
if(o is Bar && !object.ReferenceEquals(o, null))
{
}
|
|
|
|
|
I wrote this at 4AM the other morning :S I'm a little embarrassed. But it's still doing it's bit in my program.
for(int y = 0; y < bmp.biHeight; y=y+3){
for(int x = 0; x < bmp.biWidth; x=x+3){
for(int i = 0; i < 9; i++){
int yp=0, xp=0;
get_pixel(x+xp,y+yp,blank[i]);
if(xp<3)
xp++;
else{
xp = 0;
yp++;
}}
avg(blank, Merge, 9);
set_pixel(x,y,Merge);
for(int i = 0; i < 9; i++){
blank[i].rgbtBlue=0;
blank[i].rgbtGreen=0;
blank[i].rgbtRed=0;
}
clear_pixel(Merge);
yp=0, xp=0;
}}
"Sir, I protest. I am NOT a merry man!"
|
|
|
|
|
|
Did I miss something? isn't:
yp=0, xp=0;
out of scope?
|
|
|
|
|
Yeah, that was an error I have since fixed. long xp=0, yp=0 was defined before this segment. Only after did I realise I had 'int'ed them.
"Sir, I protest. I am NOT a merry man!"
|
|
|
|
|
Did you also realize that you haven't 'int'ed them, you actually have 2 new variables defined in the scope of the inner loop? :-P the long ones are only being assigned 0 every time that loop executes and thats it.
Scope is a wonderful thing, but it can also come back to bite you if you aren't careful.
|
|
|
|
|
Yes... that was the error I fixed.
"Sir, I protest. I am NOT a merry man!"
|
|
|
|
|
How fast does that run? in my experience using GetPixel and SetPixel are extremely slow
|
|
|
|
|
Fast enough that changing it would have no observable change.
But then, that may just be the fruits of overclocking.
"Sir, I protest. I am NOT a merry man!"
|
|
|
|
|
I can't even be assed looking at code formatted like that – incomprehensible! The first thing I do at work when I encounter rubbish like that is re-format it.
Steve
|
|
|
|
|
Nothing too bad there. One of the most "elegant" pieces of code I wrote in college was done after pounding down a fifth of tequila It was an amazing piece of work that worked very well.
"The clue train passed his station without stopping." - John Simmons / outlaw programmer
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
|
|
|
|
|
|
It seems the author doesn't know foreach . Nor how to cast.
|
|
|
|