Variable mit Zeichen auffüllen

[Ru!e]

moin moin!


sitz grad in der schule und mich plagt da was, leider kann mir nichtmal mein versierter banknachbar helfen.

kurz zu dem was ich machen will, denn nach meiner idee befürchte ich, dass es eine brutal lange methode werden könnte. über ideen die das ganze schlanker machen bin ich dankbar:

ich habe 4 textfelder. nach eingabe und klick auf den button "hinzufügen" wird der inhalt zusammenhängend in ein choice-feld (kombobox) eingefügt.

zwischen klicken und einfügen jedoch muss noch etwas geschehen:
- es muss in 3 der 4 textfeldern geprüft werden, dass eine bestimmte länge nicht überschritten wird.
- ist auch nur EINER DER DREI texte zu kurz, soll auf eine bestimmte länge mit leerschritten aufgefüllt werden.
- ist auch nur EINER DER DREI texte zu lang, soll einfach ab der grenze abgeschnitten werden.


hier mal so weit, wie ich bisher gekommen bin. leider ... ach naja weiß auch net, wird irgendwie nix

    public void add()
    {
        String eventdatum, eventort, eventpix, eventbemerkung;
        int fehlStelle;


        if(tf_eventdatum.getText().length() <= 10 && tf_eventort.getText().length() <= 25 && tf_eventbemerkung.getText().length() <= 50)  // Prüfung ob alle Texte kürzer sind als die Grenze
        {
            if(tf_eventort.getText().length() >25)  // Einzelprüfung jedes Textfeldes ob es kürzer ist als die Grenze
            {
                eventort = tf_eventort.getText();   // einfach eine hilfsvariable die wir bei einem versuch gebraucht haben
                
                for(int x = 0; x < (25 - tf_eventort.getText().length()); x++);  // ein kläglicher versuch....
                {
                    eventort = eventort + " ";
                }
               
            }
            choice_archiv.add(tf_eventdatum.getText() + " | " + eventort + " | " + tf_eventpix.getText() + " | " + tf_eventbemerkung.getText());   // hier hab ich es erst mal mit einem textfeld bzw. der hilfsvariable versucht.
        }
    }

hoffe mir kann jemand auf die sprünge helfen

grüße und einen sonnigen tag
ru!e

 

[Murray]

Du hast hier eine bestimmte Aufgabe ("Prüfen, ob ein String eine bestimmte länge hat; ist er länger, dann soll er abgeschnitten werden, ist er kürzer, dann soll er mit Leerzeichen aufgefüllt werden"), die mehr als einmal durchgeführt werden soll. Daher bietet es sich an, diese Aufgabe in eine Methode auszulagern.

	/**
	 * @param str - String, der auf eine bestimmte Länge gebracht werden soll
	 * @param len - gewünschte Länge
	 * @return String mit der gewünschten Länge (ggfs. abgeschnitten oder mit 
	 *         Leerzeichen aufgefüllt) 
	 */
	public String trimStr( String str, int len) {
		
		int strLen = str.length();
		if ( strLen < len) {
			/* ... */
		} else if ( strLen > len) {
			/* ... */
		} else {
			return str;
		}
		
	}

Damit wird es doch recht übersichtlich:

String s1 = tf1.getText();
String s2 = tf2.getText();
String s3 = tf3.getText();
String s4 = tf4.getText();

String s1234 = trimStr( s1, 25) + "|" + trimStr( s2, 30) + "|" trimStr( s3, 50) + "|" + s4;

 

[Ru!e]

wow das ging ja fix!!! erst mal vielen dank, ich bin schonmal stark beeindruckt.

hab das gleich mal in netbeans reingeklopft, wie folgt:

    public void add()
    {
        String s1 = tf_eventdatum.getText(); 
        String s2 = tf_eventort.getText(); 
        String s3 = tf_eventpix.getText(); 
        String s4 = tf_eventbemerkung.getText(); 

        String s1234 = trimStr( s1, 25) + " | " + trimStr( s2, 30) + " | " + s3 + " | " + trimStr( s3, 50);
        
        choice_archiv.add(s1234);
        //clear();
    }
    
    public String trimStr( String str, int len) 
    {
        int strLen = str.length(); 
        
        if ( strLen < len) 
        { 
            /* ... */ 
        } 
        else if ( strLen > len) 
        { 
            /* ... */ 
        } 
        else 
        { 
            return str; 
        }
    }

