C言語 ファイルから読み込んだデータを指示通り変数に代入できない

Question

プログラムの内容

c言語で、ファイルから読み取ったデータをもとにサーバーが動いているかどうかを確認するプログラムを作成しています。
読み込むファイル
20201019133124,10.20.30.1/16,-
20201019133125,10.20.30.2/16,1
20201019133134,192.168.1.1/24,10
20201019133135,192.168.1.2/24,5
20201019133224,10.20.30.1/16,522
20201019133225,10.20.30.2/16,1
20201019133234,192.168.1.1/24,8
20201019133235,192.168.1.2/24,15
20201019133324,10.20.30.1/16,-
20201019133325,10.20.30.2/16,3
20201019134510,192.168.1.1/24,1
20201019135050,192.168.1.2/24,15
20201019135051,10.20.30.1/16,-
20201019135052,10.20.30.2/16,3
20201019135055,10.20.30.1/16,2
20201019135100,10.20.30.2/16,2
20201019135510,192.168.1.1/24,-
20201019135550,192.168.1.2/24,15
20201019135600,192.168.1.1/24,-
20201019135601,192.168.1.2/24,15
20201019135610,192.168.1.1/24,1
20201019135611,192.168.1.2/24,15
左から＜確認日時＞,＜サーバアドレス＞,＜応答結果＞という並びになっています。
応答結果で-が返ってきた場合にサーバー故障とみなすようになっています。

このファイルを読み込み、故障状態のアドレスとそのアドレスの故障期間を出力するプログラムを作りたいと考えています。
(故障期間は、最初に-が出てから次にちゃんとしたping値がかえるまでの期間)

該当のソースコード

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

#define N 50
struct HANDAN{
  char handan_name[N];
  int kaisuu;
  long time;
  bool count;
};

struct LOG{
  long time;
  char server_name[N];
  char result[N];
};

void printTime(long kikan){
  char kikan2[14];
  sprintf(kikan2,"%ld",kikan);
  int len = strlen(kikan2);
  long sec,min,hour,day,month,year;
  year = kikan/10000000000;
  month = (kikan/1000000)%100;
  day = (kikan/10000)%100;
  hour = (kikan/100)%100;
  min = kikan/100;
  sec = kikan%100;

  if(sec>=60){
    sec = sec-60;
    min = min+1;
  }
  if(min>=60){
    sec = min-60;
    hour = hour+1;
  }
  //printf("%ld年%ld月%ld日%ld時%ld分です\n",year,month,day,hour,min);
  //printf("kikan is %ld\n",kikan);
  
  if(len<=2){
    printf("故障期間は%ld秒です\n",sec);
  }else if(len<=4){
    printf("故障期間は%ld分%ld秒です\n",min,sec);
  }else if(len<=6){
    printf("故障期間は%ld時%ld分%ld秒です\n",hour,min,sec);
  }else if(len<=8){
    printf("故障期間は%ld日%ld時%ld分%ld秒です\n",day,hour,min,sec);
  }else if(len<=10){
    printf("故障期間は%ld月%ld日%ld時%ld分%ld秒です\n",month,day,hour,min,sec);
  }else if(len<=14){
    printf("故障期間は%ld年%ld月%ld日%ld時%ld分%ld秒です\n",year,month,day,hour,min,sec);
  }
}

  
  
