|
Especially nice considering the getter... This translates to a fancy "name_ = name_", well done 
'As programmers go, I'm fairly social. Which still means I'm a borderline sociopath by normal standards.' Jeff Atwood
'I'm French! Why do you think I've got this outrrrrageous accent?' Monty Python and the Holy Grail
|
|
|
|
|
I didn't spotted that initially (perhaps I was way too attracted by try/catch). But now I've fallen from my chair, thank you. 
|
|
|
|
|
I think this is incomplete. You should add this in the try block:
if (this.Name != name_)
throw new InvalidOperationException("Bad XML !");
'As programmers go, I'm fairly social. Which still means I'm a borderline sociopath by normal standards.' Jeff Atwood
'I'm French! Why do you think I've got this outrrrrageous accent?' Monty Python and the Holy Grail
|
|
|
|
|
And before that:
if (this == null)
{
throw new NullReferenceException();
}
And from the clouds a mighty voice spoke:
"Smile and be happy, for it could come worse!"
And I smiled and was happy
And it came worse.
|
|
|
|
|
Oh yes, silly me! I forgot to check that case 
'As programmers go, I'm fairly social. Which still means I'm a borderline sociopath by normal standards.' Jeff Atwood
'I'm French! Why do you think I've got this outrrrrageous accent?' Monty Python and the Holy Grail
|
|
|
|
|
I'm just wondering in how many ways this is useless...
And from the clouds a mighty voice spoke:
"Smile and be happy, for it could come worse!"
And I smiled and was happy
And it came worse.
|
|
|
|
|
Don't forget this fella:
Microsoft.Win32.SystemEvents.SessionEnding += (a,b) =>
{
MessageBox.Show("An expected error occurred.");
}
Don't forget to rate my post if it helped! 
"He has no enemies, but is intensely disliked by his friends."
"His mother should have thrown him away, and kept the stork."
"There's nothing wrong with you that reincarnation won't cure."
"He loves nature, in spite of what it did to him."
|
|
|
|
|
And this, at least if we have multithreaded code:
catch(ThreadAbortException x)
{
throw;
}
For one thing,it would lessen the appearance of doing Pokemon exception handling (You must catch them all!) and it may have strange side effects to let this exception simply disappear.
And from the clouds a mighty voice spoke:
"Smile and be happy, for it could come worse!"
And I smiled and was happy
And it came worse.
|
|
|
|
|
It's not only Pokemon exception handling, Smurf naming convention is also widely used in the application I took the snippet from. I just removed it from class name, because it leads in some way to the source of the snippet.
|
|
|
|
|
CDP1802 wrote: if (this == null)
{
throw new NullReferenceException();
}
You don't know how hard we laughed when we found such one in code library.
|
|
|
|
|
Won't ((InspectionMethod*)BadPointer)->Resuscitate() raise the exception ?
|
|
|
|
|
While this is obviously dumb code, in C# and Delphi (and any other language with property support like VB.net), a 'simple assignment' is essentially calling a method so it is as possible to have crash-inducing code in there as it is anywhere else. Particularly if it's something like "myReallyComplicatedRemotingThingy.Connected = true".
|
|
|
|
|
The source code for the property is included. If that causes a crash, you have bigger problems.
Don't forget to rate my post if it helped!
"He has no enemies, but is intensely disliked by his friends."
"His mother should have thrown him away, and kept the stork."
"There's nothing wrong with you that reincarnation won't cure."
"He loves nature, in spite of what it did to him."
|
|
|
|
|
Yes, obviously in the posted example the developer is just an idiot. Just saying, an assignment sign doesn't automatically make a line of code crash-proof. Even a property getter (which looks totally innocuous) can break stuff, though one can argue that that is bad style in itself (Microsoft do it though, for example trying to read Length out of an unbounded stream).
|
|
|
|
|
too sad you can't override the assignment operator in c#
|
|
|
|
|
I love the error message!
VUnreal wrote: MessageBox.Show(ex.Message, "Serialization error has happened.",
MessageBoxButtons.OK, MessageBoxIcon.Exclamation);
That's assuming an error is thrown. (I've seen the computer do impossible things several times, so I wouldn't put it past it.)
Serialization isn't happening here. No information about where it is happening. Oh shoot, it has ex.Message in it. at first I thought it wouldn't even tell you what was wrong.
Oh, lets throw in an icon to show this is an important error message.
The name of the routine is interesting. You're in a type safe environment that has an invalid type passed into the string and you are going to "fix" it by passing the invalid type back into the string. Really "advanced" thinking going on here.
|
|
|
|
|
Consider a text box with auto complete checking for the existence of the char . or [
Brilliant outsource developer sitting far far away writes the following:
private TextOffsets getTextOffsets(string text, int caretIndex)
{
var heirarchyChars = HeirarchyChars.ToArray();
int previousSeparatorOffset = -1;
if (caretIndex > 0)
{
previousSeparatorOffset = text.LastIndexOfAny(heirarchyChars, caretIndex - 1, caretIndex);
}
...
And of course commits the change... 10 minutes after update contracts kick-in in run time and
ContractException was unhandled
Precondition failed: this == String.Empty || startIndex - count >= 0
Then I go  Update and get this
private TextOffsets getTextOffsets(string text, int caretIndex)
{
var heirarchyChars = HeirarchyChars.ToArray();
int previousSeparatorOffset = -1;
try
{
if (caretIndex > 0)
{
previousSeparatorOffset = text.LastIndexOfAny(heirarchyChars, caretIndex - 1, caretIndex);
}
}
catch (Exception x)
{
}
...
Absolutely brilliant. Let's not fix it.. let's circumvent it... 
By this time curses in Greek are being launched...
Next day update and...
private TextOffsets getTextOffsets(string text, int caretIndex)
{
var heirarchyChars = HeirarchyChars.ToArray();
int previousSeparatorOffset = -1;
int count = caretIndex - 2;
if (caretIndex > 0 && count > 0 )
{
previousSeparatorOffset = text.LastIndexOfAny(heirarchyChars, caretIndex - 1, count);
}
...
 what the...
Pen and paper just to make sure I am not hallucinating... yep we always go back one to check one character forward... Also we have an innovation... one would start an expression for a struct or an array with a . or a [ from position 0 without a variable first. Just brilliant...
Oh well... it should at least be...
private TextOffsets getTextOffsets(string text, int caretIndex)
{
var heirarchyChars = HeirarchyChars.ToArray();
int previousSeparatorOffset = -1;
if (caretIndex > 1)
{
previousSeparatorOffset = text.LastIndexOfAny(heirarchyChars, caretIndex - 1, 1);
}
...
Alberto Bar-Noy
---------------
“The city’s central computer told you? R2D2, you know better than to trust a strange computer!”
(C3PO)
|
|
|
|
|
Nice development environment you have there. Somebody offshore can commit a change that immediately hits production without going through test environment, not bothering with code review, your offshore idiot remains hired... (You are saving $ not paying somebody local, breakdowns of your application reduces costs of running the system for no customers because it doesn't work...)
I'm not too sure about you...
Alberto Bar-Noy wrote: yep we always go back one to check one character forward...
That's for?:
Alberto Bar-Noy wrote: previousSeparatorOffset = text.LastIndexOfAny(heirarchyChars, caretIndex - 1, count);
If caretIndex==10, count would be 8, the start of looking in the array is at 9. If text is a 5 character string, you are allowing the search to start 5 characters past the first valid index location. Same situation, except the string is 10 characters long. You can only look 1 character ahead, yet you are asking for 8, that will blow with the original error you got. (The string has to be 18 characters long or longer for that to work.)
Your statement to look only 1 character forward doesn't make sense either. Going from where you start to the end of the string makes sense to me. Why you are going back 1 character from the index passed doesn't make sense to me either. I'd want to use the last index found to find the next index to be found.
Here's an alternate version:
private TextOffsets getTextOffsets(string text, int caretIndex)
{
var heirarchyChars = HeirarchyChars.ToArray();
var cnt = heirarchyChars.Count();
var strt = caretIndex + 1;
int NewIndex = -1;
if (strt < cnt)
{
NewIndex = text.LastIndexOfAny(heirarchyChars, strt, cnt - strt);
}
Pass in 9 for the starting index for a 10 character string, cnt is 10, strt is 10, doesn't check the string at all.
Of course, since I'm perfect,  I'll go ahead and check this in without even seeing if it will compile...
|
|
|
|
|
well this code never reached production' This is what I was hired for... to kick them out. This code tries to check if the last character typed was . or [
this is an autocomplete for structs and arrays on a PLC Dev. system's only the last char needs to be checked. And in WPF the caret position is after the last character. Finally because of politics (they got their notice so they are still around till Jan. first and I can't fully touch their code.
Alberto Bar-Noy
---------------
“The city’s central computer told you? R2D2, you know better than to trust a strange computer!”
(C3PO)
|
|
|
|
|
Why pass an index field at all? Get the count, subtract 1, use that as the first integer and 1 as your second. If you can't get rid of the field, ignore it and do the same thing.
Don't know enough about WPF, but a string is a string and the caret is either the last character, earlier than the last, or doesn't exist in the string.
|
|
|
|
|
I know! Politics! I can't touch it yet 
Alberto Bar-Noy
---------------
“The city’s central computer told you? R2D2, you know better than to trust a strange computer!”
(C3PO)
|
|
|
|
|
Sorry, it's late, I'm not thinking straight. I can't think of a way to find the length of a string other than text.ToCharArray.Count()
|
|
|
|
|
Int32 contador;
contador = (Int32)0;
and a store procedure with +/- 2600 lines.
|
|
|
|
|
The declaration and initialisation could of course have been done in one line, but that makes little difference. From this I get the impression that the developer who wrote this is very 'modern'. He probably has no idea what the compiler will generate from those simple code lines and added the type cast just in case this assignment might be problematic.
Ironically, assingning zero to a variable of a numeric type is the most unproblematic case of all, since it turns out to be one or more zero bytes, no matter if we are looking at an integer type, a floating point type, signed or unsigned. The compiler knows the size (in bytes) of the variable the value is assigned to and there are no special ways to represent the number 'zero'. Therefore no special type information is needed in the assignment.
Bottom line: I see this as clumsy code, but not as a real horror. It does what it is supposed to and I see no potentially harmful side effects. However, I would also see it as an indicator that the developer should perhaps learn more about what happens under the hood, even if that's not 'modern'.
And from the clouds a mighty voice spoke:
"Smile and be happy, for it could come worse!"
And I smiled and was happy
And it came worse.
|
|
|
|
|
I think, someone may well correct me, that the compiler will ignore the second line as it is repeating the prior step because .net initialises all numbers to zero if no value is explicitly supplied.
Panic, Chaos, Destruction. My work here is done.
Drink. Get drunk. Fall over - P O'H
OK, I will win to day or my name isn't Ethel Crudacre! - DD Ethel Crudacre
I cannot live by bread alone. Bacon and ketchup are needed as well. - Trollslayer
Have a bit more patience with newbies. Of course some of them act dumb - they're often *students*, for heaven's sake - Terry Pratchett
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
