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

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

ただいまの
回答率

88.82%

C言語 ポインタと配列に関する質問です

解決済

回答 4

投稿

  • 評価
  • クリップ 1
  • VIEW 2,413

yusukee345

score 31

質問したいこと

「発生した乱数:
1 5 2 8 9
2倍した値:
2 10 4 16 18」

というような実行結果を得たい。長さ5の整数型の配列変数aに、すべての成分に0から100の乱数の値を代入して表示し、以下の条件を満たす数値を表示するプログラムを作りなさい。
(1) 5つの整数の値は、グローバル変数ではない配列変数に入れること
(2) 配列への値の表示、値の二倍は、それぞれ専用の関数を用いること。
(3) 専用の関数の引数として、整数型ポインタをを用いること。(問題文ここまで)

という問題に取り組んでいます。
以下のようなソースコードで書いてvisualstudio2013で実行したのですが望む通りの実行結果が得られません。

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

コンパイルは成功します。実行時、画面に表示されるのは
「発生した乱数:
2倍した数:」
だけです。printfにポインタpが格納されてないか、関数を呼び出せていないかだと思います。
更に実行するとエラーメッセージが出て、「Run-Time Check Failure#2-Stack around the variable 'a' was corrupted.」と表示されデバックを中断せざるを得なくなります。

該当のソースコード

include<stdio.h>

include<stdlib.h>

include<time.h>

void show(int*);
void two(int*);//どちらの関数もアドレスの形を引数にとる

void main(){
int a[5];
int i;
int* p = NULL;
p = a; //pにaのアドレスを代入
srand((unsigned)time(NULL));
printf("発生した乱数:");
for (i = 0; i < 5; i++){
*p = rand() % 101;
void show(p);//発生した乱数をshow関数を用いて表示(pをアドレスの形で引数にする)
p++;
}
printf("\n2倍した値:");
for (i = 0; i < 5; i++){
*p = rand() % 101;
void show(p);//発生した乱数を2倍したものをtwo関数を用いて表示(pをアドレスの形で引数にする)
p++;
}
printf("\n");
getchar();
}

void show(int *n){
printf("%d ", *n);//ポインタnが示す数値を表示
}

void two(int *m){
printf("%d ", (*m) * 2);//ポインタmが示す数値を2倍したものを表示
}

試したこと

関数の引数を入れる場所をアドレスではなく値型にした( void show(int) というプロトタイプで引数を代入する時は void show(*p) とした)が「*」の場所に「型指定子が必要です」と出てエラーになる。

補足情報(言語/FW/ツール等のバージョンなど)

C言語
コンパイラはVisualStudio2013

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    2016/11/25 15:03

    こちらの質問が他のユーザから「やってほしいことだけを記載した丸投げの質問」という指摘を受けました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 4

checkベストアンサー

0

ご参考として

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

void show(int*);
void two(int*);//どちらの関数もアドレスの形を引数にとる

void main(){
    int a[5];
    int i;
    srand((unsigned)time(NULL));
    printf("発生した乱数:");
    for (i = 0; i < 5; i++){
        *(a+i) = rand() % 101;
        show(a+i);//発生した乱数をshow関数を用いて表示(pをアドレスの形で引数にする)
    }
    printf("\n2倍した値:");
    for (i = 0; i < 5; i++){
        two(a+i);//発生した乱数を2倍したものをtwo関数を用いて表示(pをアドレスの形で引数にする)
    }
    printf("\n");
}

void show(int *n){
    printf("%d ", *n);//ポインタnが示す数値を表示
}

