Lösung für "dereferenzieren" void * "- Zeiger" - Warnung in struct in C?

Ich denke, Ihr design ist zum scheitern verurteilt, aber ich bin auch nicht überzeugt, dass die anderen Antworten sehe ich, voll befassen sich mit der tieferen Gründe, warum.

Scheint es, dass Sie versuchen, verwenden Sie C, um den Umgang mit generischen Typen, etwas, das immer bekommt haarige. Sie können es tun, wenn Sie vorsichtig sind, aber es ist nicht einfach, und in diesem Fall habe ich Zweifel, ob es lohnenswert wäre.

Tieferen Grund: nehmen wir an, wir bekommen Vergangenheit die bloße syntaktische (oder kaum mehr als syntaktische) Probleme. Dein code zeigt, dass T10CNT enthält 20 int und T20CNT enthält 20 long. Auf modernen 64-bit-Maschinen - andere als unter Win64 - sizeof(long) != sizeof(int). Also, der code, der in Ihrem Druck-Funktion sollte die Unterscheidung zwischen array-Dereferenzierung int arrays und long arrays. In C++ gibt es eine Regel, dass Sie sollten nicht versuchen, die Behandlung von arrays, polymorph, und diese Art der Sache ist, warum. Die CMNCNT Typ enthält 3 long Werte sind, anders als die T10CNT und T20CNT Strukturen in der Anzahl, wenn der Basistyp des Arrays entspricht T20CNT.

Stil Empfehlung: ich empfehle die Vermeidung führende Unterstriche auf Namen. In allgemein, Namen beginnend mit Unterstrich beginnen, sind reserviert für die Umsetzung zu nutzen und zu verwenden, wie Makros. Makros haben keinen Respekt für den Anwendungsbereich, wenn die Durchführung definiert ein makro _cnt es würde ruinieren Ihre code. Es sind Nuancen, welche Namen sind reserviert; ich bin nicht about to gehen Sie in diese Nuancen. Es ist viel einfacher zu denken 'beginnen im Namen mit Unterstrich beginnen, sind reserviert', und Steuern Sie klar von Schwierigkeiten.

Stil Vorschlag: Ihre print-Funktion gibt Erfolg bedingungslos. Das ist nicht sinnvoll; Ihre Funktion sollte nichts zurück, so dass der Anrufer nicht haben, um zu testen, die für Erfolg oder Misserfolg (da kann es nie nicht). Eine sorgfältige Programmierer, der stellt fest, dass die Funktion einen status testen immer die return-status und error-handling-code. Dieser code wird nie ausgeführt werden, so ist es tot, aber es ist schwer für alle, die (oder der compiler) zu bestimmen.

Oberfläche Beheben: Vorübergehend, können wir davon ausgehen, dass Sie behandeln können int und long als Synonyme; aber Sie müssen raus aus der Gewohnheit zu denken, dass Sie sind Synonyme, obwohl. Die void * argument ist der richtige Weg, zu sagen: "diese Funktion nimmt einen Zeiger von unbestimmter Art". Jedoch, innerhalb der Funktion, die Sie benötigen, zu konvertieren von einem void * auf einen bestimmten Typ, bevor Sie die Indizierung.

typedef struct _CMNCNT
{
    long    count[3];
} CMNCNT;

static void printCommonStatistics(const void *data, size_t nelem, size_t elemsize)
{
    int i;
    for (i = 0; i < nelem; i++)
    {
        const CMNCNT *cmncnt = (const CMNCNT *)((const char *)data + (i * elemsize));
        fprintf(stdout,"STATISTICS_INP: %ld\n", cmncnt->count[0]);
        fprintf(stdout,"STATISTICS_OUT: %ld\n", cmncnt->count[1]); 
        fprintf(stdout,"STATISTICS_ERR: %ld\n", cmncnt->count[2]);
    }
}

(Ich mag die Idee von einem Datei-stream genannt stout zu. Vorschlag: verwenden Sie cut ' N ' Paste auf den echten source code--es ist sicherer! Ich bin in der Regel verwenden "sed 's/^/ /' file.c" vorzubereiten code für cut ' N ' Paste in ein SO Antwort.)

