Wie zu beheben "conflicting types Fehler" in C?

Für die folgenden C-code (für das austauschen von zwei zahlen) bin ich immer die "Konflikt-Typen" Fehler für swap Funktion:

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

int main()
{
    int a,b;
    printf("enter the numbers to be swapped");
    scanf("%d%d",&a,&b);
    printf("before swap");
    printf("a=%d,b=%d",a,b);
    swap(&a,&b,sizeof(int));
    printf("after swap");
    printf("a=%d,b=%d",a,b);
    getch();   
}
void swap(void *p1,void *p2,int size)
{
     char buffer[size];
     memcpy(buffer,p1,size);
     memcpy(p1,p2,size);
     memcpy(p2,buffer,size);
     return(0);
}

Compiler Diagnose:

<source>:10:6: warning: implicit declaration of function 'swap' [-Wimplicit-function-declaration]
  swap(&a,&b,sizeof(int));

  ^~~~
program:15:6: warning: conflicting types for 'swap'
 void swap(void *p1,void *p2,int size)
     ^~~~

Kann jemand sagen warum dieser Fehler kommt?

Was ist die Lösung dafür?

InformationsquelleAutor Madhan | 2009-11-22
  1. 15

    Das problem ist, dass swap wurde nicht erklärt, bevor es verwendet wird. So erhält er eine "Standard-Unterschrift", wird in diesem Fall nicht mit seiner eigentlichen Unterschrift. Zitat Andrey T:

    Werden die Argumente durch eine Reihe
    der streng definierten Umwandlungen. int *
    Zeiger übergeben werden, wie int *
    Zeiger, zum Beispiel. In anderen Worten,
    die parameter-Typen sind temporär
    "abgeleitet" von argument-Typen. Nur
    der Rückgabetyp wird davon ausgegangen, dass int.

    Abgesehen davon, dass dein code erzeugt eine Reihe von anderen Warnungen. Wenn Sie mit gcc kompilieren mit -Wall -pedantic (oder sogar mit -Wextra), und sicher sein, befestigen Sie jeden Warnmeldung, bevor Sie fortfahren, um das Programm zusätzliche Funktionen. Vielleicht möchten Sie auch zu sagen, der compiler, ob Sie schreiben ANSI-C (-ansi) oder C99 (-std=c99).

    Einige Bemerkungen:

    • Leerzeichen nach den Kommas.
    • Machen main Gegenzug eine int.
      • Und machen es return 0 oder return EXIT_SUCCESS.
    • Importieren Sie die definition der getch: #include <curses.h>.
    • Importieren Sie die definition der memcpy: #include <string.h>.
    • Nicht wieder etwas in einem void Funktion.

    • Den Sie verwenden möchten, können malloc zum reservieren eines Puffers variabler Größe. Das funktioniert auch mit älteren Compilern:

      void swap(void *p1, void *p2, int size) {
    void *buffer = malloc(size);
    memcpy(buffer, p1, size);
    memcpy(p1, p2, size);
    memcpy(p2, buffer, size);
    free(buffer);
}
    • Es ist eine sehr Gründliche code-Analyse.
    • danke für den Hinweis. Klar, mein Blick das, was passiert ist, hat nicht mit der Realität übereinstimmen. Ich incorporated Ihre Bemerkung in der Antwort.
    • Nicht verwenden 'getch()' aus <curses.h> es sei denn, Sie sind auch vorbereitet zum initialisieren der Bibliothek (oder in Aufmachungen mit core-dumps, wenn Sie nicht).
    InformationsquelleAutor Stephan202
  2. 3

    Müssen Sie erklären tauschen, bevor es zu benutzen. Zum Beispiel, setzen swap oben, main, oder fügen Sie einen Prototyp für swap wie diese:

    void swap(void *,void *,int);
