Forum:

Servlets

Simple login form project - all attributes are null and HTTP Status 404 error in some cases

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

I'm a new Java user and I'm trying to code a simple login page. In first page (NewFile.jsp) users should enter their username and password and should click on "login", or click on "sign up".

1.) If user enters his username and password correctly, a login page (LoginPage.jsp) appears and says "welcome null" but it should show the name of that user instead of null.

2.) In that login page there is an edit button to edit profile information. When I clicked on it, every information is "null" and when I edit them and click on "Submit" button;

HTTP Status 404 -
type Status report
message
description The requested resource () is not available.
GlassFish Server Open Source Edition 3.1.2

that message appears.

3.) If I click on "Sign Up" button at the beginning, a registration jsp (SignUpPage.jsp) appears. After filling up text boxes and clicking on "Submit", same Status 404 screen appears.

I created a mysql database called "loginpage" using xampp. In that database there is a table called "users" and it has un, pass, name, surname, email and degree attributes.

Here is a screenshot of my project explorer:

Here is my code:

1. NewFile.jsp

<%@ page import="java.sql.*" %>
<%@ page import="java.io.*" %>
<%@ page import="java.util.*"%>
<%@ page language="java" contentType="text/html; charset=ISO-8859-1"
    pageEncoding="ISO-8859-1"%>

<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
<title>Login Form</title>
</head>
<body>
<h1> Welcome </h1>
<hr>

<form action = "LoginServlet" method = "post">
<fieldset style = "width: 300px">
<legend>Login to App</legend>
<table>
	<tr>
		<td>Username</td>
		<td><input type = "text" name = "un"/></td>
	</tr>
	<tr>
		<td>Password</td>
		<td><input type = "password" name = "pass"/></td>
	</tr>
	<tr>
		<td><input type = "button" value = "Sign Up" name = "signUp" onclick = "JavaScript:window.location='SignUpPage.jsp';"/></td>
	<tr>
		<td><input type = "submit" value = "Login" name = "login"></td> 
	</tr>
</table>
</fieldset>
</form>
</body>
</html>

2.LoginPage.jsp

<%-- <%@ page import "javax.servlet.http.HttpSession" --%>
<%@ page language="java" contentType="text/html; charset=ISO-8859-1"
    pageEncoding="ISO-8859-1"%>
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
<title>Welcome <%--<%=session.getAttribute("name")--%></title>
</head>
<body>

<h3>Login successful!!!</h3>  
    <form action = "LoginServlet" method = "get">
    <h4>Hello,  <%=session.getAttribute("name")%></h4>
    </form>

<%
    if ((session.getAttribute("un") == null) || (session.getAttribute("un") == "")) {
%>
You are not logged in<br/>
<%} else {
String value = request.getParameter("name"); 
session.setAttribute("name",value);
%>
<form action = "LoginServlet" method = "get">
Welcome <%= value %> <%--session.getAttribute("name") --%>
</form>
<%
    }
%>
<form>
	<input type = "button" value = "Edit" name = "edit" onclick = "JavaScript:window.location='EditPage.jsp';">
</form>
</body>
</html>

3. LoginServlet.java

package com.amzi.servlets;

import java.io.IOException;
import java.io.PrintWriter;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import com.amzi.dao.LoginDao;

/**
	 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
	 */
    
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		HttpSession session = request.getSession(false); 
		RequestDispatcher rd = request.getRequestDispatcher("/LoginServlet");  
		rd.forward(request, response); 
		String name = (String)session.getAttribute("name");
		session.setAttribute("name", name);
		System.out.println("welcome" +name); 
	}

/**
	 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
	 */
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
		HttpSession session = request.getSession(false);  
        response.setContentType("text/html");    
        PrintWriter out = response.getWriter();    
          
        String n=request.getParameter("un");    
        String p=request.getParameter("pass");   
          
        if(session!=null)  
        session.setAttribute("un", n);
        session.setAttribute("pass", p); 
  
        if(LoginDao.validate(n, p)){    
            RequestDispatcher rd=request.getRequestDispatcher("LoginPage.jsp");    
            rd.forward(request,response);    
        }    
        else{    
            out.print("<p style=\"color:red\">Sorry username or password error</p>");    
            RequestDispatcher rd=request.getRequestDispatcher("NewFile.jsp");    
            rd.include(request,response);    
        }    
        out.close();   
	}
}

4. LoginDao.java

package com.amzi.dao;

