Ошибка при добавлении записи в БД

@rviktoria · Регистрация: 05.03.2013

Author24 — интернет-сервис помощи студентам

Извиняюсь за глупый вопрос. Помогите подправить кавычки в SQL запросе.

Java

public static void InsertAddress(Address address) throws SQLException, ClassNotFoundException
{        
stat.executeUpdate("INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) "
                         + "VALUES (" + address.getFirstname() + "\", \"" + address.getLastname() + "\", \"" + address.getCountry() + "\", \"" + address.getCity() + "\", \"" + address.getAddress() + "\", " + address.getIndex() + ")");
}

Постоянно такая ошибка:
javax.servlet.ServletException: org.h2.jdbc.JdbcSQLException: Syntax error in SQL statement "INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) VALUES (Ivan"", ""Ivanov"", ""Russia"", ""Ivanovo"", ""Address 21-32[*]"", 95632)"; SQL statement:
INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) VALUES (Ivan", "Ivanov", "Russia", "Ivanovo", "Address 21-32", 95632) [42000-171]

Уже по-разному их переставляла, то не хватает одной, то две лишние и т.д.
Помогите, пожалуйста!!

@AckiyBolt · 03.05.2013, 15:26

нет предела лени человеческой. вроде так:

Java

stat.executeUpdate("INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) "
        + "VALUES ("
        + "\"" + address.getFirstname() + "\""
        + "\"" + address.getLastname() + "\""
        + "\"" + address.getCountry() + "\""
        + "\"" + address.getCity() + "\""
        + "\"" + address.getAddress() + "\""
        + "\"" + address.getIndex() + "\""
        + ")");

@rviktoria · 03.05.2013, 16:22 **[ТС]**

AckiyBolt, Вставила ваш код. Теперь выдаёт ошибку:

javax.servlet.ServletException: org.h2.jdbc.JdbcSQLException: Column "Ivan" not found; SQL statement:
INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) VALUES ("Ivan","Ivanov","Russia","Ivanovo","Address 21-32",34) [42122-171]

Не могу понять, что не так?

index.jsp

Java

<%@page import="java.util.ArrayList"%>
<%@page import="Data.Database"%>
<%@page import="Data.Address"%>
<%@page contentType="text/html" pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
    </head>
    <body>
        <form>
            <table>
                <tr><td>Firstname: </td><td><input type="text" name="Firstname"></td></tr>
                <tr><td>Lastname: </td><td><input type="text" name="Lastname"></td></tr>
                <tr><td>Country: </td><td><input type="text" name="Country"></td></tr>
                <tr><td>City: </td><td><input type="text" name="City"></td></tr>
                <tr><td>Address: </td><td><input type="text" name="Address"></td></tr>
                <tr><td>Index: </td><td><input type="text" name="Index"></td></tr>
                <tr><td></td><td><input type="submit" value="Add"></td></tr>
                
                <%   
                    ArrayList<Address> address = Database.PrintAddress();
                    
                    if(request.getParameter("Firstname") != null && request.getParameter("Lastname") != null && 
                    request.getParameter("Country") != null && request.getParameter("City") != null && 
                    request.getParameter("Address") != null && request.getParameter("Index") != null)
                    { 
                        String firstname = request.getParameter("Firstname");
                        String lastname =request.getParameter("Lastname");
                        String country = request.getParameter("Country");
                        String city = request.getParameter("City");
                        String addres = request.getParameter("Address");
                        String index = request.getParameter("Index");
 
                        Database.InsertAddress(new Address(firstname, lastname, country, city, addres, Integer.parseInt(index)));
                    }                
                %>                
            </table>
        </form>
    </body>
</html>

Address.java

Java

package Data;
 
public class Address {    
       
    private String firstname;
    private String lastname;
    private String country;
    private String city;
    private String address;
    private int index; 
    
    public Address(String firstname, String lastname, String country, String city, String address, int index)
    {
        this.firstname = firstname;
        this.lastname = lastname;
        this.country = country;
        this.city = city;
        this.address = address;
        this.index = index;
    }
    
    public String getFirstname() { return firstname; }
    
    public void setFirstname(String firstname) { this.firstname = firstname;}
    
    public String getLastname() { return lastname; }
    
    public void setLastname(String lastname) { this.lastname = lastname;}
    
    public String getCountry() { return country; }
    
    public void setCountry(String country) { this.country = country; }
    
    public String getCity() { return city; }
    
    public void setCity(String city) { this.city = city;}
    
    public String getAddress() { return address; }
    
    public void setAddress(String address) { this.address = address;}
    
    public int getIndex() { return index; }
    
