Тестирование toString, но это не удается?

Question

Тестирование toString, но это не удается?

1

Используя этот код:

 public String toString()
{

    StringBuilder s = new StringBuilder();
    for (int i = 0; i < length; i++)
    {                     
        s.append(i).append(":").append(arr[i]).append(", ");                       
    }
    s.delete(s.length()-2, s.length()-1);
    return s.toString();
}

не проходит этот тест:

@Test
public void testToString()
{
    BetterArray<String> b = new BetterArray<String>();
    assertEquals("", b.toString());
    b.add("hello");
    assertEquals("0:hello", b.toString());
    b.add("bye");
    assertEquals("0:hello, 1:bye", b.toString());
    b.add("adios");
    assertEquals("0:hello, 1:bye, 2:adios", b.toString());
}

Он должен начинаться как пустая строка (""), я делаю это правильно? Я новичок в StringBuilder. Я использую s.delete для удаления конечной запятой и пробела

Jakkie Chan 17 март 2014, в 02:35

Источник

Теги:

java

string

arrays

tostring

3 ответа

0

Вы удаляете только запятую, строка удаления должна быть s.delete(s.length()-2, s.length()); ,

(По соглашению, конечный индекс диапазона точек 1 проходит мимо последнего элемента диапазона. Это то же самое для String.substring() например.)

Karsten 16 март 2014, в 23:19

0

спасибо, что помогло, теперь все мои тесты проходят, кроме первого, я даже сделал, если (! (s.equals ("")))
Jakkie Chan 17 март 2014, в 01:05

0

Здесь один из способов сделать то, что вы хотите, не полагаясь на какие-либо математические трюки. Если длина равна нулю, метод возвращает пустую строку.

public String toString() {
    StringBuilder s = new StringBuilder();

    for (int i = 0; i < length; i++) {
        s.append(i);
        s.append(":");
        s.append(arr[i]);
        if (i < (length - 1)) s.append(", ");
    }

    return s.toString();
}

Gilbert Le Blanc 16 март 2014, в 23:00

0

о, да, умный, вот тактика, которую я не помню, чтобы убрать трейлинг персонажей
Jakkie Chan 17 март 2014, в 01:40

Ещё вопросы

спасибо, что помогло, теперь все мои тесты проходят, кроме первого, я даже сделал, если (! (s.equals ("")))
о, да, умный, вот тактика, которую я не помню, чтобы убрать трейлинг персонажей

Raul Guiu · Accepted Answer · 2014-03-16T23-45-00.000Z

0

Лучший ответ

Это приведет к возникновению java.lang.StringIndexOutOfBoundsException в случае пустой строки, поскольку у вас не будет запятой и пробелом.

s.delete(s.length()-2, s.length()-1);

Raul Guiu 16 март 2014, в 23:45

0

я добавил if (! (s.equals (""))) {s.delete (s.length () - 2, s.length ()); } и до сих пор не работает
Jakkie Chan 17 март 2014, в 00:55
0

Мы можем помочь с тем, что мы можем видеть. Если мы не можем увидеть остальную часть вашего кода, мы не можем помочь. Или, по крайней мере, скажите нам, какое утверждение неверно, и мы можем помочь.
Raul Guiu 17 март 2014, в 01:02
0

только первый сбой, где его пустая строка
Jakkie Chan 17 март 2014, в 01:06
0

StringBuilder b = new StringBuilder (); System.out.println (b.equals ( "")); /// ложно !! Так что это не решит вашу проблему. Вы должны проверить, если s.length ()> 0 вместо! S.equals ("")
Raul Guiu 17 март 2014, в 01:10
0

спасибо, это сработало. дал вам чек
Jakkie Chan 17 март 2014, в 01:14
0

Конечно, вы можете PM
Raul Guiu 17 март 2014, в 01:15
0

Хорошо, этот код: public int indexOf (X s) {for (int i = 0; i <= arr.length-1; i ++) {if (arr [i] .equals (s)) {return i; }} return -1; } не проходит этот тест: @Test public void testIndexOf () {BetterArray <String> b = new BetterArray <String> (); for (int i = 0; i <20; i ++) b.add ("str" + i); assertEquals (0, b.indexOf ("str0")); assertEquals (19, b.indexOf ("str19")); assertEquals (-1, b.indexOf ("не найдено"));
Jakkie Chan 17 март 2014, в 01:18
0

public int indexOf (String s) {for (int i = 0; i <arr.length; i ++) {if (arr [i] .equals (s)) {return i; }} return -1; }
Raul Guiu 17 март 2014, в 01:33
0

ты говоришь сменить Х? потому что X является общим, это приемлемо
Jakkie Chan 17 март 2014, в 01:36
0

Х в порядке, извините. Я пытался на месте и работал, как я написал для меня.
Raul Guiu 17 март 2014, в 01:39

Показать ещё 8 комментариев