Problem mit der Kommasetzung an der richtigen stelle

[HeiTim]

Hallo zusammen ich hab ein kleines Problem komm nicht drauf wie ich das Komma an der letzten stelle entferne also mein code lautet:

public class Array {
    public static void main (String args[]) {
    
            int[] intArray = new int[] {2, 4, 6, 8, 9, 2, 2, 5};

                for(int zahl : intArray) {
                   System.out.print(zahl+", ");
                }
            }
        }

und ausgegeben wird: 2, 4, 6, 8, 9, 2, 2, 5,
wie bekomme ich nun das Komma Zeichen an der letzeb stelle also 5, weg?

 

[Mart]

du machst eine for int i schleife und wenn i == array.length-1 ( also letzter aufruf) dann machst du im if eben kein komma hin

 

[LimDul]

Für sowas gibt es viele Möglichkeiten, wenn man es von Hand macht, sind das die zwei gängisten:

a) Bei dem ersten Eintrag kein Komma und bei allen weiteren Einträgen das Komma vor dem Eintrag erzeugen
b) Beim letzten Eintrag kein Komma erzeugen

Das setzt voraus, dass du in der Schleife weißt, was der erste bzw. letzte Eintrag ist. Den ersten Eintrag kann man noch mit einem Boolean ermitteln, was man nach dem ersten Eintrag auf false setzt. Beim letzten Eintrag (und als Alternative auch beim ersten Eintrag) kann man das machen, indem man anstelle der foreach Schleife auf eine klassische for schleife mit Index umstellt und den Index abfragt.

 

[HeiTim]

du machst eine for int i schleife und wenn i == array.length-1 ( also letzter aufruf) dann machst du im if eben kein komma hin

Bitte nochmal erklären
also ich hatte es mit einer if else versucht aber es kam nur schwachsinn bei raus ich glaube weil es sich an die anzahl der werte gerichtet hatte.
also ich hatte

if(zahl <= 10){
    System.out.print(", ");}
else
{
   System.out.print("");{
}   
}   
}

 

[HeiTim]

Für sowas gibt es viele Möglichkeiten, wenn man es von Hand macht, sind das die zwei gängisten:

a) Bei dem ersten Eintrag kein Komma und bei allen weiteren Einträgen das Komma vor dem Eintrag erzeugen
b) Beim letzten Eintrag kein Komma erzeugen

Das setzt voraus, dass du in der Schleife weißt, was der erste bzw. letzte Eintrag ist. Den ersten Eintrag kann man noch mit einem Boolean ermitteln, was man nach dem ersten Eintrag auf false setzt. Beim letzten Eintrag (und als Alternative auch beim ersten Eintrag) kann man das machen, indem man anstelle der foreach Schleife auf eine klassische for schleife mit Index umstellt und den Index abfragt.

also ich kann mir nichts darunter vorstellen kannst du mir da weiter helfen oder hast du eine website wo es mir gut erklärt wird also ich habs nicht ganz verstanden und im netz finde ich es mit Strings

 

[Mart]

Bitte nochmal erklären
also ich hatte es mit einer if else versucht aber es kam nur schwachsinn bei raus ich glaube weil es sich an die anzahl der werte gerichtet hatte.
also ich hatte

if(zahl <= 10){
    System.out.print(", ");}
else
{
   System.out.print("");{
} 
} 
}

du hast die 10 "hard gecoded" aber die ändert sich ja von array zu array...

die eigentliche bedingung ist

FALLS zahl GLEICH array Länge minus 1 DANN printe(arrayzahl) ANSONTEN printe(arrayzahl)+ ","

das ziel ist du hörst auf mit kommasetzung wenn du weist im nächsten schritt wird die for schleife abbrechen weil sie durch ist

PS: das if ist schon mal ein sehr guter ansatz nur die bedingung bissl ummodeln, du kannst natürlich auch meine Bedingung negieren und mit kleiner arbeiten

 

[Oneixee5]

