Die übergabe von 2d-char-array in Funktion

Ich versuche zu pass ein 2D-array mit strings in einer Funktion. Ich bekomme immer wieder die
erwartet Ausdruck vor ] token.

Den Punkt dieses Kodex ist zu Lesen in einem wordsearch puzzle, dann finden Sie die Wörter in das Rätsel. Ich werde schriftlich eine Funktion für Suchlauf vorwärts, backword suchen, und dann nach oben und nach unten suchen.

Wie bekomme ich diesen Fehler loszuwerden? Der Fehler ist unten, ganz unten, wo ich den Anruf weiterleiten-Funktion.

    /*Andrea Hatfield CPE 101 October 31st, 2012*/
    #include <stdio.h>
    #include <string.h>

    int forward(char words[][8], char puzzle[][11]);


    int main()
    {
    char puzzle[11][11];
    char words[8][8];
    FILE *fin, *fwords;
    int i = 0;
    int j= 0;

    fin = fopen("puzzle.in", "r"); 
    fwords = fopen("words.in", "r");


    if(fin == NULL) /*Reads in the puzzle file*/
    printf("File does not exist");
    else
    {
            while(fscanf(fin,"%s", puzzle[i])!=EOF)
            {
                    printf("%s\n", puzzle[i]);
                    i++;
            }


    }

    if(fwords == NULL) /*Reads in the words the puzzle will search for */
    printf("File does not exist");
    else
    {
            while(fscanf(fwords, "%s", words[j])!=EOF)
            {
                    printf("%s\n", words[j]);
            }
    }


    forward(&words[][8], &puzzle[][11]); /*Error at this point*/
    return(0);
    }

Herzlich willkommen auf Stackoverflow, kannst du bitte eine klare Frage?

InformationsquelleAutor hatfieldac | 2012-11-02

2d c function string

Es ist ein wenig schwer zu bekommen, was Sie wollen zu tun, sondern es ist ein problem hier:

forward(&words[][8], &puzzle[][11]);

versuchen Sie dies:

forward(words, puzzle);

Hier sind zwei weitere Optionen, sollte erwähnt werden, aus hier:

#define ROWS 4
#define COLS 5

void func(int array[ROWS][COLS]) {
  int i, j;

  for (i=0; i<ROWS; i++) {
    for (j=0; j<COLS; j++) {
      array[i][j] = i*j;
    }
  }
}

void func_vla(int rows, int cols, int array[rows][cols]) {
  int i, j;

  for (i=0; i<rows; i++) {
    for (j=0; j<cols; j++) {
      array[i][j] = i*j;
    }
  }
}

int main() {
  int x[ROWS][COLS];

  func(x);
  func_vla(ROWS, COLS, x);
}

InformationsquelleAutor 0x90

3

Müssen Sie rufen Sie direkt
```
 forward(words, puzzle);
```
Allerdings hast du einen Fehler in deinem code: du hast vergessen die j++ im lese-Zyklus beim Lesen words, so sind Sie überschreiben words[0] und nicht die Initialisierung der anderen Mitglieder der Reihe.

Auch, in einer realen situation, vielleicht finden Sie es wünschenswert, wählen Sie die array-Größe zur Laufzeit. Ein einfacher Weg, es zu tun wäre, um wählen Sie eine hinreichend große Begrenzung für Breite und Höhe, und gehen Sie mit 0x90 Lösung (Sie benötigen eine 30x20 puzzle, und Sie können es leicht zu behandeln, da Sie kompiliert mit ZEILEN und SPALTEN gleich, sagen wir, 1024).

Ein komplizierter Weg wäre, um malloc() (mit Zeiger auf Zeiger auf chars) und weisen beide arrays dynamisch, begrenzt durch den verfügbaren Speicher; Sie würde übergeben Sie dann Ihren Abmessungen der forward Funktion.
```
int forward(char **words, char **puzzle, size_t w_w, size_t w_h, size_t p_w, size_t p_h)
{

}
```
Doch, die Zuteilung wäre komplizierter gegeben, die Schleife ruft zu malloc() und die Notwendigkeit zu prüfen, den return-Wert jedes mal abzufangen, out-of-memory Bedingungen; auch in einigen Szenarien, die Sie möchten, könnte der Speicher sein dezugeordnet (z.B. zum ausführen von wiederholten Rätsel), das führt zu weiterer Komplexität.
- Nicht einverstanden mit der zweiten Hälfte von diesem post, dass macht den code deutlich komplizierter, ohne Grund. Das ursprüngliche design ist in Ordnung. Sie können übergeben, die Dimensionen des Arrays als zusätzliche Parameter, falls gewünscht.
- Tut mir Leid, @M. M , aber... ist das nicht was ich vorgeschlagen? Versorgung von außerhalb genau das meinte, dass, übergeben Sie die Dimensionen als zusätzlichen Parameter. Da gibt es zwei arrays (Worte und Rätsel), ich brauche vier zusätzliche Parameter. Ich gebe zu, dass ich erlaubt habe, für rechteckige puzzles; Platz für diejenigen, die Sie brauchen nur die Hälfte der Parameter 🙂
- Sie sind auch darauf hindeutet Wechsel zu übergeben, ein Zeiger-auf-Zeiger und Verwendung von malloc (was ich nicht bin, nur auf der Durchreise zu den Dimensionen und Beibehaltung der vorhandenen Methode der Zuweisung des Arrays)
- Ich sehe. Ja, das betrachtet werden könnte, eine zusätzliche Komplexität. Hatte es nicht in den Sinn gekommen. Ich werde formulieren Sie die Antwort entsprechend aus; vielen Dank!
InformationsquelleAutor LSerni

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.