    public void setIndex(int index) { this.index = index; }     
}

Database.java

Java

package Data;
 
import java.sql.*;
import java.util.ArrayList;
import java.util.logging.Level;
import java.util.logging.Logger;
 
public class Database {
    
    private static Connection conn = null;
    private static Statement stat = null;
    private static ResultSet res = null;
    
    public static ArrayList<Address> PrintAddress() throws ClassNotFoundException, SQLException
    {
        ArrayList<Address> address = new ArrayList();
    
        try
        {
            Class.forName("org.h2.Driver");
            conn = DriverManager.getConnection("jdbc:h2:~/dbjava/address", "username", "password");
 
            if (conn != null) 
            {                
                stat = conn.createStatement();               
                res = stat.executeQuery("SELECT * FROM Addresses.tAddress");
 
                while (res.next())
                {
                    address.add(new Address(res.getString("Firstname"), res.getString("Lastname"), res.getString("Country"), res.getString("City"), res.getString("Address"), res.getInt("Index")));
                }
            }
        }
        catch (SQLException ex)
        {
            Logger.getLogger(Database.class.getName()).log(Level.SEVERE, null, ex);
        }
 
        return address; 
    }
    
    public static void InsertAddress(Address address) throws SQLException, ClassNotFoundException
    {        
        stat.executeUpdate("INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) "
                         + "VALUES ("
                         + "\"" + address.getFirstname() + "\","
                         + "\"" + address.getLastname() + "\","
                         + "\"" + address.getCountry() + "\","
                         + "\"" + address.getCity() + "\","
                         + "\"" + address.getAddress() + "\","
                         + address.getIndex()
                         + ")");
    }
}

@AckiyBolt · 03.05.2013, 16:31

эм... забавно... вот такое попробуйте

Java

stat.executeUpdate("INSERT INTO Addresses.tAddress (Firstname, Lastname, Country, City, Address, Index) "
        + "VALUES ("
        + "'" + address.getFirstname() + "',"
        + "'" + address.getLastname() + "',"
        + "'" + address.getCountry() + "',"
        + "'" + address.getCity() + "',"
        + "'" + address.getAddress() + "',"
        + address.getIndex()
        + ");");

@rviktoria · 03.05.2013, 16:53 **[ТС]**

AckiyBolt, Спасибо большое! Всё работает!

@DTSHNick · 03.05.2013, 17:53

Попробуй еще так:

Java

//что-то там
Address ads = new Address ();
ads.setFirstname("First Name");
ads.setLastname("Last Name");
ads.setIndex(1);
ads.setCountry("Country")
ads.setCity("City");
ads.setAddress("Address");
 
SessionFactory sessionFactory=new Configuration().configure().buildSessionFactory();
Session session= sessionFactory.openSession();
 
session.save(ads)
session.getTransaction().commit();
//что-то там

Добавлено через 1 минуту
Некоторые говорят лучше так.

Лучше не использовать запросы, не знаю правда почему))

@AckiyBolt · 03.05.2013, 18:03

это использование ОРМ. не лучше, иначе. некоторые, видимо, не шарят когда что нужно использовать. в любом случае для обучения сначала нужно колупать тупые запросы

да. а лучше юзать вот это: http://docs.oracle.com/javase/... pared.html
зачем? запросы экранируются от скул инекций. типа более секьюрно

@DTSHNick · 03.05.2013, 18:05

Сообщение от AckiyBolt

это использование ОРМ. не лучше, иначе. некоторые, видимо, не шарят когда что нужно использовать. в любом случае для обучения сначала нужно колупать тупые запросы

да. а лучше юзать вот это: http://docs.oracle.com/javase/... pared.html
зачем? запросы экранируются от скул инекций. типа более секьюрно

спс, буду знать

@mutagen · 03.05.2013, 23:07

AckiyBolt, о майн гот, зачем ты научил девушку говнокодить?

@AckiyBolt · 04.05.2013, 03:43

Сообщение от mutagen

AckiyBolt, о майн гот, зачем ты научил девушку говнокодить?

а шо не так?

@mutagen · 04.05.2013, 03:54

допустим в адресе строка

SQL

1	'address'; DROP Addresses.tAddress;

и капец

@AckiyBolt · 04.05.2013, 04:13

Сообщение от mutagen

допустим в адресе строка

SQL

1	'address'; DROP Addresses.tAddress;

и капец

ну девушка получила ответ на свой вопрос. а по поводу "допустим" перед тем как критиковать нужно открывать урлы

Сообщение от AckiyBolt

это использование ОРМ. не лучше, иначе. некоторые, видимо, не шарят когда что нужно использовать. в любом случае для обучения сначала нужно колупать тупые запросы

