トップページ⇒tech
948コメント314KB

C言語なら、俺に聞け！ <10>

■ このスレッドは過去ログ倉庫に格納されています
0001Cハッカー日下部陽一NGNG
おれは42才、24年間プログラムを一生懸命作ってきた。
いろいろな言語を極めたが、一番手になじむのはCだった。
Cのことならぜんてわかるから、疑問な点があったらオレに聞け。

だが、いちいちFAQに載った質問に応えるほど暇じゃない。
課題丸投げなら別スレにいけ。

・comp.lang.c Frequently Asked Questions
http://www.eskimo.com/~scs/C-faq/top.html

・C FAQ 日本語訳
http://www.catnet.ne.jp/kouno/c_faq/c_faq.html

・前スレ C言語なら、オレに聞け！ <9>
http://pc.2ch.net/test/read.cgi/tech/1007272267/l50

Cプログラマならこれを買っとけ。
・プログラミング言語C(K&Rと呼べば通のようだぞ)
http://www.amazon.co.jp/exec/obidos/ASIN/4320026926/yfcbookshelf-22/250-3950205-1475419

K&Rで挫折するようなヤツはこれでも買え。
これでわからない、救いがたいヤツはあきらめろ。
・作ってわかるCプログラミング(アヒル本と呼べば通のようだぞ)
http://dennou.gihyo.co.jp/books/void_c/
http://www.amazon.co.jp/exec/obidos/ASIN/477411328X/qid%3D1005618315/250-3950205-1475419
0002デフォルトの名無しさんNGNG
2
0003デフォルトの名無しさんNGNG
1は童貞
0004デフォルトの名無しさんNGNG
手になじんだのは、自分のﾃｨﾑﾎﾟ
0005デフォルトの名無しさんNGNG
いつもお世話になっております。
早速ですが、ファイルポインタを引数として関数に渡そうとしてコーディング
してみたのですが、どうも上手くいきません。
方法は下記の通りです。

main(){
FILE *FP;

Init_Proc(&FP);
}
Init_Proc(FILE **FP){
*FP = fopen(・・・);
X_Proc(FP);
}

X_Proc(FILE **FP){
fprintf(*FP, "%s\n", "ABC");
}

現状ではX_Procでファイル書込を行った際にファイルポインタが破壊されて
いるようです。
ファイルポインタを引数として渡すことはあまりなくて、どのようにすれば
よいのかわかりません。
みなさん、ご教授を宜しくお願いします。
0006Kusakabe YouichiNGNG
>>5
デフォルトの名無しさん wote:
>いつもお世話になっております。

そういうことにしたいのですね。
0007デフォルトの名無しさんNGNG
>>6
つかれてきましたね？
0008Kusakabe YouichiNGNG
In article >>5, デフォルトの名無しさん/5 wrote:
> main(){FILE *FP;Init_Proc(&FP);}
> Init_Proc(FILE **FP){ *FP = fopen(・・・); X_Proc(FP);}
> X_Proc(FILE **FP){fprintf(*FP, "%s\n", "ABC");}
>
> 現状ではX_Procでファイル書込を行った際にファイルポインタが破壊されて
> いるようです。

そういう問題ではないですね。

> ファイルポインタを引数として渡すことはあまりなくて、

っていうか、こんな渡し方をしなきゃいけない場合は考えられないので、
そもそも設計がまちがっているのでしょう。

まず、fopenで帰ってくる値(ポインター)が指す実体はどこにあるのか
想像してみましょう。

# いったいどんな本を読んで勉強したのだろうか。
0009Kusakabe YouichiNGNG
In article >>6, Kusakabe Youichi/6 wrote:

> >>5
> デフォルトの名無しさん wote:
> >いつもお世話になっております。
>
> そういうことにしたいのですね。

引用の仕方がちがいますね。
0010デフォルトの名無しさんNGNG
>>8
アヒル本とK&Rをよんで勉強してきました。
0011デフォルトの名無しさんNGNG
>>5
> 早速ですが、ファイルポインタを引数として関数に渡そうとしてコーディング
> してみたのですが、どうも上手くいきません。
「どうも」じゃなくて「どう」上手くいかねぇのよ。

