Return type to return all string

Question

0.00/5 (No votes)

See more:

I have this code and it returns the member function as written but return only the last character of emp_name.
any assistance is highly appreciated.

What I have tried:

C++

#include <iostream>

using namespace std;

class emp {
public:

	int getInfo(char emp_name, int emp_id_number, int emp_age) {
		cout << "Employee Name: " << emp_name << "\n";
		cout << "Employee Id Number: " << emp_id_number << "\n";
		cout << "Employee Age: " << emp_age;
	}
};
int main() {
	emp emp1;
	emp1.getInfo('Felix', 635932, 23);
}

Posted 9-Oct-17 23:44pm

Felix Mensah

Updated 10-Oct-17 1:28am

Add a Solution

Comments

nv3 10-Oct-17 5:54am

What makes you think that type char for emp_name could hold an entire character string? It holds just a single character. Open your C++ text book and start reading the first 10 pages. Then try again.

Felix Mensah 10-Oct-17 6:10am

thanks for the adv.

4 solutions

Solution 4

dont use calling by value, try to use calling by reference.
Your function waits a char value not a string.

C++

int getInfo(char* emp_name, int emp_id_number, int emp_age)

Posted 10-Oct-17 1:28am

Member 13454019

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

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

Richard MacCutchan · Accepted Answer · 2017-10-09T23:53:00

Solution 1

C++

int getInfo(char emp_name, int emp_id_number, int emp_age) {

You have declared the emp_name parameter as a single character, rather than a character string. It should be:

C++

int getInfo(char* emp_name, int emp_id_number, int emp_age) { // char*

Posted 9-Oct-17 23:53pm

Richard MacCutchan

Comments

Felix Mensah 10-Oct-17 6:14am

it worked thanks. I had to also enclose Felix in double quotes
#include <iostream>

using namespace std;

class emp {
public:

int getInfo(char* emp_name, int emp_id_number, int emp_age) {
cout << "Employee Name: " << emp_name << "\n";
cout << "Employee Id Number: " << emp_id_number << "\n";
cout << "Employee Age: " << emp_age;
}
};
int main() {
emp emp1;
emp1.getInfo("Felix", 635932, 23);
}

Jochen Arndt · Accepted Answer · 2017-10-10T00:00:00

You probably want to pass a string a to getInfo() but you are actually passing a single character.

Change these two lines:

// Must be char* here
int getInfo(char *emp_name, int emp_id_number, int emp_age)

// Must pass a string here using double quotes
emp1.getInfo("Felix", 635932, 23);

You have defined getInfo() to return an int but you have no return statement.

I suggest to use a high warning level when compiling (-Wall with G++, /W4 in the project settings with VS). Then you would have got at least two warnings for your code.

Felix Mensah · Accepted Answer · 2017-10-10T00:15:00

C++

#include <iostream>

using namespace std;

class emp {
public:

	int getInfo(char* emp_name, int emp_id_number, int emp_age) {
		cout << "Employee Name: " << emp_name << "\n";
		cout << "Employee Id Number: " << emp_id_number << "\n";
		cout << "Employee Age: " << emp_age;
	}
};
int main() {
	emp emp1;
	emp1.getInfo("Felix", 635932, 23);
}

Return type to return all string

4 solutions

Solution 1

Solution 2

Solution 3

Solution 4

Add your solution here

Preview 0