ソート、数字二桁からの処理がうまくいきません...

Question

前提・実現したいこと

要素数が n(n<=100)である int 型の配列 vc の並びを逆順にする関数
void rev_intarray( int *vc, int n )
を作成し，main関数からキーボードから入力した要素数 nとn個の整数からなる整数型配列を引数として渡すことで呼び出して，結果を表示せよ．

という課題が出たのですが、うまく行かず手を貸していただきたいです。

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

二桁に入ってからの処理がうまくいきません

該当のソースコード

#include <stdio.h>
#include <stdlib.h>

void rev_intarray( char *vc, int n ){
	int first=0, last, temp;
	last = n;
	for(first=0;first<n/2;first++){
		temp = *(vc+first);
		*(vc+first) = *(vc+last);
		*(vc+last) = temp;

		last--;
	}
}

int main(){
    int n=0, i=0;
    char vs[100];
    
    fgets(vs,100,stdin);
    n = atoi(vs);
    
    rev_intarray(&vs[2], 2*n-2);
    for(i=2;i<=2*n;i++){
        printf("%c", vs[i]);
    }
    return 0;
}

試したこと

一桁の数字の場合はうまくいくのですが、二桁になるとうまくいきません...

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

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

Comments

[azuapricot]

(課題がわからないなら先生に聞きましょうと何度teratailで書いたことか)
(課題ってわざわざ言わなきゃいいのに)
(自分の分からないところがわかってるんだったらそこだけ抜き取って質問すればいいのに)

[cateye]

>入力した要素数 nとn個の整数からなる整数型配列を引数として〜

void rev_intarray( char *vc, int n )・・・これって、おかしくないですか？
『整数型配列』は何処に？
・・・rev_intarray(&vs[2], 2*n-2);←なぜ、「要素数 n」が「2*n-2」に成るのでしょう？

[dodox86]

題名中の"ソート"を読み、更に質問内容を読んで「を？」と思ったのですが、SORTと聞くと一般的には昇順、降順に並び替えることを指すことがほとんどだと思いますが、「一定の基準で並び替える」をSORTの定義とするならば、末端から先頭に並び替えるのもSORTと言っていいのかな？

[dice142]

退会しとりますがな。。。

Answer 1

要素数が n(n<=100)である int 型の配列 vc の並びを逆順にする関数
void rev_intarray( int *vc, int n )
を作成し，main関数からキーボードから入力した要素数 nとn個の整数からなる整数型配列を引数として渡すことで呼び出して，結果を表示せよ．

という課題とのことですが、

void rev_intarray( char *vc, int n ){

と書いていて、そもそも関数の仕様が合っていないです。

要は整数配列の並び替えなのに、文字列を受け取っているために実装が大きくズレています。

問題文を読み直して、仕様を正しく把握しましょう。

Answer 2

問題文に対する理解が足りません。
よく読んで下さい。
ヒント↓
・100個のintの配列を作る・・・vc[100];
・要素数を読み込む・・・scanf();→n
・入れ替え前の表示
・呼び出し（rev_intarray(vc, n);）
・反転・・・rev_intarray(int *vc, int n)
・入れ替え後の表示
「追記」

#include <stdio.h>
#include <stdlib.h>

static void rev_intarray(int *vc, int n)
{
    int *pos = vc + n - 1;
    while (vc < pos) {
        int tmp = *vc;
        *vc++   = *pos;
        *pos--  = tmp;
    }
}

int main(void)
{
    int vc[100];
    for (int i = 0; i < 100; i++) {
        vc[i] = i + 1;
    }
    char str[8];
    fputs("要素数 ?", stdout);
    fgets(str, sizeof str, stdin);
    int n = (int)strtol(str, NULL, 10);
    //
    rev_intarray(vc, n);
    //
    for (int i = 0; i < n; i++) {
        printf("%3d", vc[i]);
    }
    putchar('\n');

    return 0;
}

usr ~/Project/test % ./a.out
要素数 ?20
 20 19 18 17 16 15 14 13 12 11 10  9  8  7  6  5  4  3  2  1

Comments

[dice142]

乱数で初期化って質問文の何処に書いてますか？

[cateye]

いらないと思いますが、ゴミが入るので確認しにくいかと・・・削除します。
簡単なのは１〜１００で埋めることでしょうか？

[dice142]

ああなるほど、確認用にですね。
私は課題で入力値が与えられてるけど、それも抜けてると判断していました。

[cateye]

あぁ・・・なるほど・・・了解です。
１〜１００で埋めておけば、入れ替え前の表示もいらないですね？

[dice142]

1～100入れとくだけで結果見ればわかるので確認用にいいですね。
（質問者さん退会してしまいましたが。。。）

[cateye]

文章問題に弱い日本人・・・(´；ω；｀)
せっかく、作ったのに・・・

Answer 3

そもそも、文字のソートしてるだけなので、要素は１文字分だけですよ