В чём может быть проблема?

Если в 20-й строке заменить

String addString = e.getCause().getClass().getCanonicalName() + ": " + e.getCause().getMessage();

на

e.toString()

То вывод для Other тоже не будет соответствовать шаблону из условия. Из условия должно быть:

RuntimeException : java.lang.StringIndexOutOfBoundsException: String index out of range: -1 : 3#

c e.toString будет

RuntimeException : java.lang.RuntimeException: java.lang.StringIndexOutOfBoundsException: String index out of range: -1 : 3#

Зачем? Ведь проблемы только с First и Second, с Other проблем нет.

Да мне кажется, везде эти навороты лишние. Тебе уже дана строка для форматирования

final String string = "%s : %s : %s";

Надо в нее подставить подходящие значения. И ничего к ней не надо прибавлять. Что-то вроде:

return String.format(string,
        t.getName(),
        e.getClass().getSimpleName(),
        e.getCause().toString());

И всё.

Наворотов там нет. Дополнительная строка из-за того, что, кроме необходимости учитывать пробелы по сторонам от двоеточий, строка для форматирования "%s : %s : %s" создана для 3-х переменных. В заданном шаблоне требуется вывести больше данных. Например для Second:

java.lang.StringIndexOutOfBoundsException: String index out of range: -1 : StringForSecondThreadTooShortException : 2#

а) java.lang.StringIndexOutOfBoundsException - выводит конструкция e.getCause().getClass().getCanonicalName() б) String index out of range: -1 - выводит конструкция e.getCause().getMessage() в) StringForSecondThreadTooShortException - выводит конструкция e.getClass().getSimpleName() г) 2# - t.getName() Не пойму, почему не принимается решение.

Шаблон не случайно задан на 3 элемента - потому что все эти строки можно разделить как раз на 3 части. "java.lang.StringIndexOutOfBoundsException: String index out of range: -1" - вполне может быть одной частью, а не двумя кусками - e.getCause.toString()

Так-то оно так, но в этом варианте появляются лишние пробелы. Код

protected String getFormattedStringForSecondThread(Thread t, Throwable e, String string) {
//  java.lang.StringIndexOutOfBoundsException: String index out of range: -1 : StringForSecondThreadTooShortException : 2#
        String result = String.format(string,
                e.getCause().toString(),
                e.getClass().getSimpleName(),
                t.getName());
        return result;

даёт результат:

java.lang.StringIndexOutOfBoundsException:  String index out of range: -1 : StringForSecondThreadTooShortException : 2#

а нужно

//  java.lang.StringIndexOutOfBoundsException: String index out of range: -1 : StringForSecondThreadTooShortException : 2#

Появился лишний пробел между ...Exception: и String Поэтому изначально и был костыль в виде добавочной строки. Или твой вариант может быть без лишнего пробела?

Ты так говоришь, как будто этот пробел с потолка взялся, а не ты сам своими руками его прописал в getCause(). Да и в принципе не очень-то правильно хардкодить getCause() вместо того, чтобы перехваченный эксепшн передать в конструктор твоих ...TooShortException

public StringForFirstThreadTooShortException(Exception e) {
    super(e);
}

Тогда и переопределять ничего уже не надо будет, getCause и так вернет то, что нужно. Поэтому-то у тебя Other проходит нормально, а First и Second - нет.

Не совсем понимаю, где я этот пробел мог сам прописать?

@Override
public synchronized Throwable getCause() {
    return new StringIndexOutOfBoundsException(" String index out of range: -1");
}

Вот этого в задаче не было, это ты сам написал. И в начале строки лишний пробел. Но даже если бы его не было - подход неверный.

Точно. Не заметил этого пробела. Спасибо. Забыл о переопределении этого метода. Задача просто мутная. Код исправил, а проблема осталась. Создал новую тему.