for文あたりで動作が停止する

Question

前提・実現したいこと

txtファイルからデータを取得し最大のscoreを出したいです。一応、0.5以上差が開いたら、新しい最大値をだしたいのでk++にしています。
変更後のコードです

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

int max_score[100],max_time[100],k=0,j,i;

struct lv{
	float score;
	float time;
	struct lv *next;
};
struct lv *add_list(float t,float s ,struct lv *head){
		struct lv *new_head; // 宣言
		new_head = (struct lv *)malloc(sizeof(struct lv)); // 領域確保
		new_head->time = t;// 新しいデータの代入
		new_head->score = s;// 新しいデータの代入
		new_head->next = head; // 古い head を新しい head の次のデータに指定
		return new_head;
}
void max(struct lv *head){
		if(head->score>max_score[k]){
			max_score[k]=head->score;
			max_time[k]=head->time;
		}
		else{
			if(max_score[k]-head->score  >0.5){
				k++;
			}
		}
	if(head->next !=NULL)
   			 max(head->next);
	else{
		for(j=0;j==k;j++){
			printf("番号：%d,時間：%f,値：%f\n",j+1,max_time[j],max_score[j]);
		}
	}
}
void main(){
	FILE *fp;
	struct lv *lv;
	struct lv *head=NULL;
	float t,s;
	int i=1000,n=0;
	
	if ((fp=fopen("s:\jx18250_0426_100Hz.txt","r"))==NULL){
		 	printf("Cant open file %s\n", "a.txt");  /*オープンに失敗したらメッセージ表示*/ 
	 	}
	while(fscanf(fp,"%f,%f",&t,&s)!= EOF){
		head=add_list(t,s,head);
	}
	
	max(head);
	
	free(lv);
	fclose(fp); 
}

試したこと

max()のところはとまるのはなくなりましたが、時間、値が0.000のままです。txtからの時間、値がとりだせていることは確認済みです。

Comments

[cateye]

ポインタのNU L L判定は？

[tmtmmm]

そこが原因だったようです

Answer 1

