How can I make this code shorter using "?:"

Question

2.50/5 (2 votes)

See more:

this code is repeated for all columns of DGV. I think defining a lot of variables will affect performance.notice I need "x" variables because somewhere in code I use them for calculating.

C#

TimeSpan  x1 = TimeSpan.Zero;
string s1 = dataGridView1.Rows[i].Cells["MrnngEnt"].Value.ToString();
 if (!string.IsNullOrEmpty(s1))
    x1 = TimeSpan.Parse(s1= (s1 == "24:00:00") ? "23:59:59" : s1);

Posted 3-Mar-14 1:37am

mit62

Updated 3-Mar-14 1:38am

v2

Add a Solution

Comments

Vedat Ozan Oner 3-Mar-14 7:48am

I couldn't understand what your problem is exactly.

Maciej Los 3-Mar-14 7:52am

COuld you be more specific and provide more details about yoour issue?

mit62 3-Mar-14 8:05am

I don't want declare variable s1 because I think declare a lot of variables will take place memory and hit performance.is there any way to avoid declaring s1 variable?

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CPallini · Accepted Answer · 2014-03-03T02:04:00

Solution 2

I think making such sub-optimizations (such reducing the number of variables) does not make sense, because, probably:

You don't need to optimize (are you sure this part of your application is the bottleneck?)
Performance is not affected by the 'number of variables'.
The compiler is smarter than us and could optimize such code more effectively

Unless you are really sure a piece of code needs to be optimized (have you rpofiling results?), don't try to do it. A cleaner code more valuable.

Posted 3-Mar-14 2:04am

CPallini

Comments

Maciej Los 3-Mar-14 8:11am

Valuable advice, +5!

CPallini 3-Mar-14 10:47am

Thank you.

midnight_ 3-Mar-14 8:14am

+1 for question the sense

If you optimize code that way, then its a matter of readability.

Using a ternary op or IF doesn't count at all, since the compiler interpret it the same.
And that will absolutly not gain any kind of performance.

CPallini 3-Mar-14 10:47am

Thank you.

Sergey Alexandrovich Kryukov 3-Mar-14 10:37am

Great points, a 5. Way too many beginners are driven by big misconceptions on the "optimal" code, and on the value of code in general.
—SA

CPallini 3-Mar-14 10:47am

Thank you.

CHill60 · Accepted Answer · 2014-03-03T02:04:00

Well you could get all of

C#

if (!string.IsNullOrEmpty(s1))
    x1 = TimeSpan.Parse(s1= (s1 == "24:00:00") ? "23:59:59" : s1);

as a single line using the ternary operator again.

x1 = (!string.IsNullOrEmpty(s1)) ? TimeSpan.Parse(s1 = (s1 == "24:00:00") ? "23:59:59" : s1) : TimeSpan.Zero

But personally I would write a little helper function to do that data adjustment for you. E.g.

C#

private TimeSpan AdjustTime(int row, string colName)
{
    string s1 = dataGridView1.Rows[row].Cells[colName].Value.ToString();
    return ((!string.IsNullOrEmpty(s1)) ? TimeSpan.Parse(s1 = (s1 == "24:00:00") ? "23:59:59" : s1) : TimeSpan.Zero);
}

Then you would use it like this

TimeSpan x1 = AdjustTime(i, "MrnngEnt");
TimeSpan x2 = AdjustTime(i, "NextColumn");

How can I make this code shorter using "?:"

2 solutions

Solution 2

Solution 1

Add your solution here

Preview 0

Existing Members

...or Join us