Functions, What is Wrong?

[galaxy1500]

#include <stdafx.h>
#include <iostream>


using namespace std;
double calcLatefee(double, double);

int main ()
{
double overDue,
creditAmount,
surFee ;


cout << " Amount? ";
cin >> creditAmount;


cout << "Days past due? ";
cin >> overDue ;

lateFee = calcsurfee(double, double) ;



return 0 ;
}




double calcsurfee(double, double)

{if ( overDue >= 60 )
{
if ( creditAmount >= 10 || creditAmount < 50)
{
return 20 ;

}
else if ( creditAmount >= 50 || creditAmount < 100)
{
return 15 ;

}
else if ( creditAmount >= 100 || creditAmount < 250)
{
return 30 ;

}
else
{
return 70 ;

}
}

else if ( overDue >= 30)
{
if ( creditAmount >= 10 || creditAmount < 50)
{
return 10 ;

}
else if ( creditAmount >= 50 || creditAmount < 100 )
{
return 70 ;

}
else if ( creditAmount >= 100 || creditAmount < 250)
{
return 15 ;

}
else
{
return 35 ;

}

}


cout << "\nA Credit amount of " << creditAmount << " is " << overDue << " late" << "\n";
cout << "\nA fee is: " << lateFee << " dollars " "\n" ;


return 0 ;



}

[kbw]

http://www.cplusplus.com/forum/articles/1295/

[MaikCAE]

Here are some mistakes fixed which I found at the first watch.
Next time use the source-tags to quote code and also post a discription the errors which occur.
And maybe yes, read the post which kbw linked before ;)

int main ()
{
  double overDue, creditAmount, surFee ;

  cout << " Amount? ";
  cin >> creditAmount;

  cout << "Days past due? ";
  cin >> overDue ;

  // call function not with "double", use the variable name
  // lateFee is not defined, you mean surFee?
  surFee = calcsurfee(creditAmount, overDue) ;
  
  // I think you have to make this output here, because your
  // function don't know something about surFee/lateFee  
  cout << "\nA Credit amount of " << creditAmount << " is " << overDue << " late" << "\n";
  cout << "\nA fee is: " << surFee << " dollars " << "\n" ;
  
  return 0 ;
}

// You have to give names to input variables of a function
double calcsurfee(double creditAmount, double overDue)
{
  if ( overDue >= 60 )
  {
    /* If you want a value beetween 2 numbers take && instead of ||
     *  for example 100 in if ( creditAmount >= 10 || creditAmount < 50)
     *  would bring the result (true || false) which is true
     *  Taking &&: (true && false) is false
     */ 
    if ( creditAmount >= 10 && creditAmount < 50)
    {
      return 20 ;
	}
    else if ( creditAmount >= 50 && creditAmount < 100)
    {
      return 15 ;
    }
    else if ( creditAmount >= 100 && creditAmount < 250)
    {
      return 30 ;
    }
    else
    {
      return 70 ;
    }
  }
  else if ( overDue >= 30)
  {
    if ( creditAmount >= 10 && creditAmount < 50)
    {
      return 10 ;
    }
    else if ( creditAmount >= 50 && creditAmount < 100 )
    {
      return 70 ;
    }
    else if ( creditAmount >= 100 && creditAmount < 250)
    {
      return 15 ;
    }
    else
    {
      return 35 ;
    }
  }
  return 0 ;
}