import java.util.*;
import java.sql.Connection;  
import java.sql.DriverManager;  
import java.sql.PreparedStatement;  
import java.sql.ResultSet;  
import java.sql.SQLException;

import com.mysql.jdbc.Statement;
  
public class LoginDao {  
    public static boolean validate(String un, String pass) {          
        boolean status = false;  
        Connection conn = null;  
        PreparedStatement pst = null;
        ResultSet rs = null;  
        
  
        String url = "jdbc:mysql://localhost:3306/";  
        String dbName = "loginpage";  
        String driver = "com.mysql.jdbc.Driver";  
        String userName = "root";  
        String password = "";  
        try {  
            Class.forName(driver).newInstance();  
            conn = DriverManager  
                    .getConnection(url + dbName, userName, password);  
  
            pst = conn  
                    .prepareStatement("select * from users where un=? and pass=?");  
            pst.setString(1, un);  
            pst.setString(2, pass);  
  
            rs = pst.executeQuery();  
            status = rs.next();  
            
            while(rs.next()) {
            	String uname = rs.getString ("un");
            	String upass = rs.getString("pass");
                System.out.println(uname + ", " + upass);
            }

} catch (Exception e) {  
            System.out.println(e);  
        } finally {  
            if (conn != null) {  
                try {  
                    conn.close();  
                } catch (SQLException e) {  
                    e.printStackTrace();  
                }  
            }  
            if (pst != null) {  
                try {  
                    pst.close();  
                } catch (SQLException e) {  
                    e.printStackTrace();  
                }  
            }  
            if (rs != null) {  
                try {  
                    rs.close();  
                } catch (SQLException e) {  
                    e.printStackTrace();  
                }  
            }
        }  
        return status;  
    }  
}

5. SignUpPage.jsp

</body>
</html>

6. Registration.java

package com.amzi.servlets;

import java.io.IOException;
import java.io.PrintWriter;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletConfig;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import com.amzi.dao.LoginDao;
import com.mysql.jdbc.Connection;
import com.mysql.jdbc.PreparedStatement;

public class Registration extends HttpServlet{
	
	/**
	 * 
	 */
	private static final long serialVersionUID = 1L;

public void init(ServletConfig config) throws ServletException{
		super.init(config);
	
	}

public void doPost(HttpServletRequest req, HttpServletResponse res) 
				throws ServletException, IOException{
	        String url = "jdbc:mysql://localhost:3306/";  
	        String dbName = "loginpage";  
	        String driver = "com.mysql.jdbc.Driver";  
 
		
			Connection connection=null;
			ResultSet rs;
			res.setContentType("text/html");
			PrintWriter out = res.getWriter();
			
			HttpSession session = req.getSession(false);

String un = (String)session.getAttribute("un");
			String pass = req.getParameter("password");
			String name = req.getParameter("name");
			String surname = req.getParameter("surname");
			String email = req.getParameter("email");
			String degree = req.getParameter("degree");
			
			try {
				Class.forName(driver);
				connection = (Connection) DriverManager.getConnection(url + dbName, "root", ""); 
				String sql = "insert into users values (?,?,?,?,?,?)";
				PreparedStatement pst = (PreparedStatement) connection.prepareStatement(sql);
				pst.setString(1, un);
				pst.setString(2, pass);
				pst.setString(3, name);
				pst.setString(4, surname);
				pst.setString(5, email);
				pst.setString(6, degree);
				int numRowsChanged = pst.executeUpdate();
				out.println(" Welcome : ");
				out.println(" '"+name+"'");
				pst.close();
			}
			
			catch(ClassNotFoundException e){
				out.println("Couldn't load database driver: " + e.getMessage());
			}
			
			catch(SQLException e){
				out.println("SQLException caught: " + e.getMessage());
			}

catch (Exception e){
					out.println(e);
				}
			
			finally {

try {
					if (connection != null) connection.close();
				}

catch (SQLException ignored){
						out.println(ignored);
					}
			}
		}
}

7. EditPage.jsp

<br>

8. Edit.java

package com.amzi.servlets;

import java.io.IOException;
import java.io.PrintWriter;

import com.amzi.dao.EditDetails;
import com.amzi.dao.LoginDao;
import com.amzi.dao.UpdateUser;

/**
* Servlet implementation class UpdateCustomerServlet
*/
@WebServlet("/UpdateCustomerServlet")

