Methoden String übergeben in Methode

[Glox]

Hallo.

Vorab, ich bin ein totaler Neuling auf dem Gebiet und habe gerade erst mein Informatikstudium angefangen.

Wir sollen über das WE ein Programm schreiben, dass einen eingelesene Ziffernfolge (String) als Integer wiedergeben soll. Bei ungültiger Eingabe soll über die Main-Methode eine "0" ausgegeben werden.

Soweit so gut. Ich habe mich an die Arbeit gemacht, nun hänge ich aber schon m.M.n. ziemlich am Anfang fest

public class Conversion {

	public static void main (String[] args){
		
			
		println("Bitte geben Sie eine Ziffernfolge ein: ");
		
		String eingabe = readLine();
		readLine();
		
				
		
	}

	
	public static int convert (eingabe digits){
		
		int ziffer, zlaenge; 
		
		
		zlaenge = strLen(readLine());
		ziffer = strCharAt(readLine(), 0);
			
		if (zlaenge <= 0){
			println ("0");
		}
		while (ziffer <= zlaenge){
			ziffer++;
			
		int test = ziffer;	
			
	}	println(readLine());
	
	}

Ich weiß einfach nicht, wie ich den eingelesenen String an die Methoder darunter weitergebe. Wir sollen ausschließlich die "Werkzeuge" nutzen die wir bis jetzt gelernt haben und das ist denk ich nicht viel. Hab schon hier im Forum und Google gesucht, jedoch kamen immer Vorschläge die mit Funktionen etc. zu tun hatten, die wir so noch nicht gelernt haben.

Würde mich sehr über einige Tipps freuen.

Ich erwarte natürlich nicht, das mir hier jemand die Arbeit oder sonstiges abnimmt. Will es ja schließlich lernen.

Lg

 

[ARadauer]

public static int convert (eingabe digits){
nö so...
public static int convert (String digits){

methode (Datentyp variablenName){
}

warum rufst du in der Methode 3 mal readLine() auf?

 

[Glox]

Ah da habe ich etwas rumprobiert, aber vergessen die wieder rauszunehmen. Sorry


Ich habe es ein wenig verändert und hoffe es ist etwas ersichtlicher was ich vorhabe
Aber das übergeben der "eingabe" klappt iwie nicht.

public class Conversion {

	public static void main (String[] args){
		
			
		println("Bitte geben Sie eine Ziffernfolge ein: ");
		
		String eingabe = readLine();
		readLine();
		
		int fehler = 0;
		
		println(fehler);		
		
	}

	
	public static int convert (String digits){
		
		int ziffer, zlaenge, fehler = 0;
				
		
		zlaenge = strLen(eingabe);
		ziffer = strCharAt(eingabe, 0);
			
		if (zlaenge <= 0){
			println (fehler);
		}
		while (ziffer <= zlaenge){
			ziffer++;
		
		return ziffer;	
		}
	
	}	


}

 

[ARadauer]

Ja klar... weil du es nicht machst...

 

[pappawinni]

Ohoh,
deine Methode convert erhält seinen Eingangsparameter in Form des Strings digits.
D.h. wenn du aus der Main z.B.

int k = convert(eingabe);

aufrufst, weiss die so aufgerufenen Methode convert garnienichts davon, dass der Parameter in der Main den Namen eingabe hat.
In der Methode convert wird der Parameter als "digits" bezeichnet und damit musst du in der Methode convert arbeiten.
Die Variable eingabe würde nur dann in convert bekannt sein, wenn du sie ausserhalb von main und convert deklarierst (Klassenvariable) und damit in allen Methoden bekannt sein. In diesem Fall würde aber die Definition eines Parameters digits nicht mehr sinnvoll sein.
Möglicherweise kann dir eine Klassenvariable aber dabei helfen eine Fehlercode an die Main zurück zu liefern.
[EDIT]
die Methode convert soll ja nach Definition eine Integer zurückliefern. Ergo fehlte in der Methode convert die Rückgabe eines Wertes.

return irgendwas;

[/EDIT]
[EDIT]
Woher nimmst du dann auch sowas:
zlaenge = strLen(eingabe);
ziffer = strCharAt(eingabe, 0);
Doch nicht aus einer Vorlesung zu Java, oder ?
[/EDIT]

 

[Glox]

Also ich habe die Techniken aus den Folien entnommen, die wir bis jetzt bekommen haben.
Vermutlich ist auch meine ganze Rangehensweise verkehrt.


Vielleicht sollte ich erst mal erklären, wie ich es lösen wollte:

- Zuerst wollte ich in der Main Methode die Benutzereingabe einlesen ( readLine(); )
- Danach wollte ich diesen Wert in die Methode convert übergeben und mit Hilfe der von mir erstellten
Variablen "zlaenge / ziffer", wollte ich den String auf seine Länge und jede einzelne Ziffer auslesen.
-> Bei einer leeren Eingabe bzw. sollte der Fehler: 0 ausgegeben werden (Main Methode).
-> Ansonsten die Ziffernfolge in Form eines Integer

Meine Fehler die ich für mich selbst sehe sind, zum einen die fehlende Übergabe des eingelesenen Wertes und zum anderen glaube ich, dass bei den Schritten auslesen des Strings und Konvertierung was Entscheidenes fehlt - die Konvertierung. ???:L

Korrigiert mich bitte wenn ich falsch liege.
Habe wie gesagt noch keine Erfahrung in dem Bereich und außer Hello World & einen simplen Teiler-Berechner noch nicht viel Übung dahingehend gehabt.

 

[turtle]

Exceptions für so etwas zu zweckendfremden ist kein guter Stil, aber...

	private static int convert(String zahl) {
		int parseInt = 0;
		try {
			parseInt = Integer.parseInt(zahl);
		} catch (NumberFormatException e) {
			e.printStackTrace();
		}
		return parseInt;
	}

 

[Glox]

Danke für den Vorschlag, aber das dürfen wir nicht nutzen, da wir es noch nicht in der Vorlesung behandelt haben. :noe:

 

[pappawinni]

Also..was du jetzt verwenden darfst werden wir jetzt kaum erraten können.
Vielleicht etwa so, vom Prinzip ..

        String zahlString = "123" ;
        int wert = 0;
        boolean fehler = false;
        int len = zahlString.length();
        for (int i=0;i<len;i++){
          int c = zahlString.charAt(i) - 48; // Zeichen 48 ist die "0" usw.
          if (c < 0 || c > 9){
        	  fehler = true;
        	  break;
          }
          wert = wert * 10 + c;
        }
        if (!fehler){
           System.out.println(wert);
        }
        else {
           System.out.println(fehler);        	
        }

 

[Glox]

Da hast du Recht. Es würde auch etwas ausarten

Ich schätze mal das es dem schon näher kommt, aber "for" und "break" hatten wir leider auch noch nicht.
Tut mir leid

Wir haben momentan If & While in unserem Werkzeugkasten ^^

Ich werde einfach mal weiter rumprobieren und mir was zusammenbasteln.

Danke für eure Hilfe!

 

[pappawinni]

also ein letzter Versuch...

        String zahlString = "4123" ;
        int wert = 0;
        boolean fehler = false;
        int len = zahlString.length();
        if (len == 0) fehler = true;
        int i = 0;
        while (i<len){
          int c = zahlString.charAt(i) - 48; // Zeichen 48 ist die "0" usw.
          if (fehler || c < 0 || c > 9){
        	  fehler = true;
          }
          else{
              wert = wert * 10 + c;        	  
          }
          i++;
        }
        if (!fehler){
           System.out.println(wert);
        }
        else {
           System.out.println("Fehler");        	
        }

 

[Glox]

Also wenn ich es starte, erhalte ich bei jeder Eingabe die Ausgabe: "Fehler".

An sich sieht es gut aus, nur leider nutze ich dann nicht die vom Package importierte Convert Methode oder liege ich da falsch? ;(

 

[pappawinni]

Wenn du was startest ?
Woher sollen wir denn wissen, wie du das in deinen Code eingebaut hast.
Mein Beispiel zeigt doch nur das Prinzip der Konvertierung.

Poste doch mal deinen Code, Mann.

 

[pappawinni]

Denkbar wäre vielleicht sowas:

public class Conversion {

    static boolean fehler;
 
    public static void main (String[] args){
        
            
        System.out.println("Bitte geben Sie eine Ziffernfolge ein: ");
        
        String eingabe = readLine();
        int k = convert(eingabe);
        if (!fehler){
           System.out.println(k);
        }
        else {
           System.out.println("Fehler");            
        }                                        
    }
 
    
    public static int convert (String zahlString){
        int wert = 0;
        int len = zahlString.length();
        fehler = false;
        if (len == 0) fehler = true;
        int i = 0;
        while (i<len){
          int c = zahlString.charAt(i) - 48; // Zeichen 48 ist die "0" usw.
          if (fehler || c < 0 || c > 9){
              fehler = true;
          }
          else{
              wert = wert * 10 + c;           
          }
          i++;
        }
        return (wert);   
    } 
    
}

 

[Glox]

Dank dir.

Hab es jetzt bei mir editiert und es sieht so aus:

public class Conversion {

    public static void main(String[] args) {
	        
    	println("Bitte geben Sie eine Ziffernfolge ein:");
	    String eingabe = readLine();
	    int neu = convert(eingabe);
	    println(neu);
	    }
	    
    public static int convert(String folge){

    	int wert = 0;
    	int len = folge.length();
    	
    
    	int i = 0;
    	while (i < len){
    		int c = folge.charAt(i) - 48;
    		if (len == 0 || c < 0 || c > 9){
    			println("0");
    		}
    		else
    			{
    			wert = wert * 10 + c;
    		
    		}
    		i++;	
	    }
    	return (wert);
	   
	}
}

Ich habe den Boolean bewusst rausgenommen, weil wer den so noch nicht hatten. Also gelernt schon, aber nicht in dem Stil verwendet.

 

[pappawinni]

Entschuldige, aber das gibt doch Stuss..
Es wird doch jedes Zeichen durchlaufen und dann wird bei jedem Zeichen, das nicht einer Ziffer entspricht "0" ausgegeben.
Am Ende wird dann auch noch von der Main ein Wert ausgespuckt, selbst wenn vorher Fehler aufgetreten sind.
Im Fall len = 0 wird die while-Schleife nicht durchlaufen und es kommt keine "Fehlermeldung". Es sähe für den Anwender genauso aus, als wäre 0 eingegeben worden.
Was soll denn ein Anwender mit einer solchen Ausgabe anfangen?
Wenn du keine Boolean verwenden willst, dann nimm halt stattdessen eine Integer, die z.B. im Fehlerfall den Wert -1 und andernfalls den Wert 0 hat.

 

[pappawinni]

Hm.. ich hab die Aufgabenstellung nochmal gelesen.
Bei Fehler soll 0 ausgegeben werden.
Dann mach halt deine convert-Methode so:

    public static int convert(String folge){
 
        int wert = 0;
        int len = folge.length();

        int fehler = 0;
   
        int i = 0;
        while (i < len){
            int c = folge.charAt(i) - 48;
            if (fehler > 0 || c < 0 || c > 9){
                fehler++;
                wert = 0;
            }
            else
            {
                wert = wert * 10 + c;
            
            }
            i++;    
        }
        return (wert);      
    }
}

 

[Glox]

Jetzt hab ichs

Hab das mit Integer eingebaut wie du es gepostet hast und es tut was gewollt ist.

Nun muss ich das nur noch kommentieren und gut ist. :toll:

Dank dir vielmals für deine Hilfe und vor allem deine Geduld!