Тест Junit не пройден - PullRequest
0 голосов
/ 14 мая 2018

В следующем коде запрос, который я использовал, правильный, я проверил его на сервере sql, но тест завершился ошибкой. Что я делаю не так?

    public int ShipYearLaunched(Connection connection, String ship) throws 
    SQLException {
    String a="select Launched from Ships where name=?";
    PreparedStatement st=connection.prepareStatement(a);
    st.setString(1, ship);
    ResultSet rs =st.executeQuery();
    rs.next();
    int count = rs.getInt("c");
    return count; 
}

Это то, что он показывает вTrace Failure.

com.microsoft.sqlserver.jdbc.SQLServerException: The index 2 is out of range.
    at com.microsoft.sqlserver.jdbc.SQLServerException.makeFromDriverError(SQLServerException.java:206)
    at com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.setterGetParam(SQLServerPreparedStatement.java:940)
    at com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.setValue(SQLServerPreparedStatement.java:954)
    at com.microsoft.sqlserver.jdbc.SQLServerPreparedStatement.setString(SQLServerPreparedStatement.java:1532)
    at jdbc.JDBC.MostShipsSunkInBattle(JDBC.java:43)
    at jdbc.JDBC_Test.TestNumShipsSunkInBattle(JDBC_Test.java:51)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
    at java.lang.reflect.Method.invoke(Unknown Source)
    at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
    at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
    at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
    at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
    at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
    at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
    at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
    at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
    at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
    at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
    at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
    at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
    at org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
    at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:86)
    at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:538)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:760)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:460)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:206)

1 Ответ

0 голосов
/ 14 мая 2018

Сложно сказать, в чем ваша проблема, по коду и сообщению. Было бы полезно, если бы вы опубликовали SQL для создания таблицы.

Мне не нравится то, что я вижу в вашем SQL. Я бы сказал, что это должна быть таблица SHIP, а не множественное число.

Вы не следуете стандартам Java-кодирования.

Вы не закрываете свои ресурсы в области действия метода.

Я бы написал что-то вроде этого:

    private static final String SELECT_YEAR_LAUNCHED_SQL = "select yearLaunched from Ship where name = ?";


    public int getShipYearLaunched(Connection connection, String shipName) throws SQLException {
        int yearLaunched = 0;
        PreparedStatement ps = null;
        ResultSet rs = null; 
        try {
            ps = connection.prepareStatement(SELECT_LAUNCHED_SQL);
            st.setString(1, ship);
            rs = st.executeQuery();
            // Is there more than one row for a given ship name?  If yes, you should get a List<Integer> and append the value returned to the List
            while (rs.hasNext()) {
                yearLaunched = rs.getInt(1);
            }
        } finally {
            close(rs);
            close(ps);
        }
        return yearLaunched; 
    }
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...