C言語 文字列の入れ替え

Question

配列を要素数３０で二つ宣言して、標準入力から二つの文字列を入力。
そしてswapstr関数により文字列を入れ替え、入れ替え前後の配列内の値を配列名とともに標準出力するプログラムを作成したいです。

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

文字を入れ替えるところまではできたのですが、ポインタを使えていないように思えます。for(i = 0; i < 51; i++)の部分をfor(i = 0; i < count; i++)として成立するようにしたいです。

該当のソースコード

#include<stdio.h>


void swapstr(char *str1, char *str2);


int main(void) {
	char x[51], y[51];

   printf("文字列を入力してください\n");
   scanf("%s",x);
   scanf("%s",y);
	
   printf("入れ替え前\n");
   printf("配列str1=%s\n",x);
   printf("配列str2=%s\n",y);


   swapstr(x, y);


   printf("入れ替え後\n");
   printf("配列str1=%s\n",x);
   printf("配列str2=%s\n",y);

}


void swapstr(char *str1, char *str2) {
int i;
char c;
for(i = 0; i < 51; i++) {
c = str1[i]; 
str1[i] = str2[i]; 
str2[i] = c; 
}
}

Answer 1

• 文字列自体を交換する？

		str1[i] = str2[i];

ポインタを使えていないように思えます

質問者のコードは添え字変数 i を使って、配列としてアクセスしています。そこで思いつくのは、ポインタのままアクセスすることです。例えばこんな感じです。

		c = *str1;
		*str1++ = *str2;
		*str2++ = c;

コンパイラが賢くなってきた現在は、ポインタでも配列でも、どちらでも性能やコード効率などに大きな違いは無いと聞きます。つまり、どちらでも良いのではないでしょうか。ポインタを使って配列のようにアクセスする、これもポインタの使い方です。

文字列の長さに上限があるので、文字列自体を交換しても構わないでしょうし、それが題意のように読めますが、、、

• 交換するのはポインタだけで良いんじゃない？

私が最初に思いついたのは、こちらの方法です。
この問題では文字列の長さに上限がありますが、普通は長さに上限があるとは限りません。長さ10文字の文字列と100文字の文字列を、上のやり方で交換すれば、10文字しかなかった領域に100文字を納めなければなりません。これはメモリの使い方として賢くありません。

文字列自体を交換する方法に対して、ポインタだけ交換するのは、安全だし、処理時間もチョー短く済みます。なので私なら、そうできるように、データ構造や関数呼出しの関係を考えますが。
或いは、この先にポインタを交換する課題が待っているかもしれないですね。

• for (i = 0; i < count; i++) として成立するようにしたい

すでに他の方が回答しているので省略。

• 「配列を要素数３０で二つ宣言して…」

なのに、なぜ [51] なんですか？ここはつっこまずにいられない（笑）。

int main(void) {
    char x[51], y[51];

Answer 2

もしかしてこういう事?

void swapstr(char *str1, char *str2, int count)
{
	int i;
	char c;
	for(i = 0; i < count; i++) {
		c = str1[i]; 
		str1[i] = str2[i]; 
		str2[i] = c; 
	}
}

swapstr()の最後に文字列の長さを引数として追加します。
最初の宣言も

void swapstr(char *str1, char *str2, int count);

とし、呼び出す方は、swapstr(x, y, 51); みたいにする。

Answer 3

質問の意図がよく分かりません。ポインタはちゃんと使えていると思います。
51の代わりにcountと書きたいと言うことであれば、

#define count 51

ですかね。

また、このままだと51文字以上入力すると異常動作するので、scanfじゃなくてfgetsを使いましょう。