How I delete the perticular record from access database using delete button?

Question

0.00/5 (No votes)

See more:

i use delete button link to delete recrd on follwing code userlist1.jsp:

<%@page import="java.sql.DriverManager"%>
<%@page import="java.sql.ResultSet"%>
<%@page import="java.sql.Statement"%>
<%@page import="java.sql.Connection"%>

<%
String driverName = "sun.jdbc.odbc.JdbcOdbcDriver";
String connectionUrl = "jdbc:odbc:sdldsn";
String dbName = "database";
String userId = "root";
String password = "root";

try {
Class.forName(driverName);
} catch (ClassNotFoundException e) {
e.printStackTrace();
}

Connection connection = null;
Statement statement = null;
ResultSet resultSet = null;
%>
<html>
<body bgcolor="LightGray" text="white"  >
<br>
<div style="width:95%;border-style:solid;border-width:3px;border-color:orange;height:600px;border-radius:15px;background-color:white;box-shadow:-1px -1px 20px orange" >
<br>
<br>
<img src=../images/adminbar.JPG width=100% style="border-radius:15px" >



<table align="center" cellpadding="5" cellspacing="5" border="1">
<br>
<br>
<tr bgcolor="YellowBrown">
<td>UserId</td>
<td>Name</td>
<td>Gender</td>
<td>Remove</td>
</tr>
<%
try{ 


Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
Connection con=DriverManager.getConnection("jdbc:odbc:sdldsn");


statement=con.createStatement();
String sql ="SELECT * FROM usermaster";
resultSet = statement.executeQuery(sql);
while(resultSet.next()){
String uid=resultSet.getString("UserId") ;
String nm=resultSet.getString("UserName") ;
String gd=resultSet.getString("Gender") ;
String dlink="removeuser.jsp?uid="+uid;
%>

<tr bgcolor="Brown">
<td><%=uid%></td>
<td><%=nm%></td>
<td><%=gd%></td>
<td><a href='<%=dlink%>'><img src="images/delete.jpg" height=20 width=100></a></td>
</tr>
<% 

}

} catch (Exception e) {
e.printStackTrace();
}
%>
</table>
</div>
</body>
</html>

What I have tried:

i want to delete user by clicking delete button ...i create othr page for this
removeuser.jsp

<%@ page language="java" import="java.sql.*" %>
<%
String uid=request.getParameter("UserId");
Class.forName("sun.jdbc.odbc.JdbcOdbcDriver");
Connection con=DriverManager.getConnection("jdbc:odbc:sdldsn");
String msql="delete from UserMaster where UserId="+uid;	
Statement smt=con.createStatement();
smt.executeUpdate(msql);
con.close();
Thread.sleep(1000);
response.sendRedirect("userlist1.jsp");
%>

Posted 15-Feb-18 21:22pm

SHUBHAM KHANDAR

Add a Solution

Comments

CHill60 16-Feb-18 3:48am

What is wrong with the code you already have?

SHUBHAM KHANDAR 16-Feb-18 8:57am

still,the record is not deleting from database
i dont know why

CHill60 16-Feb-18 11:06am

Are you sure the value in uid is correct and that it exists on the database?

SHUBHAM KHANDAR 23-Feb-18 9:02am

yes it exists and database is also coreect

Richard Deeming 16-Feb-18 13:11pm

Your code is vulnerable to SQL Injection[^]. NEVER use string concatenation to build a SQL query. ALWAYS use a parameterized query.

Everything you wanted to know about SQL injection (but were afraid to ask) | Troy Hunt[^]
How can I explain SQL injection without technical jargon? | Information Security Stack Exchange[^]
Query Parameterization Cheat Sheet | OWASP[^]

SHUBHAM KHANDAR 23-Feb-18 9:03am

i try paranthisis query too but it is not working too same effect

Richard Deeming 23-Feb-18 9:05am

Then either the record doesn't exist in the database; or you're getting an error that you haven't told us about; or you're looking at the wrong database.

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)