|
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.
|
|
|
|
|
That is a known issue with Java, and Sun (and Oracle) have stated it won't be fixed because "it might break existing programs."
Bill Gates is a very rich man today... and do you want to know why? The answer is one word: versions.
Dave Barry
Read more at BrainyQuote[ ^]
|
|
|
|
|
To be fair, it's not just Java's problem. Once a year, any language/environment/framework/program that uses local time with daylight saving is going to have time ambiguity for an hour and once a year it'll have a nonexistent hour.
Using UTC makes a lot of sense.
 Now, if only they'd address the leap second problem...
Cheers,
Peter
Software rusts. Simon Stephenson, ca 1994. So does this signature. me, 2012
|
|
|
|
|
Yes, well this might also become a problem soon... 
One of the imports is per millisecond...
V.
|
|
|
|
|
Zac Greve wrote: That is a known issue with Java, and Sun (and Oracle) have stated it won't be
fixed because "it might break existing programs."
Exactly where is that stated?
|
|
|
|
|
V. wrote: Dating
And there was me hoping to find a match. 
Signature construction in progress. Sorry for the inconvenience.
|
|
|
|
|
On CodeProject? I don't think so 
|
|
|
|
|
|
I can't believe it took 4 posts before someone said something. 
Membrane.Inside("Insane");
|
|
|
|
|
V. wrote: What is strange though, is that there is no warning, no exceptions nothing. Just
implicit conversion.
So now is the time to get the following idea into your head...
Timestamps ALWAYS have an associated timezone.
If it isn't explicit then it is implicit.
So anytime you see a timestamp value you MUST determine how the timezone will be determined. Always.
And this has nothing to do with Java. It is true for any programming language and for that matter other types of usage as well.
V. wrote: So looking at the errors, it says it runs into a unique constraint (dts is
unique). Fine, what value was double then?
And there are other things about timestamps as well...
The resolution for timestamps is not absolute. Especially across boundaries. One example of that is that the MS SQL Server datetime resolution for milli-seconds is less than that of windows. So you can put a value into SQL Server and get it out with a different millisecond value.
Because it isn't absolute relying on it to be unique without quaranteeing the coarseness to be more than a second is probably asking for trouble.
And for Java the resolution of java.sql.Timestamp as represented by the base class java.util.Date is SECONDS, rather than nanoseconds as represented by java.util.Date itself. Read the docs on java.sql.Timestamp for specifics.
|
|
|
|
|
... when you start to code like this
var splittedKey = Key_.Split(new Char[]{'.'});
(yes|no|maybe)*
|
|
|
|
|
I like. I like very much!
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
