안녕하세요. Java Scanner에 대해 궁금합니다.

Question

안녕하세요. 제가 인터넷에서 검색해본 바로는 Scanner를 사용후 닫아야만한다, 또는 안닫는게 낫다라고 의견이 분분하더군요.. 지금은 System.in을 통해서만 사용해봤습니다. 닫아주지 않아도 문제가 없는것같구요. 근데 추후에 메모리 할당문제 등 때문에라던지, 무엇이라도 좋은점이 있다면 close를 통해서닫아야할 경우를 파악해서 닫아주는 습관을 들이고 싶어서 이렇게 질문남깁니다. 1. Scanner 사용 후 close를 통해 닫아줘야 하는지? 2. 이점이 어떤것이 있는지? 3. 어느 타이밍에 닫아줘야하는지 (메소드 안에서 Scanner를 사용하고 close로 닫아주면 다른 메소드에서 쓸 수 없다더군요.. 어느 선까지 사용하고 닫으라는건지 이해가 안갑니다.. 그래서 아래 예제에서는 어떤 분 말씀대로 모든 메소드를 사용하고 닫으려고 클래스 맨 하위에 작성했는데 오류가 뜹니다)궁금합니다.. package condition; import java.util.Scanner; public class Condition { Scanner sc = new Scanner(System.in); public void practice1(){ System.out.println("1. 입력"); System.out.println("2. 수정"); System.out.println("3. 조회"); System.out.println("4. 삭제"); System.out.println("9. 종료"); System.out.print("메뉴 번호를 입력하세요 : "); int num = sc.nextInt(); String menu = ""; switch(num) { case 1 : menu = "입력"; break; case 2 : menu = "수정"; break; case 3 : menu = "조회"; break; case 4 : menu = "삭제"; break; case 9 : menu = "종료"; break; } if(menu.equals("종료")) { System.out.println("프로그램이 종료됩니다."); } else { System.out.println(menu + " 메뉴입니다."); } } public void practice2(){ System.out.print("숫자를 한 개 입력하세요 : "); int num = sc.nextInt(); String msg = ""; if(num <= 0) { msg = "양수만 입력해주세요."; } else if (num%2 == 0) { msg = "짝수다"; } else { msg = "홀수다"; } System.out.println(msg); } public void practice3(){ System.out.print("국어 : "); int kor = sc.nextInt(); System.out.print("영어 : "); int eng = sc.nextInt(); System.out.print("수학 : "); int math = sc.nextInt(); int sum = kor + eng + math; double avg = sum / 3.0; if(kor >= 40 && eng >= 40 && math >= 40 && avg >= 60) { System.out.println("국어 : " + kor); System.out.println("영어 : " + eng); System.out.println("수학 : " + math); System.out.println("합계 : " + sum); System.out.printf("평균 : %.1f", avg); System.out.println(); System.out.println("축하합니다, 합격입니다!"); } else { System.out.println("불합격입니다."); } } public void practice4(){ System.out.print("월 입력 : "); int month = sc.nextInt(); String season = ""; switch(month) { case 1: case 2: case 12: season = "겨울"; break; case 3: case 4: case 5: season = "봄"; break; case 6: case 7: case 8: season = "여름"; break; case 9: case 10: case 11: season = "가을"; break; default : season = "해당하는 계절이 없습니다."; break; } if(month >= 1 && month <= 12) { System.out.println(month + "월은 " + season + "입니다."); } else { System.out.println(season); } } public void practice5(){ String userId = "wkddkfka1211"; String userPwd = "dldbals12#"; System.out.print("아이디 : "); String id = sc.nextLine(); System.out.print("비밀번호 : "); String pwd = sc.nextLine(); String loginState = ""; if(userId.equals(id) && userPwd.equals(pwd)) { loginState = "로그인 성공"; } else if (userPwd.equals(pwd)) { loginState = "아이디가 틀렸습니다."; } else { loginState = "비밀번호가 틀렸습니다."; } System.out.println(loginState); } public void practice6(){ } sc.close(); }

