我是C的新手.我试图使一个char指针作为函数的out参数.但功能的变化并没有体现在主要功能中.我究竟做错了什么?
void SetName( char *pszStr ) { char* pTemp = new char[10]; strcpy(pTemp,"Mark"); pszStr = pTemp; } int _tmain(int argc,_TCHAR* argv[]) { char* pszName = NULL; SetName( pszName ); cout<<"Name - "<<*pszName<<endl; delete pszName; return 0; }
解决方法
你的指针被复制到堆栈上,你正在分配堆栈指针.如果要更改指针,则需要传递指针到指针:
void SetName( char **pszStr ) { char* pTemp = new char[10]; strcpy(pTemp,"Mark"); *pszStr = pTemp; // assign the address of the pointer to this char pointer } int _tmain(int argc,_TCHAR* argv[]) { char* pszName = NULL; SetName( &pszName ); // pass the address of this pointer so it can change cout<<"Name - "<<*pszName<<endl; delete pszName; return 0; }
这将解决你的问题.
但是,这里还有其他问题.首先,在打印之前,您将取消引用指针.这是不正确的,你的指针是一个指向字符数组的指针,所以你想打印整个数组:
cout<<"Name - "<<pszName<<endl;
你现在只打印第一个字符. Also,你需要使用delete []删除一个数组:
delete [] pszName;
更大的问题在于您的设计.
那个代码是C,而不是C,即使这样也不是标准的.首先,您要查找的功能主要是:
int main( int argc,char * argv[] )
其次,您应该使用references而不是指针:
void SetName(char *& pszStr ) { char* pTemp = new char[10]; strcpy(pTemp,"Mark"); pszStr = pTemp; // this works because pxzStr *is* the pointer in main } int main( int argc,char * argv[] ) { char* pszName = NULL; SetName( pszName ); // pass the pointer into the function,using a reference cout<<"Name - "<<pszName<<endl; delete pszName; return 0; }
除此之外,如果可以的话,通常情况下只需要返回:
char *SetName(void) { char* pTemp = new char[10]; strcpy(pTemp,"Mark"); return pTemp; } int main( int argc,char * argv[] ) { char* pszName = NULL; pszName = SetName(); // assign the pointer cout<<"Name - "<<pszName<<endl; delete pszName; return 0; }
有一些东西使这更好. C有一个string class:
std::string SetName(void) { return "Mark"; } int main( int argc,char * argv[] ) { std::string name; name = SetName(); // assign the pointer cout<<"Name - "<< name<<endl; // no need to manually delete return 0; }
如果这样做可以简化,如果你愿意的话:
#include <iostream> #include <string> std::string get_name(void) { return "Mark"; } int main(void) { std::cout << "Name - " << get_name() << std::endl; }
您应该处理格式化,使事情变得更加可读.操作符之间的空间有助于:
cout<<"Name - "<<pszName<<endl; cout << "Name - " << pszName << endl;
就像英文单词之间的空格有所帮助,在你的间隙之间. 原文链接:https://www.f2er.com/c/114073.html