sleepを使ったタイマーのプログラム

Question

###実現したいこと
指定された時間が経過したことを知らせるプログラムを作りたい。
sleepを使う
###問題点

タイマーの部分がうまく動かない。
指定した時間と違う時間を刻み始める。。

###コード

#include<iostream>
#include<chrono>
#include<thread>
#include<cstdlib>
void sleep(double sec);
int main(int argc,char* argv[])
{
    int i;
    std::cout<<"0%   10   20   30   40   50   60   70   80   90  100\n";
    std::cout<<"|----|----|----|----|----|----|----|----|----|----|\n";
    std::cout<<"^"<<std::flush;
    for(i=0;i<50;i++){
        sleep(std::atof(argv[i])/50);
        std::cout<<"\b";
        std::cout<<"~";
        std::cout<<"^";
        std::cout<<std::flush;
    }
    std::cout<<"\n";
    std::cout<<std::atof(argv[1])<<"秒経過しました\n";
    return 0;
}
void sleep(double sec)
{
    int msec=sec*1000;
    std::this_thread::sleep_for(std::chrono::milliseconds(msec));
}

Comments

[coco_bauer]

質問のプログラムは、どのような引数を指定して起動するのですか？　
for(i=0;i<50;i++){ sleep(std::atoi(argv[i])/50);
となっているので、50個もの引数を指定するのだと思うのですが、そんなに大量の引数で何を指定するのですか？

Answer 1

そもそも引数いくつ受け取るプログラムでしょうか？

        sleep(std::atof(argv[i])/50);

もし引数一個だけなら、ここがargv[1]になると思いますが、iだとへんな場所を参照すると思います。
あと、何度もstd::atofするより、一回double型の変数に受けて使いまわした方がいいです。

Answer 2

推測ですが、test.exe 10 のようにタイマーの秒数を引数に指定して、プログレスバーのような形で指定した時間分カウントするプログラムが作りたかったのではないでしょうか。
だとすると、指定した秒数は argv[1] で参照できるので、sleep(std::atof(argv[i])/50); の部分は sleep(std::atof(argv[1])/50); とするべきではないでしょうか。

- sleep(std::atof(argv[i])/50);
+ sleep(std::atof(argv[1])/50);

argc, argv の仕様

• test.exe: argc = 1, argv[0] = "test.exe"
• test.exe 10: argc = 2, argv[0] = "test.exe", argv[1] = "10"
• test.exe 10 20: argc = 3, argv[0] = "test.exe", argv[1] = "10", argv[2] = "20"

修正後のコード

#include <chrono>
#include <cstdlib>
#include <iostream>
#include <thread>

void sleep(double sec);

int main(int argc, char *argv[])
{
    double time_secs = std::atof(argv[1]);

    std::cout << "0%   10   20   30   40   50   60   70   80   90  100\n";
    std::cout << "|----|----|----|----|----|----|----|----|----|----|\n";
    std::cout << "^" << std::flush;

    for (int i = 0; i < 50; i++) {
        sleep(time_secs / 50);
        std::cout << "\b";
        std::cout << "~";
        std::cout << "^";
        std::cout << std::flush;
    }
    std::cout << "\n";
    std::cout << time_secs << "秒経過しました\n";

    return 0;
}

void sleep(double sec)
{
    int msec = sec * 1000;
    std::this_thread::sleep_for(std::chrono::milliseconds(msec));
}

Comments

[SHOMI]

説明文の方argv[1]ではなくargv[0]になっていますよ

[tiitoi]

修正しました。ご指摘ありがとうございます。

[langhtorn]

できました。ありがとうございます！