int main(){
  FILE *fp;
  struct HANDAN han[N];
  struct LOG log[N]; 
  char s[N][N],*cp;
  char name[4][15]={"10.20.30.1/16","10.20.30.2/16","192.168.1.1/24","192.168.1.2/24"};
  const char *sikiri=",";
  long handan_time=0,kikan;
  int i = 0,j = 0,h=0;
  int kaisuu[4] = {};
  bool count[N]={};//countによって故障したかどうか判断
  
  fp = fopen("server.txt","r");
  
  while(fgets(s[i],N,fp)!=NULL){
    cp = strtok(s[i],sikiri);
    log[i].time=atol(cp);//ここで時間を挿入、文字列から数に変換
    int c=0;
    while(cp!=NULL){
      cp = strtok(NULL,sikiri);
      if(cp!=NULL&&c==0){
	strcpy(log[i].server_name,cp);	//ここでサーバーの名前を挿入
	c++;
	
	for(h=0;h<=j;h++){
	  if(han[h].count==true){ //if(故障していたら)故障状況をプリント
	    if(strcmp(han[h].handan_name,log[i].server_name)==0){	      
	      kikan=log[i].time-han[h].time;
	      printf("サーバー%sが故障状態です\n",han[h].handan_name);
	      printf("log.time is %ld\nhandan is %ld\n",log[i].time,han[h].time);
	      printTime(kikan);
	      han[h].count = false;
	      
	    }
	  }
	  
	}
	
      }
      
      if(cp!=NULL&&c==1){
	strcpy(log[i].result,cp);//ここで結果を挿入
	if(strncmp(log[i].result,"-",1)==0){                      //1."-"かどうか
	  for(h=0;h<4;h++){//回数計算
	    if(strcmp(log[i].server_name,name[h])==0){           //入力された値がどれかと一緒ならkaisuuを+1する 
	      if(kaisuu[h]==0){                                   //2.一回目か
		strcpy(han[j].handan_name,log[i].server_name);
		han[j].time = log[i].time;
	      }
	      kaisuu[h]++;
	    }
	  }
	  //printf("%d %d %d %d\n",kaisuu[0],kaisuu[1],kaisuu[2],kaisuu[3]);
	}else{
	  for(h=0;h<=j;h++){
	    if(strcmp(han[h].handan_name,log[i].server_name)==0){//名前が以前に出てきていないか確認
	      han[h].count=true;
	    }
	  }
	}
      }
      j++;
    }
    
    i++;
  }
  
  fclose(fp);
  return 0;
}

発生している問題

実行してみると、以下のように出力されます。

サーバー10.20.30.1/16が故障状態です
log.time is 20201019133324
handan is 20201019133124
故障期間は2分0秒です
サーバー10.20.30.1/16が故障状態です
log.time is 20201019135051
handan is 20201019133124
故障期間は19分27秒です
サーバー10.20.30.1/16が故障状態です
log.time is 20201019135055
handan is 20201019133124
故障期間は19分31秒です
サーバー192.168.1.1/24が故障状態です
log.time is 20201019135610
handan is 20201019135510
故障期間は1分0秒です

出力された結果を見ると、応答結果で-が出た後、応答結果が正常かどうかは関係なく同じアドレスのログを一つ飛ばしで参照されてしまいます。

例）

サーバー10.20.30.1/16が故障状態です

と出力された後、本当ならば
20201019133224,10.20.30.1/16,522
のログを参照して変数に代入したいのですが、
20201019133324,10.20.30.1/16,-
のログを参照してしまいます。

どう変更すればうまく参照できるようになるのでしょうか。
言葉足らずな説明で申し訳ありません。

Comments

[tatsu99]

これは、何かの課題なのでしょうか。
それとも、実際の業務で使うものなのでしょうか。

[tatsu99]

課題なら現行の方法になるかも知れませんが、
実際の業務ならもっと改善の余地があります。
言語はC言語でなく、perlとかrubyでやった方が簡単です。
現行ではserver.txtの行数が最大50行、監視対象のサーバーが４つの前提になりますが、
実際の業務ならserver.txtの行数及び監視対象のサーバー数に特に制約を設けたくないかと思います。
perlとかrubyなら制約を設けなくても簡単に実現できます。
C言語でやる場合なら、server.txtの行数は制限なしにできますが、サーバー数については、制限が必要に
した方が簡単に実現できます。（工数を増やせば制限なしにすることも可能ですが・・・）

Answer 1

とりあえず、修正しました。
Nがいろいろな用途で使用されていたので１つの用途に限定しました。

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

#define N (50)          //読み込むファイルの１行の長さ
#define H (100)         //判断テーブルhan[]の最大要素数
struct HANDAN {
    char handan_name[N];
    int kaisuu;
    long time;
    bool count;
};

struct LOG {
    long time;
    char server_name[N];
    char result[N];
};

void printTime(long kikan)
{
    char kikan2[14];
    sprintf(kikan2, "%ld", kikan);
    int len = strlen(kikan2);
    long sec, min, hour, day, month, year;
    year = kikan / 10000000000;
    month = (kikan / 1000000) % 100;
    day = (kikan / 10000) % 100;
    hour = (kikan / 100) % 100;
    min = kikan / 100;
    sec = kikan % 100;

    if (sec >= 60) {
        sec = sec - 60;
        min = min + 1;
    }
    if (min >= 60) {
        sec = min - 60;
        hour = hour + 1;
    }
    //printf("%ld年%ld月%ld日%ld時%ld分です\n",year,month,day,hour,min);
    //printf("kikan is %ld\n",kikan);

    if (len <= 2) {
        printf("故障期間は%ld秒です\n", sec);
    } else if (len <= 4) {
        printf("故障期間は%ld分%ld秒です\n", min, sec);
    } else if (len <= 6) {
        printf("故障期間は%ld時%ld分%ld秒です\n", hour, min, sec);
    } else if (len <= 8) {
        printf("故障期間は%ld日%ld時%ld分%ld秒です\n", day, hour, min,
               sec);
    } else if (len <= 10) {
        printf("故障期間は%ld月%ld日%ld時%ld分%ld秒です\n", month, day,
               hour, min, sec);
    } else if (len <= 14) {
        printf("故障期間は%ld年%ld月%ld日%ld時%ld分%ld秒です\n", year,
               month, day, hour, min, sec);
    }
}



int main()
{
    FILE *fp;
    struct HANDAN han[H];
    struct LOG log;
    char s[N], *cp;
    const char *sikiri = ",";
    long kikan;
    bool find;
    int i = 0,  h = 0;
    int han_ctr = 0;                                //han[N]に登録した件数
    fp = fopen("server.txt", "r");

    while (fgets(s, N, fp) != NULL) {
        cp = strtok(s, sikiri);
        if (cp == NULL) continue;
        log.time = atol(cp);                    //ここで時間を挿入、文字列から数に変換
        cp = strtok(NULL, sikiri);
        if (cp == NULL) continue;
        strcpy(log.server_name, cp);            //ここでサーバーの名前を挿入
        cp = strtok(NULL, sikiri);
        if (cp == NULL) continue;
        strcpy(log.result, cp);                 //ここで結果を挿入
        if (strncmp(log.result, "-",1) == 0) {     //1."-"かどうか
            find = false;
            for (h = 0; h < han_ctr; h++){      //判断テーブル検索
                if (strcmp(log.server_name, han[h].handan_name) == 0){
                    find = true;
                    break;
                }
            }
            //判断テーブル上にサーバーが存在しない場合
            if (find == false){
                if (han_ctr >= H){  //上限オーバーなら、次のログの読み込みへ
                    printf("判断テーブル上限:%d オーバー |%ld|%s|の故障監視をスキップ\n",H,log.time,log.server_name);
                    continue;
                }
                //判断テーブルに新規登録（値は初期値）
                h = han_ctr;
                strcpy(han[h].handan_name,log.server_name);
                han[h].count = false;
                han[h].time = 0;
                han[h].kaisuu = 0;
                han_ctr++;
            }
            han[h].kaisuu++;    //回数カウントアップ
            if (han[h].count == false){ //故障中でなければ、故障中にし、時刻を登録する。
                                        //故障中ならなにもしない。（回数カウントアップのみ）
                han[h].count = true;
                han[h].time = log.time;
            }
        }else{
            //"-"以外のケース
            for (h = 0; h < han_ctr; h++) {
                if (strcmp(log.server_name, han[h].handan_name) == 0) { //入力された値がどれかと一緒なら検索終了
                    if (han[h].count == true){   //故障中なら故障状況をプリント
                        kikan = log.time - han[h].time;
                        printf("サーバー%sが故障状態です\n",
                               han[h].handan_name);
                        printf("log.time is %ld\nhandan is %ld\n",
                               log.time, han[h].time);
                        printTime(kikan);
                        han[h].count = false;
                    }
                    continue;                   //次のログの読み込みへ
                }
            }
        }
    }

    fclose(fp);
    return 0;
}