> 現状ではX_Procでファイル書込を行った際にファイルポインタが破壊されて
> いるようです。
んなこたぁない。
fopen()の引数だけ直して#include <stdio.h>つけたらちゃんと動いたぞ。
省略したところに別のバグがあるんだろ。

> ファイルポインタを引数として渡すことはあまりなくて、どのようにすれば
> よいのかわかりません。
変すぎ。
00125NGNG
みなさん、有り難うございます。
以前、UNIXの環境でファイルポインタを引数にしようとした時、
『ダブルポインタにしないと渡せない』と言われてから、そういうものだと
思っていましたが、ＶＣ＋＋では違うようですね・・・
みなさんの意見を参考に試してみます。
有り難うございました。
0013Kusakabe YouichiNGNG
In article >>12, 5/12 wrote:
> みなさん、有り難うございます。
> 以前、UNIXの環境でファイルポインタを引数にしようとした時、
> 『ダブルポインタにしないと渡せない』と言われてから

「だれに」いわれたのですか?
0014デフォルトの名無しさんNGNG
>>12
> 以前、UNIXの環境でファイルポインタを引数にしようとした時、
> 『ダブルポインタにしないと渡せない』と言われてから、そういうものだと
> 思っていましたが、ＶＣ＋＋では違うようですね・・・

>>5が完全に誤解してるか、思いっきりだまされてるか、そう言った奴も完璧な厨房か。
>>5の誤解に1000ポインタ。
0015もふNGNG
バイナリのランダムアクセスファイルを作ったり読んだりする
プログラムをつくってます。ひとつのデータが下のような構造体です。

typedef struct {
int x;
long y;
} mydata;

なにかアライメントやエンディアンとかの問題があるらしいので
mydata buf;
fread( &buf, sizeof(mydata), 1, fp );
fwrite( &buf, sizeof(mydata), 1, fp );
みたいなのは駄目らしいのですが、具体的にどう回避すれば
よいのでしょうか？

アライメントの問題のほうは
fwrite( &(buf.x), sizeof(int), 1, fp );
fwrite( &(buf.y), sizeof(long), 1, fp );
とか分ければいいのかな？ と思うのですが
エンディアン云々のほうは回避策がわかりません。。。

そもそも、longとかのビット数が違ったら駄目な気もするし。。。
なにかこういうときの定番の方法とかあったら教えてくださるとうれしいです。
0016デフォルトの名無しさんNGNG
>>5（釣り師）のネタに10000ポインタ。
001716NGNG
あ、これ>>14へのレスね
0018デフォルトの名無しさんNGNG
>>15
アライメントの問題は、そのように構造体のメンバを
ひとつずつ処理しればOKですよ。

エンディアンの問題は、たとえば、こういうように一バイトずつ
読み書きするしかないでしょうね。
void write_long(FILE* fp, long n)
{
　int i;
　for (i = 0; i < 4; i++ ) {
　　fputc(n & 0xff, fp);
　　n >>= CHAR_BIT;
　}
}
0019デフォルトの名無しさんNGNG
hton*関数で強制的にネットワークバイトオーダにしてから
書く、という手もあるよ。
0020デフォルトの名無しさんNGNG
>>15
longのサイズの問題は、保証されてる32ビットしか使わない
ようにすればよいと思う。
0021Kusakabe YouichiNGNG
In article >>15, もふ/15 wrote:
> なにかこういうときの定番の方法とかあったら教えてくださるとうれしいです。

対象環境が1つか2つだったら#ifで対処、
無数の環境を意識するんだったら、
freadでcharの配列に読んで1バイトずつ処理するか、
ascii化して書くかどちらか。
0022taiNGNG
Ｃ言語で関数ごと個々のソースで分けてあるプログラムを改造しようと思っているのですが,
ひとつの関数で使ったローカル変数に入った値をプログラム全体での変数に代入したいのですが、どのようにしたら良いのでしょうか？
よろしくおねがいします。
0023デフォルトの名無しさんNGNG
グローバル変数にするのはダメなの？
0024Kusakabe YouichiNGNG
In article >>22, tai/22 wrote:
> ひとつの関数で使ったローカル変数に入った値をプログラム全体での変数に
> 代入したいのですが、どのようにしたら良いのでしょうか?

