c言語 ファイルの読み込み

Question

初C言語で躓いてしまっており、
ソースコードもおかしな点があると思いますので、教えて頂ければと思います。

前提・実現したいこと

test.cfgファイルにスペース区切りで２つの数値があります。

その数値をweight1, weight2のポインタ(float型)変数として
別の関数から呼び出せるようにしたいです。
（書き方自体が間違っていればご指摘お願いします）

まずは２つの数値をポインタ（float型）変数としてcfgファイルから読み込みたいとのですが、どのようにすれば宜しいでしょか。

該当のソースコード　修正版

回答頂いたコードを元に変更すると、コンパイル時にエラーが発生しています。
layer.cでsse_ng_gpuにweight1, weight2を渡していますが、
渡し型の問題でしょうか？

main.cの// read values from fpで間違いなければ別の質問を
作成しようと思いましたが、元々の質問と繋がっているかもしれず
こちらでまずは確認させて頂きました。

元々のコード記載ファイル(main.c)

void parse_config(float* weight1, float* weight2)
{
    char values[100+1];
    char *filename;
    long lines;

    // read file
    filename="test.cfg";
    FILE* fp = fopen(filename, "r");
    lines = fread(values, sizeof(float), 1, fp);

    // read values from fp
    sscanf(values, "%f %f\n", weight1, weight2);
    //*weight1 = 
    //*weight2 =
    fclose(fp);
}

上記を呼び出す別ファイル(layer.c)

void forward_cost_layer(cost_layer l, network net)
if
//途中コード省略
else if(l.cost_type == SSE_NG){
        float weight1 = 0.0;
        float weight2 = 0.0;
        parse_config(&weight1, &weight2);
        sse_ng_gpu(l.batch*l.inputs, net.input_gpu, net.truth_gpu, l.delta_gpu, l.output_gpu, weight1, weight2);
               }  else {
        l2_gpu(l.batch*l.inputs, net.input_gpu, net.truth_gpu, l.delta_gpu, l.output_gpu, weight1, weight2);
    }

エラー

./src/cost_layer.c: In function ‘forward_cost_layer’:
./src/cost_layer.c:147:95: error: incompatible type for argument 6 of ‘sse_ng_gpu’
 l.batch*l.inputs, net.input_gpu, net.truth_gpu, l.delta_gpu, l.output_gpu, weight1, we
                                                                            ^
compilation terminated due to -Wfatal-errors.
Makefile:103: recipe for target 'obj/cost_layer.o' failed
make: *** [obj/cost_layer.o] Error 1

該当のソースコード

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

void parse_config(float* weight1, float* weight2)
{
    char values[100+1]
    char *filename
    float *weight1;
    float *weight2;
    float lines;

    // read file
    filename="test.cfg";
    FILE* fp = fopen(filename, "r");
    lines = fread(values, sizeof(float), 100, fp);

    // read values from fp
    *weight1 =
    *weight2 =

    fclose(fp);
}

test.cfg

100 1

試したこと

上記のコードは他にも色々コードが書いてあるので、
一旦今回の目的機能のみ抜き出して下記のようにスクリプトを書いてみました。

試したことは、まずはstrtokを用いてcfgから数値を文字列として
読み取り、そこからatofを用いてstr to floatなどを行えればと思いました。

ただ、上手くcfgファイルから値が読み取れず、キャストも失敗します。
恐らくやり方自体がシンプルではないと感じてますので、
解決方法を教えて頂ければと思います。

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

int main(void) {
    FILE *fp;
    char *filename="sse_ng.cfg";
    char values[1000];
    char *value1;
    char *value2;
    float *weight1;
    float *weight2;
    float lines;

    fp = fopen(filename, "r");
    lines =fread(values, sizeof(float), 100, fp);

    value1 = strtok(values, " ");
    while(value1 != NULL) {
        value1 = strtok(NULL, " ");
        printf("%s\n", value1);
        if(value1 != NULL) {
            value2 = value1;
            }
        }

    printf("%s", value1);
    printf("%s", value2);
    weight1 = atof(value1);
    printf("%f", *weight1);
    fclose(fp);
}

よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

ubuntu

Answer 1

読み込むところまではできてるので、ちょっとの手直しでいける、と思いましたが、ちょいちょいミスってますね。

• セミコロンがない
• freadの戻り値の型
• freadに与えるサイズ
• 引数と同じ名前の変数を宣言している

これらを踏まえて修正してみました。

#include <stdio.h>

void parse_config(float* weight1, float* weight2);

int main(void)
{
	float weight1;
	float weight2;
	parse_config(&weight1, &weight2);
	printf("%f %f\n", weight1, weight2);

	return 0;
}

void parse_config(float* weight1, float* weight2)
{
    char values[100+1];
    char *filename;
    long lines;

    // read file
    filename="test.cfg";
    FILE* fp = fopen(filename, "r");
    lines = fread(values, sizeof(values), 1, fp);

    // read values from fp
	sscanf(values, "%f %f\n", weight1, weight2);

    fclose(fp);
}

Comments

[MF_19]

早速ありがとうございます。

教えて頂いたコードで試すと、別のエラーが発生しました。
void parse_config内の// read values from fp以下ですが、
下記のような定義は不要でしょうか？
*weight1= ~~~ もしくはweight1=~~~

詳細は質問に更新という形で記載します。

もし別の問題でしたら、別の質問を作成しようと思います。

[MF_19]

追加の質問は本件とは完成ありませんでした。

引数で渡すときに型指定が間違っていました。
ありがとうございました。