public class Edit extends HttpServlet {
private static final long serialVersionUID = 1L;

/**
* @see HttpServlet#HttpServlet()
*/
public Edit() {
super();
// TODO Auto-generated constructor stub
}

try {
// Get the PrintWriter object
HttpSession session = request.getSession(true);
PrintWriter writer = response.getWriter();
writer.write("<html><body>");

// Get product information from request.getParameter()
String un = (String)session.getAttribute("un");
session.setAttribute("un", un);
String pass = (String)session.getAttribute("pass");
session.setAttribute("pass", pass);
String name = (String)session.getAttribute("name");
session.setAttribute("name", name);
String surname = (String)session.getAttribute("surname"); //request.getParameter
session.setAttribute("surname", surname);
String email = (String)session.getAttribute("email");
session.setAttribute("email", email);
String degree = (String)session.getAttribute("degree");
session.setAttribute("degree", degree);

// If "UPDATE" button is clicked
if (request.getParameter("submit") != null) {
// Create an instance of ProductBean object
EditDetails editdetails = new EditDetails();
// Set the attribute values for the newly created object
editdetails.setUn(un);
editdetails.setPass(pass);
editdetails.setName(name);
editdetails.setSurname(surname);
editdetails.setEmail(email);
editdetails.setDegree(degree);

// Invoke the ProductBeanDAO's updateProduct() method
UpdateUser updateuser = new UpdateUser();
return_code = updateuser.updateUserMethod(editdetails);

if (return_code != 0)
writer.write("UPDATE successful!<br><br>");
else
writer.write("UPDATE failed!<br><br>");
writer.write("<a href=\"checkout.jsp\">Back</a> to Product Search Page.");

}

writer.write("</body></html>");
} catch (Exception ex) {
System.out.println("Error: " + ex.getMessage());
}
}

/**
* @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
*/

protected void doPost(HttpServletRequest request,
HttpServletResponse response) throws ServletException, IOException {

}
}

9. EditDetails.java

10. UpdateUser.java

package com.amzi.dao;

import java.util.*;
import java.sql.Connection;  
import java.sql.DriverManager;  
import java.sql.PreparedStatement;  
import java.sql.ResultSet;  
import java.sql.SQLException;

import com.mysql.jdbc.Statement;

public class UpdateUser {
	public int updateUserMethod(EditDetails editdetails) throws Exception {
		int return_code = 0;

String url = "jdbc:mysql://localhost:3306/";  
        String dbName = "loginpage";  
        String driver = "com.mysql.jdbc.Driver";  
        String userName = "root";  
        String password = "";  
        
		try {
			Connection conn = null;  
	        PreparedStatement pst = null;
	        ResultSet rs = null;
	        
	        Class.forName(driver).newInstance();  
            conn = DriverManager  
                    .getConnection(url + dbName, userName, password);  
		
	        pst = conn
	        	.prepareStatement("UPDATE users.pass, users.name, users.surname, users.email, users.degree");

pst.setString(1, editdetails.getUn());
	        pst.setString(2, editdetails.getPass());
	        pst.setString(3, editdetails.getName());
	        pst.setString(4, editdetails.getSurname());
	        pst.setString(5, editdetails.getEmail());
	        pst.setString(6, editdetails.getDegree());
	       
	        return_code = pst.executeUpdate();

conn.close();
		} catch (Exception ex) {
		throw new Exception("Error: " + ex.getMessage());
		}

return return_code;
		}

}

11. glassfish-web.xml

K. Tsang

Bartender

Posts: 3648

I like...

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Welcome to the Ranch.

How come you are getting the as

This practically gets the session if it already exists, which at first (eg at the login page) there is no session. You should remove the false part.

Also in the jsp pages (those need authenticated content) add in the session attribute for the @page declaration

This way subsequent jsp pages will be session-aware until user logs out or session times out.

K. Tsang CEng MBCS PMP PMI-ACP OCMJEA OCPJP

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Hi Koray,
Remove the doGet method in LoginServlet and tell the new behaviour.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Thanks for welcome K.

I have corrected the things you mentioned but still have same problems.

My code is a bit complicated because I'm trying to learn own my own from lots of websites so there can be lots of unrelated or non functional codes. Sorry for that

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Paul Ngom wrote:Hi Koray,
Remove the doGet method in LoginServlet and tell the new behaviour.

Hey Paul, nothing is changed.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

