Результат верный не проходит проверку

Вопрос по задаче

Java Multithreading, 2 уровень, 5 лекция

Решен

Подскажите, почему не проходит проверку данный код, по условию все верно выводит.

public static String getPartOfString(String string) {

      String s = "";
      int count = 0;
          for (int i = string.indexOf(" "); i < string.length() - 1; i++) {
              if(!string.substring(i + 1 , i + 2).equals(" ") && count < 4){
                  s += string.substring(i + 1 , i + 2);
              }else {
                  s += " ";
                  if(string.substring(i + 1, i + 2).equals(" ")){
                      count++;
                  }
              }
           }

      if(count < 4) throw new TooShortStringException();
      return s;

  }

Метод getPartOfString должен возвращать подстроку начиная с символа после 1-го пробела и до конца слова,
которое следует после 4-го пробела.

Пример:
"JavaRush - лучший сервис обучения Java."

Результат:
"- лучший сервис обучения"

Пример:
"Амиго и Диего лучшие друзья!"

Результат:
"и Диего лучшие друзья!"

На некорректные данные бросить исключение TooShortStringException (сделать исключением).

Требования:

Класс TooShortStringException должен быть потомком класса RuntimeException.
Метод getPartOfString должен принимать строку в качестве параметра.
В случае, если в метод getPartOfString были переданы некорректные данные, должно возникнуть исключение TooShortStringException.
Метод getPartOfString должен возвращать подстроку начиная с символа после 1-го пробела и до конца слова, которое следует после 4-го пробела.

package com.javarush.task.task22.task2202; /* Найти подстроку */ public class Solution { public static void main(String[] args) { System.out.println(getPartOfString("JavaRush - лучший сервис обучения Java.")); } public static String getPartOfString(String string) { String s = ""; int count = 0; for (int i = string.indexOf(" "); i < string.length() - 1; i++) { if(!string.substring(i + 1 , i + 2).equals(" ") && count < 4){ s += string.substring(i + 1 , i + 2); }else { s += " "; if(string.substring(i + 1, i + 2).equals(" ")){ count++; } } } if(count < 4) throw new TooShortStringException(); return s; } public static class TooShortStringException extends RuntimeException { } }

Комментарии (6)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Vlad

Уровень 40

18 апреля 2018, 17:07

Исправил метод на такой

public static String getPartOfString(String string) {


        // Считаем количество пробелов
            int spaceCount = 0;
            for(int i = 0; i < string.length(); i++){
                if(string.substring(i, i + 1).equals(" ")){
                    spaceCount++;
                }
            }

            // Разбиваем строку на слова
            String[] s = string.split("\\s+");

            // Кидаем исключение или записываем строку
        String str = "";
        if(spaceCount < 4){
            throw new TooShortStringException();
        }else{
            for(int i = 1; i < s.length; i++){
                if(i < 4) {
                    str += s[i] + " ";
                }else if(i == 4){
                    str += s[i];
                }
            }
        }


            return str;

    }

Вроде все ок, не понятно почему в этот раз не проходит только 1 тест: "В случае, если строка, переданная в метод getPartOfString содержит менее 4 пробелов должно возникнуть исключение TooShortStringException.", хотя ошибку пишет, если меньше 4 пробелов.

Roman Pryshchepa Java Developer

Expert

19 апреля 2018, 10:11

решение

На некорректные данные бросить исключение TooShortStringException (сделать исключением). Некорректные данные это не только строка с меньшим количеством пробелов. Строка может быть, например, null.

Vlad

Уровень 40

27 апреля 2018, 08:54

В условии написано, что "В случае, если строка, переданная в метод getPartOfString содержит менее 4 пробелов должно возникнуть исключение TooShortStringException.", про пустую строку не слова. Но, вот проверка если строка пустая и все равно не проходит.

if(spaceCount < 4 || string == null){
            throw new TooShortStringException();
        }else{
            for(int i = 1; i < s.length; i++){
                if(i < 4) {
                    str += s[i] + " ";
                }else if(i == 4){
                    str += s[i];
                }
            }
        }

Roman Pryshchepa Java Developer

Expert

27 апреля 2018, 09:10

Тебе IDEA должна давать подсказку на выражение:

string == null

Condition always 'false' when reached И попробуй скормить своему методу null на вход, ты получишь TooShortStringException ?

Vlad

Уровень 40

27 апреля 2018, 09:53

Пустую строку отдал, ошибку получил. Но не важно уже, добавил в начало проверку

if(string == null){
           throw new TooShortStringException();
       }

Тогда валидатор принял.

Roman Pryshchepa Java Developer

Expert

18 апреля 2018, 10:23

Как ты проверял? Сравни вывод (вверху твой):

- лучший сервис обучения      |
- лучший сервис обучения|