Problem mit strcmp

eine Warnung habe ich nicht erhalten

und wenn wir schon dabei sind:
"j < = i"

Ich mach mal nen Gegenvorschlag:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(void) {
  char (*array)[10] = NULL;
  int i = 0;

  do {
    char (*tmp)[10] = realloc(array, (i + 1) * sizeof(char[10]));

    if(tmp == NULL) {
      free(array);
      array = NULL;
      break;
    }

    array = tmp;

    if(NULL == fgets(array[i], 10, stdin)) {
      break;
    }

    printf("%s", array[i]);
  } while(strcmp(array[i++], "end\n") != 0); /* Ui, endlich mal ein Anwendungsfall für Postinkrement :D */

  if(array != NULL) {
    int j;

    for(j = 0; j < i; ++j) {
      puts(array[j]);
    }
  }

  free(array);

  return 0;
}

Natürlich wäre es effizienter, die Allokation in größeren Blöcken durchzuführen, aber das sei dir als Übungsaufgabe überlassen.

Ich weiß, ich sollte die Fehlerbehandlung nicht weglassen :). Danke für deine Mühe, einen sauberen Text zu schreiben :).

qwert_zuiopü schrieb:

eine Warnung habe ich nicht erhalten

Dann erhöhe den Warnlevel vom Compiler (alles).
Beachte und behebe die Warnungen.

SeppJ

DirkB schrieb:

qwert_zuiopü schrieb:

eine Warnung habe ich nicht erhalten

Dann erhöhe den Warnlevel vom Compiler (alles).
Beachte und behebe die Warnungen.

Das ist ein "Feature" vom GCC. Er hat vermutlich auf O0 compiliert, da hat das j einen definierten Wert. Daher bekommt man erst mit Optimierungen eine Warnung.

Also ich erhalte auch mit O2 keine Warnung.

Es scheint so, als würde jetzt der Wert von j automatisch die 0 zugewiesen bekommen.

rüdiger

qwert_zuiopü schrieb:

Also ich erhalte auch mit O2 keine Warnung.

Dann hast du Warnungen vermutlich deaktiviert. Setz mal folgende Flags: -Wall -Wextra -pedantic-errors -std=c99

(oder besser -std=c11 bzw. -std=c1x wenn du einen aktuellen GCC hast)

Danke, so funktioniert es.

Weil es sich immer noch auf das selbe Beispiel bezieht, schreibe ich meine nächste Frage nochmal hier rein:

char **array;
    array = malloc(sizeof(char *));
    int i=0;
    for(;;i++) {
        array = realloc(array,(i+1)*sizeof(char *));
        array[i] = malloc(10*sizeof(char));

        if(fgets(array[i],10,stdin) == NULL)
            return -1;

        if(strcmp(array[i],"end\n") == 0)
            break;
    }

    printf("Ihre Eingaben:\n");
    for(int j = 0 ;j<=i;j++)
        printf("%s",array[j]);

    qsort(&array[0],i,10*sizeof(char),qsort_strcmp);

    printf("\n\nSortiert:\n");
    for(int j = 0 ;j<=i;j++)
        printf("%s",array[j]);

    for(int j=0;j<=i;j++)
        free(array[j]);

    free(array);

mit

int qsort_strcmp(const void *left, const void *right) 
{ 
    return strcmp(left, right); 
}

Scheinbar rufe ich die qsort-Funktion falsch auf. Was mache ich genau falsch?

SeppJ

Das qsort ruft die Vergleichsfunktion mit Zeigern auf die Elemente auf. Deine Elemente sind hier Zeiger auf deine Zeichenketten. Also bekommt qsort_strcmp Zeiger auf Zeiger auf die Zeichenketten. strcmp erwartet aber die Zeiger auf die Zeichenketten.

Zudem ist natürlich falsch, wie du qsort an sich aufrufst, denn deine Elemente haben Größe sizeof(char*) nicht Größe 10 (sizeof(char) ist übrigens per Definition immer 1 und &array[0] ist als Funktionsparameter das gleiche wie einfach nur array).