実行結果は以下の通り
サーバー10.20.30.1/16が故障状態です
log.time is 20201019133224
handan is 20201019133124
故障期間は1分0秒です
サーバー10.20.30.1/16が故障状態です
log.time is 20201019135055
handan is 20201019133324
故障期間は17分31秒です
サーバー192.168.1.1/24が故障状態です
log.time is 20201019135610
handan is 20201019135510
故障期間は1分0秒です

ところで、年末に故障が発生し、翌年復旧するケースで例えば、以下のようなデータ
20201231235959,10.20.30.1/16,-
20210101000009,10.20.30.1/16,1
を処理すると
サーバー10.20.30.1/16が故障状態です
log.time is 20210101000009
handan is 20201231235959
故障期間は69月76日41時88697640分88697580秒です
のような結果になりますが、これはよろしいのでしょうか。
（故障期間は10秒です。・・・・これが期待する結果のような気がします）

Answer 2

回答としては不十分かと思いますが、、、

まず、中で何をしているか把握しているでしょうか? インデントが途中で合っていないところを見ると、どこからか、コピーし、張り合わせたのでしょうか。

さて、最初にある

if(cp!=NULL&&c==0){

のカッコの中(インデントが合ってないので範囲が分かりにくい)で、サーバーの名前をコピーし、for(h=0;h<=j;h++){では何をしたいのでしょう? 次の if()のコメントは、"(故障していたら)"とありますが、ここでは、故障かどうかの判定はできません。

次の

if(cp!=NULL&&c==1){

で、応答結果を見ているので、ここで故障かどうかの判定となると思います。
if(strncmp(log[i].result,"-",1)==0){ が故障だと思いますが、kaisuu[h]に入れた値は何に使うのでしょう? ここで、han[h].count=true;にすべきでは?
で、ここの }else{が故障じゃない時と思わるので、前回が故障なら、、という処理にすべきではないかと考えます。

直接、コードを書くのも良いですが、どこで何をすべきかの検討が足りてないように思われます。

また、jの変数は何を意味しているでしょうか? han[N]のインデックスにも参照されているようですが、多分、オーバーフローしてます。

具体的にどう書き換えるかは、全体の流れを見直す事が必要です。
(このコードの流れで書き換えるのはちょっと大変か?)

中のwhile() 内について、ちょっと参考コード。こんな感じかという事で。

    while(cp!=NULL){
      cp = strtok(NULL,sikiri);
      if(cp!=NULL&&c==0){
        strcpy(log[i].server_name,cp);    //ここでサーバーの名前を挿入
        //// ここから   対応するサーバー番号を探す hに設定  (無い場合は? ...)
        // for(h=0;h<4;h++){
        //   if(strcmp(log[i].server_name, name[h])==0) break;
        // }
      }

      if(cp!=NULL&&c==1){
        strcpy(log[i].result,cp);//ここで結果を挿入
        if(strncmp(log[i].result,"-",1)==0){                      //1."-"かどうか
          //// 障害が発生した場合には、それを記録  サーバー番号 h
          // if (!han[h].count) {	// ただし、最初に初期化で false としておく
          //    han[h].count = true;
          //    han[h].time = log[i].time
          // }
        }else{
          //// 前回が障害の場合の処理
          // if (han[h].count) {
          //   kikan=log[i].time - han[h].time;
          //   printf("サーバー%sが故障状態です\n",han[h].handan_name);
          //   printf("log.time is %ld\nhandan is %ld\n",log[i].time,han[h].time);
          //   printTime(kikan);
          //   han[h].count = false;
          // }
        }
      }
    }

Comments

[pepperleaf]

書き換えてみようかと思ったのですが、以下の理由で保留。
- VCでコンパイルすると、intのサイズが違うとかで動作が変わる
- 細々した箇所が気になり、大幅に手を入れたくなる

Answer 3

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。