Also man kommt ohne ein if in der Schleife aus. Der Trick ist: schreibe das erste Element des Arrays vor der Schleife und beginne die Schleife beim Index 1.

    public static void main(final String[] args) throws Exception {
        final int[] intArray = {2, 4, 6, 8, 9, 2, 2, 5};
        print(intArray);
    }

    private static void print(final int[] intArray) {
        // test ob im Array ein Inhalt vorhanden ist
        if (intArray != null && intArray.length > 0) {
            // schreibe das erste Element mit index = 0
            System.out.print(intArray[0]);
            // schreibe andere Elemente mit index > 0
            for (int i = 1; i < intArray.length; i++) {
                System.out.print(", " + intArray[i]);
            }
            System.out.println();
        }
    }

[CODE title="Ausgabe"]2, 4, 6, 8, 9, 2, 2, 5[/CODE]

 

[Neumi5694]

Ich bevorzuge, das Ganze nicht direkt auszugeben, sondern zuerst in einen Stringbuilder zu packen. Zuletzt kürze ich ihn dann um das Trennzeichen.

Eine weitere Möglichkeit ohne If-Bedingung wäre ein Iterator.
Das erste Element wird ohne Trennzeichen ausgegeben, alle anderen mit führendem Trennzeichen.

Da gibt's wirklich eine Unmenge an verschiedenen Möglichkeiten, wie du das hinkriegen kannst.

 

[Mart]

Ich bevorzuge, das Ganze nicht direkt auszugeben, sondern zuerst in einen Stringbuilder zu packen. Zuletzt kürze ich ihn dann um das Trennzeichen.

Eine weitere Möglichkeit ohne If-Bedingung wäre ein Iterator.
Das erste Element wird ohne Trennzeichen ausgegeben, alle anderen mit führendem Trennzeichen.

Da gibt's wirklich eine Unmenge an verschiedenen Möglichkeiten, wie du das hinkriegen kannst.

ich würde jemanden der mit ifs kämpft keinen iterator in die hand drücken

zusätzlich würde ich auch nicht den stringbuilder benutzten da du eine anweisung durchführst die du im nachhinein wieder rükcgängig machen musst

 

[HeiTim]

Es hat funktioniert ich danke euch für eure Schnelle Hilfe.

 

[Neumi5694]

zusätzlich würde ich auch nicht den stringbuilder benutzten da du eine anweisung durchführst die du im nachhinein wieder rükcgängig machen musst

Das ist ok, ich hab gesagt, wir ich's machen würde. Du must ja nicht. Es ist halt die effizienteste Methode.

Aber ehrlich .. was soll an einem Iterator kompliziert sein? Das ist doch die einfachste Möglichkeit überhaupt.

if (it.hasNext())  {//ok ... ein if braucht's doch,
     var e0 = it.next();
    //Ersten Wert behandeln
}
while (it.hasNext()) {
    var e = it.next();
    //Zweiten bis letzten Wert behandeln.
}

 

[mihe7]

Da gibt es gefühlt tausend Wege...

Den Iterator hatten wir schon.
Den direkten Weg auch.

Via StringBuilder

if (arr != null && arr.length > 0) {
    StringBuilder b = new StringBuilder();
    b.append(arr[0]);
    for (int i = 1; i < arr.length; i++) {
        b.append(", ").append(arr[i]);
    }
    System.out.println(b.toString());
}

Via StringJoiner:

if (arr != null && arr.length > 0) {
    StringJoiner j = new StringJoiner(", ");
    for (int value : arr) {
        j.add(Integer.toString(value));
    }
    System.out.println(j.toString());
}

Via Stream:

System.out.println(IntStream.of(arr).mapToObj(Integer::toString).collect(Collectors.joining(", ")));

Via Arrays:

String text = Arrays.toString(arr);
System.out.println(text.substring(1, text.length() - 1));

Zusätzlich: alle möglichen Abarten davon wie z. B. Zählvariable überprüfen, boolean verwenden. Noch was vergessen?

 

[Oneixee5]

Leider kann ich erst jetzt antworten, da ich mal wieder 2 Tage gesperrt war - keine Ahnung warum aber das kommt immer wieder vor.
Aber jetzt zum Thema: In meiner Java-Version gibt es keinen Iterator für Arrays von Primitiven und ich denke mit einer eigenen Implementierung ist der TE noch überfordert. Ich denke auch die Java-Entwickler haben absichtlich darauf verzichtet.

 

