Programming Forums - View Single Post

grumpy · Jun 3rd, 2006, 4:16 AM

A couple more comments

1) If you use a test case that contains two or more spaces in sequence, the histogram will not be correct. I'm guessing all your test cases only have one space between two words.

2) Something that works along the lines;

(Toggle Plain Text)

  ++x;
  if (some_condition)
  {
        --x;
        more_stuff();
  }

can be expressed more cleanly in the form;
[code]

(Toggle Plain Text)

  if (some_condition)
  {
        more_stuff();
  }
  else
  {
      ++x;
  }

Your last code snippet is in that form .....

Jun 3rd, 2006, 4:16 AM	#6
grumpy Programming Guru Join Date: Jun 2005 Location: Adelaide, South Australia Posts: 1,253 Rep Power: 5	A couple more comments 1) If you use a test case that contains two or more spaces in sequence, the histogram will not be correct. I'm guessing all your test cases only have one space between two words. 2) Something that works along the lines; (Toggle Plain Text) ++x; if (some_condition) { --x; more_stuff(); } ++x; if (some_condition) { --x; more_stuff(); } can be expressed more cleanly in the form; [code] (Toggle Plain Text) if (some_condition) { more_stuff(); } else { ++x; } if (some_condition) { more_stuff(); } else { ++x; } Your last code snippet is in that form .....