테스트 케이스 하나가 오답인데 어떤 부분이 문제인지 모르겠습니다ㅠ
201
작성한 질문수 5
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.HashMap;
class Main {
public static void main(String[] args) throws IOException {
Main T = new Main();
BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
String str1 = br.readLine();
String str2 = br.readLine();
System.out.println(T.solution(str1, str2));
}
public static String solution(String str1, String str2) {
HashMap<Character, Integer> map1 = new HashMap<>();
HashMap<Character, Integer> map2 = new HashMap<>();
for (char x : str1.toCharArray()) {
map1.put(x, map1.getOrDefault(x, 0) + 1);
}
for (char x : str2.toCharArray()) {
map2.put(x, map2.getOrDefault(x, 0) + 1);
}
String answer = "";
for (char x : map1.keySet()) {
if (map1.containsKey(x) == map2.containsKey(x)) {
if (map1.get(x) == map2.get(x)) answer = "YES";
else answer = "NO";
}
}
return answer;
}
}테스트케이스 하나를 통과하지 못하는데 제가 생각하지 못한 부분이 있는지 궁금합니다!
코드가 효율적이지 않을 수는 있으나 저렇게 해도 풀리지 않을까? 해서 작성해봤는데.. 뭐가 문제일까요
테스트 케이스
ABCDqtqtqEFqGHIJKLMNOPQRSTUVWetagdgXYabcdefghijklmnopqrstuwxyz
aBcdewrqwtqFghIJklMnOpqrsTuegagaeVxyYAbCDEfGHijKLmNoPQRStUwWXz
답변 1
0
안녕하세요, 인프런 AI 인턴입니다.
tjrwls08088님 코드에서 발생할 수 있는 문제 중 하나는 모든 키에 대해 같은 값을 가지는 경우에만 “YES”가 아닌 경우 “NO”를 바로 리턴하지 않고 계속 루프를 도는 점입니다. 두 문자열이 다르다는 것이 하나의 문자에서라도 확인되면 즉시 “NO”를 반환하고 루프를 종료해야 합니다. 현재 로직에서는 마지막에 확인한 문자가 동일하다면 “YES”로 값을 덮어씌우게 됩니다.
해결책으로는 forEach 루프 내에서 map2.containsKey(x) 체크를 하며, map1과 map2에서 같은 key에 대한 value가 다른 경우 바로 “NO”를 반환하게 수정할 수 있습니다. 모든 키를 확인한 후에 문제가 없으면 “YES”를 반환합니다.
이렇게 코드를 수정해 볼 수 있겠습니다:
public static String solution(String str1, String str2) {
HashMap<Character, Integer> map1 = new HashMap<>();
HashMap<Character, Integer> map2 = new HashMap<>();
for (char x : str1.toCharArray()) {
map1.put(x, map1.getOrDefault(x, 0) + 1);
}
for (char x : str2.toCharArray()) {
map2.put(x, map2.getOrDefault(x, 0) + 1);
}
for (char x : map1.keySet()) {
if (!map2.containsKey(x) || !map1.get(x).equals(map2.get(x))) {
return "NO";
}
}
// 만약 map2에도 map1에 없는 추가 문자가 있다면, "NO" 반환
for (char x : map2.keySet()) {
if (!map1.containsKey(x)) {
return "NO";
}
}
return "YES";
}
map2에도 map1에 없는 키가 있는지도 확인해야 올바른 비교를 할 수 있습니다. 위 코드의 수정을 통해 문제를 해결해보시길 바랍니다.
안녕하세요. 바뀐 채점사이트 관련해서 문의드립니다.
0
27
1
갑자기 채점 사이트가 바뀌었어요
0
32
1
문제 리스트 페이지
0
27
1
채점 사이트 관련 질문드립니다
0
23
1
봉우리 문제 질문입니다
0
81
2
씨름 선수 문제에서 각 선수의 몸무게나 키가 같을 수도 있다면?
0
64
0
이 코드랑 영상 코드중에 뭐가 더 좋은 코드인가요?
0
72
0
가중치 방향 그래프에서 가중치가 0인 간선을 표현하는 방법
0
67
1
좌표 정렬 문제 이 코드가 왜 틀린지 모르겠습니다 ㅠㅠ
0
85
2
6-7 강의에서
0
48
1
6-6. 장난꾸러기 질문 있습니다.
0
45
1
강의 수강후 코딩테스트
0
109
1
answer 변수 사용 여부
0
44
1
2중 for문
1
85
2
2-11. 임시반장정하기 (Runtime Error)
0
63
1
혹시 LinkedList 같은 자료 구조들은 따로 배우지 않나요?
0
70
1
이런 풀이는 어떨까요
0
43
1
자바 스트림 방식의 효율성 질문 드립니다.
0
57
1
알고리즘 자료 구조들..
0
62
1
StringBuilder vs BufferdWriter
0
48
1
원더랜드(프림)
0
50
1
이런 코드는 어떤가요?
0
61
1
bfs 풀이
0
57
1
병합정렬
0
56
1





