Vergleichen von zwei Dateien in c

Wie der Titel sagt, brauche ich zum vergleichen von 2 Dateien.Wenn eine Zeile kommt ,dass ist nicht das gleiche zwischen diesen 2 Dateien,print, Online (Von beiden Dateien).Der code gibt keine Fehler senden.Ich kann nicht scheinen, um in der Lage sein den Fehler in meinem code.

#include <stdio.h>
#include <string.h>
#define MAX 1000

int main(int argc, char *argv[]) {
    char c1, c2;
    char s1[MAX], s2[MAX];
    char *p1;
    char *p2;
    FILE *fp1;
    FILE *fp2;
    fp1 = fopen(argv[1], "r");
    fp2 = fopen(argv[2], "r");
    p1 = s1;
    p2 = s2;
    if (argc != 3) {
        printf("Wrong use of program \n ");
        return -1;
    }
    if (fp1 == NULL || fp2 == NULL) {
        printf("One or both of the files can't be used \n ");
        return -1;
    }
    while ((c1 = getc(fp1)) != EOF || (c2 = getc(fp2)) != EOF) {
        *p1 = c1;
        *p2 = c2;
        p1++;
        p2++;

        for (c1 = getc(fp1); c1 != '\n'; p1++) {
            *p1 = c1;
        }
        *p1 = '\0';

        for (c2 = getc(fp2); c2 != '\n'; p2++) {
            *p2 = c2;
        }
        *p2 = '\0';
        if (!(strcmp(s1, s2))) {
            printf("%s \n ", s1);
            printf("%s \n ", s2);
            return 0;
        }
    }
    return 0;
}
  • Ein nicht senden Fehler?
  • Kompilieren mit allen Warnungen und debugging-Informationen (z.B. gcc -Wall -g auf Linux) und lernen, wie Sie den debugger verwenden (z.B. gdb auf Linux).
  • Für den Anfang, ich würde mich der Suche an, wo ordnen Sie die Zeichenfolgen zu s1 und s2.
  • Was ist Los mit 2 fgets() und ein strcmp() im while - Schleife?
  • Sie wollen wahrscheinlich sowohl s1 und s2 beim start mit memset. Verwenden Sie dann den debugger (Schritt für Schritt), um Sie anzuzeigen. Beachten Sie, dass Ihr Programm nicht auf Dateien arbeiten, die mit großen Linien (mit mehr als 1000 Zeichen in einer Zeile). Denken Sie daran, dass sich der source-code kompiliert, ist der einfache Teil, das Debuggen ist der eigentliche Spaß.
  • ok, werde versuchen, dass zuerst.Das Programm wird kompiliert,habe nicht versucht, zu Debuggen, obwohl es
  • Auch überlegen, was passieren wird, wenn Sie Ihre Datei entweder endet Sie in einem single-line-feed, oder eine Linie, die ist nicht beendet mit einem line-feed...
  • Noch wichtiger ist, welche Art nicht fgetc zurück? int... Was ist der Bereich der möglichen Werte zurückgeben? [ EOF, 0..UCHAR_MAX ]. Unter der Annahme char signiert ist, kann es darstellen UCHAR_MAX? Angenommen char ist unsigned, wird (c2 = getc(fp2)) != EOF jemals falsch sein? Bitte, verwenden Sie einfach einen int zu speichern den Rückgabewert von getc.

