0

ファイルへのフルパスを提供するために、環境変数とプリセット文字列を組み合わせて開くファイルを取得するログ解析プログラムに取り組んでいますが、fopen で sprintf から出力を取り出すのに問題があります。環境変数とプリセット文字列を組み合わせるために使用しているので、これを適切に機能させるために何をすべきかについて誰かアドバイスを提供できるかどうか疑問に思っていましたか? ありがとう!(ここ数週間で独学で C を教え始めたばかりなので、どんなに明白なヒントであっても喜んで受け入れます)

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define _GNU_SOURCE
void main(int argc, char *argv[], char *envp[])
{
  FILE *fd; // File pointer
  char *name;
  char *filename[];
  name = getenv("MCEXEC_PLAYERNAME");
  sprintf(filename,"/home/minecraft/freedonia/playerdata/deathlog-%s.txt",name);
  char buff[1024];
  if ((fd = fopen(filename, "r")) != NULL) // open file
  {
    fseek(fd, 0, SEEK_SET); // make sure start from 0

    while(!feof(fd))
    {
      memset(buff, 0x00, 1024); // clean buffer
      fscanf(fd, "%[^\n]\n", buff); // read file *prefer using fscanf
    }
    printf("Last Line :: %s\n", buff);
  }
  else
  printf( "fail" );
}

gccを使用してコンパイル中に発生するエラーは次のとおりです

lastline.c: In function ‘main’:
lastline.c:9: error: array size missing in ‘filename’
lastline.c:11: warning: passing argument 1 of ‘sprintf’ from incompatible pointer type
/usr/include/stdio.h:341: note: expected ‘char * __restrict__’ but argument is of type   ‘char **’
lastline.c:13: warning: passing argument 1 of ‘fopen’ from incompatible pointer type
/usr/include/stdio.h:249: note: expected ‘const char * __restrict__’ but argument is of   type ‘char **’
4

1 に答える 1

3
char *filename[];

charサイズが不明なポインタの配列を宣言します。十分な既知の長さのchartoの配列が必要です。sprintf宣言する

char filename[1000];  // assuming 1000 is large enough

また

char *filename;

char名前を取得した後のmalloc十分なメモリへのポインタとして、

filename = malloc(sizeof "/home/minecraft/freedonia/playerdata/deathlog-.txt" - 1 + strlen(name) + 1);
if (!filename) exit(EXIT_FAILURE);

name予想よりも長くなった場合に不愉快な驚きを避けるため。

于 2012-05-08T23:49:02.617 に答える