Question

我正在尝试用文件的每个单词填充数组的每一行。我不想总体分配内存，所以我想知道最长的单词的长度和应该分配的行数，因此要写入文件中的单词数。我不明白代码中的问题在哪里。我认为对最长的单词进行计数应该是一个问题，因为当我在分配由函数返回的值后打印longest_file_word时，它会打印-1。

显然它不起作用。

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

int longestWord(char *file, int *nWords);
char ** Create2DStr(ssize_t numStrings, ssize_t maxStrLen);

int main(int argc, char *argv[]){
    int  file_elements_number=0 , i , j , k , z , longest_file_word , count_file_words ;
    char *filename =(char*)malloc((strlen(argv[2]) +1 )*sizeof(filename));
    strcpy( filename , argv[1]);
    for(i  =  0; i < strlen(argv[1])+1 ; i++){
        printf("%c" , filename[i]);
    }
    if(filename = NULL){
        printf("Non c'e' abbastanza memoria");
        return 1;
    }
    if(argc!=2)
    {
        printf("Errore numero parametri passati da linea di comando\n");
        return 1;
    } 
    longest = longestWord( filename , &count);
    printf("ciao %d\n%d\n", count , longest);
    char **file_words = Create2DStr(count, longest);
    FILE *file_ptr;

    const char delim[] = {" \n\t"};
    char line[260];
    char *buf = NULL;

    file_ptr = fopen( filename, "r");
    count=0;
    while(fgets(line, 260, file_ptr))
    {
        buf = strtok(line, delim);
        while(buf)
        {    
            if((strlen(buf) > 0)){
                strcpy(file_words[count], buf);
                count++;
            }
            buf = strtok(NULL, delim);
        }
    }
    for(i = 0 ; i < count ; i++){
         for( j = 0 ; j < longest ; j++){
             printf("%c" , file_words[i][j]);
         }
         printf("\n");
    }


    fclose(file_ptr);    
    free(filename);
    filename = NULL;
    return 0;


}






int longestWord(char *filename, int *nWords)
{
    FILE *file_ptr=0;
    int cnt=0, longest=0, numWords=0;
    char c;
    file_ptr = fopen(filename, "r");
    if(file_ptr){
        while ( (c = fgetc(file_ptr) ) != EOF )
        {
            if ( isalnum (c) ) {
                cnt++;  
            }
            else if ( ( ispunct (c) ) || ( isspace(c) ) || (c == '\0' ) || (c== '\n'))
            {
                (cnt > longest) ? (longest = cnt, cnt=0) : (cnt=0);
                numWords++;
            }
        }
        *nWords = numWords;
        fclose(file_ptr);
    }
    else {
        return -1;
    }

    return longest;
}

char ** Create2DStr(ssize_t numStrings, ssize_t maxStrLen){
    int i;
    char **a = {0};
    a =(char**) calloc(numStrings, sizeof(a));
    for(i=0;i<numStrings; i++)
    {
        a[i] = (char*)calloc(maxStrLen + 1, 1);
    }
    return a;
}

Answer 1

您正在做，

if(filename = NULL)

而不是

if(filename == NULL)

读取文件名后。

您应该在gcc上启用-Wall的警告的情况下进行编译。

Answer 2

结果-1表示函数longestWord无法打开指定的文件名，这可能是if(filename = NULL)的结果

除此之外，很难理解您准备使用argv[1]和argv[2]进行的工作。您根据filename的字符串长度分配内存，然后从argv[2]复制可能更长的字符串。

在访问argv[1]或argc之前，应先检查filename和argv。

用文件txt

2 个答案: