1주차 Assignment - 이승현 #9
Conversation
|
저 제출 된건가욤 ㅠㅠㅠㅠㅠ |
네! 제출은 되었습니다! 과제 확인 후 피드백드리겠습니다~~ |
| int M = scanner.nextInt(); | ||
| int N = scanner.nextInt(); | ||
|
|
||
| int n1, n2, ball = 0; |
There was a problem hiding this comment.
한 줄에 다중 변수를 선언해도 문제는 없지만 추천하지는 않습니다!
shinheekim
added
✅ 과제 통과
LEE SEUNG HYUN/assigns/Test1.java
Outdated
| System.out.printf("%.1f\n", c1.volume()); | ||
| System.out.printf("%.2f\n", c1.surfaceArea()); | ||
|
|
||
| // 객체 생성 |
LEE SEUNG HYUN/assigns/Test1.java
Outdated
|
|
||
| public static final double PI = 3.14;// 원주율 3.14를 정적 상수 PI로 선언과 동시에 초기화 | ||
|
|
||
| int x, y;// 정수형 원의 중심 좌표 x, y를 선언 |
There was a problem hiding this comment.
클래스의 필드를 아래와 같이 private으로 선언하는 것이 객체 지향 프로그래밍의 중요한 원칙 중 하나인 캡슐화를 구현하기 위한 좋은 방법입니다. 참고해주세요!
private int x, y;
private double r;
private int height = 10;
|
|
||
| public class Main { | ||
| public static void main(String[] args) { | ||
| Scanner scanner = new Scanner(System.in); |
There was a problem hiding this comment.
Scanner 잘 사용해주신 거같은데 이 Scanner가 사용하기 쉽지만 대량의 데이터를 입력 받을 때 성능 문제가 생길 수 있습니다. 더 효율적인 코드를 위해서는 BufferedReader를 사용하는 것도 추천드립니다. 아래 블로그 참고해보시면 좋을 거같습니다~!
| int T = scanner.nextInt(); | ||
| int i = 1; | ||
|
|
||
| while (i <= T) { |
There was a problem hiding this comment.
여기서 반복문 잘 사용하셨네요. T의 값을 입력받고 i를 선언 및 초기화를 해준다음 while문 안에서 i가 계속해서 증가하도록 잘 작성해주셨습니다.
| public static void main(String[] args) { | ||
| Scanner scanner = new Scanner(System.in); | ||
|
|
||
| int N = scanner.nextInt(); |
There was a problem hiding this comment.
백준에서는 N으로 변수를 지정하라 했을 수도 있긴 한데 앞으로 변수는 소문자로 작성하고 명시적인 것으로 네이밍하면 좋을 거같아요.
|
|
||
| int array1[] = new int[M]; | ||
|
|
||
| for(int i=0;i<N;i = i + 1) { |
There was a problem hiding this comment.
for 중첩 반복문 잘 활용하셨는데 앞으로 가독성을 위해 for(int i=0;i<N;i = i + 1) 보다는 for (int i = 0; i < N; i = i + 1) 이렇게 작성해주시는 걸 추천드립니다!
Description
<!— 구현 및 작업 내용을 적어주세요 —>
Important content
<!— 주의 깊게 봐줬으면 하는 부분을 적어주세요 —>
Question
<!— 궁금한 점을 적어주세요 —>
Reference
<!— 참고한 레퍼런스가 있다면 공유해 주세요 —>