関数getInputで何が問題になっていますか？

Question

私はprintf体重、ドラッグ、時間が0の場合、値が重量、ドラッグ、時間変数（構造体USER_INPUT内）に格納されていない理由を知りません。 私はプログラムの残りの部分、私は主な機能でいくつかの間違いを犯しましたか？

#include <stdio.h> 

#define G 9.8 
#define index 3 

typedef struct 
{ 
double weight; 
double drag; 
double time; 
}USER_INPUT; 

double velocities[index]; 

double getInput(USER_INPUT); 
double calculateVelocities(USER_INPUT); 

void main(void) 
{ 
double velocity; 
USER_INPUT input; 

getInput(input); 
calculateVelocities(input); 



printf("Velocities for the parachuties with weight %f\n", input.weight); 
printf("and a drag coefficient %f\n", input.drag); 

printf("\n\n Time  Velocities m/s\n"); 
printf("---------------------------------\n"); 
printf("  %f  %f\n", input.time, velocities[0]); 
printf("  %f  %f\n", input.time, velocities[1]); 
printf("  %f  %f\n", input.time, velocities[2]); 




} 

double getInput(USER_INPUT input) 
{ 
printf("Please enter weight, drag and time:\n"); 
scanf("%lf %lf %lf", &input.weight, &input.drag, &input.time); 

printf("%f %f %f\n"), input.weight, input.drag, input.time; 
} 

double calculateVelocities(USER_INPUT input) 
{ 
velocities[0]=1; 
velocities[1]=2; 
velocities[2]=3; 

} 

Answer 1

これにはいくつかの問題があります。

1. これはgetInput下のタイプミスです：

   printf("%f %f %f\n"), input.weight, input.drag, input.time; 
   

   それはこのようになります。voidの代わりdoubleを返す必要がありますあなたの機能getInputとgetVelocities

   printf("%f %f %f\n", input.weight, input.drag, input.time); 
   

2. 。 変更この：この

   void getInput(USER_INPUT); 
   void calculateVelocities(USER_INPUT); 
   

   へ

   double getInput(USER_INPUT); 
   double calculateVelocities(USER_INPUT); 
   

   そして定義に同じことを行います。

3. ポインタに代えて値で関数にUSER_INPUTを渡しています。そのフィールドの値をstructに設定したい場合は、それをポインタとして渡してから関数内で参照解除してください。値渡しとは、オブジェクトを受け取った関数が実際にオブジェクトの内容を新しいものにコピーすることを意味します。したがって、&input.widthを参照すると、main関数のオリジナルのコピーではなく、コピーオブジェクトのwidthフィールドが参照されています。

   たとえば、あなたgetInput機能は次のようになります。

   // declaration 
   void getInput(USER_INPUT*); 
   
   // stuff 
   
   // definition 
   void getInput(USER_INPUT *input) 
   { 
       printf("Please enter weight, drag and time:\n"); 
       scanf("%lf %lf %lf", &((*input).weight), &((*input).drag), &((*input).time)); 
   
       printf("%f %f %f\n", input->weight, input->drag, input->time); 
   } 
   

   過度の括弧が起こっているかについて全く明確にすることがあります。しかし、より簡単に&((*input).weight)を&(input->weight)と書くことができます。

   そして、calculateVelocities機能で同じことをしなければなりません。

   最後に、次のように行い、これらの新しく定義された関数を呼び出すには：

   getInput(&input);