Spawning threads to process data

Question

I have a class that spawns threads to process data. I am working on instrumentation code. When the process is started, time = System.currentTimeMillis(); When it completes, time = System.currentTimeMillis() - time; I have a method to retrieve this time:

/**
 * Get the time taken for process to complete
 * 
 * @return Time this process has run if running; time it took to complete if not
 */
public long getRunTime() {
   return processRunning? System.currentTimeMillis() - time : time;
}

Is this a clear use of the ternary operator? Is my javadoc comment clear?

\$\begingroup\$ should document your units \$\endgroup\$

Ron
– Ron

2011-02-03 03:03:11 +00:00
Commented Feb 3, 2011 at 3:03 — Ron
– Ron, Commented Feb 3, 2011 at 3:03

TheXenocide · Accepted Answer · 2011-02-02 19:55:09Z

Since the method has a return value even if the code is not finished processing the first line of your javadoc ought not read "Get the time taken for process to complete" but perhaps rather "Get the current or total processing time" or something of that nature? I also agree with Bert F's comment, but if there were clear comments explaining the behavior and you don't intend on exposing start and end times individually then there's no reason to waste an extra variable; you can either keep the code as it is and add comments or optionally get rid of the boolean "processRunning" and instead use your completedTime/runTime variable to establish the fact that it's still running. All minor gripes, all in all.

Loki Astari · Accepted Answer · 2011-02-02 20:33:53Z

7

I have no problem the ternary operator.

But it is hard to tell that the result is what the comments is saying. Without having a context on what time is (which is a bad variable name) it is hard to understand the result of the function.

time: Bad variable name. Time of what?

answered Feb 2, 2011 at 20:33

Loki Astari

97.7k5 gold badges126 silver badges341 bronze badges

Add a comment |

Bert F · Accepted Answer · 2011-02-02 19:23:55Z

5

It appears that the method sometimes returns a timestamp (time) and sometimes it returns a duration (difference between timestamps) (assuming that time is always a timestamp - if not, then this issue simply moves to time having a dual definition).

answered Feb 2, 2011 at 19:23

Bert F

3381 silver badge6 bronze badges

3

\$\begingroup\$ What about processRunning? System.currentTimeMillis() - startTime : completedTime - startTime; \$\endgroup\$

Michael K
– Michael K

2011-02-02 19:26:45 +00:00
Commented Feb 2, 2011 at 19:26
1

\$\begingroup\$ Works for me, as does processRunning ? System.currentTimeMillis() - startTime : runTime; \$\endgroup\$

Bert F
– Bert F

2011-02-02 19:30:48 +00:00
Commented Feb 2, 2011 at 19:30

Add a comment |

m0s · Accepted Answer · 2011-02-21 13:22:24Z

2

It is a matter of style and taste but I'd rather go with

public long getRunTime() {
   if(processRunning)
     return System.currentTimeMillis() - time; 

     return time;
}

I just think this reads easier, and its easier to comment this. Now set me on fire for multiple return statements :D

answered Feb 21, 2011 at 13:22

m0s

1212 bronze badges

\$\begingroup\$ Actually, I don't have a problem with this style. I do have a problem with multiple exit points spread throughout the function. \$\endgroup\$

Michael K
– Michael K

2011-02-21 21:14:28 +00:00
Commented Feb 21, 2011 at 21:14
\$\begingroup\$ Looks a little odd without the else, IMHO. \$\endgroup\$

Todd Lehman
– Todd Lehman

2012-01-22 08:03:57 +00:00
Commented Jan 22, 2012 at 8:03

Add a comment |

Stack Exchange Network

Spawning threads to process data

4 Answers 4

You must log in to answer this question.

Hot Network Questions

Spawning threads to process data

4 Answers 4

You must log in to answer this question.

Related

Hot Network Questions