printfが！

Question

int main(void){
      ....
      while(!(i==EOF)){
        num=0;
       while((i=fgetc(stream))!='
'){
        if(i==EOF) break;
    str[num]=i;
        num++;
    }
      kannsuu1();
      kannsuu2();
      }
      return 0;
void kannsuu1(void){

    printf("出力されない");
    for(i=0;str[i]!=NULL;i=i+2){
    printf("%c%c",str[i],str[i+1]);/*これも出力されない*/
    }
}
void kannsuu2(void){
    ....
}

簡単に上記のように記述したプログラムがあるのですが、コンパイラの警告はまったくかかりませんでしたが、関数の中のprintfがdosプロンプトに出力されないのですよ。だれか心あたりがありましたらおしえてもらえないでしょうか？
お願いいたします。

leaz024 · Accepted Answer

出力できるようになったのですね？ヨカッタヨカッタ！

原因が分からない事なんて多々あります（笑）
そのうち原因に気が付くこともあるしね。

とりあえずmarkの位置修正などでソースとのにらめっこなどが続くと思います。
頑張ってください。

leaz024 · Answer

うーん、、、Nickeeさんのソース丸々コピーして、divide()とcall_di()だけコメントにして実行したら、ちゃんと動きましたよ？（全角SPは消しましたが…）

OSはWin98、コンパイラはBorland C/C++ 5.5です。

↓これが使用したテストデータです。
あいうえお

さしすEOF

↓これが実行結果です。（DOS窓よりコピーしました）
ここからあいうえお
ここからさしす

・Nickeeさんの方の実行結果の画面って、どうなってます？できたらDOS窓のコピーをお願いします。
・とりあえずresult()以外の２つの関数呼び出しをコメントにして、↑のような超単純データでテストしてみてもらえますか？
・またresult()の代わりにputs(str)を実行してみてもらえますか？

#要求ばかり多くてゴメンなさい。。

以上、よろしくお願いします。

leaz024 · Answer

コンパイルでエラーが出たということですが、もしかしてNo.5のサンプルから、コピーしたりしました？

　サンプルはブラウザでの見栄えをよくするために、半角スペースやタブを使うところを「全角スペース」で書いています。文字列内などでない場所で全角文字を使用すると、「無効な文字がありました」のようなエラーになります。

　お使いのテキストエディタに「全角スペースを何らかの記号で表示する機能」などがあれば、有効にしておく便利ですよ。
　またソース全体を選択し、「全角スペース→半角スペース２個」の置換を実行してみるとよいでしょう。


#もし上記が見当違いの回答だとしたら、次の補足をお願いします。

>rewindを使用したら、コンパイルエラーがでて
このときに追加したコードを、そのまま見せていただけますか？

>rewindだけをけして
とは、どのようなことをされたのでしょうか？

leaz024 · Answer

>>memset(str, '\0', strlen(str));　　/* これしないと、まずくない？ */ 
>これは文字列の最後にNULLをいれるということでしょうか？いちおう、単純にstr[n]=NULL;ってしてあります. 

　例には書いてないだけで、他の雑用的な処理はなされているのですね。もしかしたら、その延長で処理してあるのかも知れませんが、一応気になったので・・・

　　str[n] = NULL;

　これって、main関数内の処理の方にあるってことですよね？
kansuu1内では、strの中を１個おきにしか調べていないので、奇数番目にヌルが入っていると、そのままループが継続されてしまう可能性があります。
（前回の入力よりも入力した文字数が少なく、文字列長が奇数の場合）

　kansuu1の処理が３個単位になっても、１個単位になってもOKなように、２個のヌルを入れるのではなく、memsetでクリアするようにしてみました。

　それからヌルの使い方なんですが、ヌルを表すNULLと'\0'には
　　・NULL … 無効ポインタ
　　・'\0' … 文字列の終端記号
という、意味の違いがあります。
（Cでは
　　#define　NULL　((void *)0)
という処理系もあるので、この場合は'\0'とは全くの別物です。）

　使ってしまえばどちらも0なので問題はないのですが、人にソースを公開する場合には解読の助けにもなるので、このような使い分けは大事ですよ。
（特に集団開発などでは、人のコードを読まされる場合なんて多々あります。）

leaz024 · Answer

　そもそもコンパイラの警告やエラーというのは

　　・プログラム自体が間違っている場合
　　　（セミコロン抜けや括弧の不整合、文法ミスなど）
　　・間違いではないが、プログラマのミスではないかと思われるような記述を見つけた場合
　　　（ifやwhileなどの条件式内で、代入のみが行われているなど）
などに発せられるもので、動作の不具合を教えてくれるものではありません。
（このような実行時の不具合を一般に「バグ」と呼びます。バグを警告してくれるコンパイラなんてあったら、世の中つまんなくなっちゃうでしょ？）

　で本題ですが、a-kumaさんがおっしゃられているように、printf関数の出力先である「標準出力（stdout）」は、デフォルトでバッファリングを行うように設定されています。
　バッファリング付き出力先に対して出力を行う際は、バッファが満タンになるか、改行（'\n'）が出力されるかまで、実際の出力は行われません。

　途中途中で改行してやるのがベターなのですが、スクロールできないDOS窓プログラムなので、大量の情報を表示するために改行をしていないのでしょう。

　こういう場合、バッファリングを切ったりするのも手なんですが、バッファリングを切らずに、しかも改行しないで出力を実行させる関数があります。
　【書式】
　　　#include <stdio.h>
　　　void rewind(FILE *fp);
　この関数を使えば、その時点でバッファの中身が出力先デバイスに送られます。
引数には、<stdio.h>内で定義されているstdoutを指定すればOKです。

　例のリストは少し見にくかったので、少々手直しをしたサンプルを載せておきます。

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

/* main()内部 */
　　　　　：
　　do {
　　　　n = fgetc(stream);
　　　　if (n != EOF && n != '\n') {
　　　　　　str[num++] = (char)n;
　　　　} else {
　　　　　　kannsuu1();
　　　　　　kannsuu2();
　　　　　　num = 0;
　　　　　　memset(str, '\0', strlen(str));　　/* これしないと、まずくない？ */
　　　　}
　　} while (n != EOF);
　　　　　：

/* kansuu1()内部 */
　　　　　：
　　printf("表示されない？");
　　for (i = 0; str[i] != NULL; i += 2) {
　　　　printf("%c%c", str[i], str[i+1]);
　　}
　　rewind(stdout);　　　　/* この関数実行後、画面に出力される */
　　　　　：

　とりあえずはrewindの行を追加するだけでも動きますので、お試しください。

selenity · Answer

まず、コンパイル環境くらいは書きましょう。
32bit Windowsなら「Win32コンソール」アプリ
ケーションにすればprintfの出力が見れます。

「関数のプロトタイプ宣言」が先であることは
分かっていますよね。

それと、
while(!(i==EOF)){ 
　num=0; 
　while((i=fgetc(stream))!='
'){ 
の使い方はあまり感心しません。

通常File I/Oでは
　while((i=fgetc(stream))!=EOF){
　　//必要な処理を書く
　}
の様に書くのが普通ではないでしょうか？

今回のケースだと、
　num=0;
　while((i=fgetc(stream))!=EOF){
　　if(i=='
'){
　　　kannsuu1();
　　　kannsuu2();
　　　num=0;
　　　//ここでstr[]の中身をpaddingする必要がある
　　}
　　else{
　　　str[num]=(char)i;
　　　num++;
　　}
　}
　if(num>0){//これがないと最後の1行が処理されない
　　kannsuu1();
　　kannsuu2();
　}
の様になるでしょう。

aaaaa · Answer

int main(void){ 
.... 
　　while(!(i==EOF)){  
　　　　while((i=fgetc(stream))!='
'){ 
　　　　　　if(i==EOF) break;
　　　　} 
　　　　kannsuu1(); 
　　} 
　　return 0;
｝ 

void kannsuu1(void)
{
　　printf("出力されない");
｝


勝手にまとめさせてもらうとこんなプログラムになりますが
これで”出力されない”が出力されないなら
素人の私では回答できません。

もしよかったら
このプログラムがどのように終了するのか
教えてください。

a-kuma · Answer

「出力されない」というのは、その関数を呼び出したタイミングで、ということ
なのでしょうね。

printf() は、普通、出力をバッファリング、つまり、関数の中で一時貯めておきます。
その貯めた中身を吐き出す、つまり、出力するのは、バッファが一杯になるか、改行(
)に
当たったときです。

「dosプロンプトに」とあるので、勝手に Visual C/C++ だと想像しますが、関数の
リファレンスに書いてあります。

これを避けるためには、方法はふたつ。

ひとつは、バッファリングされていない stderr を使うこと。printf() を
fprintf(stderr, ...) に置き換えます。

もうひとつは、setvbuf() 関数を使って、バッファリングをしない指定をすること。
stdout に対して、バッファリングしない指定 _IONBF を指定します。

詳細は、関数のリファレンスを見て欲しいのですが、多分こんな感じの関数呼び出しを
最初に一発だけしておけば良いです。

　　setvbuf(stdout, NULL, _IONBF, 0);

バッファリングをしない指定なので、バッファのアドレスやそのサイズは、
NULL と 0 を指定しておけば良いです。

kee · Answer

printfの中に
の改行マークを入れてないというのはどうでしょうか？
古い、unixならためておくこともあると思います。

あと、１３行目のreturnの後ろに}で閉じてないのは、
省略しているだけですよね。

printfが！

出力できるようになったのですね？ヨカッタヨカッタ！

うーん、、、Nickeeさんのソース丸々コピーして、divide()とcall_di()だけコメントにして実行したら、ちゃんと動きましたよ？（全角SPは消しましたが…）

コンパイルでエラーが出たということですが、もしかしてNo.5のサンプルから、コピーしたりしました？

この回答への補足

>>memset(str, '\0', strlen(str)); /* これしないと、まずくない？ */

この回答への補足

そもそもコンパイラの警告やエラーというのは

この回答への補足

まず、コンパイル環境くらいは書きましょう。

この回答への補足

int main(void){

「出力されない」というのは、その関数を呼び出したタイミングで、ということ

printfの中に\nの改行マークを入れてないというのはどうでしょうか？

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング

　うーん、、、Nickeeさんのソース丸々コピーして、divide()とcall_di()だけコメントにして実行したら、ちゃんと動きましたよ？（全角SPは消しましたが…）

　コンパイルでエラーが出たということですが、もしかしてNo.5のサンプルから、コピーしたりしました？

>>memset(str, '\0', strlen(str));　　/* これしないと、まずくない？ */

　そもそもコンパイラの警告やエラーというのは