C言語 配列の中身を反転させる方法について

Question

解決した
ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー

Comments

[SHOMI]

ソースコードは以下のように貼り付けてください。
```C
コード
```

[SHOMI]

質問文のコードを直接編集されるとどこに対する指摘かわからなくなるので避けてください。

Answer 1

ptに値を設定しないまま使用しています。

---

for (pp = pat; *pp != '\n'; pp++)

scanfで読み取った文字列には改行が含まれないので、メモリ上のどこかで\nが見つかるまでループが止まりません。

---

	for (i = 0; i < strlen(txt) / 2; i++) {
-		char swap = *(pt + i);
+		char swap = *(txt + i);
-		*(pt + i) = *(pt + txt_len - i);     //ここが入れ替えてる場所です
+		*(txt + i) = *(txt + txt_len - i - 1);     //ここが入れ替えてる場所です
-		*(pt + txt_len - i) = swap;
+		*(txt + txt_len - i - 1) = swap;
	}

---

-	for (pp = pat; *pp != '\n'; pp++)
+	for (pp = pat; *pp != '\0'; pp++)

Comments

[junnnnchan]

pt = txt + pat_len - 1;をfor文の前に置いたのですがやはり同じエラーが出ました

[SHOMI]

forの中を書き換える形としたのでコメント削除

[SHOMI]

闇雲に動かしてみるのではなく、デバッガでステップ実行してどうなっているのか確認してみては？

[junnnnchan]

for (i = 0; i < strlen(txt) / 2; i++) {
+ char swap = *(txt + i);
+ *(txt + i) = *(txt + txt_len - i - 1); //ここが入れ替えてる場所です
+ *(txt + txt_len - i - 1) = swap;
}
をやってみたのですがコアダンプとなりました