Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1주차 Assignment - 정다연 #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

firefox1234123
Copy link

Description

<!— 구현 및 작업 내용을 적어주세요 —>

  • java 기초 문제 풀이
  • Test1.java 완성

Important content

<!— 주의 깊게 봐줬으면 하는 부분을 적어주세요 —>

  • 없습니다

Question

<!— 궁금한 점을 적어주세요 —>

  • 2588번 문제를 println을 여러줄 작성하는 게 아닌, 반복문으로 처리할 수 있는 방법이 있나 궁금합니다!

Reference

<!— 참고한 레퍼런스가 있다면 공유해 주세요 —>

  • 없습니다

@shinheekim shinheekim changed the title [1주차 Assignment - 정다연] 1주차 Assignment - 정다 Mar 31, 2024
@shinheekim shinheekim changed the title 1주차 Assignment - 정다 1주차 Assignment - 정다연 Mar 31, 2024
Copy link

@djdongjae djdongjae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전반적으로 고민한 흔적이 보이는 과제였습니다! 고생하셨어요~ 과제 통과입니다.

class Cylinder {

// 원주율 3.14를 정적 상수 PI로 선언과 동시에 초기화
final static double PI = 3.14;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상수 선언 잘하셨습니다..! java에서 상수는 public static final 키워드를 붙히면 됩니다!

public class Main {
public static void main(String[] args) {
Scanner scanner = new Scanner(System.in);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

더 빠른 입력을 위해 Scanner 대신 BufferedReader를 사용해보는 것도 추천드려요!

자세한 사용법은 링크 남겨둡니다: https://rlakuku-program.tistory.com/33

}
}
for (int i = 0; i < basketArr.length; i++) {
System.out.print(basketArr[i]+" ");

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

배열의 인덱스를 고려해서 잘 푸셨습니다!
팁 하나를 드리자면 많은 양의 정보를 System.out.print()로 반복 호출하여 출력하는 것은 효율적이지 않을 수 있습니다. 이보다 StringBuilder를 사용해서 정답 문자열을 먼저 생성하고 마지막에 딱 한 번 System.out.print()를 호출하면 좋을 것 같습니다!

StringBuilder 사용법도 위 링크에 같이 있습니다..!

int caseNum = scanner.nextInt();

int[] numArray = new int[caseNum * 2];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

numA와 numB을 입력으로 받지고 않고 배열을 이용해서 입력 값을 저장하여 처리한 이유가 있을까요?
위에 선언한 변수가 밑에서는 사용되고 있지 않네요...!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

처음 코드를 작성할 때 numA와 numB에 각각 값을 받아와 addAB = numA + numB 식으로 처리하려 했습니다. 추후에 배열을 이용하며 충분히 처리할 수 있어 사용하지 않았는데, 미처 삭제하지 못했습니다...!

package bj_2439;

import java.util.Scanner;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

헷갈렸을 수도 있는데 잘 푸셨네요!


int caseNum = scanner.nextInt();
String[] strArray = new String[caseNum+1];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nextInt 다음에 nextLine을 사용하게 되면 개행 문자(엔터 값)가 nextLine으로 전달되어 nextLine이 제대로 동작하지 않습니다. 지금은 caseNum의 값을 1 증가 시켜서 개행 문자를 한 번 씹어주고 인덱스 1부터 저장되게끔 했는데 그것보다 nextInt() 다음에 nextLine을 사용해야 하는 경우에는 nextLine()을 한 번 더 써서 버퍼를 비워주는 것을 추천 드립니다.

int caseNum = scanner.nextInt();
scanner.nextLine() // 버퍼 비우기

@djdongjae djdongjae added the ✅ 과제 통과 조지미에게 확인을 받은 후의 상태입니다. label Apr 1, 2024
@giwoong01 giwoong01 closed this Jul 9, 2024
@giwoong01 giwoong01 reopened this Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✅ 과제 통과 조지미에게 확인을 받은 후의 상태입니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants