Блог

 #include "Definition.h"
#include <stdio.h>
#include "ExternalVar.h"
#include <stdlib.h>
#include <string.h>

extern int Readline(),CountWord(),CountsUpdate();


char Line[MaxLine];  /* one line from the file */


int NChars = 0,  /* number of characters seen so far */
    NWords = 0,  /* number of words seen so far */
    NLines = 0,  /* number of lines seen so far */
    LineLength;  /* length of the current line */ 

int wc = 0,
    lc = 0,
    cc = 0,
    tc = 0;


int i;

main(int argc, char *argv[])  
{
    FILE *fp;
    fp= fopen(argv[1],"r");
    if (fp) 
    {
        while (i=fscanf(fp,"%s",Line)!=EOF)
        ///printf("%s n",Line);
        cc=Readline(Line);
        printf("%d n",cc);
        fclose(fp);


    }
    return 0;  
}
  

Это моя основная функция, я передаю строку массива символов[] в функцию Readline (строка), и я хочу, чтобы она возвращала количество символов в массиве.

Вот функция Readline.c . Независимо от того, какой текстовый файл я использую в качестве аргумента, я продолжаю получать 84 в качестве возвращаемого значения. Не уверен, что я делаю неправильно или правильно, так как я новичок в программировании на языке Си. Пожалуйста, помогите.

 #include "Definition.h"
#include "ExternalVar.h"
#include <stdio.h>



int Readline(char *Line)
{   
int i;


for (i = 0; Line[i] == 'n' ; i  )
{

return i;
}
}
  

Вывод выглядит следующим образом

taj@taj:~/Desktop/2014_Summer_CIS5027_Asg2$ gcc Main.c Readline.c taj@taj:~/Desktop/2014_Summer_CIS5027_Asg2 $ ./a.out b.txt

84

 int Readline(char *Line){
    int i;
    for (i = 0; Line[i] != '' ; i  )//newline is not included in the Line. Because read by fscanf(fp,"%s",Line)
        ;

    return i;
}
  

У вас мало проблем:

1) Вы, вероятно, хотите проверять, пока не нажмете n . Таким образом, условие действительно должно быть:

 for (i = 0; Line[i] != 'n' ; i  )
  

2) Ваш ввод может не содержать n . Поэтому вам также следует проверить, достигли ли вы конца строки:

 for (i = 0; Line[i] amp;amp; Line[i] != 'n' ; i  )
  

3) Что делать, если первый символ находится n в Line ? Он вообще не войдет в цикл, и в этом случае вы ничего не возвращаете. Итак, вы должны инициализировать i=0 и вернуть его. Вы могли бы переписать функцию как:

 int Readline(char *Line)
{   
int i = 0;

  for (i = 0; Line[i] amp;amp; Line[i] != 'n' ; i  )
  {
  ;
  }

return i;
}
  

4) fscanf() with %s format не читается n , даже если ваш входной файл содержит его. Вы, вероятно, хотите использовать fgets() для чтения, который читает n символы:

 while (fgets(Line, sizeof Line, fp)) {
  ...
}
  

5) функция main() должна возвращать int : int main(int argc, char *argv[]) {... }