Help! error C2082: redefinition of formal parameter 'mang'

Question

0.00/5 (No votes)

See more:

C++

#include"iostream"
using namespace std;
typedef char* ptr;

void nhapchuoi(ptr &mang, int &n)
{
	cout<<"\nNhap so ky tu: ";
	cin>>n;
	ptr mang = new char[n];        //line error!!!!!!!!!!!!
	for(int i=0;i<n;i++)>
	{
		cout<<"\nNhap ky tu thu "<<i<<" : ";
		cin>>mang[i];
	}
}

void xuatchuoi(ptr mang, int n)
{
	cout<<"\nTa co: ";
	for(int i=0;i<n;i++)>
	{
		cout<<" "<<mang[i];
	}
	cout<<endl;
}

void chen(ptr mang, int n, char kytu)
{
	if(n==0)
	{
		n++;
		ptr mang2 = new char [n];
		mang[0] = kytu;
	}
	else
	{
		n++;
		ptr mang2 = new char[n];
		for(int i=0;i< n-1;i++)
			mang2[i]=mang[i];
		mang2[n-1]=kytu;
		delete[]mang;
		mang=mang2;
		
	}
}

void xetnguyenam(ptr mang, int n, ptr &mang2, int &n2)
{
	cout<<"\nCac nguyen am: ";
	for(int i=0;i<n;i++)>
	{
		if(mang[i]=='a' || mang[i]=='e' || mang[i]=='i' || mang[i]=='o' || mang[i]=='u')
		{
			cout<<" "<<mang[i];
			//chen(mang2,n2,mang[i]);
		}
	}
	  
}

void main()
{
	ptr mang1, mang2;
	int n1,n2;
	nhapchuoi(mang1,n1);
	xuatchuoi(mang1,n1);
	xetnguyenam(mang1,n1,mang2,n2);
	//cout<<"\nCac ky tu nguyen am: ";
	//xuatchuoi(mang2,n2);
	system("pause");
}

Posted 25-Nov-14 22:50pm

Member 11265217

Updated 25-Nov-14 22:55pm

v3

Add a Solution

Comments

Aescleal 26-Nov-14 5:24am

As well as the answer below look at using std::string or std::vector instead of raw character pointers. They're a lot easier to get right at the cost of not a lot of performance.

Additionally remember that main is int main() - void main() is only supported on non-hosted systems. Also get into the habit of initialising all your variables when you declare them, it's bound to save you some grief at some point in your learning or subsequent career.

1 solution

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)

Kornfeld Eliyahu Peter · Answer 1 · 2014-11-25T23:07:00

mang is one of the parameters of your function, so you have a variable by that name. You can not declare a second variable with that name...
First declaration:

C++

void nhapchuoi(ptr &mang, int &n)

Second declaration:

C++

ptr mang = new char[n];

If you mean to assign value to that variable than remove the type definition form the beginning of the line:

C++

ptr mang = new char[n];