close scanner (java)

Hej

Jag har min kod. Där använder jag scannern en del.
Jag har läst nu att folk rekomenderar att man ska stänga scannern med något i den här stilen

input.close();

Jag importerar min scanner
private Scanner input = new Scanner(System.in);

och använder den i flera metoder.
men vart stänger jag scannern?

nja det går inte riktigt att ha en metod där jag bara använder scannern.
jag har ett program som skapar användare, skapar mediaitems(cd,book,journal), gör ändringar på respektive sak, utför lån av items, retunerar lån etc.

så ganska många metoder som utnyttjar scannern.

jag har några metoder som hanterar input av string, int och andra variablar.
till exempel:

	private String readString(String prompt) {
		System.out.print(prompt);
		return input.nextLine();

	}


	private int readInt(String prompt) {

		System.out.print(prompt);
		String in;
		int number = 0;
		boolean correctInput = false;

		while (!correctInput) {
			in = input.nextLine();
			try {
				number = Integer.parseInt(in);
				correctInput = true;
			} catch (NumberFormatException e) {
				correctInput = false;
				System.out.print("Not a number, try again \n> ");
			}
		}
		return number;

	}

vad jag lärt mig är att inte importera scannern mer än en gång per klass, onödigt att repetera kod som jag kan göra en gång.

aha man tackar. då behöver jag inte bry mig om detta i just mitt fall.
och ja jag skrev lite tokigt om det där med att importera scannern....

men är det fel att till exempel lägga till det i en metod såhär?

	private Address readAddress() {
		String streetName = readString("Street name: ");
		int streetNo = readInt("Street Number: ");
		String zipCode = readString("Zip Code: ");
		String city = readString("City: ");
		Address address = new Address(streetName, streetNo, zipCode, city);
		input.close();

		return address;

	}

eller

	private void createReader() {
		String name = readString("Name: ");
		int readerID = readInt("Reader ID: ");
		int birthYear = readInt("Birth Year: ");
		Address address = readAddress();
		PhoneNo phoneNo = readPhoneNo();
		Email email = readEmail();
		Reader theNewReader = new Reader(name, readerID, birthYear, address,
				phoneNo, email);
		library.addReader(theNewReader);
		System.out.println("Reader " + name + " added");
		input.close();

	}

Jag går snart upp för en extra bedömning med min lärare.
Och jag letar sätt att trimma koden så den ser lite bättre ut än vid första tillfället.
Ta bort onödiga saker och hitta nya saker som kan ge ett plus i kanten

okej tack

förstår inte riktigt vad du menar med det du la till (om createReader metoden).

Såhär ser hela koden ut gällande att skapa en reader.

Sju klasser
MainLibraryProgram
Library
ReaderCollection
Reader
Address
PhoneNo
Email

	private void createReader() {
		String name = readString("Name: ");
		int readerID = readInt("Reader ID: ");
		int birthYear = readInt("Birth Year: ");
		Address address = readAddress();
		PhoneNo phoneNo = readPhoneNo();
		Email email = readEmail();
		Reader theNewReader = new Reader(name, readerID, birthYear, address,
				phoneNo, email);
		library.addReader(theNewReader);
		System.out.println("Reader " + name + " added");

	}

	private ReaderCollection readerCollection = new ReaderCollection();


	public void addReader(Reader reader) {
		readerCollection.addReader(reader);

	}

	private List<Reader> readers = new ArrayList<Reader>();


	public void addReader(Reader reader) {
		readers.add(reader);

	}

klasserna Reader, Address, PhoneNo, Email definierar då vad en reader är.

Jag tänker såhär att i LibraryMainProgram
skapar jag en reader (createReader).
sedan lägger jag till reader i listan med (addReader)

menar du att om jag har en metod som definierar hur en ny reader ska se ut (dvs sätter värden till variablar).
retunerr sedan en reader.
så kan jag med en annan metod plocka upp denna reader och göra vad jag vill med... antingen använda till att addReader
eller att använda till något annat om jag kommer på något

ja jag har en loop som printar en meny och en switch för att köra menyn.
scannern skapar jag i början av klassen utanför all metoder.

så du menar att jag skulle stänga scannern i denna metoden?

	private void start() {
		createTestReaders(); //endast för att testa programet och slippa skapa nya varje gång.
		createTestItem();
		System.out.println("---------------Welcome to The Library"
				+ "---------------\n---------Please select your "
				+ "preferd option.--------\n");

		while (true) {
			printMenu();
			handleMenuCommand();
		}

	}

Ja, jag kan inte svara på riktigt varför jag går via Library... Den agerar mer eller mindre som en router/switch till hela programet.

