2017-02-11 4 views
-2

私は単純な等級計算機で作業していましたが、 "labs"オプションで問題が発生しましたが、10進数を探していますが、非常に非常に大きな指数です。ループ変数を別の変数でインクリメントするためのC++ ++

私は問題を抱えている特定の部分は、正確にはforループであるcalclabavg()関数です。

私は正確な解決策を求めていませんが、私は自分自身で問題を解決できるように正しい方向を指しておきたいと思います。事前に

本当にありがとうございました:)を

float calclabavg(){//funciton that calculates user lab averages 

    float x, vary, pointspos, sumpointspos, sumearned; 

    cout << "How many labs in labs?" << endl; 
    cin >> x; 

    float totalpoints = 0; 

    cout << "Do the points vary per lab? (Press 1 for yes, 0 for no)" << endl; 
    cin >> vary; 

    if (vary == 0){ 
     cout << "How many points were possible on each lab?" << endl; 
     cin >> pointspos; 
     sumpointspos = x * pointspos; 
    } 

    for (int i = 0; i < x; i++){ 
     float temp; 
     cout << "What was your score on lab " << i + 1 << endl; 
     cin >> temp; 
     sumearned += temp; 
     if (vary == 1){ 
      cout << "How mant points possible on lab " << i + 1 << endl; 
      float pointsposvary; 
      cin >> pointsposvary; 
      sumpointspos += pointsposvary; 
     } 
     //pointspos = pointspos + temp; 
    } 

    return sumearned/sumpointspos; 
} 
+1

"私はちょうど正しい方向を指し示したい。" [OK]をクリックすると、正しい方向にあなたを設定するリンクが表示されます(http://stackoverflow.com/questions/388242/the-definitive-c-book-guide-and-list)。 HTH。 –

+0

おすすめの本を購入しました。 –

答えて

1

あなたがsumearnedを初期化していないかのように思えます。 calclabavgの冒頭にあるsumearned = 0.0;のステートメントでは、少なくともさらに進んでください。 BTW:他の変数も初期化することは良い習慣であり、コードはより安定しています。

+0

ありがとう!それは大いに役立ちます。私は、バリエーションの部分にまだ固執して、動作するバリエーションの部分を得ることができました。良い進歩を作る –

+0

私の他の変数の初期化は私の問題を解決しました。本当にありがとう。 –

関連する問題