2つの文字列の文字数の合計を出力する

Question

前提・実現したいこと

文字配列a,bの先頭アドレスを受け取り、２つの文字数の合計を返却値で返す関数を作成したいです。「関数内では、ポインタのインクリメントを繰り返す方法を用い、構造体、添え字[]は用いない」という条件が付いています。

発生している問題・エラーメッセージ

1>C:\Users\fit-user\Desktop\test\test.c(4,8): warning C4013:  関数 'strcat' は定義されていません。int 型の値を返す外部関数と見なします。
1>C:\Users\fit-user\Desktop\test\test.c(13,9): warning C4477: 'printf' : 書式文字列 '%s' には、型 'char *' の引数が必要ですが、可変個引数 1 は型 'int (__cdecl *)(char *,char *)' です
1>C:\Users\fit-user\Desktop\test\test.c(11,1): warning C4047: '初期化中': 間接参照のレベルが 'char' と 'char [6]' で異なっています。
1>C:\Users\fit-user\Desktop\test\test.c(12,1): warning C4047: '初期化中': 間接参照のレベルが 'char' と 'char [4]' で異なっています。

該当のソースコード

int count(char *a,char *b) {
	int i, count = 0;
	strcat(a, b);
	for (i = 0; a = "\0"; i++) {
		count++;
	}
	return(count);
}
int main(void) {
	char x = { "hello" };
	char y = { "usa" };
	printf("mojisuu=%s", count);
	return(0);
}

試したこと

参考書などを見直したのですがやり方がよくわからなかったです。

補足情報（FW/ツールのバージョンなど）

大学でｃ言語について学習しています。最近始めたばかりでわからないところが多々あります。今後のためにもどこが間違っていてどうしたら改善できるのか教えていただけると嬉しいです。間違っていた部分を重点的に復習していきます。
ちなみに使っているのは、visualstdio2019です。

Comments

[cx20]

参考にされた参考書を教えて頂けますか？

[Global4996]

「新・明解c言語　入門編」ていう参考書です。

[cx20]

参考書の提示、ありがとうございます。
参考書はかなり丁寧に書かれているようなので、悪くは無さそうに思いました。
演習問題が付いているかと思いますので、最初から１つずつこなしていくことをお勧めします。

[cx20]

目次レベルだと
第9章 文字列の基本
第11章 文字列とポインタ
あたりを復習すると良いと思います。

[Global4996]

貴重なお言葉ありがとうごさまいます。確かにまだ基本的なところがわかっていないと気づいたので提示していただいた項目の復習をがんばります。ありがとうございました！

Answer 1

２つの文字数の合計を返却値で返す関数

これをどう実現しようと考えましたか?　多分、strcat()を使ってと思いますが、 strcat()関数が何をする関数で、どう使うのか、をまず確認しましょう。
....なんですが、わざわざ、文字列をつなげる必要はなく、別個に数えて加えても良いのでは?