[Mart]

str

Leider kann ich erst jetzt antworten, da ich mal wieder 2 Tage gesperrt war - keine Ahnung warum aber das kommt immer wieder vor.
Aber jetzt zum Thema: In meiner Java-Version gibt es keinen Iterator für Arrays von Primitiven und ich denke mit einer eigenen Implementierung ist der TE noch überfordert. Ich denke auch die Java-Entwickler haben absichtlich darauf verzichtet.

kommt bei mir manchmal vor wenn ich mit meinem google account angmeldet bin

 

[Oneixee5]

str

kommt bei mir manchmal vor wenn ich mit meinem google account angmeldet bin

Den Google Account nutze ich nicht für den Login im Forum.

 

[Mart]

    private static void print(final int[] intArray) {
        // test ob im Array ein Inhalt vorhanden ist
        if (intArray != null && intArray.length > 0) {
            // schreibe das erste Element mit index = 0
            System.out.print(intArray[0]);
            // schreibe andere Elemente mit index > 0
            for (int i = 1; i < intArray.length; i++) {
                System.out.print(", " + intArray[i]);
            }
            System.out.println();
        }

das hier ist hald "idioten sicher" eine schleife , fokus auf ausgeben und nicht mehr

if (arr != null && arr.length > 0) {
    StringBuilder b = new StringBuilder();
    b.append(arr[0]);
    for (int i = 1; i < arr.length; i++) {
        b.append(", ").append(arr[i]);
    }
    System.out.println(b.toString());
}

das zb ist halt komplett übertrieben für die problemstellung.. fast ein overkill


warum nicht einfach sowas

    private static void print(final int[] intArray) {
        // test ob im Array ein Inhalt vorhanden ist
        if (intArray != null && intArray.length > 0) {
            String summary ="" + intArray[0];
            // schreibe andere Elemente mit index > 0
            for (int i = 1; i < intArray.length; i++) {
                summary+= ""+intArray[i];
            }
            System.out.println(summary);
        }

man muss keinen code über komplizieren, solange es einfacher geht solltem an es einfahcer machen

 

[Mart]

Den Google Account nutze ich nicht für den Login im Forum.

firefox?

 

[thecain]

Leider kann ich erst jetzt antworten, da ich mal wieder 2 Tage gesperrt war - keine Ahnung warum aber das kommt immer wieder vor.

Einfach keine Themen angezeigt, dafür irgend ein Fehler? Hatte ich die letzten 2 Tage auch. Dachte das wäre etwas allgemeines?

 

[Oneixee5]

Einfach keine Themen angezeigt, dafür irgend ein Fehler? Hatte ich die letzten 2 Tage auch. Dachte das wäre etwas allgemeines?

Es kommt eine Fehlermeldung, irgendwas mit: Ups ...
Das ist mir schon oft passiert. Man kann aber noch auf "Kontakt aufnehmen" klicken.
Ich glaube es handelt sich um ein bekanntes Problem mit dieser Art Forum und wird vom Betreiber einfach ignoriert. Ich meine das hängt damit zusammen, dass die Login-Tabelle(n) in der DB kaputt gehen, es gibt auch einen Fix dafür.
Für mich ist das hier nur ein Zeitvertreib aber viele Anfänger werden dringend auf Hilfe warten - für die es halt blöd.

 

[mihe7]

Einfach keine Themen angezeigt, dafür irgend ein Fehler? Hatte ich die letzten 2 Tage auch. Dachte das wäre etwas allgemeines?

irgendwas mit: Ups ...

Ja, hatte ich auch: Ooops, es ist ein Problem aufgetreten. Irgendwas in der Richtung.

Vielleicht können ja @mrBrown, @Flown oder @tbone78 Licht ins Dunkel bringen

 

[Flown]

Ich kann dazu nichts sagen. Ich bin nur ein User Administrator und hab keinen Zugriff für den Betrieb. @tbone78 müsste eine Aussage treffen

 

[Blut1Bart]

Am idiotensichersten finde ich ja immer noch:

    public static void main(String[] args) {
        int[] intArray = new int[]{2, 4, 6, 8, 9, 2, 2, 5};
        String s = Arrays.toString(intArray).replaceAll("[\\[\\]]", "");
        System.out.println(s);
    }

 

[mrBrown]

Am idiotensichersten finde ich ja immer noch:

Du hast "idiotischsten" falsch geschrieben

 

[Blut1Bart]

Du hast "idiotischsten" falsch geschrieben

Begründung?

 

[mihe7]

Begründung?

Hier lesen Kinder mit.

 

[Oneixee5]

Du hast "idiotischsten" falsch geschrieben

Das ist nicht in Ordnung. Statt Sarkasmus und Verhöhnung wäre eine sachlich Begründung angebracht. Es gibt viele die (noch) nicht verstehen, warum etwas falsch, richtig, unsinnig oder besonders gut und effektiv ist. Wenn man jemanden persönlich unsympathisch findet, dann sollte evtl. nicht persönlich auf dessen Beiträge geantwortet werden und jemand anderes sollte das übernehmen.

 

[Blut1Bart]

Das ist aber oft so, wenn man selber nicht weiterweiß, dann beleidigt man den Gegenüber persönlich.

 

[mihe7]

@Oneixee5, @Blut1Bart, Ihr solltet das Forum doch mittlerweile kennen: ein bisschen Foppen muss sein. Noch dazu, wenn die Vorlage so schön ist, kann man manchmal halt einfach nicht widerstehen.

Sachliche Begründung: es ist nicht sonderlich günstig, einen unleserlichen regulären Ausdruck zu verwenden, nur um die öffnende bzw. schließende eckige Klammer entfernt zu bekommen. Ich hatte mir oben schon überlegt, ob ich den substring-Ansatz überhaupt schreiben soll, da in meinen Augen alles Murks ist, wenn ich zuerst einen String baue(n lasse), von dem ich dann wieder etwas entfernen muss.

 

[Blut1Bart]

Du kannst natürlich aus den substring-Ansatz verwenden, aber dann hast du mehrere substring-Aufrufe, was ich als unschön empfinde.

 

[mihe7]

dann hast du mehrere substring-Aufrufe, was ich als unschön empfinde.

substring wird nur einmal aufgerufen: text.substring(1, text.length() - 1)) und es ist auch relativ deutlich, dass das erste und das letzte Zeichen entfernt werden. Der Punkt ist: warum soll man erst einen String zusammenbauen, aus dem man dann wieder etwas entfernen muss? Das ergibt wenig Sinn.

 

[Blut1Bart]

text.substring(1, text.length() - 1))

