質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

解決済

3回答

207閲覧

C言語で配列の対照を入れ替えるコードについて

退会済みユーザー

退会済みユーザー

総合スコア0

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2017/12/04 10:30

###前提・実現したいこと
C言語で配列の対照を入れ替えるコードを書いています。
以下が実行できて欲しい結果です。

a = [11, 1, 5, 6, 8, 2] a = [2, 8, 6, 5, 1, 11]

###発生している問題・エラーメッセージ
以下のようにエラーが表示されており、修正する際のアドバイスをいただけましたら幸いです。

In function ‘Invert’: warning: ‘j’ is used uninitialized in this function [-Wuninitialized]

###該当のソースコード

c

1#include <stdio.h> 2void Print(int *a, int n); // The "void" shows no return 3 4//fuction def 5void Invert(int *a, int n); 6 7int main(void){ 8 int a[6] = {11, 1, 5, 6, 8, 2}; 9 Print(a, 6); //Outputs the array. 10 Invert(a, 6); //Revert the order. 11 Print(a, 6); //Outputs the array again. 12 return 0; 13} 14 15//function 16void Print(int* a, int n){ 17 int i; 18 for (i=0; i<n; i++){ 19 printf("%d, ", a[i]); 20 } 21 printf("\n"); 22} 23 24//function 25void Invert(int *a, int n){ 26 int j, tmp; 27 tmp = a[j]; 28 a[j] = a[6-j]; 29 a[6-j] = tmp; 30}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

void Invert(int *a, int n) { int j, tmp, k; for(j = 0; j < (n/2); j++) { k = n - j - 1; tmp = a[j]; a[j] = a[k]; a[k] = tmp; } }

こうじゃろ。

投稿2017/12/04 10:39

編集2017/12/04 10:44
tkturbo

総合スコア5572

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2017/12/04 12:38

ご回答いただきましてありがとうございました。
guest

0

これはエラーじゃなくて警告ですね。想定通りに作用しないという点では同一ですが。
警告の内容は書いてあるまんまなので、英語だからと敬遠せずに読んでください。

In function ‘Invert’: warning: ‘j’ is used uninitialized in this function [-Wuninitialized]

jが初期化されていないまま使われてるよと、明確に書いてあるじゃないですか。


C

void Invert(int *a, int n){
int j, tmp;
tmp = a[j];
a[j] = a[6-j];
a[6-j] = tmp;
}

Invertは配列を反転させるための関数なのですよね。
tmpを用いている部分は典型的なスワップ処理ですが、一回で反転するわけないじゃないですか。
適切に繰り返す必要があることは言うまでもないです。

既に他の方々の回答がついているので、後はそちらを見て考えてください。
大事なのは、自分のコードが何をやっているのか知ることです。

投稿2017/12/04 10:38

編集2017/12/04 10:43
LouiS0616

総合スコア35658

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2017/12/04 12:37

ご回答いただきましてありがとうございます。 もっと経験を積みたいと思います。
guest

0

void Invert(int *a, int n) { int j, tmp; for(j = 0; j <= 2; j++) { tmp = a[j]; a[j] = a[6-j]; a[6-j] = tmp; } }

じゃないでしょうか。

投稿2017/12/04 10:36

編集2017/12/04 10:37
gazette2

総合スコア179

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

LouiS0616

2017/12/04 10:39

引数のnを使ってあげてください。
gazette2

2017/12/04 10:41

tkturboさんの答えが正解ですね。
tkturbo

2017/12/04 10:45

array lengthと交換先のIndex算出部分を修正しました。私の最初の回答でも正解になりません。 実行時にArrayIndexOutOfBoundsが発生してしまう。
退会済みユーザー

退会済みユーザー

2017/12/04 12:38

ご回答いただきましてありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問