for (i = 0; a = "\0"; i++) {

警告が出ないのが不思議。　特に、a = "\0"; これでは、aに 文字列 "\0"を代入 ... ではなく、"\0"のポインタを代入となります。多分、C言語での文字の扱いが分かってないのでは? と思います。文字列の終わりが 0で終わる事が分かってるなら、a != '\0'; .... と思ったが、この for()は、こんな感じ?　(意味、分かりますか?)

    for (; *a != '\0'; a++)	count++;
    for (; *b != '\0'; b++)	count++;

warning C4047: '初期化中': 間接参照のレベルが 'char' と 'char [6]' で異なっています。

となります。x は、charなので、1文字(半角英数1文字)しか入りません。文字列にするならば、配列とし、 char x[6] = { "hello" };のようにすべき。

次の警告、

warning C4477: 'printf' : 書式文字列 '%s' には、

関数printf()の仕様を調べましょう。
printf("mojisuu=%s", count); の%s に対応するのが、countです。 %sは引数に文字列がある事を示します。ではcountは何? コンパラは関数名と判断したようです。 関数名は文字列でないので不適切。 また、関数としたいならば、count(x, y)のように引数も付けるべきです。
ただし、この関数の戻り値は、intなので、%sに一致しません。 %d とすべきでしょう。

Comments

[Global4996]

なるほど！警告の意味がわかっていなかったので警告ごとの改善方法を教えてくださったのはとても勉強になりました。ありがとうございました！

[cx20]

> char x[6] = { "hello" };
波カッコはいらないかなと思いましたが、この書き方でもエラーにはならないのですね。少し気になって調べてみました。
＜参考＞
■ 「それを波括弧で囲んでもよい」は何の意味があるのか？
https://teratail.com/questions/92502

[pepperleaf]

確かに、、波カッコはいらなかった。コンパイル通ったので、気にしなかったけど。
他にも細々とありそうなで、これからも大変かと思った方が先。

Answer 2

warning C4013: 関数 'strcat()' は定義されていません。int 型の値を返す外部関数と見なします。

この警告は関数が見つからない場合に表示されます。strcat() は string.h に含まれている為、

#include <string.h>

とすれば、警告は消えるかと思います。（ただ、今回の文字数をカウントするだけの場合であれば strcat も使う必要はありません。）

warning C4477: 'printf' : 書式文字列 '%s' には、型 'char *'～

printf("mojisuu=%s", count);

printf() の %s は「文字列」を指定する場合の書式になります。
文字数を表示したいのであれば %d を使ってください。

printf("mojisuu=%d", count);

warning C4047: '初期化中': 間接参照のレベルが 'char' と 'char [6]' で異なっていま

「文字」と「文字列」の区別の理解が足りていないように思います。

char x = { "hello" };

１文字だけが格納できる変数に「文字列」 を代入しようとして警告が出ています。文字列として書くとしたら、以下になります。

char x[] = "hello";
// 文字列の初期化は以下の方法でも可能です。「"」でなく「'」が使われていることに注意ください。
// char x[] = {'h', 'e', 'l', 'l', 'o', '\0'};

strcat(a, b);

今回は strcat() は不要です。

for (i = 0; a = "\0"; i++) {

これは恐らく NULL 終端かどうかのチェックを使用としているのだと思いますが、これだと代入になってしまいます。書くとしたら、

for (i = 0; *a != '\0'; i++) {

といった感じでしょうか。というか i 使ってませんね。。

for (; *a != '\0'; a++) {

で良さそうです。

以下は上記の指摘を元に、修正した結果になります。

#include <stdio.h>

int count(char *a,char *b) {
    int count = 0;
    for (; *a != '\0'; a++) {
        count++; // 文字数のカウントアップ
    }
    for (; *b != '\0'; b++) {
        count++; // 文字数のカウントアップ
    }
    return count;
}

int main(void) {
    char a[] = "hello";
    char b[] = "usa";
    // 文字列の初期化は以下の方法でも可能です。「"」でなく「'」が使われていることに注意ください。
    // char a[] = {'h', 'e', 'l', 'l', 'o', '\0'};
    // char b[] = {'u', 's', 'a', '\0'};

    int mojisuu = count(a, b);
    printf("mojisuu=%d\n", mojisuu);
    return 0;
}

Comments

[Global4996]

NULL文字の使い方が間違えていたんですね。解決できました！

[cx20]

解決できたようで何よりです。
ただ、今回、間違えている箇所はNULL文字の箇所以外にも多く見受けられました。（C言語を殆ど理解していないのでは？と不安に感じました。）
他の学習で忙しいかと思いますが、正しく理解しないと後で困ると思いますので、再度、参考書を初めからやり直すことをお勧めします。

＜今回気になった点＞
・#include の指定が無い
・今回のケースでは変数「i」は不要
・今回のケースでは strcat() も不要
・NULL 文字の判定の仕方が誤っている
・文字列の初期化が誤っている
・printf の書式が誤っている
・count() 関数の戻り値を取得していない

[Global4996]

確かにnull文字以外にも多くの間違いがありますね。もう一度参考書を1から復習したいと思います。貴重なお言葉ありがとうございました。