By using this site, you agree to our updated Privacy Policy and our Terms of Use. Manage your Cookies Settings.
435,255 Members | 2,640 Online
Bytes IT Community
+ Ask a Question
Need help? Post your question and get tips & solutions from a community of 435,255 IT Pros & Developers. It's quick & easy.

Update functionality

P: n/a
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.
Jul 17 '05 #1
Share this Question
Share on Google+
3 Replies


P: n/a
>
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.
Jul 17 '05 #2

P: n/a
"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.
Jul 17 '05 #3

P: n/a
"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.
Jul 17 '05 #4

This discussion thread is closed

Replies have been disabled for this discussion.