• ベストアンサー

文字列の比較

このプログラムに問題はあるでしょうか？ ２つの文字列に対して，同じ位置にある文字が一致する箇所をカウントするプログラムです． やはりwhile分の||を&&にしなければならないのでしょうか？ #include <stdio.h> int main(void) { char mojiA[] = "Please Help Me!" ; char mojiC[] = "eeeeeeeeeeeeeeeee"; char mojiB[] = "Please give me some money" ; int count ; /* 一致した文字数 */ int i ; count = 0 ; i=0; do{ if (mojiA[i] == mojiB[i]) { 　　　　　　　　　count++ ; 　　　　　　　} i++; } while(mojiA[i]!=0 || mojiB[i]!=0); printf("%d\n", count) ; return 0 ; } ;

質問者が選んだベストアンサー

  • ベストアンサー
  • Oh-Orange
  • ベストアンサー率63% (854/1345)
回答No.2

★文字列の最後は NULL 文字です。 ・このため、mojiA、mojiB のどちらか一方が NULL 文字に到達したときに while 文のループを 　抜けるようにします。こうしないと NULL 文字以降の文字列ではないメモリとの比較・カウント 　が行われ正しくカウントされませんよ。 ・よって問題ありです。→回答者 No.1 さんのアドバイスどおりに do-while 文だと mojiA、mojiB 　が最初から『空文字列』のときに１カウントしたり、do-while 文を抜け出せなくなってしまい 　無限ループしてしまうかも…。怖いよ～無限ループは。 ・それではどうするか？ ・do-while 文から while 文または、for 文に書き直します。 ・私は、for 文の方が分かりやすいと思います。→for(初期化 ; 条件式 ; 増減式 ){ … } 手直しサンプル: int main(void) { 　char mojiA[] = "Please Help Me!"; 　char mojiC[] = "eeeeeeeeeeeeeeeee";　←これ必要なの？ 　char mojiB[] = "Please give me some money"; 　int count = 0;　←宣言と同時に初期化も可能 　int i; 　 　for ( i = 0 ; (mojiA[i] != '\0') && (mojiB[i] != '\0') ; i++ ){　←『&&』ですね 　　if ( mojiA[i] == mojiB[i] ){ 　　　count++; 　　} 　} 　printf( "%d\n", count ); 　return( 0 ); }←ここにあった『；』文字は必要ありません。 最後に: ・質問者さんも『やはり while 文の || を && にしなければ』と分かっているようですね。 ・この場合は『&&』の演算子になります。 ・また、文字列の最後を表す NULL 文字は数値の 0 ですが、一般的に『'\0'』という表現をします。 　もちろん、数値の 0 でも動作しますが…。 　あと NULL 文字の間違った表現に『NULL』という記号定数を使う人がいますが、こちらは 　『NULLポインタ』という意味です。これも、数値的には 0 だったりしますが厳密にはポインタ 　ですので NULL 文字を表現するのに使ってはいけません。注意。 ・『NULL文字』、『NULLポインタ』、『空文字列』をしっかり区別しましょう。 ・以上。おわり。

全文を見る
すると、全ての回答が全文表示されます。

その他の回答 (1)

  • koko_u
  • ベストアンサー率12% (14/116)
回答No.1

>やはりwhile分の||を&&にしなければならないのでしょうか？ うん。 これだと mojiB の方が長かった場合に(この場合まさにそう) Please Help Me! を越えてメモリを読みにいっちゃうからね。 mojiC が mojiA と mojiB の間に挟んであるけど、メモリの配置までその順になるとは保証されないと思う。 ついでに do { } while() でループをまわすと mojiA, mojiB が ""(空)の場合に mojiA[0] を終了判定せずに mojiA[1] をアクセスしてしまうので while ループにしましょう。

全文を見る
すると、全ての回答が全文表示されます。
  1. カテゴリ
  2. [技術者向] コンピューター
  3. プログラミング・開発
  4. C・C++・C#

