ストリーム内、マップなし、キーと値のペアを使用せずにテキスト内のチャート周波数をカウントして出力する

Question

テキスト内のすべての文字の頻度を数えて印刷しようとしています。 ArrayまたはArrayListを使用し、キーと値のペアを持つマップは使用しないでください。

以下のコードは好ましい結果を示します。 ループを取り除きたいmain countLetters()のすべての作業を行います。

明確にするには、forループやif文を使用しない機能的な方法でこれを実行したいと思います。 これはできますか？そしてもしそうなら、どうですか？

 
public class LetterCounter4 {
 

public static void main(String[] a) { 

    System.out.print("Input text > "); 

    int[] res = countLetters(); 

    for (int i = 0; i < res.length; i++) { 
     if(res[i] != 0){ 
      System.out.println((char) ('a' + i) + " appears " 
       + res[i] + ((res[i] == 1 ? " time" : " times"))); 
     } 
    } 
} 

private static int[] countLetters() { 


    return Arrays.stream(new Scanner(System.in).nextLine().toLowerCase() 
      .split("")) 
      .map(s -> s.charAt(0)) 
      .filter(Character::isLetter) 
      .collect(Collector.of(
        () -> new ArrayList<Integer>(Collections.nCopies(26, 0)), 
        (li, el) -> { 
         Integer oInt = li.get(el - 'a'); 
         li.set(el - 'a', ++oInt); 
        }, 
        (result1, result2) -> { 
         for (int i = 0; i < result1.size(); i++) { 
          Integer temp = result1.get(i); 
          result1.set(i, temp + result2.get(i)); 
         } 
         return result1; 
        })) 
      .stream() 
      .mapToInt(Integer::intValue) 
      .toArray();  
    } 
} 

Answer 1

を求めているものであるサンプルコードの希望です。誰かがÀまたは[a-zA-Z]範囲外の他の文字を入力すると、コードは簡単になり、爆発しません。しかし、それはMapを作成します。これはあなたが望んでいないと言います。

あなたが特に配列やリストに固執したい場合は、ここでそれを行う方法です：

public static void main(String[] args) { 
    String input = new Scanner(System.in).nextLine(); 

    int[] counts = countLetters(input); 

    IntStream.range(0, counts.length) 
      .filter(i -> counts[i] > 0) 
      .forEachOrdered(i -> System.out.printf("%c appears %s %s%n", 
        'a' + i, 
        counts[i], 
        counts[i] > 1 ? "times" : "time" 
      )); 
} 

public static int[] countLetters(String s) { 
    return s.chars() // this is better than stream(split("")) 
      .filter(Character::isLetter) // WRONG to assume that all letters are [a-zA-Z] 
      .map(chr -> Character.toLowerCase(chr) - 'a') 
      .collect(
        () -> new int[26], 
        (ary, i) -> ary[i]++, 
        (a,b) -> Arrays.setAll(a, i -> a[i] + b[i]) 
      ); 
} 

Answer 2

以下は、これはあなたが簡単な方法は、.groupingBy(c -> c, Collectors.counting())を使用することです

import java.util.Arrays; 
import java.util.Scanner; 
import java.util.stream.Collectors; 
import java.util.stream.Stream; 

public class LetterCounter4 { 


    public static void main(String[] a) { 

    System.out.print("Input text > "); 


    countLetters(); 

} 

private static void countLetters() { 
    try{ 
     Arrays.stream(new Scanner(System.in).nextLine().toLowerCase() 
       .split("")) 
    .flatMap(line -> Stream.of(line.split("\\s+"))) 
    .map(String::toLowerCase) 
    .collect(Collectors.toMap(word -> word, word -> 1, Integer::sum)) 
    .entrySet() 
    .stream() 
    .sorted((a, b) -> a.getValue() == b.getValue() ? a.getKey().compareTo(b.getKey()) : b.getValue() - a.getValue()) 
    .forEach(System.out::println); 
    }catch(Exception e){ 
     e.printStackTrace(); 
    } 

} 
}