What says the title of the LoginPage.jsp that diplays Welcome null?

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Yes it still says welcome null because of that part in LoginPage.jsp

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

You want to display the name or the username? Because there is no field passed from LoginServlet on which you can apply a request.GetParameter(). Maybe you need to connect to the database to retrieve it. You could make LoginDao.validate() to return the name which you will attach to LoginPage as LoginPage?name=nameReturned. The values set in the session are un and pass.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Tsang

Bartender

Posts: 3648

I like...

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

OK let's just focus on NewFile.jsp -> login servlet -> LoginPage.jsp.

NewFile.jsp is just a form which you got. Yet those <@page import ....> can be removed at the top.

Login Servlet:
1) can just have the doPost() method. Because ideally this servlet won't be triggered by users typing it in the address bar.
2) don't need the explicit constructor. Remove that.
3) doGet() method can be removed.

protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {  
        HttpSession session = request.getSession();    // remove false parm
        response.setContentType("text/html");      
        PrintWriter out = response.getWriter();      
            
        String n=request.getParameter("un");      
        String p=request.getParameter("pass");     
            
        if(session!=null)    { // brace it
        session.setAttribute("un", n);  
        session.setAttribute("pass", p);   
    } // add closing brace

if(LoginDao.validate(n, p)){      
            RequestDispatcher rd=request.getRequestDispatcher("LoginPage.jsp");      
            rd.forward(request,response);      
        }      
        else{      
            out.print("<p style=\"color:red\">Sorry username or password error</p>");      
            RequestDispatcher rd=request.getRequestDispatcher("NewFile.jsp");      
            rd.include(request,response);      
        }      
        out.close();     
    }

LoginPage.jsp
1) add session="true" to <%@ page
2) why need to wrap "Hello...." in a form calling LoginServlet?
3) add line <% request.getSession(); %> before using session variable

That should get you started to a more simpler page/servlet.

K. Tsang CEng MBCS PMP PMI-ACP OCMJEA OCPJP

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Paul Ngom wrote:You want to display the name or the username? Because there is no field passed from LoginServlet on which you can apply a request.GetParameter(). Maybe you need to connect to the database to retrieve it. You could make LoginDao.validate() to return the name which you will attach to LoginPage as LoginPage?name=nameReturned. The values set in the session are un and pass.

I wanna display the name but I didn't understand what do you want me to do

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

I have changed all the things you said K. Tsang. I am trying to solve these problems for days and I don't know what to do.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

status = rs.next();