int main ()

    Übrigens main sollte int nicht void und in der Regel gibt es den Wert null, es sei denn, es ist ein Fehler.

    InformationsquelleAutor
  3. 3

    First off, ist die eigentliche Fehlermeldung würde nicht Schaden.

    Zweitens, was buffer [size] funktioniert nur auf manchen Compiler (das ist ein neues feature, nicht alle Compiler haben es noch). Bist du sicher, dass deins nicht?

    Drittens müssen Sie erklären tauschen, bevor Sie ihn aufrufen. Fügen Sie einen Prototyp an den Anfang der Datei:

    void swap(void *p1,void *p2,int size);
    InformationsquelleAutor Michael Kohne
  4. 2

    Konnte Sie nicht erklären swap ausdrücklich, zwingt den compiler, um Annahmen über die Funktion an der Stelle des Aufrufs. Der compiler, in übereinstimmung mit den C-Regeln wird davon ausgegangen, dass swap ist

    int swap(int *, int *, size_t)

    Später erklären Sie Ihre swap

    void swap(void *, void *, int)

    die offensichtlich Verschieden von dem, was der compiler angenommen. Dies ist der Konflikt, den der compiler dir erzählt.

    Auch, Ihre void swap versucht return 0. Was wollten Sie damit erreichen?

    P. S. Es ist int main, nicht void main.

    P. P. S. Das Programm ist nicht garantiert, erzeugen keine Ausgabe, wenn seine Ausgabe nicht Ende in ein neue-Zeile-Zeichen.

    InformationsquelleAutor AnT
  5. 1

    Kann man sich Fragen, warum das Programm kompiliert, ohne einen Prototyp für swap(), und das ist, weil der compiler ist mehr als ein C99-tool. Es kompiliert auch C89 und K&R C-Programme.

    C89 Hinzugefügt die Prototypen. Vor C89, der compiler nicht brauchen, um zu sehen, die Deklaration (Prototyp) einer Funktion es sei denn, es kehrte etwas anderes als int und die Typen der formalen Parameter nicht bekannt waren, die für den compiler überhaupt. Der compiler einfach aufgerufen, jede Funktion mit die Typen der tatsächlichen Argumente, die Sie erhalten eine Reihe von Standard-argument-Aktionen, um die Dinge zu vereinfachen. Der Programmierer würde die lint Dienstprogramm zur überprüfung der tatsächlichen und formalen Parameter. Dieses Programm ist noch im Lieferumfang der BSD-Distributionen.

    K&R Programme und die entsprechenden code-styles sind immer noch akzeptiert, die von Ihrem compiler, so dass, wenn es sieht, eine Funktion, für die keine Prototypen gibt es nur voran geht und fordert es trotzdem.

    In diesem Fall, schalten Sie Paradigmen in der zwischen dem Aufruf und der definition der Funktion. Die K&R-C Annahmen, die der compiler über das nicht deklarierte Funktion, wenn es um die generieren ein Anruf stellte sich heraus, nicht, um gültig zu sein. Selbst wenn du geschrieben hättest das ganze Programm in der K&R Stil, der compiler hätte die gleichen Beschwerden, wenn Sie erfahren, dass die real Typen der Funktionsargumente.

    InformationsquelleAutor DigitalRoss
  6. 0

    Mit GCC, das ist eine Warnung,

    wenn Sie nicht deklarieren eine Funktion vor der Verwendung der compiler versucht zu erraten, die Deklaration über die Art des Aufrufs der Funktion.Daher das Verhalten.

    InformationsquelleAutor sud03r
  7. -1

    Gut, es kompiliert auf http://codepad.org (nach dem entfernen der getch(), welche ist irrelevant). Vielleicht hat dein compiler beschwert sich über die Verwendung von memcpy, die auf nicht-eingeschränkte Hinweise?

    In swap() p1 und p2 sind nicht garantiert keine Aliase. Dies ist wirklich ein bug warten, um zu geschehen - Aufruf von swap auf &a[i] und &a[j] könnte blow up memcpy wenn i==j. Entweder memmove (die garantiert nicht zu sprengen, auf überlappende Bereiche) oder deklarieren Sie den Zeiger restricted.

    • Die fehlende Erklärung der swap kompiliert, weil offiziell ist es nicht ein Fehler in C. Es ist UB. Aber wie Sie es geschafft zu kompilieren return 0 im void Funktion ist mir nicht klar.
    • Nun, Fragen codepad admins 🙂
    • zu argumentieren, mit Ihrer Meinung von vor 10 Jahren, formal ist es ein Fehler, in C... 😉
    InformationsquelleAutor Rafał Dowgird
Schreibe einen Kommentar