Всем привет, подскажите, пожалуйста, вроде вывод верный, что не так?

большое спасибо, через пару часов доберусь до ноута и попробую

19? с n/2 тоже не проходит

🙈 правильный ответ n+1:

0, 1, 2, ... , n

То есть ты считываешь на 1 байт больше, чем требуется. Кроме того, вот эти вложенные циклы работают не так, как ты ожидаешь:

while (stream.available() != 0) {
    for (int i = 0; i <= n; i++) {
        list.add((char)stream.read());
    }
    break;
}

Ты проверяешь, что изначально в стриме есть байты, доступные для чтения, и сразу пытаешься прочитать из него все байты, которые тебе нужны (на 1 больше, чем нужно 😉). Но при этом ты не проверяешь, не закончились ли доступные байты в процессе чтения, потому что у тебя проверка не внутри цикла чтения (for), а снаружи, в while. Но в while эта проверка выполняется только один раз, перед началом, а потом всё равно выполняется break, и while по сути у тебя работает как if, потому что всегда содержит только одну итерацию. Если ты хочешь совместить проверки, то во-первых можно убрать бесполезный while, а во-вторых перенести проверку внутрь for:

for (int i = 0; i < n && stream.available() > 0; i++) {
    // read byte;
}

Либо сделать две проверки отдельно, будет чуть более громоздко, но проще читаться:

for (int i = 0; i < n; i++) {
    if (stream.available() == 0) {
        break;
    }
    // read byte;
}

спасибо большое за объяснение, я поняла :) а про 19 - я ответила, так как там n = 38, символ char равен 2 байтам

Это кстати тоже неправильно. Кодировка UTF-8 не просто двухбайтовая, это кодировка переменной длины. Это означает, что если символ можно закодировать одним байтом, то он будет занимать один байт, а если символ больше одного байта, то тогда для его хранения будет выделяться два байта памяти. Можешь выполнить код с приведёнными мной изменениями и посчитать выведенные символы сама, их будет 38. И в условии прямо написано, что для проверки будет использоваться строка, в которой каждый символ кодируется одним байтом:

Считаем, что каждый байт соответствует одному символу.

Поэтому вообще говоря стоит либо убрать кодировку вообще из метода getBytes() и работать с дефолтной, либо, если надо было действительно учитывать кодировку, то тогда надо было бы передавать её третьим параметром внутрь метода getBytes и преобразовывать байты в символы не прямым приведением (char) b, а превращать байты в строку с использованием этой кодировки: new String(byte[], Charset), но это уже был бы другой метод и другая задача. Кстати, вообще есть более удобные способы для чтения из стрима, чем цикл с чтением по одному байту. Во-первых, если мы примем, что количество байтов равно минимуму между их изначальным количеством и n, то мы сразу можем создать массив нужной длины и прочитать в него. Это будет гораздо экономичнее, чем чтение в список, где тебе приходится для каждого байта создавать отдельный объект-обёртку, и работать будет быстрее:

public static char[] bytesToChars(ByteArrayInputStream stream, int n) {
	int length = Math.min(stream.available(), n);
	byte[] bytes = new byte[length];
	stream.read(bytes, 0, length);
	return new String(bytes).toCharArray();
}

А ещё, я не знаю, какая jdk используется в этом квесте, но начиная с jdk 11 у стрима есть более удобный метод чтения нужного числа байт, который так и называется readNBytes. Правда, он выбрасывает проверяемое исключение, поэтому его нужно оборачивать в try-catch:

public static char[] bytesToChars(ByteArrayInputStream stream, int n) {
    try {
        return new String(stream.readNBytes(n)).toCharArray();
    } catch (IOException e) {
        throw new RuntimeException(e);
    }
}