要素の入れ替えについて

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
サイト初心者でお見苦しいところがあったら申し訳ありません。
関数のint data[], int len, int x, int yを変更せず、for文を使わずに結果を12345
12435とだしたいのですが、お力を貸して頂きたいです。
変更点は関数のint data[], int len, int x, int yより下のコードです。
何卒宜しくお願い致します。

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

エラーメッセージ

該当のソースコード

void swap(int data[], int len, int x, int y);

int main(void)
{
int arr[5] = { 1, 2, 3, 4, 5 };
int length = sizeof(arr) / sizeof(int);
for (int i = 0; i < length; i++)
{
printf("%d ", arr[i]);
}
puts("");
swap(arr, length, 2, 3);
for (int i = 0; i < 5; i++)
{
printf("%d ", arr[i]);
}
puts("");
return 0;
}
void swap(int data[], int len, int x, int y)
{
int i = 1;
int temp;
for (i = 0;i < len;i++)
{
if (i == x)
{
temp = data[x];
data[x] = data[y];
data[y] = temp;
}
}
}

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Comments

[y_waiwai]

提示のコードではどういう結果になるんでしょうか

[kennzi_665]

自分が載せたソースコードでも出力結果は12345 12435となります。情報不足で申し訳ありません。

[kennzi_665]

申し訳ありません。関数のコードは変更しないと載せるのを忘れいていました。」

[dodox86]

関数のコードは変更しない、関数のint data[], int len, int x, int yは変更しない、でもfor文は使わずに、とは意味不明です。一体どうしろと。

[kennzi_665]

申し訳ありません、int data[], int len, int x, int yを変更せずにfor文を使わずに、です。誤解を招いて申し訳ありません。

[episteme]

> void swap(int data[], int len, int x, int y)
コレは何してくれる関数なんですか? 仕様をきっちり述べてください。

「daa[x] と data[y] を入れ替える」であれば len は何に使うんですか?

[kennzi_665]

長さがlenの配列dataの要素xと要素yを入れ替える関数の本体を答えるという問題です。言葉が足らずに申し訳ありません。

[dodox86]

配列の要素長lenがあるということは、実はx, yが配列の範囲を越えているかどうかをチェックせよ、という隠れた要件があるのでは？（もう質問閉じちゃってますけど）

[kennzi_665]

そのような要件は特にないと思います！皆さんコメントありがとうございました。また、何かあったら質問させていただきます。

Answer 1

for文でi==xとしていますが、必要ありません。
x,yは独立した変数です。
そのまま配列のインデックス値として扱えばよいです。
swap関数のforを削除すればOKです。

void swap(int data[], int x, int y) {
  int temp = data[x];
  data[x] = data[y];
  data[y] = temp;
}

Comments

[kennzi_665]

本当にごめんなさい。関数のint data[], int len, int x, int yは変更しないと情報を載せるのを忘れいました。せっかく回答していただいたのに申し訳ありません。

[kazuma-s]

「関数の引数の int data[], int len, int x, int y は変更しない」ということですか？
それなら、第2引数に int len, を追加するだけですよね。
引数は全部使う必要はありません。
何が問題なのですか？

[kennzi_665]

確かにその通りですね！緊張してよく読まずにコメントしていました。すみません。