while(rs.next()) {
...

This needs to be changed also otherwise you may never enter the while loop. I think an if clause is sufficient. This is in LoginDao.java
It is better to change the status variable value inside the loop like the following

If you do this, you can then call LoginDao.getName() inside LoginServlet.java

Then get the session value in the LoginPage.jsp

Hope i will be understood.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Hi Koray,
Was step 1 solved? Can we proceed to step 2 of errors?
Regards

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Hey Paul, thanks for your help. I added the things you want me to add but due to the errors, I had to add "static" for both variable and method name like, static String name = ""; and public static String getName().
It was like;
Login successful!!!
Hello, null
Welcome null
before. And now it is like;
Login successful!!!
Hello,
Welcome null.
Still name attribute isn't displayed.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Your LoginPage should look like this

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

I directly copied and pasted your code on LoginPage.jsp and it is now:
Login successful!!!

Hello,

Welcome.

Can it be because of my web.xml? I didn't change or add anything on that part.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

I also add a new jsp which shows entire database table to see if I can connect database or not. And it displays every users with every attributes.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Please post your LoginDao.java and LoginServlet.java. You also need to define the servlets mapping in web.xml if you want to call them from browser which is not our case.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Here they are:

LoginDao.java:

package com.amzi.dao;

import java.util.*;
import java.sql.Connection;  
import java.sql.DriverManager;  
import java.sql.PreparedStatement;  
import java.sql.ResultSet;  
import java.sql.SQLException;

import com.mysql.jdbc.Statement;
  
public class LoginDao {  
	static String name = "";
    public static boolean validate(String un, String pass) {          
        boolean status = false;  
        Connection conn = null;  
        PreparedStatement pst = null;
        //Statement st = null;
        ResultSet rs = null;  
        
  
        String url = "jdbc:mysql://localhost:3306/";  
        String dbName = "loginpage";  
        String driver = "com.mysql.jdbc.Driver";  
        String userName = "root";  
        String password = "";  
        try {  
            Class.forName(driver).newInstance();  
            conn = DriverManager  
                    .getConnection(url + dbName, userName, password);  
  
            pst = conn  
                    .prepareStatement("select * from users where un=? and pass=?");  
            pst.setString(1, un);  
            pst.setString(2, pass);  
  
            rs = pst.executeQuery();  
            status = rs.next();  
            
            while(rs.next()) {
            	status = true;
            	String uname = rs.getString ("un");
            	String upass = rs.getString("pass");
            	name = rs.getString("name");
                System.out.println(uname + ", " + upass);
            }

return status;  
    }
    
    public static String getName(){  
    	  return name;  
    }  
}

LoginServlet.java

package com.amzi.servlets;

import java.io.IOException;
import java.io.PrintWriter;

import com.amzi.dao.LoginDao;

/**
 * Servlet implementation class LoginServlet
 */
@WebServlet("/LoginServlet")
public class LoginServlet extends HttpServlet {
	private static final long serialVersionUID = 1L;
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		// TODO Auto-generated method stub
		HttpSession session = request.getSession();  
        response.setContentType("text/html");    
        PrintWriter out = response.getWriter();    
          
        String n=request.getParameter("un");    
        String p=request.getParameter("pass");   
          
        
         
        
        if(session!=null){ 
        session.setAttribute("un", n);  //buralarda hata olmalı
        session.setAttribute("pass", p); 
        }
  
        if(LoginDao.validate(n, p)){    
        	String name = LoginDao.getName();
        	session.setAttribute("name",name);
        	
            RequestDispatcher rd=request.getRequestDispatcher("LoginPage.jsp");    
            rd.forward(request,response);    
        }    
        else{    
            out.print("<p style=\"color:red\">Sorry username or password error</p>");    
            RequestDispatcher rd=request.getRequestDispatcher("NewFile.jsp");    
            rd.include(request,response);    
        }    
  
        out.close();   
	}
}

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Remove this line from LoginDao.java

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Remove this line from LoginDao.java

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Woohoo it worked!

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Woohoo it worked!

Well, that's good news. Can you figure out the solution for step 2 errors?

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

I fixed it a bit. It was displaying null values for attributes, now it displays name, surname etc. But it still gives status 404 error when I click on submit button.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Please post EditPage.jsp and a full description of the error message you are having.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

I found this code in the Internet and changed some parts. Also, I just noticed another error. When I clicked edit button all the attributes are coming except degree. Even I write selected: <option value="<%=session.getAttribute("degree")%>" selected>

EditPage.jsp

<br>

</body>
</html>

Registration.java

package com.amzi.servlets;

import java.io.IOException;
import java.io.PrintWriter;
import java.sql.DriverManager;
import java.sql.ResultSet;
import java.sql.SQLException;

import com.amzi.dao.LoginDao;
import com.mysql.jdbc.Connection;
import com.mysql.jdbc.PreparedStatement;

public class Registration extends HttpServlet{
	String un = "";
	String pass = "";
	String name = "";
	String surname = "";
	String email = "";
	String degree = "";
	
	/**
	 * 
	 */
	private static final long serialVersionUID = 1L;

public void init(ServletConfig config) throws ServletException{
		super.init(config);
	
	}

un = (String) session.getAttribute("un");
			un = (String) session.getAttribute("password"); //String pass = req.getParameter de olur
			name = (String) session.getAttribute("name");
			surname = (String) session.getAttribute("surname");
			email = (String) session.getAttribute("email");
			degree = (String) session.getAttribute("degree");
			
			try {
				Class.forName(driver);
				connection = (Connection) DriverManager.getConnection(url + dbName, "root", ""); 
				String sql = "insert into users values (?,?,?,?,?,?)";
				PreparedStatement pst = (PreparedStatement) connection.prepareStatement(sql);
				pst.setString(1, un);
				pst.setString(2, pass);
				pst.setString(3, name);
				pst.setString(4, surname);
				pst.setString(5, email);
				pst.setString(6, degree);
				int numRowsChanged = pst.executeUpdate();
				out.println(" Welcome : ");
				out.println(" '"+name+"'");
				pst.close();
			}
			
			catch(ClassNotFoundException e){
				out.println("Couldn't load database driver: " + e.getMessage());
			}
			
			catch(SQLException e){
				out.println("SQLException caught: " + e.getMessage());
			}

catch (Exception e){
					out.println(e);
				}
			
			finally {

try {
					if (connection != null) connection.close();
				}

catch (SQLException ignored){
						out.println(ignored);
					}
			}
		}
}

