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

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

ただいまの
回答率

90.34%

  • C

    4005questions

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

【C言語初心者】コード修正していただきたいです

受付中

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 333
退会済みユーザー

退会済みユーザー

下記コードのどこが間違っているのか教えていただきたいです。

#include<stdio.h>
#include<string.h>
typedef struct
{
 char name[64];
 int age;
 int gender;
}people;

int input(int);
int output(people,int);
int main(void)
{
 people data[3];
 int i;
 for(i = 1;i<=3;i++)
 {
  int input(i);
 }
 for(i=1;i<=3;i++)
 {
  int output(data,i);
 }
 return 0;
}

int input(int i)
{
 ptintf("名前を入力してください。\n");
 scanf("%s",data[i]->name);
 printf("年齢を入力してください。\n");
 scanf("%d",data[i]->age);
 printf("男性なら0、女性なら1を入力してください。\n");
 scanf("%d",data[i]->gender);
 return 0;
}

int output(people data,int i)
{
 printf("あなたの名前は%sです\n",data[i].name);
 printf("あなたは%d歳です\n",data[i].age);
 for(; ;)
 {
  if(data[i].gender==0||1) break;
  printf("男性なら0、女性なら1を入力してください\n");
  scanf("%d",data->gender);
 }
 if(data[i].gender==0)
 printf("あなたは男性です\n");
 if(data[i].gender==1)
 printf("あなたは女性です\n");
 return 0;
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • yohhoy

    2017/09/23 19:50

    質問中に「あなたの期待する動作」と「(期待に反する)実際の動き」を説明ください。

    キャンセル

  • LouiS0616

    2017/09/23 19:50

    どのように上手くいかないのか追記してください。コピペして実行してみる手間が生じますし、質問者様の環境に依存する問題の恐れもあります。

    キャンセル

  • 退会済みユーザー

    2017/09/24 01:15

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

回答 3

+1

input関数は変数dataを参照できません。
引数で渡す場合、意図した動作を行うには変数の渡し方を気をつける必要があるかと思います。

上記の他、関数呼び出し、配列の添字範囲、論理和判定等の不備が見つかります。

一度に全部を実装しようとせず、まずは最小限の動くコードを作ってから機能を実装してみてはいかがでしょうか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

+1

いろいろおかしいです。

  • inputの中でdataを参照している
    dataのスコープはmain関数内なので、input関数からは参照できないです。
    用途を考えると、ポインタ渡しをするべき場面かと思います。

  • ポインタ変数じゃないのにアロー演算子を使っている
    output関数に、dataを値渡ししていますよね。ドット演算子を使いましょう。

  • output関数の呼び出し方がおかしい
    引数の型はpeopleなのに、実際にはpeople *を渡しています。
    配列の受け渡し方を考え直した方がいいです。


もうちょっと初歩的な部分から勉強した方がいいです。
ポインタに関する基本的な知識が不足しているように思えます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

とりあえず、下記の3点確認してください。

・構造体の参照
・ポインタの使い方
・if条件

もう少し基礎を勉強するべきです。
また、
「期待している動作」や
提示しているコードをコンパイルした結果や実行した結果なども
記載することにより、あらためて気づくこともあります。

以上

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

  • C

    4005questions

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