Oracle JDBC where i nieprawidłowy wynik zapytania

Koziołek

2009-10-01T15:31:12+02:00

Rejestracja: dni
Ostatnio: dni
Lokalizacja: Stacktrace
Postów: 6823

0

Kod:

Kopiuj

public int myMethod(String day) throws SQLException{
  String sql = "Select count(*) from MyTable WHERE someColumn = ? ";
  Connection connection = ConnFactory.get();
  PreparedStatement prepareStatement = null;
  ResultSet resultSet = null;
  int ret = -1;
  try{
      prepareStatement = connection.prepareStatement(sql);
      prepareStatement.setString(1, day);
      resultSet = prepareStatement.executeQuery(sql);
      if(resultSet.next()){
          ret = resultSet.getInt(1);
      }
  }
  catch(SQLException sqle){
      // closing statement & ResultSet, log and throw exception
  }
  finally{
     // closing statement & ResultSet
  }
  ConnFactory.kill(connection);

  return ret;
}

Wynik wywołania 0. Prawidłowy wynik około 100. Jeżeli usunę klauzulę WHERE to zaczyna zwracać prawidłowe wyniki. Czy ktoś może mi powiedzieć dlaczego się tak dzieje?

Freakman

2009-10-01T15:47:02+02:00

Rejestracja: dni
Ostatnio: dni

0

a jakie przykladowe wartosci ma day?

Koziołek

2009-10-01T15:49:25+02:00

Rejestracja: dni
Ostatnio: dni
Lokalizacja: Stacktrace
Postów: 6823

0

na przykład 20090311 format YYYYMMDD kolumna jest typu NUMBER. Dlaczego tak? Nie pytaj, bo nie wiem.

Freakman

2009-10-01T15:56:49+02:00

Rejestracja: dni
Ostatnio: dni

0

nie wali żadnym errorem o typie kolumny?

spróbuj może setObject(1, day) zamiast setString. Ewentualnie - co głupio zabrzmi - zparsuj ten ciąg na Integera i daj setInt (jestem zmuszony pisać ostatnio pod javą 1.4 stąd te dziwne pomysly ; ))

Koziołek

2009-10-01T16:02:23+02:00

Rejestracja: dni
Ostatnio: dni
Lokalizacja: Stacktrace
Postów: 6823

0

Błędów brak. Próbowałem z setObject i setInt i nadal nic...

2009-10-01T19:08:48+02:00

Rejestracja: dni
Ostatnio: dni

0

Trochę ten kod mi się nie podoba ale...

Czy zapytanie na pewno działa
Czy nie powinnieneś zacząć od 0 w setString( 0, day ) ?

__krzysiek85

2009-10-02T00:35:56+02:00

Rejestracja: dni
Ostatnio: dni
Postów: 1019

0

A co daje wpisane bezpośrednio
Select count(*) from MyTable WHERE someColumn=20090311
?

Jeżeli MyTable ma klucz główny (id), to spróbuj jeszcze:
Select count(id) from MyTable WHERE someColumn=20090311

Usuń też spacje wokół '='

getInt() zwraca też 0, gdy zapytanie zwróciło NULLa.

Freakman

2009-10-02T09:12:12+02:00

Rejestracja: dni
Ostatnio: dni

0

a jaka to baza? Oracle?

//edit: aha - tytuł ; )

2009-10-02T09:59:30+02:00

Rejestracja: dni
Ostatnio: dni

0

Spróbuj :

Kopiuj

public int myMethod(String day) throws SQLException{
  String sql = "Select count(*) as ilosc from MyTable WHERE someColumn = ? ";
  Connection connection = ConnFactory.get();
  PreparedStatement prepareStatement = null;
  ResultSet resultSet = null;
  int ret = -1;
  try{
      prepareStatement = connection.prepareStatement(sql);
      prepareStatement.setString(1, day);
      resultSet = prepareStatement.executeQuery(sql);
      if(resultSet.next()){
          ret = resultSet.getInt("ilosc");
      }
  }
  catch(SQLException sqle){
      // closing statement & ResultSet, log and throw exception
  }
  finally{
     // closing statement & ResultSet
  }
  ConnFactory.kill(connection);

  return ret;
}

Oraz podstaw na sztywno coś za zmienna day. Moze tu tkwi problem

Freakman

2009-10-02T10:34:24+02:00

Rejestracja: dni
Ostatnio: dni

0

według mnie trzeba drążyć temat wstawiania przy pomocy setString gdy w bazie jest typ number - sam trafiłem na przypadek gdy wstawiając stringa z id i porównując go z typem liczbowym waliło mi errorem o typie kolumny. Ty błedów nie masz - nie wiem czy to zależy od wersji bazy czy czegoś innego.

2009-10-04T11:37:32+02:00

Rejestracja: dni
Ostatnio: dni

0

Jaka wersja Javy? Jaka wersja sterownika OJDBC ? Która wersja Oracle ?

Kopiuj

      ...
      //prepareStatement.setString(1, day);
      prepareStatement.setInt(1, 20090311);   // czy zwraca poprawny wynik ?
      //prepareStatement.setInt(1, Integer.valueOf(day)); // albo to poprawny

Od strony Oracle włączyłbym trace'a i patrzył co trafia do parsera. Możliwe, że sterownik JDBC ma błąd.

Zamiast prepared statement sprobowałbym czy działa zwykły statement.

Kopiuj

 Statement stmt = conn.createStatement();
 ResultSet rset = stmt.executeQuery("Select count(*) from MyTable WHERE someColumn =  20090311 ");

I jeszcze wersja z PreparedStatement, ale z jawnym castowaniem:

Kopiuj

public int myMethod(String day) throws SQLException{
  String sql = "Select count(*) from MyTable WHERE someColumn = TO_NUMBER(?) ";
  Connection connection = ConnFactory.get();
  PreparedStatement prepareStatement = null;
  ResultSet resultSet = null;
  int ret = -1;
  try{
      prepareStatement = connection.prepareStatement(sql);
      prepareStatement.setString(1, day);
      resultSet = prepareStatement.executeQuery(sql);
      if(resultSet.next()){
          ret = resultSet.getInt(1);
      }
  }
  catch(SQLException sqle){
      // closing statement & ResultSet, log and throw exception
  }
  finally{
     // closing statement & ResultSet
  }
  ConnFactory.kill(connection);

  return ret;
}

pozdr,
yarel

2009-10-04T11:44:19+02:00

Rejestracja: dni
Ostatnio: dni

0

I jeszcze jedno wydaje mi się dziwne:

Kopiuj

    resultSet = prepareStatement.executeQuery(sql);

Dlaczego dajesz jako parametr sql, a nie korzystasz z samego executeQuery() ?

pozdr,
yarel