dabei hast du aber zwei Stringmethoden-Aufrufe, das ist unschön.

 

[kneitzel]

dabei hast du aber zwei Stringmethoden-Aufrufe, das ist unschön.

Könntest Du das noch erläutern? Es ist durchaus üblich, dass man bei einer String Operation auch mal die Länge braucht. Und die Alternative wäre dann etwas wie:

int textLength = text.length();
String whatever = text.substring(1, textLength - 1));

Also der reine Aufruf text.substring(1, text.length() - 1)) ist aus meiner Sicht besser lesbar als das Aufsplitten - bei dem Aufsplitten ändert sich an der eigentlichen Zeile kaum etwas (der "." und die "()" gehen und das "l" wird zu "L") und man hat eine weitere Zeile, die man mit berücksichtigen muss.

Und der Vorwurf mit mehreren Aufrufen ist auch schwer nachzuvollziehen, denn im Beispiel mit den regulären Ausdrücken sind ja auch mehrere Aufrufe auf einer Zeile.

Aber vielleicht habe ich ja etwas übersehen und etwas anderes ist gemeint gewesen.

Aber ansonsten gilt natürlich: lasst euch bei eurem gegenseitig anflaumen nicht stören ... mein Popcorn ist noch nicht alle, also bitte weiter machen.

 

[mihe7]

Aber ansonsten gilt natürlich: lasst euch bei eurem gegenseitig anflaumen nicht stören ... mein Popcorn ist noch nicht alle, also bitte weiter machen.

Ah, das war @mrBrown, der alte Hetzer

Mir gefällt weder substring noch replaceAll, aber wenn ich zwischen beiden wählen müsste, würde ich substring wählen, weil es einfacher zu lesen ist.

 