연습용더미1 · Answer

스캐너 close를 위한 메소드를 만들어서 그 안에서 스캐너를 닫으면 됩니다. 스캐너 다 썼을 때 그 메소드를 불러와서 close를 하게 하면 되죠. 저 상황에서는 practice6에다 sc.close(); 를 넣어도 되죠.

winfly · Answer

1. Scanner 사용 후 close를 통해 닫아줘야 하는지? 2. 이점이 어떤것이 있는지? 3. 어느 타이밍에 닫아줘야하는지 (메소드 안에서 Scanner를 사용하고 close로 닫아주면 다른 메소드에서 쓸 수 없다더군요.. 어느 선까지 사용하고 닫으라는건지 이해가 안갑니다.. 그래서 아래 예제에서는 어떤 분 말씀대로 모든 메소드를 사용하고 닫으려고 클래스 맨 하위에 작성했는데 오류가 뜹니다)&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;위처럼 질문 주셨는데 할 거 없는 학생이지나가다가 아는 대로 답변 해 보겠습니다. 1. 닫을 필요는 없어 보입니다. 이유는 System.in으로 외부로부터입력을 받는데 스트림을 이용해 입력을 받지만 거의 모든 입력에 스트림 인스턴스 사용 후 실제 닫을 필요가 없다는 공식 사이트 글을 봤습니다.리소스가 IO 채널일 때(외부 네트워크, 파일 등)만 스트림을 닫아주면 된다고 합니다.아래는 공식 문서에서 관련 내용을 가져와 봤습니다.Streams have a BaseStream.close() method and implement AutoCloseable, but nearly all stream instances do not actually need to be closed after use. Generally, only streams whose source is an IO channel (such as those returned by Files.lines(Path, Charset)) will require closing. Most streams are backed by collections, arrays, or generating functions, which require no special resource management. (If a stream does require closing, it can be declared as a resource in a try-with-resources statement.) 2. 일단 파일 리소스의 경우 하나의 스레드가 스트림을 열어 사용하고 있다면다른 스레드는 접근할 수 없게 되는 경우가 있습니다. 또한 connection fool 이 다 차서 더이상 접근이 불가한 경우도 있습니다(DB의 경우)위 같은 상황을 방지하기 위해 닫아주는게 아닐까 라는 생각이 듭니다. 제가 부족해 이 부분의 상세한 내용은 여기 참고해보시면 좋을 듯 합니다. https://madplay.github.io/post/mistakes-when-using-java-streams 3. 일단 main 메서드 없이 클래스에 메서드를 만들어 놓으신 걸 보니외부 main 메서드에서 인스턴스를 생성, 참조해 사용하시는 듯 합니다.하지만 Scanner 객체를 모든 메서드에서 공통으로 사용할 수 있도록 멤버변수로 생성해 놓으셨습니다. 그렇기 때문에 Scanner 인스턴스를사용하는 메서드의 모든 실행이 종료된 후 Scanner를 닫아야 할 거 같습니다.정확히는 System.in을 닫는 것입니다. Scanner 객체는 입출력에 대한 처리를System.in을 통해 진행하기 때문입니다.다시 돌아와서 정상적으로 Scanner를 닫아 마무리 하고 싶으시다면 public void scannerClose() { sc.close(); }이렇게 메서드를 만들어 맨 main 메서드 맨 밑에(모든 메서드의 실행이 끝난 후)호출해 Scanner를 닫아주시는게 어떨까 생각됩니다!&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;제가 정답이 아닙니다. 틀렸을 수 있으니 다른 분들의 의견도참고하고, 검색을 통해 문제 해결 하시기 바랍니다.또한 제가 틀린 부분이 있다면 알려주시면 감사하겠습니다!

winfly · Answer

