Исключения с оператором prepare в Java для обновления MySQL - PullRequest
1 голос
/ 02 мая 2011

У меня есть следующая строка:

public static final String UPDATESONG = "update songs SET " +
        "name = ?, artist = ?, albumName = ?, genre = ?, " +
        "time = ?, trackNum = ?, year = ?, numlikes = ?, numdislikes = ? where song_id = ?";

Затем я использую следующий код для подготовки и использования этого утверждения:

ps = conn.prepareStatement(AppConstants.Queries.UPDATESONG);
        ps.setString(1, name);
        ps.setString(2, artist);
        ps.setString(3, albumName);
        ps.setString(4, genre);
        ps.setInt(5, time);
        ps.setInt(6, trackNumber);
        ps.setString(7, year);
        ps.setInt(8, numberOfLikes);
        ps.setInt(9, numberOfDislikes);
        ps.setInt(10, id);
        ps.executeUpdate();

Когда я запускаю тест Junit, я получаю следующую ошибку:

com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: у вас есть ошибка в вашем синтаксисе SQL; проверьте руководство, соответствующее вашей версии сервера MySQL, на предмет правильного синтаксиса, который можно использовать около 'time = 240, trackNum = 11, year =' 2007 ', numlikes = 0, numdislikes = 0, где s' в строке 1

Похоже, что подготовленное выражение добавляет a перед предложением where. Я думаю, что это является причиной проблемы, но я не уверен, почему это проявляется. У кого-нибудь есть понимание этого вопроса?

Редактировать: Я исправил проблему пропущенной запятой после жанра =? и до сих пор проблема Спасибо, что указали на эту проблему.

Ошибка, которую я получаю после добавления новой запятой:

com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'time = 240, trackNum = 11, year = '2007', numlikes = 0, numdislikes = 0, where s' at line 1
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39)
at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
at com.mysql.jdbc.Util.handleNewInstance(Util.java:407)
at com.mysql.jdbc.Util.getInstance(Util.java:382)
at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:1052)
at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3603)
at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3535)
at com.mysql.jdbc.MysqlIO.sendCommand(MysqlIO.java:1989)
at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2150)
at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2626)
at com.mysql.jdbc.PreparedStatement.executeInternal(PreparedStatement.java:2119)
at com.mysql.jdbc.PreparedStatement.executeUpdate(PreparedStatement.java:2415)
at com.mysql.jdbc.PreparedStatement.executeUpdate(PreparedStatement.java:2333)
at com.mysql.jdbc.PreparedStatement.executeUpdate(PreparedStatement.java:2318)
at SongDAO.updateSong(SongDAO.java:155)
at SongTest.testUpdateSong(SongTest.java:58)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at junit.framework.TestCase.runTest(TestCase.java:168)
at junit.framework.TestCase.runBare(TestCase.java:134)
at junit.framework.TestResult$1.protect(TestResult.java:110)
at junit.framework.TestResult.runProtected(TestResult.java:128)
at junit.framework.TestResult.run(TestResult.java:113)
at junit.framework.TestCase.run(TestCase.java:124)
at junit.framework.TestSuite.runTest(TestSuite.java:243)
at junit.framework.TestSuite.run(TestSuite.java:238)
at junit.framework.TestSuite.runTest(TestSuite.java:243)
at junit.framework.TestSuite.run(TestSuite.java:238)
at org.junit.internal.runners.JUnit38ClassRunner.run(JUnit38ClassRunner.java:83)
at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:49)
at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:467)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:683)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:390)
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:197)

Ответы [ 3 ]

3 голосов
/ 02 мая 2011

"name = ?, artist = ?, albumName = ?, genre = ? " последняя запятая отсутствует.

Ваш SQL будет выглядеть как ... , albumName = ?, genre = ? time = ?, trackNum = ?, ...

2 голосов
/ 02 мая 2011

Вы забыли запятую после genre = ? и до time = ?

0 голосов
/ 02 мая 2011

Вы пропустили запятую после установки жанра.

  public static final String UPDATESONG = "update songs SET " +
    "name = ?, artist = ?, albumName = ?, genre = ?, " +
    "time = ?, trackNum = ?, year = ?, numlikes = ?, numdislikes = ? where song_id = ?";
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...