MQL4のFileReadString()でCSVファイルを読み取り、変数に値を代入したいがログが空白になる。

Question

実現したいこと

FileReadString()でCSVファイルを読み取り、変数に値を代入したい。

発生している問題・エラーメッセージ

ログにて、sRowの右が空白になってしまいます。csvファイル(MQL4>Filesに保存)は、A列１行目にddddとのみ、入力してあります。ログのこの部分に、ddddと表示されるかと思いましたが、されませんでした。

該当のソースコード

void OnStart()
{

string sRow;

int fileHandle = FileOpen("test.csv", FILE_READ | FILE_CSV);

if(fileHandle != INVALID_HANDLE)
{
while(FileIsEnding(fileHandle) == false)
{string sRow = FileReadString(fileHandle);}
Print("sRow ",sRow);
FileClose(fileHandle);
}

}

Answer 1

ログにて、sRowの右が空白になってしまいます。

MQL4と言うプログラミング環境については詳細は知らないのですが、おおむねC言語/C++がベースになっていることから考えると、string sRowが2つ宣言されてしまっているのと、while文のブロックがFileReadString()実行後に即、閉じてしまっているのが問題なのだと考えられます。これでは冒頭で宣言した変数string sRowに読み込んだCSVのデータ行が入りません。

要望の動きとしては恐らく以下のようなコードになるのではないでしょうか。

void OnStart()
{
    string sRow;

    int fileHandle = FileOpen("test.csv", FILE_READ | FILE_CSV);
    if (fileHandle != INVALID_HANDLE)
    {
        while (FileIsEnding(fileHandle) == false)
        {
            // NG: ここでstring sRowを再度宣言してしまっているのと、
            // while文のブロックが閉じている
            //string sRow = FileReadString(fileHandle);}
            sRow = FileReadString(fileHandle);
            Print("sRow ",sRow);
        }

        FileClose(fileHandle);
    }
}

※コードはちゃんと整形しましょう。そうして確認することで無用なバグを極力減らすことができます。

Comments

[dodox86]

> これでは冒頭で宣言した変数string sRowに読み込んだCSVのデータ行が入りません。

反対に、string sRowをwhileのブロック内でのみ使うのであれば、冒頭での宣言はいらないと思います。変数の使用可能な範囲（スコープ）に注意しましょう。

[mokopi]

ご回答ありがとございます。
お陰様で、問題については解決しました。

さらに、スコープという考えについても今覚えました。
とても助かりました。ありがとうございます。