0と1のファイルへの書き込み、出力

質問

c言語で0と1の乱数を生成し、ファイルに出力したいのですが、出力がおかしくなってしまいます。
正しく0と1をファイルに書き込み、出力するにはどうしたらいいでしょうか。
わかる方いらっしゃいましたら教えてください。
お願い致します。

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

#define max 3000000
#define nrand(n) (int)((double)n*rand()/RAND_MAX) /* 0以上n未満の整数を返す */

int main()
{
　　FILE *Wf;
　　char W_filename[20];
　　int *W=(int *)malloc(sizeof(int) * max);

　　printf("入力ファイル名 : "); scanf("%s",W_filename);

　　if( ( Wf=fopen(W_filename,"w") ) == NULL ) printf("ファイルを開けません\n");
　　else
　　{
　　　 for(i=0;i<max;i++)
　　　 {
　　　　W[i]=nrand(2);
　　　　fprintf(Wf,"%d ",W[i]);
　　　}
　　}

　　fclose(Wf);
　　free(W);

}

出力が
‰‱‱‰‱‱‰‱‱‰‰‰‰‱‰‱‰‱‰‰
のようになってしまいます。
Wにはちゃんと0か1が入っているみたいなのですが・・・

回答No.4

Lchan0211 · ベストアンサー

出力したファイルの内容をどうやって確認していますか?
おそらく、
0 1 1 0 1 1 0
と正しく出力されているのに、
そのファイル内容の文字コードを
UTF-16と見なした上でそれをShift-JISに
変換した結果が表示されていると思います。

たぶん
UTF-16 Shift-JIS
-----------------------
0x2030 → 0x81f1(‰)
0x2031 → 0x8241
-----------------------
に変換されているような雰囲気。
http://ja.wikipedia.org/wiki/Unicode%E4%B8%80%E8%A6%A7_2000-2FFF

回答No.6

Tacosan

本題とは全く関係ありませんが, ここでの
int main()
は「引数を持たないで int を返す関数 main」を意味します＞#5.
JIS を見ればわかりますが, 宣言のときと定義のときでは意味が異なります.

回答No.5

postal0x02

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

#define WRITE_DATA_MAX 3000000
#define nrand(n) (int)((double)n*rand()/RAND_MAX) /* 0以上n未満の整数を返す */

int main(void)
{
　　FILE *Wf;
　　char W_filename[128];
　　int i;

　　printf("入力ファイル名 : ");
　　scanf("%s",W_filename);

　　if( ( Wf=fopen(W_filename,"w") ) == NULL )
　　{
　　　　printf("ファイルを開けません\n");
　　　　return -1;
　　}
　　else
　　{
　　　　for(i=0;i<WRITE_DATA_MAX;i++)
　　　　{
　　　　　　fprintf(Wf, "%d ", nrand(2));
　　　　}
　　}

　　fclose(Wf);

　　return 0;
}
修正してみました。

>#define max 3000000
max というマクロは既に定義されています。
定数は大文字が一般的です。
例）#define WRITE_DATA_MAX 3000000

>#define nrand(n) (int)((double)n*rand()/RAND_MAX)
n にカッコがありません。
もし呼び出し側が nrand(1+2) とした場合に
(int)((double)1+2*rand()/RAND_MAX)
と意図しない結果に置き換えられます。
↓のようにカッコでくくる癖をつけましょう(全体もくくるとなおいいです)。
#define nrand(n) ((int)((double)(n)*rand()/RAND_MAX))

>int *W=(int *)malloc(sizeof(int) * max);
この領域は今回のプログラムでは必要ありません。

>char W_filename[20];
領域が小さい気がします。
ユーザに入力させるなら、128 位にしたほうがいいです。

>int main()
引数が省略されていますが、void 型にはなりません。
引数が不定いう別の意味になりますので、C言語では省略しないでください。
（C++言語では省略は自動で void です。あと、この情報は古いかも知れません）

戻り値が宣言させているのに、
main 関数は戻りを返していません。
アプリケーションが何かしら戻り値を勝手に返しています。
どこのアドレスの何を返してるか分かったものではありません。

>printf("入力ファイル名 : "); scanf("%s",W_filename);
２行に分けたほうがいいです。

回答No.3

sankaku197

もう本当に参考程度なんですけど。

ファイルからの出力の仕方に問題があるだけでは？
少なくとも、ファイル（txtですか？）を開いて正常に見えるなら、ファイルへの読み込みは問題ないはずです。

メモ帳で見える羅列と、（Stirlingなどの）バイナリエディタで見える羅列の両方を比較してみると問題点が分かるかもしれません。

回答No.2

Yanch

●ポイント１
変数 i が宣言されていないので、コンパイルできません。

●ポイント２
＞　　int *W=(int *)malloc(sizeof(int) * max);
エラー処理してないですが、メモリ確保に失敗していませんか？

●ポイント３
環境を書きましょう。

●ポイント４
異様に横長のテキストファイルになりますけど、テキストエディタは対応していますか。

●ポイント５
iを宣言して、コンパイルを通るようにしてみましたところ、
拙者宅の環境だと、それらしい実行結果になりましたよ。
どこかタイプミスしているのでは？

確認環境：
gcc (GCC) 3.4.2 (mingw-special)
Windows 2000 Pro
TeraPad(テキストファイルを確認用)

回答No.1

asuncion

１）出力ファイル名は何ですか？
２）乱数の初期化をしましょう。今の状態では、毎回同じ出力結果を得てしまいます。それでかまわなければ、別にいいのですけれど。
３）今回のプログラムで、領域を動的に確保する必要がありますか？ nrand(2)で求めた0か1を、直接ファイルに書けばいいだけではないですか？

補足

回答ありがとうございます。
１）出力ファイルはテキストファイルです。
２）元のプログラムから抜粋したのもなので、書くのを忘れてしまいましたが、
srand((unsigned int) time(NULL));
をファイル入力前に書いてあります。
３）上記のとおり、元の長いプログラムからの抜粋であり、ここに書いていない処理で０か１の乱数の入った配列が必要なのです。その配列をファイルに書き出したいのです。

元のプログラムはこの部分以外は正常に動作します。