• maxの最初のfor(i=0;i<100;i++){は98～100回同じことを繰り返しているだけなので不要です。
• for(j=0;j=i;j++){この処理の意味が全てのmax_time、max_scoreを表示するというものであれば、jは* max_time/scoreの添え字なので、この配列の要素数(100)未満という条件であるべきだと思います。そうでない場合、恐らくfor文である必要はありません。どちらにせよ、場所を移動するか実行するかどうかの条件を付けるべきです。
• max(head->next);は最終的にnullになるためnullチェックが必要です。
• struct lv *lv;使用していないため不要です。
• headおよびhead->nextはmallocされていますが、freeされていません。

Comments

[tmtmmm]

返信ありがとうございます。３つめの最大値未満という条件であるべきというところがどうすればよいのかわかりません。ご教授おねがいします。nullチェックは修正ずみです。

[moredeep]

すみません、表現が間違っていました。配列の要素数未満が正しいですね。
配列はmax_score[100]と100個の要素を持っているので、今回は100未満です。

>時間、値が0.000のままです
こちらはif文の条件の各値がどのようになっているかを確認してみたほうがいいかと思います。

Answer 2

can110 さんご回答の最後の一文が的と思いますので, 再帰で無いコードを^^;
ただ, 結果が何を表すものなのか全然分かりませんので, 合っているのやら…

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

//#define TEST //テスト時定義

#define ARRAY_MAX 100
float max_score[ARRAY_MAX], max_time[ARRAY_MAX];
int max_len=0;

struct lv{
  float score;
  float time;
  struct lv *next;
};

struct lv *add_list(float t, float s ,struct lv *head){
  struct lv *new_head; // 宣言
  new_head = (struct lv *)malloc(sizeof(struct lv)); // 領域確保
  new_head->time = t;// 新しいデータの代入
  new_head->score = s;// 新しいデータの代入
  new_head->next = head; // 古い head を新しい head の次のデータに指定
  return new_head;
}

struct lv *readfile(char *filename) {
  struct lv *head=NULL;

#ifdef TEST
  head = add_list( 1.0, 100.0, head);
  head = add_list( 2.0,  99.0, head);
  head = add_list( 3.0, 101.0, head);
  head = add_list( 4.0, 110.0, head);
  head = add_list( 5.0,  88.0, head);
  head = add_list( 6.0,  80.0, head);
  head = add_list( 7.0, 100.0, head);
  head = add_list( 8.0,  95.0, head);
  head = add_list( 9.0, 103.0, head);
  head = add_list(10.0,   0.0, head);
#else
  float t,s;
  FILE *fp;

  if((fp = fopen(filename, "r")) == NULL) {
    printf("Can't read file. errno=%d filename='%s'\n", errno, filename);
    return NULL;
  }
  while(fscanf(fp, "%f,%f", &t, &s) != EOF) {
    head = add_list(t, s, head);
  }
  fclose(fp);
#endif

  return head;
}

void max(struct lv *head) {
  for(struct lv *p=head; p!=NULL; p=p->next) {
    if(p->score > max_score[max_len]){
      max_score[max_len] = p->score;
      max_time[max_len] = p->time;
    } else if(max_score[max_len]-p->score > 0.5) {
      if(++max_len >= ARRAY_MAX) return;
    }
  }
}

void free_list(struct lv *head) {
  for(struct lv *p=head, *next; p!=NULL; p=next) {
    next = p->next;
    free(p);
  }
}

void main() {
  struct lv *head = readfile("s:\jx18250_0426_100Hz.txt");
  if(head == NULL) exit(1);

  max(head);

  free_list(head);

  for(int i=0; i<max_len; i++) {
    printf("番号：%d,時間：%f,値：%f\n", i+1, max_time[i], max_score[i]);
  }
}

Answer 3

時間、値が0.000のままです。

printf("番号：%d,時間：%f,値：%f\n",j+1,max_time[j],max_score[j]);
int値に対して書式%fを使うのは誤りです。
というかmax_timeはintでよいのでしょうか？？

Answer 4

if(head->next !=NULL)
    max(head->next)

---

何をしようとしているのかわかりませんのでnullチェックここに入れるのが適切かわかりませんが
とにかくnullチェックが抜けてます。

あと全体的にwarning出まくりなので(使ってない変数あったり条件式がj=iだったり)
ちゃんとコンパイラの言うこと聞いてあげてください。

Comments

[tmtmmm]

返信ありがとうございます。nullチェックをしたところうごきました。ですが、時間、値が０をずっと繰り返しています。txtからのデータ取得はかくにんずみです。

[ozwk]

C言語でないと駄目なとき以外はC言語はもはや使わないほうが労力的にいいんですが
Cじゃないと駄目なんですか?

課題だっていうなら頑張ってくださいとしか言えませんが。

Answer 5

詳しくは見ていませんが、maxの再帰呼び出しから抜け出せてないと思います。
maxの先頭あたりで、headがNULLなら即returnするようにしてみては。

Comments

[tmtmmm]

返信ありがとうございます。時間、値が０をずっと繰り返していて最大値が取得できていないのですがどうすればいいでしょうか？

Answer 6

ぱっとみただけですが、

for(j=0;j=i;j++){

    printf("%d,時間：%f,値：%f",j+1,max_time[j],max_score[j]);
}

このfor文、条件のところが間違ってます

Comments

[tmtmmm]

返信ありがとうございます。
j==iということですかね？

[y_waiwai]

そこで何をしようとしてるところまでは見てないですが、、
j==i としたなら、jとiが同じ時だけループするという動作になりますが、そんでいいのでしょうか

[tmtmmm]

ご指摘ありがとうございます。jとiが同じ時だけループするという動作でだいじょうぶでしたが数がおおいので100で制限しましたが最大値、時間が0.00000となってしまいます。ｔｘｔからのしゅとくはできています

[y_waiwai]

えーと、、同じ値の時だけループする、とすると、i が０の時に、一回だけループする、それ以外の時は全くループされない、という動作になると思いますが。