|
StringComparison.IgnoreOrdinalCase
In Word you can only store 2 bytes. That is why I use Writer.
|
|
|
|
|
digimanus wrote: StringComparison.OrdinalIgnoreCase
FTFY.
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Thank you
In Word you can only store 2 bytes. That is why I use Writer.
|
|
|
|
|
If they don't accept UpperCase in username/email, that's weird but still not too bad. Imagine they do accept it, AND insist on case-sensitivity when you log-in... "Bernie" != "bernie" - which one did I use?
E.g. with postgresql, string comparison is case-sensitive by default...
|
|
|
|
|
PIEBALDconsult wrote: upper-case letters in a username/email-address.
There's your first problem. 
But yes, a simple problem to fix.
There are only 10 types of people in the world, those who understand binary and those who don't.
|
|
|
|
|
|
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???
|
|
|
|
|
|
null means it'll fall back to some default or setting, this is from the source:
modifiedOptions.RequireEncryption =
modifiedOptions.RequireEncryption
?? serviceClient.DefaultRequestOptions.RequireEncryption
?? BaseDefaultRequestOptions.RequireEncryption;
TableRequestOptions.cs:135[^]
Perhaps an enumeration would've made it clearer:
enum RequireEncryption
{
Default,
Yes,
No
}
|
|
|
|
|
But obviously the default for a bool (at the end of the day) will be false, if not stated otherwise...And if stated otherwise, than it should be part of the documentation...
Skipper: We'll fix it.
Alex: Fix it? How you gonna fix this?
Skipper: Grit, spit and a whole lotta duct tape.
|
|
|
|
|
It's not a boolean, it's a Nullable<bool>. Assigning a value to the property basically says "ignore all settings and defaults and do whatever I tell you to do", whether that's true or false. So leave it to null to allow some configuration option to determine whether to require encryption, or assign a value to make sure it either requires encryption or not.
If it were a regular boolean, it would be impossible to determine whether the developer specified false or whether it's just the default value of a boolean. Nullable<bool> does allow for that.
Personally I would've chosen an enumeration, as it's much more verbose, but a Nullable<bool> does the trick as well.
|
|
|
|
|
+1 for the enumeration. It brings a lot more room for expansion and it isn't a PITA in serialization or marshalling.
GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++* Weapons extension: ma- k++ F+2 X
If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver
|
|
|
|
|
It could be good explanation if it wasn't around bool...and if wasn't around this specific code...
Did you see what the default value of the property (in BaseDefaultRequestOptions)? NULL...
But actually in the code (line 158) it turns to false...
Also the documentation states clearly that only true and false has meanings...
Such nullable bool has meaning in cases where you want to force the developer to make a choice, but in this code is a clear case of wasting CPU cycles...
Skipper: We'll fix it.
Alex: Fix it? How you gonna fix this?
Skipper: Grit, spit and a whole lotta duct tape.
|
|
|
|
|
The method at line 158 is called ApplyDefaultsAndClearEncryption, which is internal as well. true and false do have meaning, as well as leaving it undefined a.k.a. leave it to some setting to determine whether the value will be true or false.
I agree the code is unclear (as well as the documentation), but it does serve its purpose. How would you otherwise suggest to fall back to a default (which may either be true or false)?
|
|
|
|
|
Maarten Kools wrote: How would you otherwise suggest to fall back to a default (which may either be true or false)?
By deciding, what the default is! (it is decided to be false...)
The problem of this code (and it of course doesn't mean that nullable bool to be banned at all) is that, at the end of the road there is nothing about null value...It not used in any way to define a different behavior, but simply changed to false behavior...
Skipper: We'll fix it.
Alex: Fix it? How you gonna fix this?
Skipper: Grit, spit and a whole lotta duct tape.
|
|
|
|
|
For now. It is extendable / patchable for future developements, which means that in version x+1.0 default may become true or there will be a mechanism to decide independently. It leaves a door opened to change - that's why I and other programmers would have used a enum, still having it nullable prevents errors, even when interfacing with different data sources.
Only the things that REALLY must exist or any reasoning is pointless should be non-nullable... unless you're developing a system which must be extremely picky about its inputs.
GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++* Weapons extension: ma- k++ F+2 X
If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver
|
|
|
|
|
Of course it is impossible to future version to change default value (or behavior) - that will break applications in an instance...You not even ca nto introduce a new meaning for null, as it is today interpreted as false and changing that will also break applications...
If you are aiming to future options, you have to use enum, and add new values for new features...
Skipper: We'll fix it.
Alex: Fix it? How you gonna fix this?
Skipper: Grit, spit and a whole lotta duct tape.
|
|
|
|
|
That's fine, if you bother to document what the null actually means.
Without that, its a bug waiting to happen.
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.
|
|
|
|
|
Kornfeld Eliyahu Peter wrote: obviously the default for a bool (at the end of the day) will be false, if not stated otherwise...And if stated otherwise, than it should be part of the
IMO, this is the most significant statement in this thread.
The thing that drew me into this thread was its subject, which made me wonder how a bool could be considered nullable. When I see the word "nullable," I assume that the context is an object, rather than a primitive type such as bool, int, long, uint, ulong, double, etc.
David A. Gray
Delivering Solutions for the Ages, One Problem at a Time
Interpreting the Fundamental Principle of Tabular Reporting
|
|
|
|
|
It means not set. It can trigger an auto-discover method, an error or a default behaviour.
GCS d--- s-/++ a- C++++ U+++ P- L- E-- W++ N++ o+ K- w+++ O? M-- V? PS+ PE- Y+ PGP t++ 5? X R++ tv-- b+ DI+++ D++ G e++>+++ h--- ++>+++ y+++* Weapons extension: ma- k++ F+2 X
If you think 'goto' is evil, try writing an Assembly program without JMP. -- TNCaver
|
|
|
|
|
Exactly...So the question remains...Why to use it in this code, where nothing happens and NULL interpreted as false...
Skipper: We'll fix it.
Alex: Fix it? How you gonna fix this?
Skipper: Grit, spit and a whole lotta duct tape.
|
|
|
|
|
|
Which is there to aid cases of data bound control and required filed...but in the code above NULL simply interpreted as false and not as missing value...
Skipper: We'll fix it.
Alex: Fix it? How you gonna fix this?
Skipper: Grit, spit and a whole lotta duct tape.
|
|
|
|
|
... discovered in a legacy app.
public string GetCode(string ProjectCode, string id)
{
try
{
int len = ProjectCode.Length + id.Length;
int rem = 10 - len;
switch (rem)
{
case 1:
ProjectCode = ProjectCode + "0" + id;
break;
case 2:
ProjectCode = ProjectCode + "00" + id;
break;
case 3:
ProjectCode = ProjectCode + "000" + id;
break;
case 4:
ProjectCode = ProjectCode + "0000" + id;
break;
case 5:
ProjectCode = ProjectCode + "00000" + id;
break;
case 6:
ProjectCode = ProjectCode + "000000" + id;
break;
}
}
catch (Exception ex)
{
}
return ProjectCode;
}
|
|
|
|
|
I have to compliment the author on writing bulletproof code - the catch all is awesome!
/ravi
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