EditDetails.java

UpdateUser.java

package com.amzi.dao;

import java.util.*;
import java.sql.Connection;  
import java.sql.DriverManager;  
import java.sql.PreparedStatement;  
import java.sql.ResultSet;  
import java.sql.SQLException;

import com.mysql.jdbc.Statement;

public class UpdateUser {
	public int updateUserMethod(EditDetails editdetails) throws Exception {
		int return_code = 0;

conn.close();
		} catch (Exception ex) {
		throw new Exception("Error: " + ex.getMessage());
		}

return return_code;
		}
}

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

1
Number of slices to send:

Optional 'thank-you' note:

Send

Try this for EditPage.jsp and tell me if the field degree shows. I assume the possible values of degree in table are associatedegree, bachelordegree, masterdegree and doctoraldegree.

<%@ page language="java" contentType="text/html; charset=ISO-8859-1"  
    pageEncoding="ISO-8859-1"%>  
      
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">  
<html>  
<head>  
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">  
<title>Edit</title>  
</head>  
<body>  
<h2> Edit </h2>  
<hr>  
<form name="frm1" action= "Edit" method=post>  
<table>  
 <tr>  
 <td>Username</td>  
 <td><input readonly="readonly" name="un" value="<%=session.getAttribute("un")%>" disabled="disabled"></td>  
 </tr>  
 <tr>  
 <td>Password</td>  
 <td><input type ="text" name="password" value="<%=session.getAttribute("pass")%>"></td>  
 </tr>  
 <tr>  
 <td>Name</td>  
 <td><input type ="text" name="name" value="<%=session.getAttribute("name")%>"></td>  
 </tr>  
 <tr>  
 <td>Surname</td>  
 <td><input type ="text" name="surname" value="<%=session.getAttribute("surname")%>"></td>  
 </tr>  
 <tr>  
 <td>E-Mail</td>  
 <td><input type ="text" name="email" value="<%=session.getAttribute("email")%>"></td>  
 </tr>  
 <tr>  
 <td>University Degree</td>  
 <td> <select name="degree">  
   <option value="<%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("associatedegree"))?" selected":""%>">Associate Degree</option>  
   <option value="<%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("bachelordegree"))?" selected":""%>">Bachelor's Degree</option>  
   <option value="<%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("masterdegree"))?" selected":""%>">Master's Degree</option>  
   <option value="<%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("doctoraldegree"))?" selected":""%>">Doctoral Degree</option>  
   </select></td>  
   </tr>  
   </table>  
  
<br/>  
  
<input type = "button" value = "Back" name = "back1" onclick = "JavaScript:window.location='LoginPage.jsp';"/>  
<input type = "submit" value = "Submit" name = "submit1" onclick = "JavaScript:window.location='LoginPage.jsp';"/>  
</form>  
  
</body>  
</html>

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

It always shows Associate Degree is selected.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Sorry for the previous post, try this for EditPage.jsp

<%@ page language="java" contentType="text/html; charset=ISO-8859-1"  
    pageEncoding="ISO-8859-1"%>  
      
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">  
<html>  
<head>  
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">  
<title>Edit</title>  
</head>  
<body>  
<h2> Edit </h2>  
<hr>  
<form name="frm1" action= "Edit" method=post>  
<table>  
 <tr>  
 <td>Username</td>  
 <td><input readonly="readonly" name="un" value="<%=session.getAttribute("un")%>" disabled="disabled"></td>  
 </tr>  
 <tr>  
 <td>Password</td>  
 <td><input type ="text" name="password" value="<%=session.getAttribute("pass")%>"></td>  
 </tr>  
 <tr>  
 <td>Name</td>  
 <td><input type ="text" name="name" value="<%=session.getAttribute("name")%>"></td>  
 </tr>  
 <tr>  
 <td>Surname</td>  
 <td><input type ="text" name="surname" value="<%=session.getAttribute("surname")%>"></td>  
 </tr>  
 <tr>  
 <td>E-Mail</td>  
 <td><input type ="text" name="email" value="<%=session.getAttribute("email")%>"></td>  
 </tr>  
 <tr>  
 <td>University Degree</td>  
 <td> <select name="degree">  
   <option value="associatedegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("associatedegree"))?" selected":""%>>Associate Degree</option>  
   <option value="bachelordegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("bachelordegree"))?" selected":""%>>Bachelor's Degree</option>  
   <option value="masterdegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("masterdegree"))?" selected":""%>>Master's Degree</option>  
   <option value="doctoraldegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("doctoraldegree"))?" selected":""%>>Doctoral Degree</option>  
   </select></td>  
   </tr>  
   </table>  
  