Eftersom jag har ett UML diagram som jag utgår ifrån när jag gör mitt program så var detta den enklaste vägen som jag förstod det samt att min lärare tyckte det var bra.
Till en början hade jag inte det på detta vis och då fick jag smäll på fingrarna och jag la då till Library klassen.

Här är mitt UML diagram
http://imgur.com/EYMph

ReaderCollection är mest där för att representera en databas om jag nu skulle göra ett stort program.
Idag är den lite fjuttig med bara sin arrayList.
Fast det var ett bra upplägg enligt läraren när han bedömde diagramet då vi visar hur man skulle kunna göra om programet skulle vara större och mer avancerat.

En sak som slog mig gällande min kod.
Jag saknar möjlighet att säga åt användaren att skapa ett unikt ID, när metoden createReader körs.

Jag har metoden searchReader och då söker den efter ID.
Så den kan jag använda för att kolla om id jag försöker använda för att skapa en reader redan finns.
Men det jag fastnar på är hur jag formulerar en if sats med detta.

så här ser min searchReader metod ut.

	public Reader searchReader(int readerID) {
		for (Reader reader : readers) {
			if (reader.getReaderID() == readerID) {
				return reader;
			}
		}
		return null;
	}

såhär ser min createReader metod ut.

	private void createReader() {
		String name = readString("Name: ");
		int readerID = readInt("Reader ID: ");
		int birthYear = readInt("Birth Year: ");
		Address address = readAddress();
		PhoneNo phoneNo = readPhoneNo();
		Email email = readEmail();
		Reader theNewReader = new Reader(name, readerID, birthYear, address,
				phoneNo, email);
		library.addReader(theNewReader);
		System.out.println("Reader " + name + " added");
	}

Jag tänker mig att jag ska kunna göra såhär i createReader metoden.

	private void createReader() {
		String name = readString("Name: ");
		int readerID = readInt("Reader ID: ");		
		Reader reader = library.searchReader(readerID);
		if (readerID == reader){
			System.out.println("That reader ID does already exist, try again.");
		}else{
		int birthYear = readInt("Birth Year: ");
		Address address = readAddress();
		PhoneNo phoneNo = readPhoneNo();
		Email email = readEmail();
		Reader theNewReader = new Reader(name, readerID, birthYear, address,
				phoneNo, email);
		library.addReader(theNewReader);
		System.out.println("Reader " + name + " added");
		}
	}

Kan du ge mig en hint om vad jag behöver göra för att få det att fungera?

doh tack för ett väldigt direkt svar.
det löste saken snabbt och nu när jag ser det är det super självklart.

um detta såg så bra ut, men det går inte

försöker nu med

		Reader reader = library.searchReader(readerID);
		if (reader.getReaderID() == readerID){
			System.out.println("That ID does already exist.");
		}

tack för tipset. tyvärr ska jag redovisa imorgon och vill inte ändra för mycket i koden

jag söker nu bara efter lösningen på detta.

just nu funkar detta:

	private void createReader() {
		String name = readString("Name: ");
		int readerID = readInt("Reader ID: ");		
		Reader reader = library.searchReader(readerID);
		if (reader.getReaderID() != readerID){
			int birthYear = readInt("Birth Year: ");
			Address address = readAddress();
			PhoneNo phoneNo = readPhoneNo();
			Email email = readEmail();
			Reader theNewReader = new Reader(name, readerID, birthYear, address,
					phoneNo, email);
			library.addReader(theNewReader);
			System.out.println("Reader " + name + " added");
		}else{
			System.out.println("That ID does already exist.");
		}
		
	}

det funkar till den grad att jag kan slå in ett id som redan finns och den säger då åt mig att jag inte kan skapa en användare med det ID numret.

men skapar jag en användare med ett helt nytt ID så krashar programet.
med detta fel:
Exception in thread "main" java.lang.NullPointerException
at MainLibraryProgram.createReader(MainLibraryProgram.java:330)
at MainLibraryProgram.handleMenuCommand(MainLibraryProgram.java:154)
at MainLibraryProgram.start(MainLibraryProgram.java:94)
at MainLibraryProgram.main(MainLibraryProgram.java:14)

jösses... jag har ju gjort detta på flera andra ställen och så ser jag inte lösningen
tack
tack
tack

ska jag debattera med en moderator... hm ja lite
Tråden har olika frågor... första handla om att stänga en scanner, nästa problem handla om en if sats.

men det är sak samma. det passar fint här då första delen ändå har besvarats och nu inte varit aktiv på ett tag.

samt nu är problem 2 löst också
god jul

toppen
då fortsätter vi så