request for member "..." in "..." which is of non-class type "..." [duplicate]

Question

I've been hunting around for hours trying to figure out what I'm doing wrong, and I've gotten rid of most of my problems but when I try to compile my code in main() it comes up with this same error message:

request for member "..." in "..." which is of non-class type "..."

and it repeats itself for all of the functions I try to call in main(). What is the problem? I can't figure out where my error is.

I'm using Terminal on a macbook to compile the code.

Here is my main function:

//Program1.cpp
//Program1Math test function
#include "Program1Math.h"

int main()
{
  //Create a Program1Math object
  Program1Math myProgram1Math();

  myProgram1Math.setNumber1();
  myProgram1Math.setNumber2();

  myProgram1Math.displayMultiple();
  myProgram1Math.displaySine1();
  myProgram1Math.displayTangent1();
  myProgram1Math.displaySine2();
  myProgram1Math.displayTangent2();
}

Here are the member-function definitions for the class:

//Program1Math.cpp
//Program1Math member-function definitions.
#include <iostream>
#include <cmath>
#include "Program1Math.h"
using namespace std;

//constructor makes a Program1Math, adds an blank line
Program1Math::Program1Math()
{
  cout << "/n";
}

//function to assign the first integer to its appropriate location
void Program1Math::setNumber1()
{
  cout << "Please enter the first integer number /n";
  int numberSpot;
  cin >>numberSpot;
  static_cast<double>(numberSpot);
  number1 = numberSpot;
}

//function to assign the second integer to its appropriate location
void Program1Math::setNumber2()
{
  cout << "Please enter the second integer number /n";
  int numberSpot;
  cin >>numberSpot;
  static_cast<double>(numberSpot);
  number2 = numberSpot;
}

//function to find the sine value for a specified number
void Program1Math::calculateSine( double inputNumber )
{
  sineValue = sin( inputNumber ); 
}

//function to find the tangent value for a specified number
void Program1Math::calculateTangent( double inputNumber )
{
  tangentValue = tan( inputNumber );
}

//function to determine if the user-inputted numbers are multiples of each other
void Program1Math::calculateModulus()
{
  int number1Int = static_cast<int>(number1);
  int number2Int = static_cast<int>(number2);
  int modulusValue = number1Int % number2Int;
  if ( modulusValue == 0 )
    multiple = true;
  else
    multiple = false;
}

//function to display the whether the numbers are multiples or not
void Program1Math::displayMultiple()
{
  if( multiple == true )
    cout<< number1 << " is a multiple of " << number2 << "!/n";
  else
    cout<< number1 << "is not a multiple of " << number2 << "./n";
}

//function to display the sine value of the first number
void Program1Math::displaySine1()
{
  calculateSine( number1 );
  cout << "Sine(" << number1 << ") = " << sineValue << "/n";
}

//function to display the sine value of the second number
void Program1Math::displaySine2()
{
  calculateSine( number2 );
  cout << "Sine(" << number2 << ") = " << sineValue << "/n";
}

//function to display the tangent value of the first number
void Program1Math::displayTangent1()
{
  calculateTangent( number1 );
  cout << "Tan(" << number1 << ") = " << tangentValue << "/n";
}

//function to display the tangent value of the second number
void Program1Math::displayTangent2()
{
  calculateTangent( number2 );
  cout << "Tan(" << number2 << ") = " << tangentValue << "/n";
}

here is the header file:

#include <cmath>
using namespace std;

class Program1Math
{
 public:
  Program1Math();
  void setNumber1();
  void setNumber2();
  void calculateSine( double );
  void calculateTangent( double );
  void calculateModulus();
  void displayMultiple();
  void displaySine1();
  void displaySine2();
  void displayTangent1();
  void displayTangent2();
 private:
  double number1;
  double number2;
  double sineValue;
  double tangentValue;
  bool multiple;
};

Answer 1

the line Program1Math myProgram1Math(); is interpreted as a function declaration myProgram1Math() returning Program1Math;

Just use

 Program1Math myProgram1Math;

Only use () when a constructor accepts an parameter (without a default argument).

EDIT : you must compile all the source (.cpp) files that comprise your program. That will produce object files of the same name with a different extension (on windows this is .obj ,for you it's .o).

After that all those .o-files must be linked togetcher with some compiler-provided libraries to make your executable program.

Answer 2

You should create the Program1Math instance like this:

Program1Math myProgram1Math;

or to allocate the object on the heap, use the new keyword:

Program1Math *myProgram1Math = new Program1Math();