関連するQ&A

  • 文字列

    下のプログラムは何をするためのプログラムなのか教えてください。 個人的にはJISコードに関係していると思うのですがさっぱりわかりません。 どなたか詳しい説明お願いします。 #include <stdio.h> #define LEN 255 int main(void) { char str[LEN]; char cipher[LEN]; int ikey; int i,n; printf("文字列を入力せよ : "); scanf("%s", str); printf("鍵を入力 : "); scanf("%d", &ikey); i = 0; while (str[i]!='\0') { n = (str[i]-'A'+ikey)%26; cipher[i] = 'A'+n; ++i; } cipher[i] = '\0'; printf("%s\n", cipher); return 0; }

  • C言語　文字列の操作

    文字Cが含まれる個数を求めたいです #include <stdio.h> int str_chnum(const char str[],int c) { int i; int count=0; for(i=0;str[i]!="\0";i++) if (str[i]==c) count++; return(count); } int main(void) { char st[100]; printf("検索文字列を入力してください:"); scanf("%s",st) ; printf("検索文字列数は%uです。\n",st,int str_chnum(const str[],int c)); return(0); } コンパイルできません。なぜですか？printfの行がたぶん間違っていると思うんですが。。。

  • 文字列について

    textファイルをいくつかに分割して保存しようと思い、以下のプログラムを作りましたが、うまくいきません。 予定では "0000.txt", "0001.txt", ....と文字列を生成するはずです。 #include <stdio.h> #include <string.h> #define LENGTH 4 //番号の桁数 void filename(int n) { 　　　char name[LENGTH +4 +1]; 　　　char text[5] = ".txt"; 　　　int i; 　　　for(i = 0; i <= LENGTH; i++) 　　　　　　name[i] = '0'; 　　　i = LENGTH; 　　　//自然数を文字列に変換 　　　do{ 　　　　　　name[i] = (char)(n%10 + 48); 　　　　　　i--; 　　　　　　n /= 10; 　　　}while(n != 0 && i >= 0); 　　　strcat(name, text); 　　　printf("filename:%s", name); } int main(void) { 　　　int i; 　　　for(i = 0; i < 5; i++){ 　　　　　　filename(i); 　　　　　　printf("\n"); 　　　} 　　　return 0; } 実行結果は filename:00000@.txt filename:00001.txt filename:00002.txt filename:00003.txt filename:00004.txt このように、文字化けしています。 何が原因なのでしょうか。 よろしくお願いします。

  • 文字列の入力

    文字列(名前)と誕生日を読み込み、表示するプログラムです。文字列を読み込むときに空行を入力すると、読み込みを終了して表示するようにしたのですが、うまくいきません。読み込む時に使う関数が違うのだと思います。scanfとgets、fgetsは試したのですが、だめでした。 教えてください。 #include <stdio.h> #include <stdlib.h> #include <string.h> #define DATA 3 int main() { char *name[DATA]; char work[31]; int month[DATA]; int day[DATA]; int count1,count2; printf("Data #1\n"); printf("name ="); fgets(work,31,stdin); for(count1=0;(work[0]!='\n')&&(count1<DATA);count1++){ printf("month ="); scanf("%d",&month[count1]); printf("day ="); scanf("%d",&day[count1]); name[count1]=(char *)malloc((strlen(work)+1)*sizeof(char)); strcpy(*(name+count1),work); printf("\n"); if(count1<DATA-1){ printf("Data #%d\n",count1+2); printf("name ="); fgets(work,31,stdin); } } for(count2=0;(count2<DATA)&&(count2<count1);count2++){ printf("Data #%d\n",count2+1); printf(" name: %s\n",*(name+count2)); printf(" birthday: %2d %2d\n",month[count2],day[count2]); } }

  • 文字列中に含まれる文字の個数をカウントするプログラムについて…

    文字列、１文字が与えられたとき、これをポインタで入力し文字列中に含まれる文字の個数を計算するプログラムを作成せよ。 と、いう課題がだされたんですけど、ユーザが任意の文字列と１文字を入力できるようにすることができません…。 多分main関数の部分をちょっといじくればよいと思うのですが…。 どなたかアドバイスをお願いします。 #include <stdio.h> int count(const char *str, const char ch) { int cnt=0; while (*str!='\0') { if (*str==ch) cnt++; str++; } return cnt; } int main() { const char *str="hello,world!"; const ch='o'; int cnt; cnt=count(str, ch); printf("%s中に%cは%d個です\n", str, ch, cnt); return 0; }

  • 文字列をポインタに入れる方法を教えてください。

    どなたか文字列をポインタに入れる方法を教えてください。下のプログラムのa=count(name1);とreverse(name1,name2,a);のとこにエラーが表示されます。ちなみに＆を入れてもダメでした。整数の場合は＆を入れたらできるみたいですが、文字列となるとやり方が違うのでしょうか？ 問題としてはローマ字で名前を入力して表示し、文字数と名前を逆順にする２つの関数countとreverseを各自定義してそれぞれ定義しプログラムを組みなさい。姓と名の間は１文字空白を入れ、空白は文字数に含めないこと。 下のプログラムを実行させると・・・ 名前を入力しなさい。 yamada hirosi //自分で入力 yamada hirosi 文字数は12 逆順にした後はisorih adamay となるはずなんですが、なりません。どなたか宜しくお願いします。 #include "stdafx.h" #include <stdio.h> int count(char *name1[]) { int i,b=0; for(i=0;*name1[i]!='\0';i++) { b++; } b--; return b; } void reverse(char *name1[],char *name2[],int a) { int w; for(w=0;w<=a;w++) { *name2[w]=*name1[a-w]; } *name2[w]='\0'; } int main(int argc, char* argv[]) { char name1[80],name2[80]; int a; printf("名前を入力しなさい\n"); gets(name1); printf("%s\n",name1); a=count(name1); printf("文字数は%d\n",a); reverse(name1,name2,a); printf("逆順にした後は%s\n",name2); return 0; }

  • 関数に文字列を渡すことについて

    参考書にあったプログラムなのですが、 ------------------------------------------- #include<stdio.h> void strout(char ss[　]); int main(void) { char st[　]="ABCDEF"; strout(st); strout("ABab12"); return 0; } void strout(char ss[ ]) { int i; printf("ss=%s\n",ss); i=0; while(ss[i]){ printf("%X ",ss[i]); ++i; } printf("\n"); } ---------------------------------------------- ------------実行結果--------------- ss=ABCDEF 41 42 43 44 45 46 ss=ABab12 41 42 61 62 31 32 ----------------------------------- 初心者という事で、いろいろと疑問があるのですが、 ◎１「stとして、文字配列をss[ ]に渡すのと、"ABab12"として直接文字列をss[ ]に渡すのはとは、どういうことなのかということ。そしてその時ss[ ]はどうなっているか？」 ◎２「実行結果で、最初のprintfからループさせなくてもss=ABCDEF、ss=ABab12が何故2つとも表示され、2つとも16進数が表示されるのか？」 ◎３「while(ss[i])だけで何故、'\0'でない間ループするという事が出来てしまうのか？」 以上のような疑問があります。 先頭のアドレスを渡すといったような説明はあるのですが、いまひとつ分かりません。 教えていただけると嬉しいです。

  • C言語で　小文字を大文字に変えるプログラム

    C言語で小文字を大文字に変えるプログラムを造ったのですが、エラーが出てしまいます。　どこが間違っているのか教えてください。 #include<stdio.h> main() { char a[100]; scanf(%c,&a[i]); int i=0; while(i<100){ i++; if(a[i]>60){ printf("%x-20",a[i]); }else{ printf("%x",a[i]) } } }

  • 文字列の扱い方

    初歩的な質問ですみません… str文字列からcという文字を見つけたら添字を返すという関数を作ったのですが、 iにこの関数を代入して、if文の制御式にiを使って比較するまでは正常なのですが、 真文にiを使うと何故か偽文（という言い方でいいのでしょうか…この場合("そんな値はありません。"というところです）が実行されてしまいます。 よろしければご教授お願い致します。 #include <stdio.h> int str_char(const char str[],int c) { int len = strlen(str); int i; for (i = 0;i < len;i++) { if (str[i] == c) return i; } return -1; } int main() { char str[64] = "Fucking Brutal Death Metal"; int ch,i; printf("どの文字を調べますか？"); scanf("%c",&ch); i = str_char(str,ch); if (i >= 0) printf("その文字は%d番目にあります。",str_char(str,ch) + 1); //何故かiだと動かない else printf("そんな値はありません。"); return 0; }

  • 文字列を後ろから1文字ずつ表示する

    初歩的なプログラミングです。 C言語の勉強中で、タイトルの通りの プログラムを作りたいのですが、 何かヒントを頂けないでしょうか。 自分で書いてみて失敗したのは下記の 通りです。 char s[11]; int i=0; printf("文字列入力(10文字以内-->)"); gets(s); printf("s=%s",s); while(s[i]!='\0'){ i++; } while(i=0){ printf("%c\n",s[i-1]); i--; } 宜しくお願いします。

注目のQ&A

カテゴリ

専門家に質問してみよう

職業から探して質問する

あなたにピッタリな商品が見つかる！ OKWAVEセレクト

質問する