ふつうに代入する。
0025もふNGNG
皆さんありがとうございます。

なるほど、0xffと & を取ればエンディアンに関係なくいちばん「桁」の
小さいところのバイトを取れるんですねー。

いちおう、こんなのでやってみます。。。

int read_mydata( FILE *fp, mydata *dp ){

if( read_int( fp, &(dp->x) ) == 0 ) return 0;
if( read_long( fp, &(dp->y) ) == 0 ) return 0;

return 1;
}

int read_long( FILE *fp, long *l ){

int i, c;
*l = 0;

/* 32bitまで */
for( i=0; i<4; i++ ){
if( ( c = fgetc( fp ) ) == EOF ){
return 0;
}
*l |= c;
*l <<= 8;
}

return 1;
}
0026Kusakabe YouichiNGNG
In article >>25, もふ/25 wrote:
> いちおう、こんなのでやってみます。。。
> int read_mydata( FILE *fp, mydata *dp ){
> if( read_int( fp, &(dp->x) ) == 0 ) return 0;
> if( read_long( fp, &(dp->y) ) == 0 ) return 0;
> return 1;
> }
エンディアンの問題はなんとかなっても、構造体のメンバーの間に
隙間があったらどうするつもりなの?
0027デフォルトの名無しさんNGNG
>>26
>エンディアンの問題はなんとかなっても、構造体のメンバーの間に
>隙間があったらどうするつもりなの?

メンバをひとつずつ処理してるから、問題ないのでは?
0028Kusakabe YouichiNGNG
In article >>27, デフォルトの名無しさん/sage/27 wrote:
> >エンディアンの問題はなんとかなっても、構造体のメンバーの間に
> >隙間があったらどうするつもりなの?
>
> メンバをひとつずつ処理してるから、問題ないのでは?

「処理」とは?
0029デフォルトの名無しさんNGNG
>>28
>「処理」とは?

if( read_int( fp, &(dp->x) ) == 0 ) return 0;
if( read_long( fp, &(dp->y) ) == 0 ) return 0;

read_int(),read_long()こういう関数をコールして、
ファイルから、int,longの単位で読み出してることです。
0030デフォルトの名無しさんNGNG
.＿＿＿＿＿＿＿＿＿
|　　　　　　　　　　　　　|
|　　ここでボケて　　 ｜
|＿＿＿＿　＿＿＿＿|
　　 ∧,,∧　||
⊂ミ.,,ﾟДﾟ彡つ
0031Kusakabe YouichiNGNG
In article >>29, デフォルトの名無しさん/29 wrote:
> >「処理」とは?
>
> if( read_int( fp, &(dp->x) ) == 0 ) return 0;
> if( read_long( fp, &(dp->y) ) == 0 ) return 0;
>
> read_int(),read_long()こういう関数をコールして、
> ファイルから、int,longの単位で読み出してることです。

ってことは、書くときにもそうするってことですよね?(そこが重要)。
0032デフォルトの名無しさんNGNG
>>31
>ってことは、書くときにもそうするってことですよね?(そこが重要)。

ああ。
読むときに、int,long単位でやってるから、書き込みも
同じに処理するって、思い込んでましたよ。
初心者相手には、そういう配慮をしたほうがいいですかね。
0033もふNGNG
情報が足らなくてすみません。

もちろん、書くときにも同じ順番でひとつずつ write_int , write_long などとやりますが、
それなら大丈夫ですよね？
0034Kusakabe YouichiNGNG
In article >>32, デフォルトの名無しさん/32 wrote:
> ああ。
> 読むときに、int,long単位でやってるから、書き込みも
> 同じに処理するって、思い込んでましたよ。

まあ、自分でやるときはそうするから...は甘いですね:)

> 初心者相手には、そういう配慮をしたほうがいいですかね。

