【C言語】文字列を３回点滅させるプログラムを作ろうと思っているのですが、行き詰ってます。

Question

###前提・実現したいこと
自作関数を用いて文字列を3回点滅させたいのです。

###私の書いたのソースコード

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

int sleep(unsigned long);
int bput(const char *s,int d,int e, int n);

int main(void)
{
    bput("AIUEO",500,500,3);
    return 0;

}


int sleep(unsigned long x)/*x秒時間を経過させる関数*/
{
    clock_t start=clock(),end;

    do
    {
        if((end=clock())==(clock_t)-1)
        {
            return 0;
        }
    }while(1000.0*(end-start)/CLOCKS_PER_SEC<x);
    return 1;
}


int bput(const char *s,int d,int e, int n)/*文字列sを表示して、dミリ秒後にスペースで埋めてeミリ秒待機というセットをn回繰り返す*/
{
    int i,j;

    for(i=0;i<n;i++)
    {
        printf("%s",s);
        sleep(d);
        putchar("\r");
        for(j=0;j<strlen(s);j++)
        {
            printf(" ");
        }
        sleep(e);
        puts("\r");

    }


}

これが結果です。

AIUEO� 
AIUEO� 
AIUEO� 

###疑問
どこを修正すれば成功しますか,,,,(´；ω；｀)

Answer 1

for文の中のみですが、puts,putcharを全てprintfに、行出力後バッファに残ったままになる可能性があるのでfflushでちゃんと出力させてみました。

    for(i=0;i<n;i++)
    {
        printf("%s",s);
        fflush(stdout); //出力バッファを出力させる
        sleep(d);
        printf("\r");
        for(j=0;j<strlen(s);j++)
        {
            printf(" ");
        }
        fflush(stdout);
        sleep(e);
        printf("\r");

    }

こんな感じでどうでしょうか。

Comments

[nullpurge]

ありがとうございます！解決いたしました!
fflush活用していこうと思います。

Answer 2

これが結果です。

動的な挙動の説明を静的な記述で「これ」と言い切っちゃう所にちょっとウケました (^^;

int bput(const char *s,int d,int e, int n)/*文字列sを表示して、dミリ秒後にスペースで埋めてeミリ秒待機というセットをn回繰り返す*/
{
    int i,j;

    setvbuf(stdout, NULL, _IONBF, BUFSIZ);
    for(i=0;i<n;i++)
    {
        printf("%s",s);
        sleep(d);
        for(j=0;j<strlen(s);j++)
        {
            printf("\b");
        }

        for(j=0;j<strlen(s);j++)
        {
            printf(" ");
        }
        sleep(e);
        for(j=0;j<strlen(s);j++)
        {
            printf("\b");
        }

    }
    setvbuf(stdout, NULL, _IOFBF, BUFSIZ);

}

sleep()の実装にそこはかとない気持ち悪さは感じますが、直接のバグではないので取り敢えず傍観しときます。

Comments

[nullpurge]

fflush(stdout)を忘れていたので静的に動作してしまったようですね（;＾△＾）

[....]

バッファリングを有効にしてfflushを使うという手と、setvbufでバッファリング自体を無効にして即座に出力する方法がある、ということです。

また標準出力(stdout)はバッファリングされるのが普通ですが標準エラー出力(stderr)はバッファリングされない処理系がほとんどだと思うので、fprintf(stderr, ・・・)を使えばfflushもsetvbufもしなくても元のソースでも所望の動作になります。

fflushとsetvbufの使い分けですが、マルチスレッドで複数のスレッドが非同期に標準出力に出力しているような場合だと、fflushをする前にバッファがいっぱいになって出力が始まってしまって、なおかつ割り込みやシグナルで意図しないタイミングでCPUを奪われるとかして、自分ではfflushのタイミングで吐き出しているつもりなのに挙動が想定と合わないといった状態になることも考えられます。

Answer 3

putcharは文字列ではなく文字を受け取ります。
putsは改行を付与してしまいます。
ですので、以下のように修正してはどうでしょうか？

	for (i = 0; i<n; i++)
	{
		printf("%s", s);
		sleep(d);
		putchar('\r');
		for (j = 0; j<strlen(s); j++)
		{
			printf(" ");
		}
		sleep(e);
		putchar('\r');

	}

Comments

[nullpurge]

「AIUEO� �AIUEO� �AIUEO� �」という結果になってしまいました,,,

[hmmm]

@nullpurge [文字列ではなく文字]です。ダブルクォーテーションではなく、シングルクォーテーションで\rを囲ってください。