void two(int *m){
    printf("%d ", (*m) * 2);//ポインタmが示す数値を2倍したものを表示
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/10/21 12:23

    質問に答えて下さった方々ありがとうございます。皆様のアドバイスを元にこう改良したら望み通りの結果が得られました。自分では大丈夫と思ってても抜けがたくさんあり、勉強になりました。


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

    void show(int*);
    void two(int*);//どちらの関数もアドレスの形を引数にとる

    void main(){
    int a[5];
    int i;
    int* p = NULL;
    srand((unsigned)time(NULL));
    printf("発生した乱数:");
    for (i = 0; i < 5; i++){
    a[i] = rand() % 101;
    }
    p = a;
    for (i = 0; i < 5;i++){
    show(p);//発生した乱数をshow関数を用いて表示(pをアドレスの形で引数にする)
    p++;
    }
    p = a;
    printf("\n2倍した値:");
    for (i = 0; i < 5; i++){
    two(p);//発生した乱数を2倍したものをtwo関数を用いて表示(pをアドレスの形で引数にする)
    p++;
    }
    printf("\n");
    getchar();
    }

    void show(int *n){
    printf("%d ", *n);//ポインタnが示す数値を表示
    }

    void two(int *m){
    printf("%d ", (*m) * 2);//ポインタmが示す数値を2倍したものを表示
    }

    キャンセル

  • 2016/10/21 13:34

    少しはお役に立ててよかったです。

    キャンセル

0

関数の呼び出し時は
void show(p);
はなく
show(p);
で大丈夫です

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/10/20 23:29

    追記ですが、最初pにはaのアドレスをセットしており、またaはサイズはintの5つ分確保、
    2倍した値の時には、pを初期化していないので、aの確保以降のアドレスに踏み込んでいってますね。

    キャンセル

0

Pを5つ増やした後、リセットされていないですね。
関数twoも呼ばれていないような。。。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

mainでaを参照するのにわざわざアドレス参照する必要はないと思います。
普通に配列を使えばよろしいかと。
ただこれは問題ではなく、問題点はshowの呼び方と、twoを呼んでいないことと、2倍にする前にまた乱数で初期化してしまっていることですね。
ということを踏まえて修正したのが以下です。

#include <stdio.h>
#include <time.h>
void show(int*); 
void two(int*);//どちらの関数もアドレスの形を引数にとる

int main(void){ 
    int a[5]; 
    int i; 
    srand((unsigned)time(NULL)); 
    printf("発生した乱数:"); 
    for (i = 0; i < 5; i++){ 
        a[i] = rand() % 101; 
        show(&a[i]);//発生した乱数をshow関数を用いて表示(pをアドレスの形で引数にする) 
    } 
    printf("\n2倍した値:"); 
    for (i = 0; i < 5; i++){ 
        two(&a[i]);//発生した乱数を2倍したものをtwo関数を用いて表示(pをアドレスの形で引数にする) 
    } 
    printf("\n"); 
    getchar(); 

    return 0;
}

void show(int *n){ 
    printf("%d ", *n);//ポインタnが示す数値を表示 
}

void two(int *m){ 
    printf("%d ", (*m) * 2);//ポインタmが示す数値を2倍したものを表示 
}

ただ、質問文を読むと、関数化すべきなのは「配列の値の表示」と「値の二倍」なので、答えはこうじゃないでしょうか。

#include <stdio.h>
#include <time.h>
void show(int*,int); 
void two(int*,int);

int main(void){ 
    int a[5]; 
    int i; 
    srand((unsigned)time(NULL)); 
    for (i = 0; i < sizeof(a)/sizeof(int); i++){ 
        a[i] = rand() % 101; 
    } 

    printf("発生した乱数:"); 
    show(a,sizeof(a)/sizeof(int));
    printf("\n");

    // 2倍
    two(a,sizeof(a)/sizeof(int));

    printf("2倍した値:"); 
    show(a,sizeof(a)/sizeof(int));
    printf("\n");
    getchar(); 

    return 0;
}

void show(int *n, int len){ 
    int i;
    for(i = 0; i < len; i++){
        printf("%d ", *n);//ポインタnが示す数値を表示 
        n++;
    }
}

void two(int *m, int len){ 
    int i;
    for(i = 0; i < len; i++){
        *m = *m * 2;
        m++;
    }
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.82%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る