Rewriting getline with streams

[thames]

Good afternoon. How can I write the function nonOpGetLine:

 

#include <stdio.h> 


#ifndef MAXLINE
  #define MAXLINE 100
#endif  

int nonOpGetLine(char* s, int lim); 

int nonOpGetLine(char* s, int lim)
{ 
  int c;
  int i = 0; 
  while(i < lim - 1) 
  { 
     c = getchar();
     if(c == EOF) 
      break;
     if(c == '\n') 
      break; 
  
    s[i++] = c; 
  }
  s[i] = '\0';
  return i;       
}     


int main()
{ 
  int n;
  char s[MAXLINE];
  while( (n = nonOpGetLine(s, MAXLINE)) > 0)
   fputs(s, stdout);
  putchar('\n');      
  return 0;    
}

with streams like fgetc(stdin)? I tried to replace every getchar() with it but it didn't work how I was expecting.

 
#include <stdio.h> 


#ifndef MAXLINE
  #define MAXLINE 100
#endif  

int nonOpGetLine(char* s, int lim); 

int nonOpGetLine(char* s, int lim)
{ 
  int i = 0; 
  while(i < lim - 1) 
  { 
     fgetc(stdin);
     if(fgetc(stdin) == EOF)
      break;
     if(fgetc(stdin) == '\n') 
      break;
     
     s[i++] = fgetc(stdin);    
  }
  s[i] = '\0';
  return i;       
}     


int main()
{ 
  int n;
  char s[MAXLINE];
  while( (n = nonOpGetLine(s, MAXLINE)) > 0)
   fputs(s, stdout);
  putchar('\n');      
  return 0;    
}

input:
thames
output:
m

[qny]

From man fgetc:

fgetc() reads the next character from stream and returns it as an unsigned char cast to an int, or EOF on end of file or error.

getc() is equivalent to fgetc() except that it may be implemented as a macro which evaluates stream more than once.

getchar() is equivalent to getc(stdin).

You replaced your original getchar() by a bunch of fgetc() calls. One call would be reasonable ... and keep the int c variable too.

[thames]

I read earlier the man fgetc but I think I wasn't able to picture the logic. Many thanks, now I fixed it:

 
#include <stdio.h> 


#ifndef MAXLINE
  #define MAXLINE 100
#endif  

int nonOpGetLine(char* s, int lim); 

int nonOpGetLine(char* s, int lim)
{ 
  int i,c; 
  i = 0;
  
  while(i < lim - 1) 
  { 
     c = fgetc(stdin);
     if(c == EOF) 
      break;
     if(c == '\n') 
      break;
       
    s[i++] = c;    
  }
  s[i] = '\0';
  return i;       
}     


int main()
{ 
  int n;
  char s[MAXLINE];
  while( (n = nonOpGetLine(s, MAXLINE)) > 0)
   fputs(s, stdout);
  putchar('\n');      
  return 0;    
}