|
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 
"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!
|
|
|
|
|
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
|
|
|
|
|
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.
|
|
|
|
|
If you want job security, be an ninja. You can pretend you were nowhere near any bad incidents or right in the middle of good ones.
|
|
|
|
|
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.  (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?
|
|
|
|
|
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.
|
|
|
|
|
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. 
|
|
|
|
|
It gets better, I use one of the 26 pickable records where both dates aren't null and I run to the same catch error
|
|
|
|
|
Some one wanted to have this: "I have 12 text-boxes for entering Rate-Of-Intrest for different months. If user enters rateOfIntrest for January and August then rateOfInterest for February to July should be same as of January and rest should be as for August. If data for only January is entered then all the months will be same as that of January"
Suggestion was:
in this case when first text box value can given by the user then bellow method can help u.
TextBox1.Text ="10";
if (TextBox2.Text == "")
{
TextBox2.Text = TextBox1.Text;
}
if (TextBox3.Text == "")
{
TextBox3.Text = TextBox2.Text;
}
if (TextBox4.Text == "")
{
TextBox4.Text = TextBox3.Text;
}
if (TextBox5.Text == "")
{
TextBox5.Text = TextBox4.Text;
}
if (TextBox6.Text == "")
{
TextBox6.Text = TextBox5.Text;
}
if (TextBox7.Text == "")
{
TextBox7.Text = TextBox6.Text;
}
if (TextBox8.Text == "")
{
TextBox8.Text = TextBox7.Text;
}
if (TextBox9.Text == "")
{
TextBox9.Text = TextBox8.Text;
}
if (TextBox10.Text == "")
{
TextBox10.Text = TextBox9.Text;
}
if (TextBox11.Text == "")
{
TextBox11.Text = TextBox10.Text;
}
if (TextBox12.Text == "")
{
TextBox12.Text = TextBox11.Text;
}
if TextBox3 text can change then it now consider TextBox3 values..
Oh! Come on...
|
|
|
|
|
Yeah, OK, so what's the problem? How did you fix it?
|
|
|
|
|
Ok, since you asked, there can be ways. To start with, I will have a UI of dropdown and corresponding textbox and a '+' icon (to add another dropdown + textbox)
Since user is aware of the fact that month selection will drive rate of interest, all they need to do is select a month(in dropdown) and then set rate of interest from that month(in textbox). If they want to change for any coming month, use the '+' icon to add another dropdown-textbox to add the values.
UI is short and clean, so will be the implementation.
|
|
|
|
|
Does the user need to add February in order to add March?
|
|
|
|
|
BTW, your comment suggests that you think the way suggested was correct. Is so?
Having a if for all the 12 textboxes with individual conditions, followed by rest of the textbox after the current one?
|
|
|
|
|
Sandeep Mewara wrote: suggests that you think the way suggested was correct
No, but the only issue appears to be a small amount of duplicated code, so I wouldn't bother changing it just for that. Especially if the resulting code is more complex.
But the post also doesn't have much context -- if this code is called by the TextChanged events of the TextBoxes I could see performance issues as well.
|
|
|
|
|
The problem is the obvious repetition of code, and the answer to fulfil the actual request is as simple as
TextBox[] boxes = new TextBox[] { TextBox1, TextBox2, ... };
string previousContent = string.Empty;
foreach(TextBox box in boxes) {
if(string.IsNullOrEmpty(box.Text)) box.Text = previousContent;
else previousContent = box.Text;
}
|
|
|
|
|
BobJanova wrote: obvious repetition of code
Yeah, so what? There's not much of it and it's clear.
I totally agree with using an array. But I'd use a for loop starting at 1.
|
|
|
|
|
If you don't see why 30 lines where 5 would do is a problem then I'm not sure we have the same opinion on code cleanliness ...
|
|
|
|
|
I agree with you. Just to add on, it's first 30 lines and not the end of whole code. It was just based on first textbox value, similar if for other following textbox was suggested.
Complete answer says: if TextBox3 text can change then it now consider TextBox3 values..
It's too many code lines!
|
|
|
|
|
BobJanova wrote: 30 lines where 5 would do is a problem
Well I wouldn't write it that way, but I don't consider it a problem (certainly a smell though) and wouldn't necessarily change it if it works properly -- it ain't broke, there's probably more important things to do than to clean this up.
However, the OP says that this is just one small example, so perhaps revisiting all of it is worthwhile in this case.
|
|
|
|
|
Sandeep Mewara wrote: bellow method can help
It helps me...
bel·low/ˈbelō/
Verb: (of a person or animal) Emit a deep loud roar, typically in pain or anger: "he bellowed in agony";
It was broke, so I fixed it.
|
|
|
|
|
Using a control array might be better here - you could then loop through your checks and would also benefit from using the same event handler for all elements of the array rather than 12 seperate ones.
|
|
|
|
|
I hope this counts as a subtle bug and can help those encountering something similar.
I'm reading in a bunch of data from files. Basically a datetime (dts) and a float value. Because we're talking about 1.3 million records which I read in with several files, create a list of insert queries and execute them by 100 at the time. Everything worked except for some batches.
So looking at the errors, it says it runs into a unique constraint (dts is unique). Fine, what value was double then? none.
 is my state of mind by now. So I continue looking and print out the failed batches and sure enough there are double values in there. Checking the corresponding files : no doubles. But what does my little eye spy? double dts values start with 3 'o clock and only at the end of march! Daylightsaving (DTS).
The bug: I use java's GregorianCalendar and if you don't specify the timezone it will convert the 2:00 until 3:00 to 1 hour +. so 2:30 becomes 3:30 etc... What is strange though, is that there is no warning, no exceptions nothing. Just implicit conversion.
Perhaps it is my misunderstanding of the object, but if you encounter something similar, you just might want to check in this direction.
PS: I considered this forum to be the replacement of subtle bugs.
[EDIT]
The solution is to set the timezone to UTC.
If you use de SimpleDateFormat class to format the dates to string. This also has a timezome, which you also need to set to UTC to avoid having the date converted.
[/EDIT]
V.
modified 11-Jul-12 2:45am.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
