스트림, 맵, 키 - 값 쌍을 사용하지 않고 텍스트의 문자 빈도 계산 및 인쇄

Question

텍스트의 모든 문자 빈도를 계산하여 인쇄하려고합니다. Array 또는 ArrayList을 사용하고 키 - 값 쌍이있는지도가 필요하지 않습니다.

아래 코드는 바람직한 결과를 제공합니다. for 루프를 제거하고 싶습니다. 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(); 
    } 

} 
}