да. а лучше юзать вот это: http://docs.oracle.com/javase/... pared.html
зачем? запросы экранируются от скул инекций. типа более секьюрно

@mutagen · 04.05.2013, 12:14

Сообщение от AckiyBolt

а по поводу "допустим" перед тем как критиковать нужно открывать урлы

ага сначала научил как конкатенить, а потом кто там глянет в сложный препаред, когда тут всё так няшно и удобно работает )))

просто если приводить демо по конкату, надо его снабжать сразу же инжектом, чтоб новис сразу озаботился.
ато щя возможно кроме нас эту тему уже никто и не читает

@DTSHNick · 04.05.2013, 12:33

Читает, но что-то я ничего не понял

@AckiyBolt · 04.05.2013, 14:17

Сообщение от mutagen

ага сначала научил как конкатенить, а потом кто там глянет в сложный препаред, когда тут всё так няшно и удобно работает )))

просто если приводить демо по конкату, надо его снабжать сразу же инжектом, чтоб новис сразу озаботился.
ато щя возможно кроме нас эту тему уже никто и не читает

хм. тоже верно. след раз буду сразу посылать учить препаред

@mutagen · 04.05.2013, 14:42

Сообщение от DTSHNick

Читает, но что-то я ничего не понял

там обьединиены 2 sql выражения в адресе, получается чёт типа:

SQL

1 2	INSERT INTO bla.bla .....; DROP bla.bla;

и после этого удаляется таблица bla.bla

@AckiyBolt 653 / 402 / 35 Регистрация: 19.02.2013 Сообщений: 1,072 Записей в блоге: 2
	04.05.2013, 03:43	10
	Сообщение от mutagen AckiyBolt, о майн гот, зачем ты научил девушку говнокодить? а шо не так? 0

@rviktoria 1 / 1 / 1 Регистрация: 05.03.2013 Сообщений: 41
	03.05.2013, 16:53 [ТС]	5
	AckiyBolt, Спасибо большое! Всё работает! 0

@AckiyBolt 653 / 402 / 35 Регистрация: 19.02.2013 Сообщений: 1,072 Записей в блоге: 2
	03.05.2013, 18:03	7
	это использование ОРМ. не лучше, иначе. некоторые, видимо, не шарят когда что нужно использовать. в любом случае для обучения сначала нужно колупать тупые запросы да. а лучше юзать вот это: http://docs.oracle.com/javase/... pared.html зачем? запросы экранируются от скул инекций. типа более секьюрно 1

@DTSHNick 9 / 9 / 3 Регистрация: 29.05.2011 Сообщений: 74
	03.05.2013, 18:05	8
	Сообщение от AckiyBolt это использование ОРМ. не лучше, иначе. некоторые, видимо, не шарят когда что нужно использовать. в любом случае для обучения сначала нужно колупать тупые запросы да. а лучше юзать вот это: http://docs.oracle.com/javase/... pared.html зачем? запросы экранируются от скул инекций. типа более секьюрно спс, буду знать 0

@mutagen 2586 / 2259 / 257 Регистрация: 14.09.2011 Сообщений: 5,185 Записей в блоге: 18
	03.05.2013, 23:07	9
	AckiyBolt, о майн гот, зачем ты научил девушку говнокодить? 0

@mutagen 2586 / 2259 / 257 Регистрация: 14.09.2011 Сообщений: 5,185 Записей в блоге: 18
	04.05.2013, 12:14	13
	Сообщение от AckiyBolt а по поводу "допустим" перед тем как критиковать нужно открывать урлы ага сначала научил как конкатенить, а потом кто там глянет в сложный препаред, когда тут всё так няшно и удобно работает ))) просто если приводить демо по конкату, надо его снабжать сразу же инжектом, чтоб новис сразу озаботился. ато щя возможно кроме нас эту тему уже никто и не читает 0

@DTSHNick 9 / 9 / 3 Регистрация: 29.05.2011 Сообщений: 74
	04.05.2013, 12:33	14
	Читает, но что-то я ничего не понял 0

@AckiyBolt 653 / 402 / 35 Регистрация: 19.02.2013 Сообщений: 1,072 Записей в блоге: 2
	04.05.2013, 14:17	15
	Сообщение от mutagen ага сначала научил как конкатенить, а потом кто там глянет в сложный препаред, когда тут всё так няшно и удобно работает ))) просто если приводить демо по конкату, надо его снабжать сразу же инжектом, чтоб новис сразу озаботился. ато щя возможно кроме нас эту тему уже никто и не читает хм. тоже верно. след раз буду сразу посылать учить препаред 0