Last Requirement not satisfied

Question about the task

Vowels and consonants

Java Syntax, Level 9, Lesson 11

Under discussion

Any ideas what could be the reason of this bug? I have added the spaces at the end of Strings.

Write a program that reads a string from the keyboard.
The program should display two lines:
1. The first line contains only the vowels from the entered string.
2. The second contains only the consonants and punctuation marks from the entered string.
The letters should be separated by a space, each line must end with a space.

Example input:
Sam I Am.

Example output:
a I A S m m .

Requirements:

The program must read data from the keyboard.
The program should display two lines.
The first line should contain only the vowels from the input string, separated by a space.
The second line should contain only the consonants and punctuation marks from the input string, separated by a space.
Each line must end with a space.

package com.codegym.task.task09.task0923; import java.io.BufferedReader; import java.io.InputStreamReader; /* Vowels and consonants */ public class Solution { public static char[] vowels = new char[]{'a', 'e', 'i', 'o', 'u'}; public static BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); public static void main(String[] args) throws Exception { String str = reader.readLine(); char[] temp = str.toCharArray(); String vowels = ""; String notVowels = ""; for(int i = 0;i<temp.length;i++) { if(isVowel(temp[i])){ vowels += String.valueOf(temp[i])+" "; } else if(temp[i]==(' ')){ } else { notVowels += String.valueOf(temp[i])+" "; } } System.out.println(vowels); System.out.println(notVowels); } // The method checks whether a letter is a vowel public static boolean isVowel(char c) { c = Character.toLowerCase(c); // Convert to lowercase for (char d : vowels) // Look for vowels in the array { if (c == d) return true; } return false; } }

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

ddi

Level 18 , Dortmund, Germany

29 July 2019, 07:49

Have you tried to put the buffered reader inside the main method?

Ewerton Backend Developer

10 July 2019, 13:32

Maybe it is that '}' on line 36 ?

Michał Kwaśny

Level 22 , Krakow, Poland

10 July 2019, 13:58

It is not it for sure. This looks like a bug to me because only without the char[] temp = str.toCharArray(); it is passing the last requirement.

Renat Mukhametshin

Level 16 , Pervouralsk, Russain Federation

2 August 2019, 08:38

hmm.. it's really intersting - why it's not passing last test. Your code on the first look haven't any errors. I don't know what's the reason it, but my code passes the all check:

.......
public class Solution {
    public static char[] vowels = new char[]{'a', 'e', 'i', 'o', 'u'};

    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String str = reader.readLine();

        List<Character> vowelsChar = new ArrayList<>();
        List<Character> consonantsChar = new ArrayList<>();

        for(Character c : str.toCharArray()){
            if(isVowel(c)){
                vowelsChar.add(c);
                vowelsChar.add(' ');
            } else if (c != ' ') {
                consonantsChar.add(c);
                consonantsChar.add(' ');
            }
        }

        show(vowelsChar);
        System.out.println();
        show(consonantsChar);

    }

    // The method checks whether a letter is a vowel
    public static boolean isVowel(char c) {
        c = Character.toLowerCase(c);  // Convert to lowercase

        for (char d : vowels)   // Look for vowels in the array
        {
            if (c == d)
                return true;
        }
        return false;
    }

    //The method output list
    public static void show(List<Character> list){
   .......
}