配列を関数に渡す

Question

c言語で「好きな言葉を入力すると、その言葉を一文字ずつ出力する（文字と文字の間は０．１秒）」というプログラムを作ろうと思い、以下のコードを書いてみました

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

int sleep(unsigned long x){

  clock_t c1=clock(),c2;

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

  void gput(char *s,int speed){
    int c,i;
    c=strlen(s);
    for(i=0;i<c;i++){
      printf("%c",s[i]);
      sleep(speed);
    }

}

int main(void){

  char str[50];
  printf("好きな言葉を入力してください:");
  scanf("%s",str);
  gput(str,100);

  return 0;

}

ですが、これを実行すると、言葉を入力した後に、少し間があった後全ての文字が一気に表示されてしまいます
何が問題なのでしょうか？

それと、「関数に配列を渡す」ということが、いまいちわかっていないので私のやり方（関数に配列の最初のポインタを渡す）に問題があれば、そちらもご教授願います

Answer 1

nanosleep(), setvbuf() をつかって書いてみました。

#include<stdio.h>
#include<time.h>
#include<string.h>
#include <sys/time.h>

void gput(char * s, int speed) {
    int c = strlen(s);
    struct timespec ts;
	  ts.tv_sec = speed;
	  ts.tv_nsec = 0;

    setvbuf(stdout, 0, _IONBF, 0); // バッファリング無しにする
    for(int i = 0; i < c; i++) {
      printf("%c", s[i]);
      nanosleep(&ts, NULL);
    }
}

int main(void) {
  char str[50];
  printf("好きな言葉を入力してください:");
  fgets(str, sizeof(str), stdin);
  gput(str, 1);  // 1 秒
  return 0;
}

Answer 2

for(i=0;i<c;i++){
printf("%c",s[i]);
fflush(stdout); // <-これを追加
sleep(speed);
}

上のように、fflush(stdou); を入れてみて下さい。

「関数に配列を渡す」はその理解で正しいです。

Comments

[FumiakiNakao]

回答ありがとうございます
下の方とほぼ同じ回答をされていたので、先の方の回答者様をベストアンサーにさせていただきました　すみません

「関数に配列を渡す」ということについて、あっていてよかったです

Answer 3

2点気になったことがあります。

• 環境によっては、標準出力がバッファリングされていて、1文字単位で出ないことがあります。fflush(stdout)として強制出力させましょう。
• ASCIIの範囲内の文字ならこれでいいのですが、日本語などで複数バイト文字がある場合に、1バイトだけ出力するとどうなるのか気になりました。

Comments

[FumiakiNakao]

fflush(stdout)を打ち込むと解決しました
バッファの掃き出しは、参考書で見たのですが、軽く読み飛ばしていました。今回重要性に気づくいい機会になったと思います

打ち込む文字は数字、アルファベットのみを想定していたので、２点目については全く考えていませんでした
ちなみに打ち込んだ場合、網模様（？）がしばらく出力されます