Strlen возвращает фактическую длину + 2

Question

Почему strlen возвращает 6 вместо 4, хотя файл содержит четырехбуквенное слово без пробелов после слова?

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

void leggi_file(FILE *in, char *s);
char * vect(char *s);
int pal(char *);

int main(void){
    FILE *in;
    char *s[100], *v;
    int n;
    in = fopen("testfile", "r");
    leggi_file(in,*s);
    printf("%s",*s);
    v = vect(*s);
    printf("%s",v);
    n = pal(*s);
    printf("%d",n);
    int dim = strlen(*s);
    printf("%d",dim);
    return 0;
}

void leggi_file(FILE *in, char *s){
    int c, count = 0;
    while((c=getc(in))!=EOF && count < 100){
        s[count] = c;
        count++;
    }
}

char * vect(char *s){
    char *vect;
    int dim = strlen(s);
    vect = malloc((dim+1)*sizeof(char));
    vect = s;
    return vect;
}

int pal(char *s){
    int i, tmp, dim;
    dim = strlen(s);
    tmp=dim-1;
    for(i=0; i<(dim/2); i++){
        if(*(s+i) != *(s+tmp))
            return 0;
        else
            tmp--;
    }
    return 1;
}

свидетельство содержит "анна"

Answer 1

Вы, вероятно, хотите этого.Объяснение в комментариях.Тем не менее, еще есть возможности для дальнейшего улучшения.

Я не уверен, что цель вызова vect.

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

void leggi_file(FILE *in, char *s);
char *vect(char *s);
int pal(char *);

int main(void) {
  FILE *in;
  char s[100], *v;   // just use s, you want an array of char, not an array of
  int n;             //  pointers to char
  in = fopen("data.txt", "r");
  if (in == NULL)  // check if file could actually be opened
  {
    printf("Can't open file\n");
    exit(1);
  }
  leggi_file(in, s);
  fclose(in);       // close file
  printf("%s", s);
  v = vect(s);
  printf("%s", v);
  n = pal(s);
  printf("%d", n);
  int dim = strlen(s);
  printf("%d", dim);
  free(v);           // free memory
  return 0;
}

void leggi_file(FILE *in, char *s) {
  int c, count = 0;
  while ((c = getc(in)) != EOF && count < 100) {
    if (c == '\n')  // stop on first end of line
      break;
    s[count] = c;
    count++;
  }

  s[count] = 0;   // put string terminator, omitting this
}                 // was one of the major problems in your code

char * vect(char *s) {
  char *vect;
  int dim = strlen(s);
  vect = malloc((dim + 1) * sizeof(char));
  strcpy(vect, s);    // copy string
  return vect;
}

int pal(char *s) {
  int i, tmp, dim;
  dim = strlen(s);
  tmp = dim - 1;
  for (i = 0; i < (dim / 2); i++) {
    if (*(s + i) != *(s + tmp))
      return 0;
    else
      tmp--;
  }
  return 1;
}

Также (*(s + i) != *(s + tmp)) более читабельно, если вы пишете (s[i] != s[tmp]), что полностью эквивалентно.