2

これにはいくつかのエントリがあることを知っていますが、それらを調べたところ、目的の場所にたどり着くことができませんでした。

ディレクトリの内容を読み取り、通常のファイルを別の関数に渡してアーカイブを構築できる関数を構築しています。

関数に渡すことができるitemsというアイテムのようなargv[]を作成しようとしています。問題は、ファイル名を入力する必要があることです。（多くの例で見たように）静的に宣言するのではなく、ファイル数に基づいてmallocを使用したいと思います。

クイックの関数ヘッダーは次のとおりです。

void quick(int argc, char *argv[])

追加機能は次のとおりです。

void append(char* argv[]){
    struct dirent *dp;
    int count, i = 3;
    char *files;
    char items [*files][255];
    DIR *dirp = opendir(".");
    if(dirp == NULL){
        fail('f');
    }

    printf("ar: adding files in current directory to archive: %s", argv[2]);

    while((dp = readdir(dirp)) != NULL){
        if(dp->d_type == DT_REG){
            count++;
        }
    }

    rewinddir(dirp);
    files = malloc(count*sizeof(char));

    //copy argv[2] archive name, into files, so we can pass to quick
    strcpy(items[2], argv[2]);

    while((dp = readdir(dirp)) != NULL){
        errno = 0;
        dp = readdir(dirp);

        //leave is dir is empty
        if(dp == NULL)
            break;

        // Skip . and ..
        if (strcmp(dp->d_name, ".") == 0 || strcmp(dp->d_name, "..") == 0)
            continue;

        //if file is not the archive and a regular file add to list
        if(strcmp(dp->d_name, argv[2]) != 0 && dp->d_type == DT_REG){
            strcpy(items[i], dp->d_name);
            i++;
        }
    }
    closedir(dirp);

    quick(i, items);
}

互換性のないポインター型としてitemsargでエラーが発生し、それらの謎をまだマスターしていないため、malloc（および場合によっては配列）を正しく実行しなかったと推測しています。

4

4 に答える 4

3

の宣言char items [*files][255]が正しくありません。

基本的に、あなたfilesは配列の配列になりたいです。char**したがって、ポインタからポインタ（つまり）タイプである必要があります。次に、次のように、実際の文字列を保持するために、その配列内の各配列を割り当てる必要があります。

char** files;
files = malloc(count * sizeof(char*));      // allocate the array to hold the pointer
for (size_t i = 0; i < count; i += 1)
    files[i] = malloc(255 * sizeof(char));  // allocate each array to hold the strings

使い終わったら、メモリを適切に解放します。

for (size_t i = 0; i < count; i += 1)
    free(files[i]);
free(files);
于 2013-02-02T23:13:57.053 に答える
1

「文字列」の「配列」を増やす方法は次のとおりです。

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

int AddString(char*** strings, size_t* count, const char* newStr)
{
  char* copy;
  char** p;

  if (strings == NULL ||
      newStr == NULL ||
      (copy = malloc(strlen(newStr) + 1)) == NULL)
    return 0;

  strcpy(copy, newStr);

  if ((p = realloc(*strings, (*count + 1) * sizeof(char*))) == NULL)
  {
    free(copy);
    return 0;
  }

  *strings = p;

  (*strings)[(*count)++] = copy;

  return 1;
}

void PrintStrings(char** strings, size_t count)
{
  printf("BEGIN\n");
  if (strings != NULL)
    while (count--)
      printf("  %s\n", *strings++);
  printf("END\n");
}

int main(void)
{
  char** strings = NULL;
  size_t count = 0;
  PrintStrings(strings, count);
  AddString(&strings, &count, "Hello World!");
  PrintStrings(strings, count);
  AddString(&strings, &count, "123");
  AddString(&strings, &count, "ABCDEF");
  PrintStrings(strings, count);
  return 0;
}

出力（ideone）：

BEGIN
END
BEGIN
  Hello World!
END
BEGIN
  Hello World!
  123
  ABCDEF
END
于 2013-02-02T23:24:24.337 に答える
0

ここを見て：

char *files;
char items [*files][255];

malloc初期化された後に使用することをお勧めしfilesます。

私が理解しているように、これは例えば

char items [count][255];

intC99スタイル。

ところで、初期化されていない状態でcount使用されます。最初はゼロに設定してください。

于 2013-02-02T23:01:08.383 に答える
0

列の長さが静的にわかっているので、割り当てることもできます

char (*items )[255];
// obtain row count
items = malloc(rowCount * sizeof(*items));

へのポインタへのメモリchar[255]。そうすることで、連続したメモリブロックが得られ、局所性が向上する可能性があり、必要なfreeポインタは1つだけです。

可変長配列を使用して、行数が多すぎない場合は、

rowCount = whatever;
char items[rowCount][255];

ただし、C99以降がサポートされている場合は、最適なオプションになる可能性があります。

于 2013-02-02T23:31:06.437 に答える