[temi]

lasst euch bei eurem gegenseitig anflaumen nicht stören

Anflaumen ist doch nett!

Ich stell mir das schön weich vor

 

[mihe7]

@temi, Konrad mag es lieber weich und kuschelig.

 

[kneitzel]

Mir gefällt weder substring noch replaceAll, aber wenn ich zwischen beiden wählen müsste, würde ich substring wählen, weil es einfacher zu lesen ist.

Nunja - da es hier wohl um eine Anfänger Aufgabe geht, baut man sich das entsprechend zusammen in einer Schleife und dann ist da nichts zu entfernen. Aber das ist im Thread ja auch als Erstes behandelt worden.

Und wenn man die aus "den Vollen" schöpfen kann/darf, dann würde mir da auch weniger Arrays.toString einfallen sondern eher ein Stream mit StringJoiner oder so.

@temi, Konrad mag es lieber weich und kuschelig.

Ja, daher wundere ich mich immer, wenn Leute so getroffen aufschreien - ich werfe doch nur mit Wattebällchen! Die können doch gar nicht weh tun!

 

[mihe7]

ich werfe doch nur mit Wattebällchen

bis die Leute bluten!

 

[Blut1Bart]

Na ja, es geht bei der Aufgabe darum, eine Lösung zu haben, die leicht verständlich ist und möglichst wenige Aufrufe enthält, denn anderes wäre das Gegenteil von leicht verständlich.

Sicher, man könnte sich noch darüber streiten, ob der regex weniger komplex gestaltet werden könnte. Das ändert aber nichts daran, dass A) die regex-Variante 2 Aufrufe braucht und B) die substring-Variante 3. Im Review würde die substring-Variante sicher durchfallen, das ist der Unterschied.

Solche fundamentals/basics sollten eigentlich jeder, der die Softwareentwicklung etwas verstehen möchte, beherrschen. Ich sehe da zwischen richtig und falsch eigentlich wenig Interpretationsspielraum.

 

[Blut1Bart]

ach mist, heute nicht auf meine Rechtschreibung achten

 

[mihe7]

ach mist, heute nicht auf meine Rechtschreibung achten

Wenn ich #38 lese, ist die Rechtschreibung das geringste Problem.

 

[Blut1Bart]

Wenn ich #38 lese, ist die Rechtschreibung das geringste Problem.

hier für dich:

Das ist aber oft so, wenn man selber nicht weiterweiß, dann beleidigt man den Gegenüber persönlich.

 

[kneitzel]

Sicher, man könnte sich noch darüber streiten, ob der regex weniger komplex gestaltet werden könnte. Das ändert aber nichts daran, dass A) die regex-Variante 2 Aufrufe braucht und B) die substring-Variante 3. Im Review würde die substring-Variante sicher durchfallen, das ist der Unterschied.

Und genau das sehe ich nicht. Die RegEx Lösung würde ich eher anmeckern als die Substring Variante.

Und ich hoffe, dass Du nicht ernsthaft die Methodenaufrufe zählst und da anfängst wegen einem Aufruf weniger zu optimieren (In den Bereich kommen wir da wohl).

Solche fundamentals/basics sollten eigentlich jeder, der die Softwareentwicklung etwas verstehen möchte, beherrschen. Ich sehe da zwischen richtig und falsch eigentlich wenig Interpretationsspielraum.

Nunja - ich sehe da durchaus Interpretationsspielraum. Aber wie Du am Thread erkennen kannst, fällt meine Beurteilung in diesem Thread auch relativ eindeutig aus - nur eben sieht die anders aus. Die Lesbarkeit ist ein wichtiges Argument, das schon gebracht wurde. Aber ich sehe auch die Komplexität des Algorithmus, der angewendet wird. Du willst vorne und hinten ein Zeichen wegnehmen und greifst diesbezüglich zu regulären Ausdrücken. Incl. möglichen Nebeneffekten (hier sind nur Zahlen, abwe was ist, wenn das zu Strings verändert wird und da dann plötzlich auch [ ] vorkommen in den Strings?)

