[novich1]

Change the clock from a 24-hour clock to a 12-
hour clock.

public class ClockDisplay

{
	private NumberDisplay hours;
	private NumberDisplay minutes;
	private String displayString;	// simulates the actual display
	public ClockDisplay()
	{
		hours = new NumberDisplay(1-12);
		minutes = new NumberDisplay(0-59);
		updateDisplay();
	}
  public ClockDisplay(int hour, int minute)
	{
		hours = new NumberDisplay(1-12);
		minutes = new NumberDisplay(0-59);
		setTime(hour, minute);
	}
  public void timeTick()
	{
		minutes.increment();
		if(minutes.getValue() == 0) {  // it just rolled over!
			hours.increment();
		}
		updateDisplay();
	}
 public void setTime(int hour, int minute)
	
	{
		hours.setValue(hour);
		minutes.setValue(minute);
		updateDisplay();
	}
 public String getTime()
	{
		return displayString;
	}
	
	/**
	 * Update the internal string that represents the display.
	 */
	private void updateDisplay()
	{
		displayString = hours.getDisplayValue() + ":" + 
						minutes.getDisplayValue();
	}
	public String getDisplayValue(String value)
	{
		value = displayString;
		return  value + " ";
	}
	
}

I think i make some mistake somewhere,but i can't find him?


***added code tags -jjsaw5***

[jjsaw5]

Could you please provide us with some more information. Like any error messages or a description of what your code is not doing that it is supposed to.


Remember the more information you provide us with the better we will be able to help you.


Thank you!

[novich1]

There is no error messages but when i call String get.Display.value and insert "01:22"and expect to receive same result i receive "00:00".I use BlueJ.

[n00bc0der]

Your code is a mess and it's not all here.... I can't even help you because I need to see the rest of the code to figure out what's wrong.

[NeoTifa]

I would do your tick method after your set time method. Have an algorithm for detecting am and pm. Say if hours > 12, then hours - 12 is currentHour or something. Also, check to make sure nobody puts in something funky like 25:76 for the time. Roll it over maybe. 60 == 00, so 76 == 16. Hope that helped a little.