InformationsquelleAutor Lind | 2013-03-16
  1. 1

    Auf die beiden for Schleifen Sie die getc auf die Initialisierung Teil der Schleife, es wird ein Charakter in c1 und c2 einmal und führen Sie dann die Schleife, bis Sie über die Grenze s1 und s2 durch Ihre p1 und p2 Zeiger. Da die c1 != '\n' ist nie erfüllt, es sei denn Ihr erstes Zeichen in der Zeile ein "\n "" es wird wahrscheinlich werfen Sie einen segmentation fault Fehler.

    Auf die while zumindest mit gcc und auf meinem system, es hat lazy evaluation (EDIT: Nicht faul, aber short-circuit-evaluation, siehe Kommentare) und die c2 = getc(fp2) Teil wird nicht ausgeführt, wenn der andere Teil war wahr.

    Du auch nicht zurücksetzen p1 und p2 nach jeder Zeile.

    strcmp gibt 0 zurück, wenn beide strings gleich sind, und C 0 ist false und ungleich null ist wahr, so sind Sie verlassen sich auf das erste match.

    Hier eine kaum funktionierende version des Codes, die Sie noch brauchen, um daran zu arbeiten und berücksichtigen die Fälle, wie eine Datei, die kürzer als die andere, eine Zeile in einer Datei, die größer als 1000 Zeichen (so wie es jetzt ist, würde es überlaufen s1 oder s2), etc.

    #include <stdio.h>
#include <string.h>
#define MAX 1000

int main(int argc, char *argv[]) {
  char c1, c2;
  char s1[MAX], s2[MAX];
  char *p1;
  char *p2;
  FILE *fp1;
  FILE *fp2;
  p1 = s1;
  p2 = s2;
  if (argc != 3) {
    printf("Wrong use of program \n ");
    return -1;
  }
  fp1 = fopen(argv[1], "r");
  fp2 = fopen(argv[2], "r");
  if (fp1 == NULL || fp2 == NULL) {
    printf("One or both of the files can't be used \n ");
    return -1;
  }
  c1 = getc(fp1);
  c2 = getc(fp2);
  while ((c1 != EOF) && (c2 != EOF)) {
    for (; c1 != '\n'; p1++) {
        *p1 = c1;
        c1 = getc(fp1);
    }
    *p1 = '\0';

    for (; c2 != '\n'; p2++) {
        *p2 = c2;
        c2 = getc(fp2);
    }
    *p2 = '\0';
    if ((strcmp(s1, s2)) != 0) {
        printf("%s\n", s1);
        printf("%s\n", s2);
        return 0;
    }
    c1 = getc(fp1);
    c2 = getc(fp2);
    p1 = s1;
    p2 = s2;
  }
  if (c1 != EOF || c2 != EOF)
    printf("One of the files ended prematurely\n");
  return 0;
}
    • Vielen Dank das Wars was ich brauchte.Ich weiß, dass wenn in einer Zeile größer ist als 1000 verursacht es einen Fehler,ich wollte nur testen Sie den code mit weniger Zeilen als jetzt.
    • Um es klarzustellen: die Sprachen C und C++ definieren, was passiert, ausführen, wenn (x || y) wenn x nicht null ist - speziell, y wird nicht ausgewertet (es muss nicht sein, denn x ist wahr'). Es ist nicht "lazy evaluation" und es ist nicht "zumindest mit gcc und auf meinem system". Es ist definiert in der Sprache; es ist immer.
    • Vielen Dank für die Klarstellung. Meine schlecht, ich bekam das falsche Konzept, es nennt sich "short-circuit-evaluation", ist es nicht?
    • Ja, das ist es. Es gilt zu && an | | aus, und der ternäre operator ? wie in (a > b) ? a : b.
    InformationsquelleAutor Jorge Núñez
  2. 2

    Wenn die beiden Dateien sind text-Dateien, dann, ganz ehrlich, ich würde von vorne anfangen mit ein viel einfacheres Programm, das mit fgets() statt getc() und vergleicht zeilenweise mit strcmp() anstelle von Zeichen. Es sind einfach zu viele Fehler im code, wie es ist - Sie werden die Aufgabe besser, wenn Sie schnell verwerfen, was Sie haben und beginnen Sie wieder mit der einfacheren Lösung.

    Durch die Art und Weise, strcmp gibt null zurück, wenn die beiden Zeichenfolgen identisch sind, nicht null, wenn Sie anders sind.

    InformationsquelleAutor jarmod
Schreibe einen Kommentar