配慮というか「警戒」というか。
0035Kusakabe YouichiNGNG
In article >>33, もふ/sage/33 wrote:
> もちろん、書くときにも同じ順番でひとつずつ write_int , write_long などとやりますが、
> それなら大丈夫ですよね?

その読む関数と書く関数がつねにセットになっていて不整合がなく、
片方だけ修正しわすれることがなければね。

そのためにはなるべく近くに置くのがコツ。
(#defineマクロなどで共通化もはかる)
0036デフォルトの名無しさんNGNG
>>33
たいていのアーキテクチャでは整数を表現するのに2の補数を使ってるけど、稀に1の
補数を使ってるアーキテクチャもある。その場合には >>25 だと NG だね。
（ま、そんなアーキテクチャを使う予定がなければ、気にしないのが正解だけどさ）

あと、構造体ごとに read_xxx 関数を作ると面倒になるから、一つ抽象化したほうがいい
と思われ。printf() みたく

　(1) ダンプするデータを指定する文字列 (c で char, i で int, l で long とか)
　(2) 実際のデータ

を列挙して処理できる関数を、一つ用意するとかね。

inline int read_mydata(FILE *fp, mydate *dp)
{
　　unpack(fp, "il", &dp->x, &dp->y);
}

たとえば、こんな感じで。これだけだと有り難味が実感できないかもしれんが、ユーザ定義の
コールバック関数を取れるように (1) を拡張したりすると、データが複雑になったときの保守
性に差が出てくる。
0037デフォルトの名無しさんNGNG
>>26
>稀に1の補数を使ってるアーキテクチャもある。その場合には >>25 だと NG だね。

1の補数でもOKのように見えますけど、どこがいけないんでしょうか?
0038デフォルトの名無しさんNGNG
＞37
２の補数の機械で、-50を保存
そのデータを１の補数の機械へもっていき、読みこむ

とどうなるか、というお話ではないだろうか?
0039デフォルトの名無しさんNGNG
>>38
なるほど。
いろいろあるなぁ。
互換性を考えると、テキストデータが一番いいかな。
(って、文字コードもいろいろあるのか)
0040デフォルトの名無しさんNGNG
Ｃ言語ですが,ひとつの関数で出来上がったint のデータを
クリップボードに移すにはどのようにやればよいのでしょうか？
0041デフォルトの名無しさんNGNG
>>40
Generic な C では「クリップボード」は定義されてないから、何とも。VC の話なら VC スレで聞いたほうが
いいと思うぞ。
0042デフォルトの名無しさんNGNG
レスありがとうです。
なんか下記のようなサンプルがあるんだけどこれを独立したｃソースの関数とすると、
コンパイル後に「ＨＧＬＯＢＡＬは定義されていない識別子です」とでてエラーがでます。
定義はどこですればいいのでしょうか？
あとＣではクリップボードはつかわないほうがいいのですか？


void SetClipText(void)
{
HGLOBAL hText;
char *pText;

hText = GlobalAlloc(GMEM_DDESHARE | GMEM_MOVEABLE, 4);
pText = GlobalLock(hText);
lstrcpy(pText, "abc");
GlobalUnlock(hText);

OpenClipboard(NULL);
EmptyClipboard();
SetClipboardData(CF_TEXT, hText);
CloseClipboard();
}
0043デフォルトの名無しさんNGNG
>>42
ヘッダをインクルードしなさい。
てか、C言語の質問とちゃうでしょ。
0044ネタ？NGNG
「Ｃではクリップボードはつかわないほうがいいのですか？」
ｹﾞﾗｹﾞﾗ
0045デフォルトの名無しさんNGNG
C特有のヘボを指摘して、その次Win32スレに移行だな
0046デフォルトの名無しさんNGNG
古い話題を掘り返してすまんが、
>>8
> っていうか、こんな渡し方をしなきゃいけない場合は考えられないので、
Init_Proc()で開いたファイルストリームを、main()で触りたい場合は？
戻り値で返せってか、まぁその通りなんだが。

> まず、fopenで帰ってくる値(ポインター)が指す実体はどこにあるのか
> 想像してみましょう。
これはマジか？ネタか？
5をひっかけて喜んでるのか？
想像の入る余地があんのかよ。

> # いったいどんな本を読んで勉強したのだろうか。
あんたがな。
0047デフォルトの名無しさんNGNG
>>46
>これはマジか？ネタか？
>5をひっかけて喜んでるのか？

なんのことだろ。
日下部氏が間違ってるようには見えないけど?
0048デフォルトの名無しさんNGNG
>>47

・fopenで帰ってくる値(ポインター)が指す実体
> fopenが返す値(FILE*型)の実体は、標準関数が内部管理する領域(FILE型)

・fopenが返した値を格納した領域
> Init_Proc(FILE** FP)へ引数としてアドレス(FILE**型)が渡された、
main()内のFILE*型変数FP。

で、何が問題？
fclose()してない事？
0049デフォルトの名無しさんNGNG
＞46
普通、構造体で包んで渡したりするとおもうよ。
FILE**が出てくるソースは、俺はあんまし見たことない。

＞47
じゃ、とりあえず、５のどこが間違っているのか、説明してみてよ。
0050デフォルトの名無しさんNGNG
スマソ、sageてなかった。

>>49
まぁ、その通りだし、私もそうするけど、5の
> っていうか、こんな渡し方をしなきゃいけない場合は考えられないので、
“考えられない”ってのが言い過ぎだという反例です。
0051デフォルトの名無しさんNGNG
>>48-49
ポインタのポインタなんてしないで、
普通に FILE*fpで受け渡しすればいいのでは?
5が動くといっても、無駄に処理を複雑にしてるだけに見えるよ。
0052デフォルトの名無しさんNGNG
＞51
とりあえず、その「普通にFILE*fpで受け渡し」してみたコードを載せてみ。
0053デフォルトの名無しさんNGNG
>52　意地悪(ｗ
0054デフォルトの名無しさんNGNG
>>52
検証の余地なんてないと思うけど、なんで?
なんか勘違いしてない?
005546NGNG
>>54
それに必然性があるか否かはともかく、
5のソースではFILE*型変数FPの実体はmain()内にあるのさ。
で、Init_Proc()でfopen()してる。
さて、Init_Proc()からmain()に戻ったとき、
FPの値は何やろね？

という事を、52は言いたいのだと思われ。

あんまこんな事をつついていると、元凶から
“そもそも設計がまちがっているのでしょう。”
とか返されそうでやだな。
0056デフォルトの名無しさんNGNG
char *p;

p = malloc(sizeof(char)*256);


これをコンパイルするとエラーになります。

エラー E2034 testf.cpp 8: 'void *' 型は 'char *' 型に変換できない(関数 main() )

コンパイラのせい？
0057デフォルトの名無しさんNGNG
>>56
コンパイラのおかげです。
0058デフォルトの名無しさんNGNG
>>56

なんで、p=(char *)malloc(256);
にしないの？
0059デフォルトの名無しさんNGNG
５６＞５８
知らなかったから。
ありがと。無事コンパイルできたよ
0060デフォルトの名無しさんNGNG
void *型は全てのポインタ型と互換性があるんじゃないの？
0061デフォルトの名無しさんNGNG
>>60
c++コンパイラでコンパイルしたんじゃない？
0062デフォルトの名無しさんNGNG
５６＞６０．６１
そう、コンパイラはBorland C++ です。
006361NGNG
まあ、拡張子が.cだったらcでコンパイルされるみたいだけど(bcc32)。
c++はある型をtypeとすると
type*からvoid* へは普通にキャストできるけど
void*からtype* へのキャストは明示的に(type*)とキャストしないと
コンパイラに叱られる。
0064デフォルトの名無しさんNGNG
c++なら (type*) ではなく static_cast<type*>() だ、
と知ったかぶってみる板違い。
0065デフォルトの名無しさんNGNG
質問です。
Winで\nてやるとWin用の改行コードを吐きますよね？
それを、無理矢理UNIX系OSの改行コードを吐かせる方法ってありますか？
0066デフォルトの名無しさんNGNG
BC++でも、ソースの拡張子を.cにすれば、Cのソースとして理解してくれるんじゃないの？
0067デフォルトの名無しさんNGNG
>>64
それ以前に new 使うと思うが。（これならキャスト不要だよ）
0068デフォルトの名無しさんNGNG
>>65
良く分かってないのに“吐く”だなんて言葉使うな。
“\nてやると”ってなんだよ！

“テキストモード”と“バイナリモード”で調べろ。
阿呆
0069デフォルトの名無しさんNGNG
工房が吠えてます（ﾜﾗ
0070デフォルトの名無しさんNGNG
>65
あ
ほ
。
0071デフォルトの名無しさんNGNG
学校で丁寧に習わないと先へ進めなかったあほですか？
0072名無しさん＠お腹いっぱい。NGNG
>Winで\nてやるとWin用の改行コードを吐きますよね？
UNIXでもDOSでも吐きますが何か?
>それを、無理矢理UNIX系OSの改行コードを吐かせる方法ってありますか？
何を無理するんでしょうか、すいませんが私も「吐く」の意味が把握出来ません
0073自作自演NGNG
お前ら、そろそろ勘弁してやれよ（ｗ
0074デフォルトの名無しさんNGNG
>>65
バイナリモードでオープン
0075デフォルトの名無しさんNGNG
そろそろ勘弁て、俺速攻で教えてるやんかー
> “テキストモード”と“バイナリモード”
しかも工房呼ばわり。

ところで無理矢理ってｲｲね。
無理矢理履かせる、黒ﾆｰｿｸｽを
無理矢理穿かせる、ﾀｲﾄﾐﾆを
0076デフォルトの名無しさんNGNG
厨房見るとハイエナの如くハンターが集まってくるね(＾＾;)こわっ
0077デフォルトの名無しさんNGNG
>>75
> しかも工房呼ばわり。
そりゃあ 65=69 で、質問したら馬鹿にされて悔しかったんだろ（ｗ
0078　NGNG
おおかたファイル操作で躓いてんじゃねーの？＜厨房
0079デフォルトの名無しさんNGNG
>>72
自分で「吐きます」とつかっているじゃないか。
0080opopiNGNG
c:\program files\microsoft visual studio\myprojects\report\report.cpp(2) : fatal error C1083: ｲﾝｸﾙｰﾄﾞ ﾌｧｲﾙがｵｰﾌﾟﾝできません。'plot10.h': No such file or directory
こういうエラーがでるんですけど、なぜかわかりませんか？
ちなみにｐｒｏｊｅｃｔ−設定でｐｌｏｔｃ２ｆをいれたんですができません
0081Kusakabe YouichiNGNG
In article >>76, デフォルトの名無しさん/sage/76 wrote:

> 厨房見るとハイエナの如くハンターが集まってくるね(＾＾;)こわっ

っていうか「正しい教え方」どころか、
自己満足のために相手しているみたいだし。

私みたいに正しく対処すればいいのにね。
0082デフォルトの名無しさんNGNG
>>80
'plot10.h': No such file or directory
って自分で書いてるじゃねぇか。

> ちなみにｐｒｏｊｅｃｔ−設定でｐｌｏｔｃ２ｆをいれたんですができません
意味不明

正しい対処とやらは誰かさんがしてくれるじゃろ。
俺は毒づくだけ。

阿呆
0083デフォルトの名無しさんNGNG
ストレスが溜まっているからね
0084Kusakabe YouichiNGNG
In article >>80, opopi/80 wrote:

>'plot10.h': No such file or directory
> こういうエラーがでるんですけど、なぜかわかりません

考えなさい。
0085デフォルトの名無しさんNGNG
マニュアル読む癖付けろよー
と言うかwarez臭いな(ｗ
0086デフォルトの名無しさんNGNG
VC++6で下記のように書いても、エラーになりませんでした。
バグ?

while(...) {
...
} while(...);
0087デフォルトの名無しさんNGNG
＞86
なにが言いたいのかよくわからないな。
Whileが２つあるだけではないのか？
while(1) i++;
while(1) ;
を、君はエラーにしてほしい、とでも言うのか？
0088デフォルトの名無しさんNGNG
>>86
どこかで見たな、その質問。
ネタか？ﾂﾏﾗｿ
0089デフォルトの名無しさんNGNG
>>86
ネタかも知れんが、一応解説しといてやる。
structとかと違って複文ブロックのあとに;は不要。
0090デフォルトの名無しさんNGNG
cat
dog
cow
hors
dog
bird
cat

こういう感じで、一行に単語がひとつ書いてある
テキストファイルを読み込んで、重複してある
単語を取り除いて、ユニークな単語数を数えると
いうプログラムを作りました。
いったんは完成したのですが、いろいろいじっている
うちに、うまく動かなくなって、元にもどせなくなって
しまいました。
どうすれいいですか?
009190の続きNGNG
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>

char *table[1000];
int table_tail = 0;

void table_add(char *s)
{
　char *p;
　p = malloc(strlen(s)+1);
　if (p == NULL ) {
　　exit(1);
　}
　strcpy(p, s);
　table[table_tail] = p;
　table_tail++;
}

void word_count()
{
　int i, j;
　int kazu;
　int flag;
　kazu = 0;
　j = 0;

start:
　flag = 0;
　for (i = j; i < table_tail; i++) {
　　if (strcmp(table[j], table[i]) == 0) {
　　　flag = 1;
　　}
　}
　if (flag == 1 ) {
　　table[j] = NULL;
　}
　j++;
　if( j >= table_tail-1 ) {
　　goto end;
　}
　goto start;
end:

　for (i = 0; i < table_tail; i++) {
　　if (table[i] != NULL) {
　　　kazu++;
　　}
　}

　printf("%d\n", kazu);
}
009291の続きNGNG

int main()
{
　char s[256];
　char *p;
　FILE *fp;

　fp = fopen("word.txt","r");
　if (fp == NULL) {
　　exit(1);
　}

　for (;;) {
　　p = fgets(s,256,fp);
　　if (p == NULL) {
　　　goto end;
　　};
　　table_add(s);
　}
end:
　word_count();
　return 0;
}
0093デフォルトの名無しさんNGNG
>>90
ソース読むのウザイ、後でみてやるかもしれない。

> いったんは完成したのですが、いろいろいじっている
> うちに、うまく動かなくなって、元にもどせなくなって
> しまいました。
ソースは履歴を取っとけ。
“いったんは完成した”というのなら、
そういった時点でな。
0094デフォルトの名無しさんNGNG
＞いろいろいじっている
＞うちに、うまく動かなくなって、元にもどせなくなって
＞しまいました。
＞どうすれいいですか?

CVSを導入しましょう。
0095デフォルトの名無しさんNGNG
goto使うなよぅ･･･
0096デフォルトの名無しさんNGNG
gotoを全くつかうなとは言わん。(宗教論争はｶﾝﾍﾞｿ)
でも、お前がgoto使ってる個所は総て、forかwhileループ、
およびbreak文で書ける、というか俺ならそうする。

>うまく動かなくなって
何が問題が書け！
コピペしてmakeして実行してみろと？
しかもバグ付きソースを！？

アクセス違反が起きる、というなら、
strcmp(table[j], table[i])にNULLを突っ込んでいるのが原因やろ。

阿呆
0097デフォルトの名無しさんNGNG
ブレイク！
ブレイク！
0098デフォルトの名無しさんNGNG
このプログラムじゃ、いらんよな>goto
while forとbreakで十分。
0099デフォルトの名無しさんNGNG
>>90-92
なんの目的でやってるの?
プログラミングのお勉強?
それなら、とりあえず goto なくせ。
必要ない flag も使うな。

問題があるのは strcmp(table[j], table[i]) のところで
NULL を渡すことがあるのがいけんのだと思う。
0100デフォルトの名無しさんNGNG
ｹｺｰﾝだ。
010199NGNG
かぶった…
■ このスレッドは過去ログ倉庫に格納されています