|
The following is not a useful error message:
Quote from upgrade log: src\Libraries\WindowsAPICodePack\Shell\Shell.csproj: Error on line 551291232. Expected 'ENCODING' but found 'utf-8'.
The file is 296 lines, not 551291232 lines. 
Visual Studio 2013 Premium.
That error was nowhere near the issue (an unmatched tag on line 15)
Getting information off the Internet is like taking a drink from a fire hydrant.
- Mitchell Kapor
|
|
|
|
|
I've been doing some work with accessing a Queue on multiple threads (to see just how non-thread-safe they are) and had one report a count of -10.  But it worked just fine.
|
|
|
|
|
PIEBALDconsult wrote: and had one report a count of -10. But it worked just fine.
Ouch. That's probably even worse than it simply failing, because you could have production code giving bad results without an exception. 
.-.
|o,o|
,| _\=/_ .-""-.
||/_/_\_\ /[] _ _\
|_/|(_)|\\ _|_o_LII|_
\._. |\_/|"` |_| ==== |_|
|_|_| ||" || ||
|-|-| ||LI o ||
|_|_| ||'----'||
/_/ \_\ /__| |__\
|
|
|
|
|
WTF? That is rather strange. I wonder what else the framework has in store for this forum?
Getting information off the Internet is like taking a drink from a fire hydrant.
- Mitchell Kapor
|
|
|
|
|
|
And sadly the interface doesn't match the regular queue's.
|
|
|
|
|
The day you find an error message that accurately describes the error, then you can be surprised.
|
|
|
|
|
Getting information off the Internet is like taking a drink from a fire hydrant.
- Mitchell Kapor
|
|
|
|
|
Sad but true... 
|
|
|
|
|
Brisingr Aerowing wrote: Visual Studio 2013 Premium.
The extra lines come from being a Premium version. 
|
|
|
|
|
Getting information off the Internet is like taking a drink from a fire hydrant.
- Mitchell Kapor
|
|
|
|
|
"Error on line 551291232"
Now would be a good time to argue for payment by lines of code. Even at a penny a line, you'd be doing well.
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.
|
|
|
|
|
It was a C# project file that I had modified manually (changed framework version, removed file upgrade-related tags, fixed references) but I had accidentally removed a required end tag on line 15, which caused the error.
Getting information off the Internet is like taking a drink from a fire hydrant.
- Mitchell Kapor
|
|
|
|
|
He called for my help when he couldn't understand a piece of code.
After 2 hours digging, we finally understood what the code does. A convoluted piece of junk that just shows a string on a field on the page.
But the most facepalming thing we found was this piece of "code":
public string GetLimitMessage(int total, int limit){
if(total > limit)
try
{
return "The Limit has been reached!"
}
catch(Exception)
{
return "";
}
else
return string.Empty;
}
Aside from translation (the original is on Portuguese), that's the exactly code we found. We are still facepalming.
|
|
|
|
|
Do you know the one wrote this? I want to hire him! My office is too dirty lately... 
I'm not questioning your powers of observation; I'm merely remarking upon the paradox of asking a masked man who he is (V).
|
|
|
|
|
He doesn't work here anymore (actually can be anyone of 4 people, but none of them work here now).
The thing is that all of them bragged about how their code was better and how they were the best
I don't want to see the worst. 
|
|
|
|
|
Of course they bragged about their code. They used the advanced feature "try-catch" so it must be great code! Well, great in their minds, anyway.
Just because the code works, it doesn't mean that it is good code.
|
|
|
|
|
So I assume all over the codebae you have something like this?
if (GetLimitMessage(a, b).Length > 0)
MessageBox.Show(GetLimitMessaage(a, b));
The good thing about pessimism is, that you are always either right or pleasently surprised.
|
|
|
|
|
Are you mad! That would be far too risky to write it like that, far better would be:
if (!GetLimitMessage(a, b).equals(GetLimitMessage(1, -1)) {
if (!String.IsNullOrEmpty(GetLimitMessage(a, b)) {
MessageBox.Show(GetLimitMessaage(a, b));
}
else
{
MessageBox.Show("The monkey's dead!");
}
}
speramus in juniperus
|
|
|
|
|
Now I need some more mind bleach...
|
|
|
|
|
Surely a MessageBox is rather "old hat"? How do you display a Fail Whale in a MessageBox?
|
|
|
|
|
Woah up there, you're missing a major source of defensive coding here...
try {
if (GetLimitMessage(a,b) != null && GetLimitMessage(a,b).Equals(GetLimitMessage(1, -1))) {
if (!string.IsNullOrEmpty(GetLimitMessage(a,b)) {
try {
MessageBox.Show(GetLimitMessage(a,b));
}
catch (Exception ex) {
throw;
}
}
}
}
catch (Exception ex) {
MessageBox.Show("The monkey's dead!");
}
"If you don't fail at least 90 percent of the time, you're not aiming high enough."
Alan Kay.
|
|
|
|
|
It's an APS.NET MVC 3 Application, so it's:
Controller:
if(Helper.GetLimitMessage(total, 100))
{
ViewBag.LimitMessage = Helper.GetLimitMessage(total, 100);
ViewData["LimitMessage"] = Helper.GetLimitMessage(total, 100);
}
View:
@if(ViewBag.LimitMessage != "")
{
@Helper.GetLimitMessage(total, 100)
}
else if (ViewData["LimitMessage"] != null)
{
@ViewData["LimitMessage"]
}
Even my coworker who normally doesn't rant on code is cursing this one.
|
|
|
|
|
There isn't a  large enough for this.
|
|
|
|
|
Is this the real life?
Is this just fantasy?
caught in a landslide
no escape from reality
open your eye's
look up to the ceiling and stare
I'm just a poor coder, I get no sympathy
because I make simple things, make them work
respect my deadlines, don't be a jerk
Any way management blows, doesn't really matter to meeeee
To meeeeeee
MAMAAAAA I just killed my coworker
put a gun against his head
pulled my trigger, now he's dead...
.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