Aber das kannst Du gerne anders sehen, damit kann ich gut leben. Ich beschäftige mich auch lieber weiter mit meinem Hotspot und digitalen Betriebsarten (Und mich wurmt es gerade, dass es da 3 Standards gibt und ich nur C4FM und D-STAR machen kann und ich noch kein Gerät habe, dass einfaches DRM/DRM+ kann. Muss ich noch ein drittes Handfunkgerät kaufen ...)

 

[mihe7]

@Blut1Bart, das war keine Beleidigung sondern die sachliche Feststellung, dass in #38 die Problematik der Rechtschreibung bei weitem nicht so schwer wiegt wie der Inhalt aus fachlicher Sicht.

 

[Blut1Bart]

intArray

int Arrays, hier ging es eigentlich nur um int Arrays...

wenn das zu Strings verändert wird

dann würde es noch ganz andere Probleme auftreten...


Na ja wurscht, schönen Abend.

 

[kneitzel]

Das es hier um int Werte ging, habe ich ja auch geschrieben, aber es ist nun einmal so, dass AAlgorithmen gerne kopiert und angepasst bzw. so verändert werden, dass diese universeller benutzt werden. Und da wird so ein Ansatz wie Du ihn gewählt hast, eher Probleme bereiten, da Dein "alle '[' und ']' ersetzen" eben deutlich mehr macht als "erstes und letztes Zeichen entfernen". Und das war ja der Kern bei eben diesen Argument.

Das nur zum Verständnis dieses einen Arguments (neben dem noch zwei Argumente standen: Komplexität des verwendeten Algorithmus und die Lesbarkeit, die bei regulären Ausdrücken oft weniger gut ist).

Wenn regulärer Ausdruck, dann wäre evtl. etwas sinnvoll wie ^\\[(.*)\\]$ wobei ich da noch der Gruppe einen Namen geben würde zur besseren Lesbarkeit. Aber auch das finde ich immer noch unleserlich. Dieses Escapen und so macht Code halt nicht wirklich lesbar.

 

[thecain]

Meiner Meinung nach ist die Diskussion eh müssig. Entweder ist die Aufgabe das "selber" zu machen, dann wird ein Schleife gefordert sein oder es ist realitätsnah und dann gibt es eigtl kein Argument gegen StringJoiner.

 

[Blut1Bart]

wie ^\\[(.*)\\]$ wobei

Wenn schon, dann bitte auch korrekt:

    public static String toStringGeneric(Object[] oa) {
        return Arrays.toString(oa).replaceAll("(^\\[)|(\\]$)", "");
    }

    public static void main(String[] args) {
        Integer[] intArray = {2, 4, 6, 8, 9, 2, 2, 5};
        String[] strings = {"]", "[", "foo", "bar"};
        System.out.println(toStringGeneric(intArray));
        System.out.println(toStringGeneric(strings));
    }

Btw "(^\\[)|(]$)" würde genügen, ist aber nicht so gut lesbar.

 

[Blut1Bart]

StringJoiner

Es soll Fälle geben, wo das nicht verfügbar oder nicht erwünscht ist.

 

[LimDul]

Es soll Fälle geben, wo das nicht verfügbar oder nicht erwünscht ist.

Dann ist aber die Variante mit RegEx mit Sicherheit auch nicht gewünscht

Wie @thecain schrieb - Entweder es geht um eine Übungsaufgabe (Dann sind eigentlich alle Utilities Methoden raus) oder es geht um eine echte Anwendung - dann kann ich immer StringJoiner verwenden, ist schließlich eine Standard-JDK Klasse.

 

[kneitzel]

Wenn schon, dann bitte auch korrekt:

Ich werde meinen Regex jetzt nicht austesten, aber ich denke, dass er durchaus aus korrekt ist.

Du hast aber sehr gut bewiesen, dass RegEx unleserlich sind. Denn bei meinem Regel wäre es eben nicht um ein reines Entfernen von unerwünschten sondern um ein Herauslesen von dem Erwünschten gegangen (was doch eigentlich auch gut am RegEx Ausdruck erkennbar ist.

Daher ist einfach nicht nachvollziehbar, dass diese RegEx Lösung preferiert wird. Aber ich schrieb schon einmal: Deine Meinung, darfst du gerne haben.