'strcpy' was declared deprecated warning

[rahulsk1947]

Hi,
i was writing a blank console aplication in ms. visual c++ 2005 express edition

#include <iostream>
#include <string>
using namespace std;

int main(void){
	char *a="test";
	char *b="east";
	strcpy(a,b);
	cout<<a<<endl;
	cout<<b<<endl;

	int y=0;
	cin>>y;
}

the above was code.
when i compile it,i get''' 'strcpy' was declared deprecated''' warnings. how do i get rid of them? some1 pls spot my error..thanks alot

[vart]

You can disable this specific warning in the IDE settings

[anon]

    char *a="test";

Wouldn't this particular line create a pointer to a read-only string, meaning you can't strcpy to it?

Anyway, if you include <string>, why don't you use std::string? If you really need the C-style char arrays, the proper header would be <cstring> (but <string> usually includes it on its own).

#include <iostream>
#include <string>
using namespace std;

int main(void){
	string a("test");
	string b("east");
	a = b;
	cout<<a<<endl;
	cout<<b<<endl;

	cin.get();
}

[Salem]

A better question is why you're using char arrays in a C++ program to begin with.

You didn't even include the correct header file for strcpy.

Plus, had you tried to run it, it would have crashed in the attempt to modify a string constant. Yet another reason to get used to using std::string for all your C++ string needs.

[Daved]

As has been said already, you should probably be using C++ strings and if you do use C style strings you can't copy into a constant string.

However, the reason that warning comes up is because Microsoft considers strcpy and other "dangerous" functions to be deprecated. That function is not currently deprecated by the C++ standard. The warnings can be annoying, since Microsoft's solution is to use their non-portable but safer alternatives. Just search disable and the warning number for specific instructions, or even better switch to C++ strings instead.