When to check for null ?
-
CDP1802 wrote:
You are wrong. VB PHP is the problem
FTFY :laugh: On a more serious note, I don't think any language is the problem. The problem is the people using them...
PHP? That's Triassic Park! :) Every language may have a share of bad coders, but some have been advertised to them as being especially 'easy to learn'. They were called and they came. The languages may have evolved over time, but many of the coders have not.
At least artificial intelligence already is superior to natural stupidity
-
That's a great idea. Now if only I had the access to do that...
Sasha Laurel wrote:
That's a great idea. Now if only I had the access to do that...
In this case, yes, suicide is a very tempting alternative. Homicide, on the other hand, mmm... ;) ;)...sweet I've found a nice way to punish the original coders when I come across stuff like this. Ask them what they intended to do in that portion(s) of the code, and WHY they did it so. It's not offensive, and it's perfectly justifiable. Now, one of two things may happen: - You, made to look like a fool if they can come up with a good explanation why they did what they did (which is ok with me, 'cause at least I can understand, and I don't mind admitting sometimes I'm just plain dumb), or - The 'WTF-did-I-do-here-can't-remember-can't-justify look on their faces, which is purely priceless (it may not help in untangling that code, but my ego will feel a really nice tickle-tingle) Now, in your specific case, I have no idea what would help, short of modifying the whole codebase :( :(
-
Another project from our "star" developer who is apparently beyond contempt in every way. I was given this website (an ASP.Net Webforms project written in VB10) to add some additional metrics. To my horror I found that even though written very recently (within the last 2-3 years) and then updated to use a new Linq-to-Entities data source, there was no regard for architecture, and all of the data access code was written directly in the code behind of the aspx pages. That's not even the real horror. I was getting (seemingly) random errors while trying to run it/understand it, and found that through-out all of the loads of Linq statements that all of the nullable types (e.g. Integer?, Double?, DateTime?, etc.) were being treated as if they were not nullable at all! Here is an example to help you understand (table/field names changed slightly to protect the guilty):
Dim partTransactionTypeThingsQuery As IQueryable(Of SomeTransactionRelatedItem) = \_ From t In baseQuery Join tji In dataContext.OtherTransactionItems \_ On tji.SomeTransactionID Equals t.ID Where (t.LastUpdatedDate >= startDate \_ And t.LastUpdatedDate < endDate) \_ And (tji.Type = "Part") \_ And (tji.TotalDue <> 0) Select tjiLastUpdatedDate is a Nullable(Of Date) and TotalDue is a Nullable(Of Double) and when it translates to SQL it will work just fine.. The problems come about when the context switches from Linq-To-Entities to Linq-To-Objects (I have to watch closely for a .ToArray() or .ToList() or anything else that executes the SQL and fetches results) and much of the aggregation is doing exactly that. I don't even know how many of these problems there are... Anyways, thanks for letting me rant a bit. Any suggestions? Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
-
This is probably actually Microsoft's fault: a Linq query should work the same for a query against the database, or a query against an in-memory object that was created by evaluating a Linq query on that same database.
Interesting idea! I wonder what the implications would be for an IQueryable provider. Oh well, I'm mostly just wanting to piss and moan at the moment.. ;)
-
Another project from our "star" developer who is apparently beyond contempt in every way. I was given this website (an ASP.Net Webforms project written in VB10) to add some additional metrics. To my horror I found that even though written very recently (within the last 2-3 years) and then updated to use a new Linq-to-Entities data source, there was no regard for architecture, and all of the data access code was written directly in the code behind of the aspx pages. That's not even the real horror. I was getting (seemingly) random errors while trying to run it/understand it, and found that through-out all of the loads of Linq statements that all of the nullable types (e.g. Integer?, Double?, DateTime?, etc.) were being treated as if they were not nullable at all! Here is an example to help you understand (table/field names changed slightly to protect the guilty):
Dim partTransactionTypeThingsQuery As IQueryable(Of SomeTransactionRelatedItem) = \_ From t In baseQuery Join tji In dataContext.OtherTransactionItems \_ On tji.SomeTransactionID Equals t.ID Where (t.LastUpdatedDate >= startDate \_ And t.LastUpdatedDate < endDate) \_ And (tji.Type = "Part") \_ And (tji.TotalDue <> 0) Select tjiLastUpdatedDate is a Nullable(Of Date) and TotalDue is a Nullable(Of Double) and when it translates to SQL it will work just fine.. The problems come about when the context switches from Linq-To-Entities to Linq-To-Objects (I have to watch closely for a .ToArray() or .ToList() or anything else that executes the SQL and fetches results) and much of the aggregation is doing exactly that. I don't even know how many of these problems there are... Anyways, thanks for letting me rant a bit. Any suggestions? Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
Sasha Laurel wrote:
Any suggestions? Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
No, just consider it job security :rolleyes:
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
-
Sasha Laurel wrote:
Any suggestions? Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
No, just consider it job security :rolleyes:
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
Job Security! Just the positive spin that I needed. Thank you sir!
-
Job Security! Just the positive spin that I needed. Thank you sir!
Sasha, you might have to keep reminding higher up management that it will take time to eventually get it all taken care of if they start playing whiny games about it taking time.
"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon
-
Another project from our "star" developer who is apparently beyond contempt in every way. I was given this website (an ASP.Net Webforms project written in VB10) to add some additional metrics. To my horror I found that even though written very recently (within the last 2-3 years) and then updated to use a new Linq-to-Entities data source, there was no regard for architecture, and all of the data access code was written directly in the code behind of the aspx pages. That's not even the real horror. I was getting (seemingly) random errors while trying to run it/understand it, and found that through-out all of the loads of Linq statements that all of the nullable types (e.g. Integer?, Double?, DateTime?, etc.) were being treated as if they were not nullable at all! Here is an example to help you understand (table/field names changed slightly to protect the guilty):
Dim partTransactionTypeThingsQuery As IQueryable(Of SomeTransactionRelatedItem) = \_ From t In baseQuery Join tji In dataContext.OtherTransactionItems \_ On tji.SomeTransactionID Equals t.ID Where (t.LastUpdatedDate >= startDate \_ And t.LastUpdatedDate < endDate) \_ And (tji.Type = "Part") \_ And (tji.TotalDue <> 0) Select tjiLastUpdatedDate is a Nullable(Of Date) and TotalDue is a Nullable(Of Double) and when it translates to SQL it will work just fine.. The problems come about when the context switches from Linq-To-Entities to Linq-To-Objects (I have to watch closely for a .ToArray() or .ToList() or anything else that executes the SQL and fetches results) and much of the aggregation is doing exactly that. I don't even know how many of these problems there are... Anyways, thanks for letting me rant a bit. Any suggestions? Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
Sasha Laurel wrote:
Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
Depends on what you call viable. If you want to get out of this problem as quickly as possible, no matter the cost, then yes. Personally, I'd say the cost is too high to consider it. :-D (That's disregarding the subsequent funeral costs you'll impose on someone else.) PS your quote had really weird markups that I haven't seen code project do before. I stripped the verbose "span" markup surrounding every lower-case "i" in your note.
-
Job Security! Just the positive spin that I needed. Thank you sir!
-
Quote:
You are wrong. VB is the problem.
I'm afraid not. Go ahead and open your mouth and insert your shoe. In this case it is a combination of ignorance and arrogance that is the problem (something that you also seem to be displaying by this comment). Since you know so much about the culture of my workplace, why don't you expound upon that?
Sasha Laurel wrote:
Go ahead and open your mouth and insert your shoe
No, thank you. But I would like to test Newton's third law. How would you like it? Shall I take that shoe and let it collide with your rear side or shall I just keep writing? What I did say was, that VB was always advertised as 'easy to learn' and 'ideal for beginners'. If I may translate that: If you are too lazy to learn or too dumb for anything else, then use VB. What I did not say was that all people who use VB are of that kind. Anyway, what you call arrogance is what I call self defense. I do not think that I can stop an army of the lazy and the dumb from doing vtheir evil, and therefore I do not preach. What I do is to stay away from them and not allow them to drag me into their desasters. It's remarkable how often things turned ugly when I did not manage to avoid VB. Indeed I just have to look into source control, my bosses' chamber of horrors, to find 10+ years of madness. It was of course the bosses' pointy-hairedness that got them into this in the first place. It took some years for them to realize that it might help if they hire developers that actually learned their job. It's amazing how many of those failures were written in VB, or perfectly good projects converted to VB and then mercilessly butchered. Or former VB 'developers' who had turned 'professional' and looked for new places to commit the same old sins. The only projects that showed a better quality were programmed in C++, which does not have the reputation to be 'easy to learn' or 'ideal for beginners'. What a coincidence.
Sasha Laurel wrote:
Since you know so much about the culture of my workplace, why don't you expound upon that?
For your sake, I hope that this is the culture of your workspace. For my part, I prefer to be called ignorant and arrogant before voluntarily getting myself into such a mess. At least that helped me to get together a longer list of projects that did not fail, which I can arrogantly point to. The irony is , that VB has evolved to a point where it had become usable, but less so its greatest users and supporters.
At least artificial intelligence already is superior to natural stupidity
-
The problem with Job Security is: There isn't a reliable method found, that does so. (At least any that I know of.) Just ask the tons of unemployed programmers who wrote unreadable code to get it.
-
Sasha Laurel wrote:
Go ahead and open your mouth and insert your shoe
No, thank you. But I would like to test Newton's third law. How would you like it? Shall I take that shoe and let it collide with your rear side or shall I just keep writing? What I did say was, that VB was always advertised as 'easy to learn' and 'ideal for beginners'. If I may translate that: If you are too lazy to learn or too dumb for anything else, then use VB. What I did not say was that all people who use VB are of that kind. Anyway, what you call arrogance is what I call self defense. I do not think that I can stop an army of the lazy and the dumb from doing vtheir evil, and therefore I do not preach. What I do is to stay away from them and not allow them to drag me into their desasters. It's remarkable how often things turned ugly when I did not manage to avoid VB. Indeed I just have to look into source control, my bosses' chamber of horrors, to find 10+ years of madness. It was of course the bosses' pointy-hairedness that got them into this in the first place. It took some years for them to realize that it might help if they hire developers that actually learned their job. It's amazing how many of those failures were written in VB, or perfectly good projects converted to VB and then mercilessly butchered. Or former VB 'developers' who had turned 'professional' and looked for new places to commit the same old sins. The only projects that showed a better quality were programmed in C++, which does not have the reputation to be 'easy to learn' or 'ideal for beginners'. What a coincidence.
Sasha Laurel wrote:
Since you know so much about the culture of my workplace, why don't you expound upon that?
For your sake, I hope that this is the culture of your workspace. For my part, I prefer to be called ignorant and arrogant before voluntarily getting myself into such a mess. At least that helped me to get together a longer list of projects that did not fail, which I can arrogantly point to. The irony is , that VB has evolved to a point where it had become usable, but less so its greatest users and supporters.
At least artificial intelligence already is superior to natural stupidity
Quote:
Shall I take that shoe and let it collide with your rear side
The "foot in mouth" thing is figurative, but since you seem to be speaking literally, I would hate to deprive you of the opportunity. I live in Salt Lake City, UT near Liberty Park. We can meet at that park at any time of your choosing where we can test Newton's third law to your hearts content. As long as we can agree that you've conceded your point that "VB is the problem" by saying that the real problem is the devs who never get past it, then I don't think we are in disagreement at all. As a side note, you can attack VB6 and prior all you want, I don't even care if you make silly blanket statements about those versions.
Quote:
For your sake, I hope that this is the culture of your workspace.
In my defense, I didn't really have any better options at the time. I would prefer to work in an environment that "separates the men from the boys" so to speak, but for now, that will just have to wait. Even still, we have plenty of very successful products, all written in some form of VB (mostly .Net). You come across as very angry and mean, so if I offended you please accept my apologies. I'll rescind my comment about you being ignorant and arrogant, as long as it is fully obvious that you are neither.
-
Sasha Laurel wrote:
Is suicide a viable alternative to fixing thousands of lines of VB Linq statements of this nature?
Depends on what you call viable. If you want to get out of this problem as quickly as possible, no matter the cost, then yes. Personally, I'd say the cost is too high to consider it. :-D (That's disregarding the subsequent funeral costs you'll impose on someone else.) PS your quote had really weird markups that I haven't seen code project do before. I stripped the verbose "span" markup surrounding every lower-case "i" in your note.
You make me realize that I sound extremely insensitive to people you who really are suicidal. OOPS! No offense intended. As for the weird markup, my company targets IE as our main browser, so a lot of the time I'm in IE tab for chrome. I wonder if it has something to do with that?
-
Quote:
Shall I take that shoe and let it collide with your rear side
The "foot in mouth" thing is figurative, but since you seem to be speaking literally, I would hate to deprive you of the opportunity. I live in Salt Lake City, UT near Liberty Park. We can meet at that park at any time of your choosing where we can test Newton's third law to your hearts content. As long as we can agree that you've conceded your point that "VB is the problem" by saying that the real problem is the devs who never get past it, then I don't think we are in disagreement at all. As a side note, you can attack VB6 and prior all you want, I don't even care if you make silly blanket statements about those versions.
Quote:
For your sake, I hope that this is the culture of your workspace.
In my defense, I didn't really have any better options at the time. I would prefer to work in an environment that "separates the men from the boys" so to speak, but for now, that will just have to wait. Even still, we have plenty of very successful products, all written in some form of VB (mostly .Net). You come across as very angry and mean, so if I offended you please accept my apologies. I'll rescind my comment about you being ignorant and arrogant, as long as it is fully obvious that you are neither.
You are right. Time to stop. It's been 32 years since I have been in Salt Lake City and if I hop into a plane to fly half way around the world, then I would hopefully come up with a better reason. And no, you did not offend me. When somebody has a problem, he can come right in. Then we close the door and we will never lose a word again about anything that will be said in the next ten minutes. It may be unfair and certainly impolite, but that does not matter. With those things out of the way it's usually no big deal anymore to figure out what to do.
Sasha Laurel wrote:
As long as we can agree that you've conceded your point that "VB is the problem" by saying that the real problem is the devs who never get past it, then I don't think we are in disagreement at all.
Ok, lets agree on that.
Sasha Laurel wrote:
As a side note, you can attack VB6 and prior all you want, I don't even care if you make silly blanket statements about those versions.
No need for that. I don't like to make a religion out of my work and don't want to participate in any holy war over it.
Sasha Laurel wrote:
You come across as very angry and mean
Angry, yes, but not at you. I did not really notice that I had built up so much steam, but now I'm back on my way to change this.
At least artificial intelligence already is superior to natural stupidity
-
You make me realize that I sound extremely insensitive to people you who really are suicidal. OOPS! No offense intended. As for the weird markup, my company targets IE as our main browser, so a lot of the time I'm in IE tab for chrome. I wonder if it has something to do with that?
Sasha Laurel wrote:
you really are suicidal
That threw me for a bit. What did I say to make you think I was suicidal? Then because of context I realized "you" should have been "who". :) Today I'm following a bug, no real detail on how to get there so I "best guess" it. This it a multiple pull-down app. Pick something at random. Code blows up, try-catch retrieves the error. Tried to retrieve a value from a nullable datetime field. Threw the error because the field was null. Looks like 94% of the choices leads to that. Start a SQL trace, it goes there when it starts up, but doesn't when I make my pick and blows up. So the code should be able to know what it can allow and modify the pull-downs to match. It's doing it elsewhere in the same pull-downs. Just one more area where the code isn't helpful. (Especially when I'm fairly new and don't really know what is meant by "2012-01-23-D2302-Private-Repro-Attempt-7-v-brleon") OK, it's dated, by a specific person, created while looking into something. But what's really important is that I've picked a type of report that should have allowed me to pick 26 of the 428 selections that should work. That or replace the null date with today's date in the other 400 possibles.
-
Sasha Laurel wrote:
you really are suicidal
That threw me for a bit. What did I say to make you think I was suicidal? Then because of context I realized "you" should have been "who". :) Today I'm following a bug, no real detail on how to get there so I "best guess" it. This it a multiple pull-down app. Pick something at random. Code blows up, try-catch retrieves the error. Tried to retrieve a value from a nullable datetime field. Threw the error because the field was null. Looks like 94% of the choices leads to that. Start a SQL trace, it goes there when it starts up, but doesn't when I make my pick and blows up. So the code should be able to know what it can allow and modify the pull-downs to match. It's doing it elsewhere in the same pull-downs. Just one more area where the code isn't helpful. (Especially when I'm fairly new and don't really know what is meant by "2012-01-23-D2302-Private-Repro-Attempt-7-v-brleon") OK, it's dated, by a specific person, created while looking into something. But what's really important is that I've picked a type of report that should have allowed me to pick 26 of the 428 selections that should work. That or replace the null date with today's date in the other 400 possibles.
LOL, good catch! I'm glad you understood what I was saying.
-
Sasha Laurel wrote:
you really are suicidal
That threw me for a bit. What did I say to make you think I was suicidal? Then because of context I realized "you" should have been "who". :) Today I'm following a bug, no real detail on how to get there so I "best guess" it. This it a multiple pull-down app. Pick something at random. Code blows up, try-catch retrieves the error. Tried to retrieve a value from a nullable datetime field. Threw the error because the field was null. Looks like 94% of the choices leads to that. Start a SQL trace, it goes there when it starts up, but doesn't when I make my pick and blows up. So the code should be able to know what it can allow and modify the pull-downs to match. It's doing it elsewhere in the same pull-downs. Just one more area where the code isn't helpful. (Especially when I'm fairly new and don't really know what is meant by "2012-01-23-D2302-Private-Repro-Attempt-7-v-brleon") OK, it's dated, by a specific person, created while looking into something. But what's really important is that I've picked a type of report that should have allowed me to pick 26 of the 428 selections that should work. That or replace the null date with today's date in the other 400 possibles.
That sounds crazy! I'm using MySQL so not even sure if I can run a trace on that. I just use ObjectQuery.ToTraceString() to get the SQL that EF generates. Anyway, best of luck with that one. I'm sitting in a TDD class right now wishing that my company practiced TDD. :doh:
-
You are right. Time to stop. It's been 32 years since I have been in Salt Lake City and if I hop into a plane to fly half way around the world, then I would hopefully come up with a better reason. And no, you did not offend me. When somebody has a problem, he can come right in. Then we close the door and we will never lose a word again about anything that will be said in the next ten minutes. It may be unfair and certainly impolite, but that does not matter. With those things out of the way it's usually no big deal anymore to figure out what to do.
Sasha Laurel wrote:
As long as we can agree that you've conceded your point that "VB is the problem" by saying that the real problem is the devs who never get past it, then I don't think we are in disagreement at all.
Ok, lets agree on that.
Sasha Laurel wrote:
As a side note, you can attack VB6 and prior all you want, I don't even care if you make silly blanket statements about those versions.
No need for that. I don't like to make a religion out of my work and don't want to participate in any holy war over it.
Sasha Laurel wrote:
You come across as very angry and mean
Angry, yes, but not at you. I did not really notice that I had built up so much steam, but now I'm back on my way to change this.
At least artificial intelligence already is superior to natural stupidity
Cheers mate. If you ever do make it out here I'll buy you a beer (or OJ, or a cuppa-tea, whatever) We can test Newton's 3rd while we clink the glasses. :)
-
That sounds crazy! I'm using MySQL so not even sure if I can run a trace on that. I just use ObjectQuery.ToTraceString() to get the SQL that EF generates. Anyway, best of luck with that one. I'm sitting in a TDD class right now wishing that my company practiced TDD. :doh:
-
Cheers mate. If you ever do make it out here I'll buy you a beer (or OJ, or a cuppa-tea, whatever) We can test Newton's 3rd while we clink the glasses. :)
