入力ファイルの内容を検証する

Question

防御的なプログラミングの問題があり、解決方法がよくわかりません。

ファイルパスとテーブルのサイズ（行数/列数）を引数として受け取るこの関数があり、入力ファイルを検証するより良い方法を探しています。この関数の引数は常に正しいと仮定します。 size ファイルに格納されているテーブルの「小さい方の側」を表します。

例えば ​​：

1 2 3 4 
5 6 7 8 

サイズ= 2は正しいです

1 2 3 4 5
5 6 7 8 9

サイズ= 2が正しくありません

このようなファイルも拒否できるようにしたいです

1 2 3 4 5 6 7 8

サイズ = 2 (fscanf を通じて受け入れられます)

拒否できるようにしたい別の種類のファイルは次のとおりです。

1 2 3
4 5 6

サイズ = 2

今のところ、私の唯一のセキュリティは、ファイルの要素が本当に数値であるかどうかを確認することです。

これまでに行ったコードは次のとおりです。

void import(float** table, int size, char* path)
{
    FILE* data = fopen(path, "r");
    assert(data);
    int i,j;
    int st;

    for (i=0; i<size; i++)
    {
        for(j=0; j<(size*2)-1; j++)
        {
            st = fscanf(data, "%f", &table[i][j]);
            if (!st)
            {
                printf("Error while importing the file.\n");
                fclose(data);
                return -1;
            }
        }
    }
    fclose(data);
}

どこからどのように始めればよいのか全く分かりません。私は C にあまり熟達していません。やりたいことを実行するための関数やメカニズムはたくさんあるようですが、それらはすべて非常に複雑に見え、実際にコードよりも長いものもあります。提供された。

誰かが私に正しい方向を示してくれれば、それは素晴らしいことです。

Answer 1

for ループは次のようになります。

char line[1000], *token;
for (i = 0; i < size; i++) // for each line
{
    if (fgets(line, 1000, data) != NULL) // read line
    {
        token = strtok (line," ");
        for (j = 0; j < (size * 2) - 1; j++) // for each number from line
        {
            if (sscanf(token, "%f", &table[i][j]) <= 0)
            {
                // there are columns missing:
                printf("Error while importing the file.\n");
                fclose(data);
                return -1;
            }
            token = strtok (NULL," ");
        }
    }
    else
    {
        // there are rows missing:
        printf("Error while importing the file.\n");
        fclose(data);
        return -1;
    }
}

また、次の点にも注意してください assert(data); は次のようなものに置き換える必要があります。

if (!data)
{
    printf("Error while openning the file [filePath=\"%s\"].\n", filePath);
    cleanExit();
}

Answer 2

行末を簡単に検出することはできません scanf(), したがって、それを直接使用すると基準を満たしません。

おそらく行全体を読む必要があります(fgets() 多分 getline())、各行を順番に処理します。ライン処理で使用できるのは、 sscanf(), を使用できます。 %n 指令も。概要をまとめると、次のようになります。

for (line_num = 0; line_num < size; line_num++)
{
    ...read line from file into buffer line, checking for EOF...
    start = line;
    for (i = 0; i < 2 * size; i++)
    {
        if (sscanf(start, "%f%n", &value, &offset) != 1)
            ...ooops - short line or non-numeric data...
        else
        {
            start += offset;
            table[line_num][i] = value;
        }
    }
}
...check that there's no clutter after the last expected line...

Answer 3

ファイル全体のチェックサムを計算することもできます。問題は、あなたがどれだけ真剣にそれに取り組んでいるかです。XOR チェックサムを作成するのは簡単ですが、衝突に対して完全に安全というわけではありません。重要な場合は、おそらく sha-1 のようなものを使用するのが最善です。