Loops in a user defined date class

[Josiah smith]

I am having some problems with the loops in this code that should delete different dates from an array. Can you find what I am doing wrong??

import java.util.*;
import javax.swing.*;

public class ArrayListofDates {
    public static void main(String[] args) {
	String input;
	int size;
	int userAnswer;

	ArrayList myDatesList = new ArrayList();

	input = JOptionPane.showInputDialog( "Enter number of random dates" );
	size = Integer.parseInt( input );

	//**array list of random Dates
	for (int i=1; i<=size; i++) {
	    myDatesList.add(generateRandomDate());  //**pseudo data...
	}

	//iterate over them
	Iterator it=myDatesList.iterator();
	System.out.println();
	System.out.println("Here are your dates: ");

	Date thisDate;
	while (it.hasNext()) {
	    //reference to the Date in the list.
	    //***next() returns Object, must cast to Date:
	    thisDate = (Date)(it.next());
	    //thisDate = (it.next());  	  //**failure to cast would be compile error
	    System.out.print(thisDate);
	    //***access a method of the Date object:
	    System.out.println("     Month is: " + thisDate.getMonth());
	}
	System.out.println();




	//Used to search array for duplicate dates from users input
	//Used to delete date if user selects delete
	userAnswer = JOptionPane.showConfirmDialog(null,
						   "Do you want to look for duplicate dates in your list?",
						   "Find Duplicates?",
						   JOptionPane.YES_NO_OPTION);
	if (userAnswer == JOptionPane.YES_OPTION) {
	    String stringDate = JOptionPane.showInputDialog("Enter date as mm/dd/yyyy");
	    Date testDate = new Date(stringDate);
	    for (int i=1; i<=size; i++) {
                if (myDatesList.contains(testDate)) {
    		    userAnswer = JOptionPane.showConfirmDialog(null,
						   "Do you want to delete this duplicate date?",
						   "Delete Duplicate",
						   JOptionPane.YES_NO_OPTION);
		    if (userAnswer == JOptionPane.YES_OPTION) {
                        myDatesList.remove(it);
		    }
                }
	    }
	}
	//Outputs a new listing of the dates after searching/deleting
	it=myDatesList.iterator();
	System.out.println();
	System.out.println("Here is a list of your revised dates: ");
	while (it.hasNext())
	    System.out.println(it.next());
	System.out.println();



    }




    //generate a random Date.  Note this is not a method of the Date class.
    //it creates and returns a Date object.
    public static Date generateRandomDate() {
	int year, month, day=0;

	year = (int)(Math.random()*20+1995);  // +/- 10 years from 2005
    	month = (int)(Math.random()*12+1);
	switch (month) {
	case 1:
	case 3:
	case 5:
	case 7:
	case 8:
	case 10:
	case 12:
	    day = (int)(Math.random()*31+1);
	    break;
	case 4:
	case 6:
	case 9:
	case 11:
	    day = (int)(Math.random()*30+1);
	    break;
	case 2:
	    if (year%4==0 && year%100!=0 || year%400==0)  //leap year test
  	        day = (int)(Math.random()*29+1);
	    else
  	        day = (int)(Math.random()*28+1);
	    break;
	}
	//instantiate a Date object using the 3-arg constructor,
	//then return the object to caller.
        return new Date(month,day,year);
        }
    }

[Jan Cumps]

Hi Josiah.

Can you tell us how the behaviour differs from what you expect?

[Josiah smith]

Yes,
First If I select no to search for a date: it works fine.
If I select yes and enter a date that would be a duplicate it continues to have me enter more dates instead of deleting the date and prompting if you would like to search for a date again. I think that I have the loops set up wrong and somehow need a Do while loop. It probably is an easy one, just not popping out at me... Thanks for your help

[George Black]

you could Search the method ' list.contains()' in the API ,i think you will find it.