Thread: Simple function programme

Im writing a function that asks someone to put in a range from one to five and the function is to be passed through a string. I;d likke the function to return the input value if in range, or minus one if out of range. to do this, i'd like it to do something like this...

Code:

 value=getval("Please enter a value in the range 1 to 5")

Code:

 
#include <iostream>
using namespace std;

void getval ()
{
  char string[1];
  
  cout << "please enter a value between 1-5: ";
  cin >> string;
  cout << endl <<string <<endl;
}

int main ()
{
  getval ();
  
  system("pause");
}

Can anyone help?

Code:

void getval ()

Not really sure what it is that you are trying to achieve, but you need to change the line above to both take and return an argument if you intend to use it like

Code:

value=getval("Please enter a value in the range 1 to 5")

Code:

int getval ()
{
  char string[1];
  
  cout << "please enter a value between 1-5: ";
  cin >> string;
  cout << endl <<string <<endl;

  if(string<1 || string >5)
  return -1;
  else
  return string;

}

>> char string[1];
That is a one character character array. When you read into it from cin, cin assumes it is a C style string. C style strings have a null character that tells you where they end, so you have to leave room for that. That means that your string doesn't have any room for any real characters.

If you want to read in a single character, use a char variable, not a char array. If you want to read in a string, use the C++ string class. If you want to read in a number between 1 and 5, you might want just use an int, which is meant for integer numbers.

yeah sort of but we have to use the

Code:

 Value=Getval("Please enter number");

and input is a string.

hey thanx i got it working in the end. this is the code i used

Code:

#include <iostream>
using namespace std;

int GetVal(char prompt[256]);

void main()
{
int Value;
Value=GetVal("Please Enter a number in the range of 1-5");

system("pause");
}
int GetVal(char information[256])
{
   int value;

   cout<<information<<"\n";
   cin>>value;

   if (value < 1 || value > 5)
   {
      return -1;
   }
   else
   {
      return value;
   }
}

now i have managed to get they layout of the function i have another small problem.

i need to change the function that i have got and put in Lower, Upper and Outstr in as perameters, once that bit is done i then need to output the outstr (which would be the text you send to the function) and lower and upper are the range
e.g. lower = 1, upper = 5 so the number has to be within 1 and 5.

At the moment i have (i think) got the Outsrt working but am unsure about the rest.

this is what i have got soo far

Code:

#include <iostream>
#include <string>

using namespace std;

int GetVal(string);

int main()
{
int Value=GetVal("Please Enter a number in the range of 1-5");

system("pause");
return 0;
}

int GetVal(string Outstr)
{
   int value;

   cout << Outstr <<endl;
   cin>>value;

   if (value < 1 || value > 3)
   {
   cout << "Lower";
   return -1;
   }
   
   else if (value <3 || value >5)
   {
   cout << "upper";
   return -1;
   }
   
   
   else if (value >5)
   {
   return 0;
   }

   else   
      return value;
}

if anyone can help it would be great.

Step one is to add Lower and Upper as arguments to your function and make sure to pass a value for lower and upper when you call the function from inside main. You don't have to use the alues yet, just get the parameter passing compiling first.