文字照合の回数を計算するプログラム

Question

以下のような関数を作成したいです。

int char_a(char *x, char *y)
{
int a = 0;
char *xa, *xb;

for( (1)){
		if(xa == xb)
		{
			(2)
		}
}
return a;

}

Comments

[kazuma-s]

> どうしてもこの部分だけがわかりません。

他の部分は全部分かっているのですか？
別々の文字列を指している xa と xb が等しくなることはありません。
a と count の関係は？

[Totette]

countではなくaでした。
大変申し訳ございませんでした.

[WoodenHamlet]

①「(1), (2), (3), (4)にどのような意味の記述を入れればいいのかはわかるが、書き方がわからない」
②「(1)～(4)にどんな意味の記述を入れたらいいかも分からない」
③「for文がわからない」
④「for文が2回続いていることの解釈がわからない」

等いろいろな「わからない」が考えられます。考えた結果、どこまでわかって、どこからわからないかが分かれば、助言のしようもあるかと思います

Answer 1

if(xa == xb)

が間違っているため、この穴埋めは不可能です。

if(*xa == *xb)ならば可能です

---

for(xa = x; (1); (2)){ /文字列xの最後まで/
for(xb = y; (3); (4)){ /文字列yの最後まで/

それぞれ、
(1),(3)にポインタが指す文字が終端ではない
(2),(4)にポインタを1文字分後方に動かす
を補ってやればいいです

Comments

[Totette]

ご指摘ありがとうございます。

[Totette]

ということは、(2)と(4)はそれぞれxa++,xb++ということですか？

[asm]

そうなりますね

[Totette]

ですが、(1),(3)の書き方がわかりません。

[asm]

C言語の文字列はヌル終端文字列といい
「文字列の末尾には0 == '\0'をくっつける」という約束があります。

[Totette]

'\0'の所までfor文を繰り返すということですか？

[asm]

そうですね。
まぁ、失敗したところでちょびっとループが多くなるだけなので
(大抵の環境でxaもしくはxbが読み取り不可能なアドレスだった場合にエラーが出てとまる)
試してみてはどうでしょうか？

[Totette]

for(px = x; x != '\0'; px++){
　for(py = y; y != '\0'; py++){
このように作成したのですが、無限ループしてしまいます...

[asm]

x,yはいくらループを回しても動かないのでそりゃ失敗しますね。
(無限ループするのは若干不思議ですが)

最初に提示した通りpxとpyが同じ文字を指しているかを検出する場合は*px == *pyです。
同様に、pxおよびpyについて'\0'を指していないかを条件にしてやります。

[Totette]

解決しました!
ありがとうございました。

[asm]

解決したのであれば
https://teratail.com/help#resolve-question
質問を「解決済み」にしてください

Answer 2

穴埋め問題の回答ではありませんが、標準関数の strpbrk を使うと、

#include <stdio.h>   // printf
#include <string.h>  // strpbrk

int char_a(char *x, char *y)
{
    int a = 0;
	for (; x = strpbrk(x, y); x++) a++;
    return a;
}

int main(void)
{
	printf("%d\n", char_a("ababc", "ac"));
}

Answer 3

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

int charcount(const char* src, const char* pattern) {
    int count = 0;
    for ( ; *src; ++src ) {
        if ( strchr(pattern, *src ) ) ++count; // 文字*srcがpattern内にあるなら+1
    }
    return count;
}

int main(void) {
    int count = charcount("ababc", "ac");
    printf("%d\n", count);
    return 0;
}