überprüfungsschwierigkeit bei der klasse Buch

[nata]

ich habe buch getestet und liefert mir fehler bei der zeile 53 und 58 was mache ich falsch?

public class Buch{
	private String titel;
	private String autor;
	private double preis;
	private String isbn;
	
	public Buch(String titel, String autor, double preis, String isbn)throws Exception{
		setTitel(titel);
		setAutor(autor);
		setPreis(preis);
		setIsbn(isbn);
	}
	
	public void setTitel(String titel)throws Exception{
		if(titel == null){
			throw new Exception("Titel darf nicht null sein");
		}
		if(titel.length() < 1 || titel.length() > 30){
			throw new Exception("Titel darf bis 30 zeichen lang sein");
		}
	
		this.titel = titel;	
	}
	
	public void setAutor(String autor)throws Exception{
		if(autor == null){
			throw new Exception("Autor darf nicht null sein");
		}
		if(autor.length() < 1 || autor.length() > 30){
			throw new Exception("Autor darf bis 30 zeichen lang sein");
		}
	
		this.autor = autor;	
	}
	
	public void setPreis(double preis)throws Exception{
		
		if(preis < 3.5 || preis > 350){
			throw new Exception("preis soll zwischen 3 euro und 350 euro sein");
		}
	
		this.preis = preis;	
	}
	
	public void setIsbn(String isbn)throws Exception{

		if(isbn.length() < 7 || isbn.length() > 12){
			throw new Exception("Isbn nummer darf nur 7 bis 12 zeichen enthalten");
		}
		
		for(int i = 0 ; i < isbn.length() -1 ; i++){
			char c = isbn.charAt(i);
			if(!Character.isDigit(c) || Character.charAt(i) != '-'){
				throw new Exception("Isbn soll bis letzte buchstaben nur ziffern oder bindestriche enthalten");
			}
						            	
		}
		if(!Character.isDigit(isbn.charAt(isbn.length())) || Character.charAt(isbn.length()) != '-' || Character.charAt(isbn.length()) != 'X'){
				throw new Exception("letzte zeichen soll entweder zahl ,bindestrich oder X sein");
			}
			
			this.isbn = isbn;
		
	}
	
	public String getIsbn(){
		return isbn;
	
	}
	
	public String toString(){
	
		return autor + "  " + titel + "  " + isbn + "  "+ preis;	
	
	}
	
	
}

gegeben ist folgende : String isbn - ist zwischen 7 und 12 Zeichen lang, darf nur aus Ziffern und
Bindestrichen bestehen, und kann mit einem „X“ enden

 

[Landei]

Character hat keine Methode charAt

 

[icarus2]

Die Klasse String hat eine Methode charAt(int index).

Steht in der API: String: Method chartAt(int index)

 

[nata]

danke jungs.
also ich habe Statt Character. ... isbn. ... geschrieben und soweit liefert keinen fehler

 

[nata]

doch mich zu fruh gefreut ;(

public class Buch{
	private String titel;
	private String autor;
	private double preis;
	private String isbn;
	
	public Buch(String titel, String autor, double preis, String isbn)throws Exception{
		setTitel(titel);
		setAutor(autor);
		setPreis(preis);
		setIsbn(isbn);
	}
	
	public void setTitel(String titel)throws Exception{
		if(titel == null){
			throw new Exception("Titel darf nicht null sein");
		}
		if(titel.length() < 1 || titel.length() > 30){
			throw new Exception("Titel darf bis 30 zeichen lang sein");
		}
	
		this.titel = titel;	
	}
	
	public void setAutor(String autor)throws Exception{
		if(autor == null){
			throw new Exception("Autor darf nicht null sein");
		}
		if(autor.length() < 1 || autor.length() > 30){
			throw new Exception("Autor darf bis 30 zeichen lang sein");
		}
	
		this.autor = autor;	
	}
	
	public void setPreis(double preis)throws Exception{
		
		if(preis < 3.5 || preis > 350){
			throw new Exception("preis soll zwischen 3 euro und 350 euro sein");
		}
	
		this.preis = preis;	
	}
	
	public void setIsbn(String isbn)throws Exception{

		if(isbn.length() < 7 && isbn.length() > 12){
			throw new Exception("Isbn nummer darf nur 7 bis 12 zeichen enthalten");
		}
		
		for(int i = 0 ; i < isbn.length() -1 ; i++){
			char c = isbn.charAt(i);
			if(!Character.isDigit(c) && isbn.charAt(i) != '-' ){
				throw new Exception("Isbn soll bis letzte buchstaben nur ziffern oder bindestriche enthalten");
			}
						            	
		}
		if(!Character.isDigit(isbn.charAt(isbn.length())) || isbn.charAt(isbn.length()) != '-' || isbn.charAt(isbn.length()) != 'X'){
				throw new Exception("letzte zeichen soll entweder zahl ,bindestrich oder X sein");
			}
			
			this.isbn = isbn;
		
	}
	
	public String getIsbn(){
		return isbn;
	
	}
	
	public String toString(){
	
		return autor + "  " + titel + "  " + isbn + "  "+ preis;	
	
	}
	
	
}

public class BuchMain{
public static void main(String args[]) {
		try {
			Buch b1= new Buch("dssdf", "dhdj", 23, "28965437");
			b1.toString();
			 
		} catch (Exception e) {
			System.out.println(e.getMessage());
		}
	}

}

diese fehlermeldug was mir compiler liefert verstehe ich nicht.wo liegt mein fehler?

 

[eRaaaa]

der fehler ist in der zeile 58
[c] if(!Character.isDigit(isbn.charAt(isbn.length())) || isbn.charAt(isbn.length()) != '-' || isbn.charAt(isbn.length()) != 'X')[/c]

isbn.lenght()-1 musst du abfragen.
desweiteren: meinst du nicht, du müsstest das gane UND verknüpfen, statt ODER (bzw, so wie du es jetzt hast) ? 🙂

char last = isbn.charAt(isbn.length() - 1);
		if (!Character.isDigit(last) && last != '-' && last != 'X') {
			throw new Exception(
					"letzte zeichen soll entweder zahl ,bindestrich oder X sein");
		}

/edit: @main-test-programm: b1.toString(); <-- du willst das doch sicher ausgeben oder? das gibt dir ja schließlich einen string zurück:
[c] System.out.println(b1.toString);[/c] bzw, toString kann man sich in dem fall sparen (weil das durch print, bzw String.valueOf automatisch aufgerufen wird) -->
[c] System.out.println(b1);[/c]

/edit2 :

 if(isbn.length() < 7 && isbn.length() > 12){
            throw new Exception("Isbn nummer darf nur 7 bis 12 zeichen enthalten");
        }

hmm?? auch hier wieder operatoren verwechselt 😀

 

[nata]

in bezug zu operatoren.ja ich mache oft fehler,vieleicht weil ich in kopft irgendwie auf meine sprache übersetzte und in georgische logik klingt das so.😉
danke dir vielmals für deine unterstützung.hast mir viel geholfen.habe getestet und ales klappt(party)