미리 풀어 봤는데, 이 코드는 그다지 좋은 코드는 아니려나요?

Question

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()로 출력했습니다.   이런 접근은 안 좋으려나요?

김태원 · Answer

안녕하세요^^ 코딩테스트 시험 플랫폼의 스타일이 solution() 메서드에서 답을 리턴하는 방식이니 리턴하는 방식으로 연습하는게 좋습니다.

인프런 AI 인턴 · Answer

안녕하세요, 인프런 AI 인턴입니다. 11 1님. 질문하신 코드는 입력 문자열의 각 문자를 대문자와 비교하여 소문자로 변환하거나 그대로 출력하는 방식으로 구현되어 있습니다. 전체적으로 동작은 하도록 작성되었으나, 더 깔끔하고 효율적인 코드로 개선할 수 있는 여지가 있습니다. 특히 String 의 toUpperCase 메서드를 호출하여 대문자 변환을 두 번 수행하는 것은 불필요한 반복 작업이 됩니다. 이를 피하고 한 번의 변환 만으로 원하는 결과를 얻을 수 있습니다. Character.toLowerCase() 와 Character.toUpperCase() 를 적절히 사용하여, 불필요한 문자열 변환을 줄이는 방식으로 개선할 수 있습니다. 만약 이와 관련하여 더 많은 정보를 원하시거나, 다른 유사한 질문에 대한 해결 방법을 참고하시려면 아래의 링크를 확인하실 수 있습니다: replaceAll을 이용해서 풀어도 괜찮을까요?? : 이 질문에서는 문자열 처리를 효율적으로 할 수 있는 방법에 대한 다른 접근 방식을 논의하였습니다. 질문 주신 코드의 개선 포인트는 다음과 같습니다: 1. 불필요한 변환을 줄이는 방법으로 Character 클래스를 사용하는 것이 보다 코드의 명확성을 높이면서 효율성을 가져올 수 있습니다. 2. 코드의 효율성을 고려하여 불필요한 반복 작업을 줄이도록 한다면, 더 좋은 코드로 개선될 것입니다. 저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다. 현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏 추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.