인프런 커뮤니티 질문&답변
작성한 질문수
미리 풀어 봤는데, 이 코드는 그다지 좋은 코드는 아니려나요?
해결된 질문
작성
·
111
·
수정됨
0
import java.util.*;
public class Main {
public void solution(String str) {
String upper = str.toUpperCase();
for (int i = 0; i<str.length(); i++) {
if (str.charAt(i) != upper.charAt(i)) {
System.out.print(upper.charAt(i));
} else {
System.out.print(Character.toLowerCase(str.charAt(i)));
}
}
}
public static void main(String[] args) {
Main main = new Main();
Scanner sc = new Scanner(System.in);
String str = sc.next();
main.solution(str);
}
}정답 처리는 됐습니다.
저는 solution() 메서드의 리턴 타입을 void로 두고, 이 메서드 내부에서 한 글자씩 print()로 출력했습니다.
이런 접근은 안 좋으려나요?
답변 2
1
0
안녕하세요, 인프런 AI 인턴입니다. 11 1님.
질문하신 코드는 입력 문자열의 각 문자를 대문자와 비교하여 소문자로 변환하거나 그대로 출력하는 방식으로 구현되어 있습니다. 전체적으로 동작은 하도록 작성되었으나, 더 깔끔하고 효율적인 코드로 개선할 수 있는 여지가 있습니다.
특히 String의 toUpperCase 메서드를 호출하여 대문자 변환을 두 번 수행하는 것은 불필요한 반복 작업이 됩니다. 이를 피하고 한 번의 변환 만으로 원하는 결과를 얻을 수 있습니다. Character.toLowerCase()와 Character.toUpperCase()를 적절히 사용하여, 불필요한 문자열 변환을 줄이는 방식으로 개선할 수 있습니다.
만약 이와 관련하여 더 많은 정보를 원하시거나, 다른 유사한 질문에 대한 해결 방법을 참고하시려면 아래의 링크를 확인하실 수 있습니다:
- replaceAll을 이용해서 풀어도 괜찮을까요??: 이 질문에서는 문자열 처리를 효율적으로 할 수 있는 방법에 대한 다른 접근 방식을 논의하였습니다.
질문 주신 코드의 개선 포인트는 다음과 같습니다:
1. 불필요한 변환을 줄이는 방법으로 Character 클래스를 사용하는 것이 보다 코드의 명확성을 높이면서 효율성을 가져올 수 있습니다.
2. 코드의 효율성을 고려하여 불필요한 반복 작업을 줄이도록 한다면, 더 좋은 코드로 개선될 것입니다.
저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.
답변 감사합니다.