문자열이 올바르게 토큰 화되지 않았습니다.

Question

IP 주소를 통해 하나의 수식을 적용하고 있습니다. 그러나 때때로 나에게 무작위적인 결과를 주는데, 그것은 우연히 일어난다. 때로는 올바르게 토큰 화되지 않고 때로는 0 값을 부여합니다. 오류 재연의 비율은 1000 샘플에서 거의 2-5입니다.
호출 함수 = 수식 (IPADDRESS); 내가 초당 수백 개의 다른 IP를 사용하고있다.

int formula (char ip[]){ 

      char *token = NULL; 
      unsigned long value; 
      int finalv; 
      char ipaddress[16]; 
      char delims[]=".";   
      int octet=3; 
      value = 0; 
      FILE *fp; 
      fp = fopen(LOGFILE, "a"); 
      strcpy(ipaddress,ip); 
      fprintf(fp, "%s\n",ipaddress); 
      token = strtok(ipaddress,delims); 

      while(token != NULL) { 
       fprintf(fp,"%d",(atoi(token)); 
       if (atoi(token) != 0){ 
        if(octet ==3) 
          value = value + (255 * 255 * 255 * (atoi(token))); 
        else if(octet ==2) 
          value = value + (255 * 255 * (atoi(token))); 
        else if(octet ==1) 
          value = value + (255 * (atoi(token))); 
        else if(octet ==0) 
          value = value + (atoi(token)); 
       } 
        octet--; 
       token = strtok(NULL,delims); 
      } 

      finalv = value % 9999; 

      fprintf(fp, " -- %d \n",(int)finalv); 
      if(fp) 
        fclose(fp); 
      return (int)v; 
    } 

OUTPUTFILE

: 는

172.17.82.255 
172 0 1 -- 1983 
-- 
172.254.254.254 
172 0 0 0 -- 1728 
-- 
172.255.255.225 
172 0 -- 1728 
-- 
172.255.255.255 
172 21 0 1 -- 7390 
-- 
172.255.84.255 
172 0 8 -- 3768 

Answer 1

strtok()을 반복하는 대신

int a=0, b=0, c=0, d=0 ; 
int value ; 
sscanf(ip, "%d.%d.%d.%d", &a, &b, &c, &d); 
value = ((a * 255 + b) * 255 + c) * 255 + d ; 
finalv = value % 9999 

나는 같은 대답을해야한다고 생각합니다. (비록 코멘트에서 지적한 바와 같이, 255은 256을 의미 할 수 있음).

Answer 2

당신은 token를 재 선언하고 (난 단지 잘못된 출력을 감안할 때 한) : 나중에 코드에서

 char *token = NULL; // <-- here 
     unsigned long value; 
     int finalv; 
     char ipaddress[16]; 
     char token[]="."; // <-- and here 

, 다음, 당신은 약간의 혼동에 대한이 어떤 문맥에서 token이 사용되는 것으로 가정합니다. 여기서 특히 명백합니다 :

... 여기서 첫 번째 토큰은 아마도 포인터와 두 번째 배열을 의미합니다.

컴파일 된 것조차 마음에 들지만, 컴파일러가 적어도 그것에 대해 경고하지는 않는지 조금 놀랍습니다.

EDIT : token는 정의되지 않은 동작에 이르게하는 NULL 때 "업데이트"코드에서

fp = fopen(LOGFILE, token); 

가 실행된다. 당신은

fp = fopen(LOGFILE, "w"); 

을 의미 했습니까?

Answer 3

변화

token = strtok(NULL,token);

token = strtok(NULL,".");

PS에 : 초당 번이 기능 수백를 호출하기 때문에, 당신은 더 나은 개방 거라고 및 기능

외부 파일을 닫습니다