Condition condition &#61; new Condition(); condition.practice1(); condition.practice2(); condition.practice3(); condition.practice4(); condition.practice5(); condition.scannerClose(); // 이런식으로 만들어 사용해 봤습니다.모든 메서드(practice 1 ~ 5)에서 Scanner 인스턴스를 사용하고 있다 보니그 전에 스트림을 닫아 버리면 외부 입력을 읽어오지 못해 에러가 난 거 같습니다.이렇게 실행 흐름의 맨 마지막에 sc.close(); 를 호출해 실행하는 scannerClose() 를만들어 호출하면 모든 실행 흐름이 종료된 후 스트림을 닫고 종료할 수 있을거 같습니다.감사합니다!

오늘도5분 · Answer

&#64;winfly먼저 정말 친절한 답변 감사드립니다. 입문자 입장에서 'Scanner를 닫아주지 않아도 되지만, 닫아주기를 권장한다.' 또는 많은 구글 검색 시Scanner 사용 후 close()를 통해 닫아주어야 한다라는게 보여서 어떤게 맞고, 이점이 있는지 알아 본 후 그에 맞게 사용해야겠다는 생각을 했습니다. 하지만 생각보다 알아보기가 쉽지 않았습니다. 반면 winfly님은 여러가지 문서와 git 등을 살펴보시는게 학생이 아니라 이미 개발자 같습니다..! 그모습이 멋지시고 저도 winfly님을 본받아 그럴 수 있도록 하고 싶네요! 아직 저는 git이나 문서를 보진 못했고,구글링정도로 찾아보려는 노력밖에 해보지 못했네요 ㅠ 혹시 꼭 답변을 안해주셔도 되지만 추가적으로 몇가지 궁금증이 생겼는데, 한번 이야기해보겠습니다.1. Scanner 객체를 class변수로 선언하는 것 보다 지역변수로 선언하는게 좋은거 아닐까? 라는 생각입니다. 보통 공통으로 사용하는 것을 class안에 멤버변수(?)로 빼서 사용하 던데 Scanner도 그렇게 사용하는 경우를 봤습니다. 이 경우 사용의 편리성은 있겠지만 class변수로 사용 시, nextInt가 사용된 메소드 호출 후 nextLine이 사용된 메소드를 호출하게되면 enter 때문에 오류가 생기게됩니다. 이 때, nextLine을 사용하는 모든 메소드에 한번 더 nextLine을 사용해줄수 있지만 프로그래밍을 잘 모르는 제 입장에서는 비효율적 or 까먹을 수 있지 않을까 하는 우려가 듭니다. 또 지역변수로 Scanner 객체를 메소드 안에서 호출하게된다면 따로 close를 해주지 않아도 메소드가 끝나면 객체가 사라지므로(?) close나 관리를 해줄필요가 없는것이 아닌가하는 생각이 듭니다. 사실 정확히 모르는 상태라 틀린말이 많을 수 있긴 하지만 이런생각이 들었고, scanner를 메소드마다 중복 사용 시 class의 멤버변수로 객체를 선언해 쓰는게 맞을지, 지역변수로 메소드마다 선언해 사용하는게 맞을지 궁금했습니다. 2. 지역변수로 Scanner를 선언 시, 각 메소드마다 sc.close를 기재해둔다면 main class에서 2개이상의 메소드를 호출 시 두번째 메소드부터 에러가 뜹니다. 이게 댓글로 말씀해주신 System.in 스트림을 닫아버려서 더이상 사용할 수 없다는 의미 같습니다. 그렇다면 말씀해주신 것을 잘 이해한건지 모르겠지만 main쪽 or Scanner 메소드 사용이 모두 끝난 후 더이상 사용할 일이 없을때, 마지막 사용한 클래스에 close()를 기재하는게 적절하다고 말씀하신건지 궁금합니다. 한번 저도 더 검색을 해보고 있겠습니다. 정확하지 않더라도 생각을 말씀해주시면 큰 도움이 될것같습니다. 도와주셔서 감사합니다!

오늘도5분 · Answer

