delete files from a directory if datetime stamp matches

[Mei Jones]

I have the below code and eventhough the file name and datetime stamp matches it does not delete the file.

Can anyone shed some light ?

Here's the results I got


deleteme.docx File modification date and time : 23-07-2012 07:55
23-07-2012 07:55File not deleted!

I know there should be a issue with my comparison here --> if ((sdf.format(d)).toString() == s)

it gets compiled with no data type errors

import java.io.*;
import java.util.*;
import java.text.*;

public class FileDelete {
  public static void main(String[] args) throws IOException {



    DateFormat df = new SimpleDateFormat("dd-MM-yyyy");
    File f1 = new File("deleteme.docx");
    String s = null;
    try {
                                  Date date ;
				  df = new SimpleDateFormat("dd-MM-yyyy HH:mm");
				  date = (Date)df.parse("23-07-2012 07:55");
				  s = (df.format(date)).toString();
				 


    File file = new File(c:\deleteme.docx");
    System.out.println(Folder1 + Folder2 + "deleteme.docx");


    if (file.exists()) {
      long t = file.lastModified();
      Date d = new Date(t);
      SimpleDateFormat sdf = new SimpleDateFormat("dd-MM-yyyy HH:mm");
      System.out.println("Directory modification date and time : "
          + sdf.format(d));


      if  ((sdf.format(d)).toString() ==  s){
               boolean success = f1.delete();
               if (success) System.out.println("File deleted!");
               else System.out.println("File not deleted!");
		   }
          else System.out.println(s + "File not deleted!");
    } else {
      System.out.println("File not found!");
    }

    } catch (ParseException e)
             {System.out.println("Exception :"+e);  }
  }
}

[Jeff Verdegan]

Mei Jones wrote:
I know there should be a issue with my comparison here --> if ((sdf.format(d)).toString() == s)

When comparing objects' states ("contents"), use equals(), not ==. This includes comparing Strings for equality. The == operator tests if two references have the same value (both point to the same object or both are null). The equals() method, if properly written, tells if two objects have equal "contents" by whatever equality semantics are defined for the class in question.

Also, you don't need the toString() there, since format() returns a String.

Also, if you're comparing dates, you should compare them as Dates, not as Strings.

[Mei Jones]

Thank you,.. Yes it worked..