-
Notifications
You must be signed in to change notification settings - Fork 13
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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; |
There was a problem hiding this comment.
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); | ||
|
There was a problem hiding this comment.
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]+" "); |
There was a problem hiding this comment.
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]; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
numA와 numB을 입력으로 받지고 않고 배열을 이용해서 입력 값을 저장하여 처리한 이유가 있을까요?
위에 선언한 변수가 밑에서는 사용되고 있지 않네요...!
There was a problem hiding this comment.
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; | ||
|
There was a problem hiding this comment.
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]; | ||
|
There was a problem hiding this comment.
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() // 버퍼 비우기
Description
<!— 구현 및 작업 내용을 적어주세요 —>
Important content
<!— 주의 깊게 봐줬으면 하는 부분을 적어주세요 —>
Question
<!— 궁금한 점을 적어주세요 —>
Reference
<!— 참고한 레퍼런스가 있다면 공유해 주세요 —>