- ベストアンサー
read()で読みこんだ文字列の比較について
- みんなの回答 （9）
- 専門家の回答
質問者が選んだベストアンサー
コードを拝見しました。２点程問題があるようです。 (1) read() の fd に相当する変数名が sock_ipadrc となっていますが、ソケットの場合、ファイルと違って必ず sizeof(Buf) 分読めるとは限りませんので（読める場合もあるので、コードは正しいと思い込まないようにしてください））、sizeof(Buf) に満たなかった場合、再度読み込むというような処理が必要です。 (2) で、Buf に sizeof(Buf) 分読み込めたとして、以下のようにすれば strncmp() を使えば、最後の '?0' を気にしなくても大丈夫です。 if( !strncmp( "GETADR", Buf, strlen( "GETADR" ) ) { ... }
その他の回答 (8)
- ussii46
- ベストアンサー率0% (0/4)
質問者様の質問の範疇からは外れてしまいますが… No.7様が提示されたコードは、質問者様の想定している状況では恐らく動きますが、 実際のプログラム中に組み込むにはかなり大きな問題を含んでいます。 １．sizeof(Buf)分のデータを読み込むとBuf[sizeof(Buf)]に0を書き込むことになり、 他のメモリ領域を破壊します。 ２．readがエラー終了した場合、Buf[-1]に書き込むことになり 他のメモリ領域を破壊します。 なお、lenの値による分岐を設ければ他のメモリ領域破壊は回避できますが １はBufを全て使い切り、?0を入れる領域が無くなります。 そのため、strcmp以外のライブラリ関数を使ったり、 バッファを他の領域にコピーし直すような対処が必要となります。
- moritan2
- ベストアンサー率25% (168/670)
int len; len = read(sock_ipadrc,&Buf,sizeof(Buf)); Buf[len] = 0; とすればいいんじゃない？
- rinkun
- ベストアンサー率44% (706/1571)
No.1のお礼からするとBufの値をデバッガで確認していますか? Buf[6]が何になっているか確認ください。 これが'\0'でなければ文字列は等しくないです。 ソケット(?)から読み込んで終端文字('\0')まで読み込んでいる可能性は高くないと考えます。 あと一般論ですが、readの返値は変数に取ってチェックした方が良いと思いますよ。
- Trick--o--
- ベストアンサー率20% (413/2034)
readって\0いれたっけ……？
- ussii46
- ベストアンサー率0% (0/4)
この辺をチェックすればよいかと。 ・Bufはread前にnull初期化されているか readは読んだデータの最後にnullを付けません。 つまり、strcmpはBufをnullが出てくるまでBufを評価し続けます。 ・Buf内のデータがunicode文字列だったりしないか ・readの返り値を見て正しくread出来ているかをチェック エラーが起こっているようならerrnoやperrorで調べる
- ko_kinta
- ベストアンサー率39% (43/109)
readした文字列の終端に改行があるのでは？ read()の戻り値がBufに読み込まれた文字列長なので、それで確認してみてください。 文字列長が間違いないとしたら、一文字ずつループして比較してみるしかないですね。
- salf
- ベストアンサー率42% (27/64)
＞確認してみました。 ＞やっぱりどうみても等しいですね。 文字列の前後にスペースが入ってるとかないですか？
- salf
- ベストアンサー率42% (27/64)
比較結果が等しくならないんですよね？ とりあえず、Buf に入っている文字を表示してみて確認されてはどうでしょうか？
関連するQ&A
- 文字列の比較に関する質問
文字列をif文で比較したいのですが、まず下記の例だと char *a a="a"; if(a=="a"){ printf("等しい"); }else{ printf("等しくない"); } 文字列は等しくなるのですが下記の例ではなりません 何故なのでしょうか。 read(s_sock,&buf,sizeof(buf)) if(a=="GET"){ printf("TRUE") }else{ printf("FALSE"); }
- ベストアンサー
- C・C++・C#
- バイナリ文字列の比較について
テキストをバイナリで読み込んで文字列を比較する際に 下記のロジックだと2バイトずつ読み込んで比較していくのですが 比較したい文字列が"He"だと合致するのですが1バイトずれの "el"だと合致しません。 ストリーム内の任意の文字列を判定したいときはどのように修正すればよいでしょうか？ ----sample.txt--- Hello World!! ---------------- byte[] buf = new byte[2]; FileInputStream in = new FileInputStream("sample.txt"); int b; while ((b = in.read(buf)) != -1) { if ("He".equals(new String(buf))) { System.out.println("HelloのHeはOK"); } if ("el".equals(new String(buf))) { System.out.println("HelloのelはNG"); } }
- ベストアンサー
- Java
- strcmp関数などでの複数の文字列の比較
以下の例はランダムなiの値で文字列にアクセスして その文字列がどの文字列であるかを判定しているプログラムです。 char *string[] = { "aaa", "bbb", "ccc", } ; int i = rand() % 3; if ( strcmp( string[ i ], "aaa" ) == 0 ) { printf("aaaです"); } if ( strcmp( string[ i ], "bbb" ) == 0 ) { printf("bbbです"); } if ( strcmp( string[ i ], "ccc" ) == 0 ) { printf("cccです"); } するとこのようにif文の羅列になってしまいます。 （strstr関数を使う場合などでも同じような感じです。） 複数の文字列を判定する場合などにもっと良い手法は無いでしょうか？
- ベストアンサー
- C・C++・C#
- 文字列を比較
文字列を比較して 一つでも一致したらファイルに書き込まず抜ける 一つも一致しなかった場合一度書き込んで抜けるという操作を 下記のコードで実現したいんですけど、 "cPid[n] = cStoRegiAdd; for(j = 0;j < MAX;j++){ if(cPid[n] != cRegiTab[j]){ iflag = 1; } } if(iflag == 1){ fprintf(fh,"#define %s 0\n",cStoRegiAdd); }" の部分で何かおかしいとこはありますか？？ もしあれば教えていただきたいんですけど？？ #include<stdio.h> #include<string.h> #include<stdlib.h> #define MAX 1024 char *cRegiTab[MAX]; // 文字列登録表 int n = 0; int main() { rightsidehfunction(); leftsidehfunction(); } /* 左辺の値をとって.hファイルに書き込む関数 */ int leftsidehfunction() { FILE *ft; FILE *fh; char buf[MAX]; char buf1[MAX]; char *cPid[MAX]; char *cWorkRegiAdd; char *cStoRegiAdd; int istr = 0; int k; char *p; char *q; int s; int i; int j; int iflag = 0; ft = fopen("test.txt","r"); fh = fopen("test.h","a+"); if (fh == NULL || ft == NULL) { printf("ファイルを開けません。\n"); return 1; } /* ファイルを1行ずつ読み込む処理 */ while (fgets(buf, sizeof buf, ft) != NULL){ cWorkRegiAdd = strtok(buf, "="); for(k = 0;k < n;k++){ if(*cPid[k] != *cWorkRegiAdd)continue; if(strcmp(cPid[k],cWorkRegiAdd) == 0)break; } if(k < n) continue; cStoRegiAdd = malloc(strlen(cWorkRegiAdd) + 1); strcpy(cStoRegiAdd,cWorkRegiAdd); cPid[n] = cStoRegiAdd; for(j = 0;j < MAX;j++){ if(cPid[n] != cRegiTab[j]){ iflag = 1; } } if(iflag == 1){ fprintf(fh,"#define %s 0\n",cStoRegiAdd); } n++; } fclose(fh); fclose(ft); return 0; } /* 右辺の値をとって.hファイルに書き込む関数 */ int rightsidehfunction() { FILE *ft; // 入力ファイルポインタ FILE *fh; // 出力ファイルポインタ int c; //読み込み文字 char buf[MAX]; //バッファ領域 char buf1[MAX]; char cPid[MAX]; // 採取したPID char *cWorkRegiAdd; // 採取文字列作業域アドレス char *cStoRegiAdd; // 文字列格納域アドレス int iflag = 0; // 0/1; 右/左のクオート int ichs = 0; // chrの文字数 int istr = 0; // strの文字数 int k; // ntrの探索index int iOid; //OID char *p; ft = fopen("test.txt", "r"); fh = fopen("test.h", "w"); if (fh == NULL || ft == NULL) { printf("ファイルを開けません。\n"); return 1; } //ファイルを1文字ずつ読み込む while ((c = fgetc(ft)) != EOF) { switch (c) { case '"': iflag = 1 - iflag; /*右のクオート*/ if (iflag == 0) { cPid[ichs] = '\0'; cWorkRegiAdd = cPid; for(k = 0;k < istr;k++){ if(*cRegiTab[k] != *cWorkRegiAdd) continue; if(strcmp(cRegiTab[k],cWorkRegiAdd) == 0) break; } if(k == istr){ cStoRegiAdd = malloc(strlen(cWorkRegiAdd) + 1); strcpy(cStoRegiAdd,cWorkRegiAdd); cRegiTab[istr] = cStoRegiAdd; istr++; fprintf(fh, "#define %s 0\n", cStoRegiAdd); } ichs = 0; } break; case '\n': if (iflag == 1) { cPid[ichs] = '\0'; iflag = 0; } ichs = 0; break; default: if (iflag == 1) { cPid[ichs] = c; if (ichs < MAX - 1){ ichs++; } } break; } } /* ファイルの最後が改行じゃなかった場合 */ if (ichs>0) { cPid[ichs]='\0'; } fclose(ft); fclose(fh); return 0; }
- 締切済み
- C・C++・C#
- 文字列の中の1文字を比較するには？
XP,Studio.NETでC++を書いています。 文字列の中の１文字を比較したいのですがどのようにしたらいいのかわかりません。 今以下のような文字列がstring[300]に入っているとします。 「\nは改行コードです。printf("");では"から"までの文字が画面に表示されます。」 このとき、１文字ずつを取り出し、文字を比較したいのですが (iを増加) if(string[i]=='\') flag=1; //処理→次にnが来る。 if(string[i]=='"') flag=2; //処理→文字はダブルコーテーション という処理をしたいのですが、 エラー：定数が多すぎます。 エラー：定数が2行目に続いています。 と出ます。どうしたらいいのでしょうか？ どなたか教えていただけると幸いです。
- ベストアンサー
- C・C++・C#
- 一番大きい奇数を表示する
scanf関数を使用して、文字列を１０回入力し一番大きい文字列を表示するプログラムを作ったのですが、 一番大きい「奇数」を表示するように条件を加えた場合どうすればよいのでしょうか？ 偶数＝割り切れる 奇数＝割り切れない ということまでは分かるのですがその先が分かりません。 一応一番大きい文字列を表示するプログラムを貼っておきます。 #include <stdio.h> int main(void) { char str[1024]; char buf[10]; int i; printf("文字列を10回入力して下さい：\n"); memset(str, 0, sizeof(str)); for (i = 0; i < 10; i++) { memset(buf, 0, sizeof(buf)); printf("input>\n"); scanf("%s", buf); } if (strcmp(buf, str) > 0) { strcpy(str, buf); } printf("output>\n%s\n" , str); getchar(); }
- ベストアンサー
- C・C++・C#
- 文字列を比較するプログラム
キーボードより文字列aと文字列bを入力し、比較する（どちらが辞書並びで先かを表示する）プログラムを作れ。但し、strcmp関数を用いてはならない。 という問題があるんですが文字列の比較のしかたがまったくわかりません。わかりやすく教えていただけるとありがたいです。
- 締切済み
- 大学・短大
- 文字列比較の演算子
Perl の世界から PHP にやってまいりました。 Perl では、文字列の比較には専用の演算子がありまして、 str1 eq str2 とすると、二つの文字列が同値かどうか調べられますが、PHP にはこのような演算子はないのでしょうか？ www.php.net のマニュアルを見ても、そのような演算子がありません。 == 演算子が使えるものかと <?php $str1 = "this is string" ; $str2 = "this is string" if( $str1 == $str2 ) { echo "two string is same" ; } ?> などを試してみるとうまくいくようですが、www.php.net のマニュアルには「文字列を数値化して比較する」と書いてあり、本当に文字列比較が出来るか確信がもてず…。 結局、文字列を比較したいときには strcmp を呼び出していますが、これは PHP 的には正しいのでしょうか？
- ベストアンサー
- PHP
- c言語 文字列の比較
ファイルから読み込んだ文字列と入力した文字列を比較して同じなら次の処理を行いたいんですが、入力が例えば同じ１２１２でもstrcmp()で１８１がかえってきます。原因は何なのでしょうか？ #define BUFMAX=45; char smsg[BUFMAX]; char pass[BUFMAX]; printf("pass=%s\n",pass); printf("smsg=%s\n",smsg); c=smsg; stc=strcmp(pass,c); passにはscanfで1212を読み込みます。 smsgにはfgetsでファイルから一行読み込みます。 テキストのなかみは1212が一行はいっています。 自分では全く見当がつかないので、回答していただける方がいれば、ヒントでもいいのでお願いします。
- 締切済み
- C・C++・C#
お礼
確認してみました。 やっぱりどうみても等しいですね。