No clue, I fixed the logic. Now, it's returning the password from the database. But, I still feel the program logic can get a lot better than this one.

Any clue, How I can write a more efficient logic ? making use the User class?

package com.rony.forgotpass;

public class ForgotPass {

	public static void main(String[] args) {

		String password = (String) Forgotit.forgotpass("lin@yahoo.co.in");
		
		System.out.println(password);
	}
}

package com.rony.forgotpass;

import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;

public class Forgotit {

	static Connection conn = null;
	static PreparedStatement ps = null;
	static ResultSet rs = null;

	public static String forgotpass(String email) {

		String sql = "SELECT password FROM User WHERE email='" + email + "'";

		try {
			conn = DBUtil.getConnection();

			if (conn != null) {
				ps = conn.prepareStatement(sql);
				rs = ps.executeQuery();

				while (rs.next()) {
					String password = rs.getString("password");

					return password;
				}

			}

		} catch (SQLException e) {
			e.printStackTrace();
		} finally {
			DBUtil.closeConnection();

		}
		return null;

	}

}

package com.rony.forgotpass;

public class User {
	
	private String forgotEmail;

	public String getForgotEmail() {
		return forgotEmail;
	}

	public void setForgotEmail(String email) {
		this.forgotEmail = email;
	}

}

Thank You... it worked....

Ohh ok... Thanks

Yep, thats what I was doing too... I will have to look more into the efficient way to doing things. Is there a "technical term" for using the Point class constructor in another class?

Ahh, that's what I was going to do then I saw this piece of code. So both ways can work but this is more efficient.