Programm: Text ohne "b"

[andrO]

Hallo zusammen,

ich habe ein kleines Programm geschrieben, dass eine Fehlermeldung ausgeben soll, wenn im Text, der zuvor eingegeben werden muss, ein 'b' vorhanden ist. Ist kein 'b' im Text, wird die Nachricht "Ihr Text beinhaltet kein 'b'"

Jedoch funktioniert es nicht so wie ich es haben möchte
Kann es sich einer mal schnell anschauen ?


andrO

Fehler.java

  public class Fehler
  {
       private String text;
       public Fehler()
       {

       }
       
       public Fehler(String text) throws ExceptionText
       {
           for(int i= 0;i == 20; i++)
           {
              if(text.charAt(i) == 'b' || text.charAt(i) == 'B')
              {
                 throw new ExceptionText(text);
              }
              else
              {
                 System.out.println("" );
                 System.out.println(" Ihr Text beinhaltet kein 'b'");
              }
           }
       
       }

  }

ExceptionText.java

  public class ExceptionText extends Exception
  {
       public ExceptionText()
       {
            super();
       }
       
       public ExceptionText(String t)
       {
            super(" Fehler: Der Text enthaelt ein 'b'");
       }
  }

Aufruf.java

import java.io.*;
  public class Aufruf
  {
     public static void main(String argv[])
     {
         BufferedReader input = new BufferedReader(new InputStreamReader(System.in));

         // Variablendeklaration
         String eingabe = null;
         do
         {

            try
            {
                System.out.println("");
                System.out.println(" Bitte geben Sie einen Text ohne b ein:");
                System.out.print(" Text: ");
                eingabe = input.readLine();
                Fehler f = new Fehler(eingabe);
            }
            catch(ExceptionText e)
            {
                System.out.println("");
                System.out.println(" " +e);
            }
            catch(Exception e)
            {
                System.out.println("");
                System.out.println(" " +e);
            }
         }while(eingabe.equals("ende") || eingabe.equals("Ende"));

     }
  
  }

 

[Wildcard]

Und was funktioniert nicht?

 

[Gelöschtes Mitglied 5909]

ich würde das so machen:

public boolean containsB(String text) {
	for (int i = 0; i < text.length(); i++) {
		if (test.charAt(i) == 'b' || test.charAt(i) == 'B') {
			return true;
		}
	}
	return false;
}

eine eigene exception für sowas halte ich für unsinnig, ebenso wie eine eigene klasse für den Fehler....


hab das mal eben getestet...

import java.io.*;
public class B {
	
	public static void main(String argv[]) throws IOException {
   	
  	BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
    String eingabe = "";
       
    while(!eingabe.equalsIgnoreCase("ende")) {
    	System.out.println("Bitte geben Sie einen Text ohne b ein:");
      eingabe = input.readLine();
      System.out.println("B im text: " + containsB(eingabe));
    }
	}
	 
  public static boolean containsB(String text) {
		for (int i = 0; i < text.length(); i++) {
			if (text.charAt(i) == 'b' || text.charAt(i) == 'B') {
				return true;
			}
		}
		return false;
	}
}

C:\>java B
Bitte geben Sie einen Text ohne b ein:
test
B im text: false
Bitte geben Sie einen Text ohne b ein:
asdaasdfasdfsadf
B im text: false
Bitte geben Sie einen Text ohne b ein:
hier is aber ein b drinn !!!
B im text: true
Bitte geben Sie einen Text ohne b ein:
ende
B im text: false

C:\>

 

[Wildcard]

ich würde das so machen:

public boolean containsB(String text) {
	for (int i = 0; i < text.length(); i++) {
		if (test.charAt(i) == 'b' || test.charAt(i) == 'B') {
			return true;
		}
	}
	return false;
}

und ich würde das so machen:

text.toLowerCase().contains("b");

:bae:

 

[SlaterB]

gibts nicht auch noch containsIgnoreCase()?

übrigens:

 for(int i= 0;i == 20; i++)
           {
...
           }

it keine korrekte Schleife,

 for(int i= 0;i < 20; i++)
           {
...
           }

ist wohl gemeint?

------------

und

else
              {
                 System.out.println("" );
                 System.out.println(" Ihr Text beinhaltet kein 'b'");
              }

kannn man doch nicht innerhalb der Schleife schon an Position 1 feststellen,
da muss man erstmal die Schleife komplett durchlaufen lassen,

idealerweise auch nicht genau bis 20, sondern genau so lange wie der Text lang ist,
dann vergißt man nicht die letzten Zeichen und kriegt auch keine Exception, falls der Text mal kürzer als 20 Zeichen ist!!

------

}while(eingabe.equals("ende") || eingabe.equals("Ende"));

heißt 'wiederhole solange 'ende' oder 'Ende' eingegen wurde'.

Das Gegenteil ist wohl gemeint.
Und der Einfachheit wegen besser mit "ende".quals(eingabe.toLowerCase()) oder equalsIgnoreCase arbeiten.
Vielleicht noch abfangen ob eingabe 'null' sein kann, aber ist vielleicht gar nicht möglich.

 

[Leroy42]

übrigens:

 for(int i= 0;i == 20; i++)
           {
...
           }

it keine korrekte Schleife

<klugscheiß>Aber sicher ist die korrekt; halt nur nicht besonders ergiebig</klugscheiß>

 

[SlaterB]

genau wie dein Posting
nene, Scherz

 

[Leroy42]

nene, Scherz

Wieso? Stimmt doch!

(Mußte halt nur meinen Freitagsfrust loswerden)