&#64;연습용더미1고맙습니다. 해당 방법을 참고해서 사용해보도록 하겠습니다. 좋은하루되세요!

winfly · Answer

제가 오늘 지지리 공부가 하기 싫은 관계로 부족 하나마답변 달아보도록 하겠습니다. 절대 제가 맞다고 생각하지 마시고꼭 검증 해보고 이해 안되는 부분은 또 말씀 부탁 드리겠습니다.&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;첫 번째 질문을 두 가지로 나눠 봤습니다.1 - 1. Scanner 객체를 지역 변수로 사용 하는 건 어떨지.1 - 2. nextInt() 후 nextLine() 시 공백 문제점. 그럼 답변 드리겠습니다.1 - 1. Scanner 객체를 일일이 지역변수에 선언해 사용하는 것은 좋은 방법은 아니라 생각됩니다.첫 번째 이유는 코드의 중복 입니다. 코드의 중복은 유지 보수를 힘들게 합니다.물론 지금 이 코드에서 유지 보수를 논하기는 힘듭니다. 하지만 이는 습관의 문제로어떤 경우에서든 코드의 중복은 최대한 피하는 쪽으로 코딩하는 습관을 들이는 것을 추천 드립니다.정말 어쩔 수 없을 땐 그렇게 진행을 하지만 지금과 같은 상황에서는 전역 변수에 Scanner 인스턴스를 생성 하고거기서 발생하는 다른 문제를 해결하는 것이 더 좋을 듯 합니다.이는 우리가 변수를 사용하는 이유와 같습니다. 그냥 값을 사용해도 되지만계속 중복 해서 값을 입력해줘야 하며, 같은 값에 대한 변경이 필요할 때사용된 곳을 일일이 찾아 다니면서 변경해야 하는 번거로움 때문에 변수를 선언해 사용하듯이 Scanner 도 이런 부분에서 전역변수로 생성해사용하는 것이 좋을 거 같습니다. 1 - 2. 이 부분은 nextInt() 대신 int 값을 nextLine() 으로 입력을 받고바로 Integer.parseInt(); 를 사용해 int 값으로 변환해 주면 해결될 듯 합니다.이렇게 되면 모든 입력을 nextLine() 으로 받을 수 있어 일관성이 생기며int, double 등의 값이 필요할 때만 타입을 변경해 사용하면 된다 라는규칙성을 가지기 때문에 추후 프로그래밍에도 더 편할 거 같습니다. public void example() { int value &#61; Integer.parseInt(sc.nextLine()); }위 같이 사용할 수 있을 듯 합니다. 2. 네, 이해하신 부분이 맞습니다. 조금 덧붙이자면 전역 변수로 Scanner 인스턴스를생성해 사용하기 때문에 Scanner 인스턴스가 사용 되는 모든 메서드의실행이 완료된 후 닫아주는 것이 맞다고 생각 됩니다. 만약 전역변수가 아닌 지역변수로 사용 하신다면 그 때 그 때 닫아도 되겠지요.하지만 위 1 - 1 에서도 말씀 드렸듯 객체 생성 코드와 Scanner 를 닫는 코드가 모든 메서드에 중복 되므로 가독성도 떨어지고 유지 보수도 힘들어 집니다.&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;&#61;충분한 답변이 됐을 지 모르겠습니다. 부족한 부분 or 이해 안되시거나, 궁금한 점 있으시다면언제든 말씀 주세요! 이와는 별개로 정말 제대로 공부하고 계신 듯 합니다.계속 이렇게 공부하신다면 분명 빠르게 성장 하실꺼에요! 화이팅 입니다!

오늘도5분 · Answer

&#64;winfly친절한 답변 정말 감사합니다. 앞으로 열심히 해서 좋은 개발자가 되겠습니다..! 좋은하루 되시고 좋은일 많으시길 바랄게요. 화이팅입니다..!!

안녕하세요. Java Scanner에 대해 궁금합니다.

7개의 답변

Q&A 목록