Click here to Skip to main content
Search within:



Next

I need a suggestion for the the line that starts with "ratio"... it's not compiling
Sign Up to vote bad
good
See more: C++
The following program line that starts with "ratio" is supposed to = 
5√M/m
where M is maxSpeedRpm and m minSpeedRpm. Please help, I am going cross eyed trying to work this out.
 

 

 

#include <iostream>
#include <cmath>
using namespace std;
int
main0()
{
    int maxSpeedRpm, minSpeedRpm;
    double ratio;
    
    cout << "Enter the maximum speed in revolutions per minute";
    cin >> maxSpeedRpm;
    cout << "Enter the minimum speed in revolutions per minute";
    cin >> minSpeedRpm;
    
    ratio = sqrt((pow(maxSpeedRpm/minSpeedRpm),(1/5)));
 
    cout << "The calculated ratio is" << ratio;
       
    cin.ignore();
    cin.get();
    return 0;
}
Posted 21 Feb '13 - 19:26
Sarah Trattner321
Add a Solution

Comments
Sergey Alexandrovich Kryukov - 22 Feb '13 - 1:35
Not a question. Gibberish. Very bad. You did not even report the error message. —SA
Sarah Trattner - 22 Feb '13 - 1:38
whatever...I'll wait for someone that can identify the problem.
Sergey Alexandrovich Kryukov - 22 Feb '13 - 1:42
"Can identify"... Amazing... Declare all variables you use... —SA
Sarah Trattner - 22 Feb '13 - 1:38
15 C:\Dev-Cpp\ratio.cpp no matching function for call to `pow(int)'
Sergey Alexandrovich Kryukov - 22 Feb '13 - 1:46
Do you at least understand what's power?!! It's bx, where b is the base, x is exponent. So, there should be two parameters, not one! See http://www.cplusplus.com/reference/cmath/pow/ —SA
enhzflep - 22 Feb '13 - 8:14
Additionally, something you may not realize, is that in C/C++, an int divided by an int gives an int. So, 3/10 = 0. If you want to get 0.3, you'll need to declare them as floats or doubles. Also, Code::Blocks is a fairy lightweight IDE that uses MinGW (like DevCPP does) It's much better AND currently maintained. It's well worth the download. :) P.s - Look at Sergey's rep - it's over 10,000 this month! He's probably forgotten more this month than you and I combined have learned in the past year. And you know, technically, he was right insofar as there isn't a question in your initial post. ;) Welcome to the madhouse! :)

2 solutions

If you intersperse some blanks you can see for yourself why it doesn't compile:
ratio = sqrt( (pow(maxSpeedRpm/minSpeedRpm), (1/5)) );
So you got the parenthese wrong. You probably meant to write:
ratio = sqrt (pow ((maxSpeedRpm/minSpeedRpm), 0.2));
  Permalink  
Posted 22 Feb '13 - 0:58
nv315.6K
Comments
Mike M.00 - 22 Feb '13 - 12:05
Yes, white space is your friend - computers don't care, but brains see the structure better. HINT to the unseasoned or occasional programmer: Pay close attention to at least the first error message generated by the compiler - make sure you know what it means as it will lead you in the direction of your solution. In this case "no matching function for call to 'pow(int)'" was trying to tell you the compiler could not find a function called 'pow' which took one integer parameter (see hemantrautela's Solution 1). Another trick I like to use to debug complicated math expressions, is to define a series of temporaries and build up the final expression one step at a time. This allows me to see all the intermediary values and it ends up isolating any error messages to exactly and only the offending item, e.g.: int t1 = maxSpeedRpm/minSpeedRpm; // Here you would see t1 is zero for some data like 5/7, so this needs to be float. float t2 = pow ( t1, 0.2 ); // And actually pow() needs a double as first argument, but comilers convert it for you. float t3 = sqrt ( t2 ); A debugger allows you to see each value and understand what's working, what's not. Mike M.
nv3 - 22 Feb '13 - 12:35
Very good points, Mike. I hope the original poster will heed your advice!
pow() function having two arguments...as below
c++ , cmath
double pow (      double base,      double exponent );
long double pow ( long double base, long double exponent );
      float pow (       float base,       float exponent );
     double pow (      double base,         int exponent );
long double pow ( long double base,         int exponent );
 

Thanks
Asp.Net C# Help Blog[^]
  Permalink  
Posted 21 Feb '13 - 19:48
hemantrautela1.8K
Add a Solution

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

  Print Answers RSS
Your Filters
Interested
Ignored
     
0 OriginalGriff 206
1 Maciej Los 146
2 Richard MacCutchan 145
3 Tadit Dash 140
4 Sergey Alexandrovich Kryukov 135
0 Sergey Alexandrovich Kryukov 10,264
1 OriginalGriff 7,957
2 CPallini 4,201
3 Rohan Leuva 3,522
4 Maciej Los 3,155


Advertise | Privacy | Mobile
Web03 | 2.6.130523.1 | Last Updated 22 Feb 2013
Copyright © CodeProject, 1999-2013
All Rights Reserved. Terms of Use
Layout: fixed | fluid