Was bedeutet, dass cast line tun? Ich bin froh, dass Sie gefragt...

• Der erste Vorgang ist die Umwandlung von const void * in eine const char *; dies können Sie tun, byte-Größe, die Operationen auf die Adresse. In den Tagen vor der Standard C char * verwendet wurde anstelle von void * als die Universelle Adressierung Mechanismus.
• Die nächste operation fügt die richtige Anzahl von bytes zu erhalten, um den start der ite element des array von Objekten der Größe elemsize.
• Der zweiten cast dann sagt der compiler "Vertrauen Sie mir - ich weiß, was ich Tue" und "behandeln Sie diese Adresse als die Adresse eines CMNCNT Struktur".

Von dort, der code ist einfach genug. Beachten Sie, dass seit dem CMNCNT Struktur enthält long Wert, den ich verwendet %ld die Wahrheit zu sagen,fprintf().

Da Sie nicht über die zum ändern der Daten in dieser Funktion, ist es nicht eine schlechte Idee, verwenden Sie die const qualifier wie ich es Tat.

Beachten Sie, dass, wenn Sie gehen, um treu sein zu sizeof(long) != sizeof(int), dann müssen Sie zwei separate code-Blöcke (ich würde vorschlagen, getrennte Funktionen) mit dem deal Reihe von int' und 'array von long' Struktur-Typen.

InformationsquelleAutor Jonathan Leffler

Ändern Sie die Deklaration der Funktion char * etwa so:

static int printCommonStatistics(char *cmncnt, int cmncnt_nelem, int cmncnt_elmsize)

den void-Typ nicht davon ausgehen, keine Besondere Größe in der Erwägung, dass ein char, nehmen eine byte-Größe.

InformationsquelleAutor Adam Pierce

Die Funktion

static int printCommonStatistics(void *cmncntin, int cmncnt_nelem, int cmncnt_elmsize)
{
    char *cmncntinBytes;
    int ii;

    cmncntinBytes = (char *) cmncntin;
    for(ii=0; ii<cmncnt_nelem; ii++)
    {
        CMNCNT *cmncnt = (CMNCNT *)(cmncntinBytes + ii*cmncnt_elmsize);  /* Ptr Line */
        fprintf(stdout,"STATISTICS_INP: %d\n",cmncnt->_cnt[0]);
        fprintf(stdout,"STATISTICS_OUT: %d\n",cmncnt->_cnt[1]); 
        fprintf(stdout,"STATISTICS_ERR: %d\n",cmncnt->_cnt[2]);
    }
    return SUCCESS;
}

Funktioniert für mich.

Das Problem ist, dass auf der Linie, sagte: "Ptr-Line" fügt der code einen Zeiger auf eine Ganzzahl. Da unser pointer ist ein "char *" wir bewegen uns in Erinnerung sizeof(char) * ii * cmncnt_elemsize, das ist, was wir wollen, da ein char ist ein byte. Ihr code versucht, eine gleichwertige Sache voran sizeof(void) * ii * cmncnt_elemsize, aber leere nicht eine Größe haben, damit der compiler gab Sie den Fehler.

Würd ich ändern T10CNT und T20CNT sowohl mit int oder long anstelle von einer mit jedem. Sie sind abhängig von sizeof(int) == sizeof(long)

InformationsquelleAutor David Norman

Ihren Ausdruck

(CMNCNT *)&cmncntin[ii*cmncnt_elmsize]

versucht, die Adresse von cmncntin[ii*cmncnt_elmsize] und dann die Besetzung, die Zeiger auf Typ (CMNCNT *). Es kann nicht die Adresse von cmncntin[ii*cmncnt_elmsize] weil cmncntin hat den Typ void*.

Studie C-operator prš und einfügen von Klammern, wo nötig.

InformationsquelleAutor Windows programmer

C ist nicht meine Tasse o'java, aber ich glaube, dein problem ist, dass "void *cmncnt" sollte CMNCNT *cmncnt.

Fühlen Sie sich frei, mich zu korrigieren, jetzt C-Programmierer, und Sag mir, das ist der Grund, warum java-Programmierer können nicht schöne Dinge haben.

InformationsquelleAutor Ari Ronen