Output correction

Question about the task

Vowels and consonants

Java Syntax, Level 9, Lesson 11

Resolved

Hi Guys, Can you help out? My current output is: [a, I, A] [S, m, m, .] How can I trim the commas and brackets?

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; import java.util.ArrayList; /* Vowels and consonants */ public class Solution { public static char[] vowels = new char[]{'a', 'e', 'i', 'o', 'u'}; public static void main(String[] args) throws Exception { // write your code here BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String s = reader.readLine(); ArrayList<Character> vowels = new ArrayList<>(); ArrayList<Character> consonantsAndPunctuationMarks = new ArrayList<>(); char[] charString = s.toCharArray(); char space = ' '; for (int i = 0; i < charString.length;i++){ char vowel = charString[i]; if (vowel == space) continue; else if (isVowel(vowel)){vowels.add(vowel);} else {consonantsAndPunctuationMarks.add(vowel);} } System.out.println(vowels); System.out.println(consonantsAndPunctuationMarks); } // 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 (6)

Popular
New
Old

You must be signed in to leave a comment

Heroboy Blue

Level 10

13 December 2019, 03:20

package zh.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 void main(String[] args) throws Exception {
        // write your code here
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String inPutString = reader.readLine();

        char []vowel = new char[100];
        char []others = new char[100];

        int i = 0, j = 0;

        for(int k = 0; k < inPutString.length(); k++){
            if(isVowel(inPutString.charAt(k)) == true){
                vowel[i++] = inPutString.charAt(k);
            }else if(inPutString.charAt(k) != ' '){
                others[j++] = inPutString.charAt(k);
            }
        }

        for(int t = 0; t < i; t++){
            System.out.print(vowel[t] + " ");
        }
        System.out.println();

        for(int t = 0; t < j; t++){
            System.out.print(others[t] + " ");
        }

    }

    // 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;
    }
}

Ivan

Level 22 , Nope, Bulgaria

6 November 2019, 09:53

Guys, false alarm ... was stupid enough to not properly print. Should've done the following prior posting ... : for (int i = 0; i < vowels.size(); i++){ System.out.print(vowels.get(i) + " "); } System.out.println(" "); for (int i = 0; i < consonantsAndPunctuationMarks.size();i++){ System.out.print(consonantsAndPunctuationMarks.get(i) + " "); }

Guadalupe Gagnon

Level 37 , Tampa, United States

6 November 2019, 15:28

good catch

Ivan

Level 22 , Nope, Bulgaria

6 November 2019, 15:37

Thank you :) you are awesome, always helping out! hope on higher levels you'll continue as it will get tougher

MecoS Full Stack Developer at Microsoft

10 December 2019, 12:54

This is my solution. I copy a part of yours. Great Job.

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

        char[] inserted = reader.readLine().toCharArray();
        char space = ' ';
        ArrayList<String>vowelsArr = new ArrayList<>();
        ArrayList<String> consonantArr = new ArrayList<>();
        for (int i = 0; i <inserted.length; i++) {
            if(inserted[i]== space) continue;
            else if(isVowel(inserted[i])){
                vowelsArr.add(String.valueOf(inserted[i]));
            }
            else consonantArr.add(String.valueOf(inserted[i]));
        }
        for (String vov:vowelsArr
             ) {
            System.out.print(vov + " ");
        }
        System.out.print("\n");
        for (String conson:consonantArr
             ) {
            System.out.print(conson  + " ");
        }
    }

Ivan

Level 22 , Nope, Bulgaria

10 December 2019, 15:06

Really glad it helped out! Yes, i started to use fewer temp variables to do the trick and the for each forgot to use here :D