Getting chosen date to JDBC instead of current date

Question

My problem is the date entered in database from JSP is today's date instead of date chosen by user.

JSP file:

<td><input type="date" name="availableDate" /></td>

Servlet :

String[] presentationID = request.getParameterValues("selectavailability");
String[] availableDay =  request.getParameterValues("availableDay"); 
String[] availableStart =  request.getParameterValues("availableStart");
String[] availableEnd =  request.getParameterValues("availableEnd");
String[] availableDate = request.getParameterValues("availableDate");  

SimpleDateFormat availDate = new SimpleDateFormat("yyyy-MM-dd");

try
{

 for (int i = 0; i < availableDay.length; i++) 
 {

   AvailabilityBean available = new  AvailabilityBean();         

   available.setLecturerID(request.getParameter("lecturerID"));
   available.setAvailableDay(availableDay[i]);
   available.setAvailableStart(availableStart[i]);
   available.setAvailableEnd(availableEnd[i]);

   Date chosenDate = availDate.parse(availableDate[i]);
   available.setAvailableDate(chosenDate);
   available = AddAvailableDAO.addavailable(available);
   }
   }

DAO:

 String lecturerID = Abean.getLecturerID();
    String availableDay = Abean.getAvailableDay();
    String availableStart = Abean.getAvailableStart();
    String availableEnd = Abean.getAvailableEnd();
   // Date availableDate = Abean.getAvailableDate();
    String presentationID = Abean.getPresentationID();

try{  


    currentCon = JavaConnectionDB.getConnection();

    Date availableDate = Abean.getAvailableDate();           
   // java.util.Date date = new java.util.Date();
    java.sql.Date sqlDate = new java.sql.Date(availableDate.getTime()); //I THINK SOMETHING IS WRONG HERE

    PreparedStatement ps=currentCon.prepareStatement("insert into availability (availableID,lecturerID,availableDay,availableStart,availableEnd,availableDate,presentationid) values (availabilityseq.nextval,?,?,?,?,?,?)");  
            ps.setString(1,Abean.getLecturerID());  
            ps.setString(2,Abean.getAvailableDay());  
            ps.setString(3,Abean.getAvailableStart()); 
            ps.setString(4,Abean.getAvailableEnd()); 
            ps.setDate(5,sqlDate);        
            ps.setString(6,Abean.getPresentationID()); 
            ps.executeUpdate();

 }

Bean:

    private String availableID;
    private String lecturerID;
    private String availableDay;
    private String availableStart;
    private String availableEnd;
    private Date availableDate;
    private String presentationID;

How can I solve the problem?

EDIT: I added some part of my servlet which consist of parsing the value of availableDate and DAO

Answer 1

This code is wrong:

java.util.Date date = new java.util.Date();

Here you are creating a new Date object, its value will be always the current date.

You should extract the user selected date value from your AvailabilityBean object in Dao method

. Considering your code, I assume the changed code should be like below:

java.sql.Date sqlDate = new java.sql.Date(Abean.getAvailableDate().getTime());

Answer 2

Here you only creating current date object and you are not setting the user selected date value.

java.util.Date date = new java.util.Date();
java.sql.Date sqlDate = new java.sql.Date(date.getTime()); 

Try the following code:

//1. assuming Abean returning date in string value.
String availableDate = Abean.getAvailableDate();

// parsing the availableDate to string to date object.
DateFormat df = new SimpleDateFormat("MM/dd/yyyy"); // you can use your own date format here instead of 'MM/dd/yyyy'
Date choosenDate = df.parse(availableDate);

// coverting java date object to java.sql date object.
java.sql.Date sqlDate = new java.sql.Date(choosenDate.getTime());

EDIT: If availableDate is a date object then you can directly use the following:

Date availableDate = Abean.getAvailableDate();
java.sql.Date sqlDate = new java.sql.Date(availableDate.getTime());

Now you will have the user selected date in sqlDate variable.

Answer 3

you are creating a new Date, just before creating the 'PreparedStatement' in:

java.sql.Date sqlDate = new java.sql.Date(date.getTime()); 

With this code you are getting the 'Today date' and later, in 'ps' you add in the parameter # 5.

PreparedStatement ps=currentCon.prepareStatement("insert into availability (availableID,lecturerID,availableDay,availableStart,availableEnd,availableDate,presentationid) values (availabilityseq.nextval,?,?,?,?,?,?)");  
        ps.setString(1,Abean.getLecturerID());  
        ps.setString(2,Abean.getAvailableDay());  
        ps.setString(3,Abean.getAvailableStart()); 
        ps.setString(4,Abean.getAvailableEnd()); 
        **ps.setDate(5,sqlDate**);        
        ps.setString(6,Abean.getPresentationID()); 
        ps.executeUpdate();

That is the reason you are getting in the DB the 'Today´s date'. Use in the Parameter # 5 the call to the appropiate method:

ps.setDate(5,Abean.getAvailableDate());

Answer 4

The other answers are correct and informative. But they use old outmoded classes.

java.time

The java.time framework supplants the old date-time classes bundled with the earliest versions of Java. The old classes are poorly designed, confusing, and troublesome.

Do not use java.util.Date, java.util.Calendar, java.text.SimpleDateFormat and their siblings. Sun and Oracle gave up on those classes, deciding to build-in java.time classes as their replacement.

The java.time classes are built into Java 8 and later. See Oracle Tutorial. Much of their functionality has been back-ported to Java 6 & 7 and to Android. Defined by JSR 310, inspired by Joda-Time, and extended by ThreeTen-Extra. The java.time framework is the official successor to Joda-Time, though you can employ both within a project if you are careful with your import statements as some classes share the same name. Search Stack Overflow for much more info about java.time.

LocalDate

Among the new classes is LocalDate to represent a date-only value without time-of-day and without time zone.

LocalDate today = LocalDate.now( ZoneId.of( "America/Montreal" ) ) ;

You can instantiate by parsing a string or specifying numbers.

LocalDate localDate = LocalDate.parse( "2016-01-23" );
LocalDate localDate = LocalDate.of( 2016 , 1 , 23 );

SQL & JDBC

Hopefully JDBC drivers will be updated to directly use the java.time types. Until then we must use java.sql types to move data in/out of databases. Find new methods added to the old classes to facilitate converting to/from java.time types.

java.sql.Date sqlDate = java.sql.Date.valueOf( localDate );

Going the other direction.

LocalDate localDate = sqlDate.toLocalDate();

Minimize your use of java.sql.Date; use only for transfer with database. Use the java.time for your business logic.