<br/>  
  
<input type = "button" value = "Back" name = "back1" onclick = "JavaScript:window.location='LoginPage.jsp';"/>  
<input type = "submit" value = "Submit" name = "submit1" onclick = "JavaScript:window.location='LoginPage.jsp';"/>  
</form>  
  
</body>  
</html>

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

It still shows Associate Degree for all users. That problem is not that important. You can skip this one

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

That problem is not that important. You can skip this one

No, let's not skip it. What are the possible values of degree column in table?

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

That column is varchar(30) in table if you need that. Combo box consists of Associate Degree, Bachelor's Degree, Master's Degree and Doctoral Degree.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

You have not given the actual values stored in degree column. Are they associatedegree, bachelordegree,masterdegree or doctoraldegree?

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Yes but there are "s" in both "bachelorsdegree" and "mastersdegree". I directly copied and pasted this combo box from my SignUpPage.jsp. I didn't mention them in anywhere else.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

From your indication, the values of degree in table are associatedegree, bachelorsdegree, mastersdegree and doctoraldegree. I am not talking about the content of the combo box. So the EditPage.jsp should then read:

<%@ page language="java" contentType="text/html; charset=ISO-8859-1"  
    pageEncoding="ISO-8859-1"%>  
      
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">  
<html>  
<head>  
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">  
<title>Edit</title>  
</head>  
<body>  
<h2> Edit </h2>  
<hr>  
<form name="frm1" action= "Edit" method=post>  
<table>  
 <tr>  
 <td>Username</td>  
 <td><input readonly="readonly" name="un" value="<%=session.getAttribute("un")%>" disabled="disabled"></td>  
 </tr>  
 <tr>  
 <td>Password</td>  
 <td><input type ="text" name="password" value="<%=session.getAttribute("pass")%>"></td>  
 </tr>  
 <tr>  
 <td>Name</td>  
 <td><input type ="text" name="name" value="<%=session.getAttribute("name")%>"></td>  
 </tr>  
 <tr>  
 <td>Surname</td>  
 <td><input type ="text" name="surname" value="<%=session.getAttribute("surname")%>"></td>  
 </tr>  
 <tr>  
 <td>E-Mail</td>  
 <td><input type ="text" name="email" value="<%=session.getAttribute("email")%>"></td>  
 </tr>  
 <tr>  
 <td>University Degree</td>  
 <td> <select name="degree">  
   <option value="associatedegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("associatedegree"))?" selected":""%>>Associate Degree</option>  
   <option value="bachelorsdegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("bachelorsdegree"))?" selected":""%>>Bachelor's Degree</option>  
   <option value="mastersdegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("mastersdegree"))?" selected":""%>>Master's Degree</option>  
   <option value="doctoraldegree" <%=(((String)session.getAttribute("degree"))!=null && ((String)session.getAttribute("degree")).equals("doctoraldegree"))?" selected":""%>>Doctoral Degree</option>  
   </select></td>  
   </tr>  
   </table>  
  
<br/>  
  
<input type = "button" value = "Back" name = "back1" onclick = "JavaScript:window.location='LoginPage.jsp';"/>  
<input type = "submit" value = "Submit" name = "submit1"/>  
</form>  
  
</body>  
</html>

Try test it again and tell what happens.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Still gives Associate Degree, though user's degree is Bachelor's Degree.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

What! Please let's do some debugging. Insert in page EditPage.jsp the following code:

and let us know the value of degree in session.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Koray,
Please tell the value of session variable degree that is printed after you have inserted the code i gave you.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

K. Karacakaya

Ranch Hand

Posts: 53

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

It gives Bachelor's Degree.

Paul Ngom

Ranch Hand

Posts: 355

posted 10 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

OIC, then the values in your options are not correct unless you change them before inserting inside the database in the Edit class.
That is why, i was asking that you give the possible values of the column degree in table. Run a 'select distinct degree from users;' in database and post the results.

I am always surprised at the added knowledge i can get from others when i give my opinion on a topic.

Consider Paul's rocket mass heater.