das auskommentierte clear(); ist einfach eine methode die die textfelder wieder leer macht danach, unwichtig.

bei der letzten klammer der methode trimStr krieg ich´n fehler mit dem argument "missing return statement".
oder geht das erst, wenn ich die if´s ausgearbeitet hab, was entsprechend mit dem strlen passieren soll?

 

[Murray]

Na klar, das war ja auch nur ein Denkanstoß - die "/* ... */"-Kommentare müssen natürlich noch durch den Code ersetzt werden, der den String abschneidet bzw. auffüllt. In beiden Zweigen muss dann am Ende ein return-Statement stehen.

 

[SlaterB]

ob ausgearbeitet oder auf den Kopf gefallen oder wie auch immer:
am Ende der Operation muss ein return stehen,

ein bisschen schlau ist der Compiler, wenn du also ein if/ else drin hast, dann reicht es, wenn sowohl im if als auch im else-Fall ein return steht

aber ohne return gehts nicht

 

[Ru!e]

ahso alles klar. sorry aber bin halt in manchen gebieten noch net soooo fit.

ja gut also von den beiden methoden is mir nun alles klar. auch wie ich den string abschneide sollte mir recht schnell klar werden. aber wo nichtmal mein banknachbar weiter weiß, ist das auffüllen mit leerzeichen.

da hatten wir das eben mit einer schleife versucht, sah auch sehr logisch aus - fehlerfrei compiled aber aufgefüllt hat er nix

 

[Murray]

Der Ansatz im ersten Post sah doch schon nicht schlecht aus - zeigt mal den kompletten Code.

 

[Ru!e]

das war komplett
oder meinste das komplette programm? das sind 800 zeilen code mittlerweile *g*

 

[Murray]

Hast du nicht die "Auffüllschleife" mittlerweile in die Methode trimStr() geschrieben?

 

[Ru!e]

jetzt gerade eben hab ich mal bissle was zusammengeklopft, weil ich auf "concat" gestoßen bin. hier mal mein ergebnis:

    public String trimStr( String str, int len) 
    {
        int strLen = str.length(); 
        
        if ( strLen < len) 
        { 
            int puffer;
            String gefuellt = str;
            
            puffer = len - strLen;
            
            for(int x=0; x <=puffer; x++)
            {
                gefuellt.concat(" ");
            }
            return gefuellt;
        } 
        else if ( strLen > len) 
        { 
            int puffer;
            String gekuerzt;
            
            puffer = strLen - len;
            gekuerzt = str.length() - puffer;
            
            return gekuerzt;
        } 
        else 
        { 
            return str; 
        }
    }

nur das gekuerzt = str.length() - puffer; funktioniert nicht, da schreib ich wohl was an der syntax falsch *grübel*.
aber ich bin sicher es ist zu erkennen was mein plan ist *g*

 

[SlaterB]

gekuerzt = str.length() - puffer;

String = int - int?


wieso überhaupt str.length() aufrufen, du hast doch strLen

 

[Ru!e]

ja hm bin auch grad am überlegen was ich da eigentlich gemacht hab *g*

ich wollt eigentlich dem string "gekürzt" den inhalt aus dem übergegebenen string geben, aber minus der anzahl zeichen, die den unterschied zwischen soll und ist darstellen.

dumm formuliert wa? hmmm mist.


ne einfachere erklärung mit falschem code wäre so:
gekuerzt = str - puffer

 

[Murray]

String sind immutable, ein puffer.concat( " ") verändert den in puffer gespeicherten Wert nicht, sondert liefert einen neuen, verlängerten String zurück. Also wenn schon, dann

puffer = puffer.concat( " ");

(Das ist aber auch nicht viel besser als

puffer +=  " ";

)

 

[Ru!e]

ok danke murray. und was mach ich mit meinem problem weiter unten?

 

[Murray]

Was soll denn hier

            gekuerzt = str.length() - puffer;
           
            return gekuerzt;

auch passieren? gekuerzt ist doch eine String-Variable; der kann man doch nicht das Ergebnis der Subtraktion zweier Zahlen zuweisen.

Was du brauchst, ist vielmehr eine Stringoperation, die dir einen Teilstring eines übergebenen Strings zurückliefert. Such mal in der API-Doku nach String#substring.