Syntax error on token

values('"+v.vare_id+ "',  '"+ v.navn + "', " + v.pris + ")");

Du får ikke lukket dine '" + something + "'

Hvis vare_id er et tal skal det nok være:

values("+v.vare_id+ ",  '"+ v.navn + "', " + v.pris + ")");

har smidt din linie ind, men får fortsat samme fejl. Jeg har kodestykket liggende i en anden klasse hvor jeg ikke får fejl på den, derfor undrer det mig....

Jeg kan ikke se andre fejl end at dine '" ikke var lukket ordentligt.

Men hmmm - det jeg nævner burde egentlig ikke gi' compiler fejl, men først give fejl når databasen prøver at udføre sql sætningen...

Det er i følgende kodestykke jeg førsøger at indsætte koden:
------------------------------

import java.sql.*;
import oracle.jdbc.*;


public class OracleTest {
   
final static String connString = "jdbc:oracle:thin:@don:1521:JacobDB";

    public static void main(String[] args) {
        try {
       
        DriverManager.registerDriver(new oracle.jdbc.OracleDriver());
       
        }
       
               
        catch (SQLException e) {
            System.out.println(e.getMessage());
           
        }
        try{
            Connection conn = DriverManager.getConnection(
              connString,"Handelssystem","Handelssystem");
         
            conn.commit();
                     
    Vare v = new Vare(123456, "Kylling", 1234); 
       
            }       
        catch (SQLException e) {
            System.out.println("nummer zwei");
            System.out.println(e.getMessage());
        }
       
       
    }
}
------------------------------
Vare klassen er således ud:
-----------------------------
import java.sql.SQLException;

public class Vare
{

    int vare_id;   
  String navn;
  double pris;

  public Vare(int vare_id, String navn, double pris)
  {
      this.vare_id = vare_id;
    this.navn = navn;
    this.pris = pris;
  }


   
  public String toString()
  {
    return navn+": "+pris+" kr.";
  }
}
--------------------------------
Hvor indsætter jeg kodestykket? Det har måske noget med placeringen at gøre....

Jeg vil ikke indsætte din indsætVare metode i ovenstående kode. Det er jo din main-klasse!
Jeg ville lave en klasse og kalde den noget a la DatabaseManager eller lignende og lade den stå for alt der har med databasen at gøre.

Udkast til en databasemanager:

import java.sql.*;

public class DatabaseManager
{
    private final String connString = "jdbc:oracle:thin:@don:1521:JacobDB";
    private Connection conn;
   
    public DatabaseManager()
    {
        try
        {
            DriverManager.registerDriver(new oracle.jdbc.OracleDriver());
        }
       
               
        catch (Exception e) {
            System.out.println(e.getMessage());
           
        }
       
        try
        {
            conn = DriverManager.getConnection(connString,"Handelssystem","Handelssystem");
        }       
        catch (SQLException e) {
            System.out.println("nummer zwei");
            System.out.println(e.getMessage());
        }
    }
   
    public void indsætVare(Vare v) throws SQLException
    {
        conn.createStatement().executeUpdate("insert into VARE (VARE_ID, NAVN,PRIS) values('"
                          +v.vare_id+ ",  "+ v.navn + "', " + v.pris + ")");
    }
}

Og så kommer din main-klasse til at ligne noget a la det her:

public class OracleTest
{
    public static void main(String[] args)
    {
        try
        {
            Vare v = new Vare(123456, "Kylling", 1234); 
       
            DatabaseManager dm = new DatabaseManager();
            dm.indsætVare(v);       
        }
        catch (Exception ex)
        {
            ex.printStackTrace();
        }
    }
}

Jeg får nu denne fejl:

java.lang.NullPointerException
    at DatabaseManager.indsætVare(DatabaseManager.java:24)
    at OracleTest.main(OracleTest.java:34)
Exception in thread "main"

Min DatabaseManger ser således ud:
--------------------
import java.sql.SQLException;
import java.sql.Statement;


public class DatabaseManager {
   
    private static Statement stmt;
   
    public void indsætVare(Vare v) throws SQLException
              {
     
                stmt.executeUpdate("insert into VARE (VARE_ID, NAVN,PRIS) values("+v.vare_id+ ",  '"+ v.navn + "', " + v.pris + ")");
                System.out.println("varene er:  " +v.vare_id+ ": " +v.navn);
              }
}

Du kan jo heller ikke ovenstående!

Dit Statement stmt er jo null i og med at du ikke initialiserer den nogle steder.

Prøv at brug den databasemanager jeg har lavet. Eller i hvert fald brug den som inspiration.

Du skal jo først og fremmest ha' oprettet forbindelse til databasen og ud fra den Connection kan du så hive et Statement ud.

det må du undskylde, jeg havde ikke lige fået opdateret siden og set dit udkast.
Godt udkast, for nu virker det;-)


Jeg har lige 2 hurtige.
1. values("+v.vare_id+ ",  '"+ v.navn + "', " + v.pris + ")");
Hvordan skal det læses? Er i tvivl om hvor jeg skal sætte"', bare så jeg selv kan gøre det næste gang.
2. Hvis jeg har oprettet noget som varchar2 og char i databasen, hvad hiver jeg det så ud som? String, int?

" + etallerandet + "    == et tal (int, double, long)
'" + etallerandet + "'  == en tekst (String)


varchar == tekst == String

Fremragende!
Mange tak for hjælpen, smid et svar.

Der er dog et mindre problem i at bruge ovenstående, nemlig at du i den tekst du indsætter ikke må bruge tegn som ' da den vil forstyrre sql sætningen og databasen vil ikke kunne håndtere det.

Hvis du har brug for at indsætte tekster med ' skal du bruge det der hedder et PreparedStatement.

http://java.sun.com/j2se/1.4.2/docs/api/java/sql/PreparedStatement.html

svar

ok, tak. Jeg kigger nærmere på det.