Update functionality

[Mohammed Mazid]

Hi guys,

I was trying to program an application that would be used to update
the flight. Basically using a JSP form I am updating a flight and its
associated details. However, it does not do it as it cannot pass the
flightNo parameter. Therefore can anyone please suggest correction to
that please and add in a code that would work efficiently?

Here is the code snippet:

class FlightDAO
{
void update(Flight flight)
{
//Assume already statements there

// Create the UPDATE update_stmt
PreparedStatement update_stmt = null;

try
{
// Get a database connection
connection = connectionPool.getConnectionNoWait();

// Create SQL UPDATE statement
update_stmt = connection.prepareStatement(UPDATE_STMT);
update_stmt.setDate(2, flight.flightDate);
update_stmt.setString(3, flight.destination);
update_stmt.setTime(4, flight.arrTime);
update_stmt.setTime(5, flight.depTime);

// Perform the SQL UPDATE
update_stmt.executeUpdate();

// Handle any SQL errors
}
catch (SQLException se)
{
//Assume some catch statements
}

private static final String UPDATE_STMT
= "UPDATE Flight"
+ "SET FlightDate = ?, Destination = ?, ArrivalTime = ?,
DepartureTime = ?"
+ "WHERE FlightNo = ?";
}

public class FlightService
{
private FlightDAO flightDataAccess;

public FlightService()
{
flightDataAccess = new FlightDAO();
}

public Flight updateFlight(String flightNo, Date flightDate, String
destination,
Time depTime, Time arrTime)
{
// Create the Flight object
Flight flight = new Flight(flightNo, flightDate, destination,
arrTime, depTime);

// Perform the DB transaction
flightDataAccess.update(flight);

return flight;
}

It is not the whole program, just part of the source code is shown.
Can anyone please show me how to pass the flightNo parameter and also
correction of the update function so that it would work in the
database?

I think there is no need to show ConnectionPool source code and so on.
There is only update and the passing of the flightNo parameter
correction is required.

Much appreciated.

[SteveE]

>

private static final String UPDATE_STMT
= "UPDATE Flight"
+ "SET FlightDate = ?, Destination = ?, ArrivalTime = ?,
DepartureTime = ?"
+ "WHERE FlightNo = ?";

Check your update statement. As it stands, it reads:

"UPDATE FlightSET FlightDate = ?, Destination = ?, ArrivalTime = ?,
DepartureTime = ?WHERE FlightNo = ?"

You need a couple of spaces in there.

Cheers,
Steve.

[Ryan Stewart]

"Mohammed Mazid" <ka******@hotmail.com> wrote in message
news:7c*************************@posting.google.co m...
*snip*

update_stmt = connection.prepareStatement(UPDATE_STMT);
update_stmt.setDate(2, flight.flightDate);
update_stmt.setString(3, flight.destination);
update_stmt.setTime(4, flight.arrTime);
update_stmt.setTime(5, flight.depTime); *snip* private static final String UPDATE_STMT
= "UPDATE Flight"
+ "SET FlightDate = ?, Destination = ?, ArrivalTime = ?,
DepartureTime = ?"
+ "WHERE FlightNo = ?";

*snip*
Aside from the problem pointed out by SteveE, your set statements are all
wrong. Flight date is not the second argument, and you never set a first.

[Ryan Stewart]

"Ryan Stewart" <zz********@gSPAMo.com> wrote in message
news:y-********************@texas.net...

"Mohammed Mazid" <ka******@hotmail.com> wrote in message
news:7c*************************@posting.google.co m...
*snip*

update_stmt = connection.prepareStatement(UPDATE_STMT);
update_stmt.setDate(2, flight.flightDate);
update_stmt.setString(3, flight.destination);
update_stmt.setTime(4, flight.arrTime);
update_stmt.setTime(5, flight.depTime);

*snip*

private static final String UPDATE_STMT
= "UPDATE Flight"
+ "SET FlightDate = ?, Destination = ?, ArrivalTime = ?,
DepartureTime = ?"
+ "WHERE FlightNo = ?";

*snip*
Aside from the problem pointed out by SteveE, your set statements are all
wrong. Flight date is not the second argument, and you never set a first.

Oh, and if you want answers, post to comp.lang.java.help or
comp.lang.java.programmer in the future. The former is for beginners. The
latters is for more advanced topics. This group is not carried by all
servers.