百思不得其解，求助于各位达人！

[hotbee]

#include<iostream>
#include<string>
using namespace std;
class String
{public:
String(){p=NULL;}//定义构造函数
String(char *str){p=str;}//重载构造函数的定义
bool operator>(String &string2);
void display();
private:
char *p;
};

void String::display()
{cout<<p;}

bool String::operator>(String &string2)
{String string1;
if (strcmp(string1.p,string2.p)>0)return true;
else return false;
}

int main()
{String string1="Hello",string2="Bo";
cout<<(string1>string2)<<endl;
}
上面的程序是我写的不用友元函数重载>运算符的程序，编译可以通过无错误提示，但一运行就程序崩溃，无解，茫然！

[freestyIe]

可以使用this指针，调用自身对象进行操作，修改如下：
bool String::operator>(String &string2)
{
//String string1;
if (strcmp(this->p,string2.p)>0)return true;
else return false;
}

[deltamaster]

改正错误，顺便帮整理一下代码。
以后写好看点啊。

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

class String 
{
public: 
	String()
	{
		p = NULL;
	}
	String(char *str)
	{
		p = str;
	}	
	bool operator>(String&); 
	void display(); 
private: 
	char* p; 
}; 

void String::display() 
{
	cout << p;
}

bool String::operator>(String& string) 
{
	//String string1; 运算符左边的表达式是对象本身
    if (strcmp(this->p, string.p) > 0) return true; 
	else return false; 
} 

int main() 
{
	String string1="Hello", string2="Bo"; 
	cout << (string1 > string2) << endl; 
	return 0;
} 

[deltamaster]

另外还要注意下构造函数的写法，现在这样的写法不是很好。
一般这样的类需要自定义复制构造函数和赋值运算符函数，否则也会造成问题。

[goodname]

bool String::operator>(String &string2)
{
if (strcmp(this->p,string2.p)>0)return true;
else return false;
}

[goodname]

你写的主要是string1使用的是局部变量，所以string1.p是空指针
而strcmp使用空指针则崩溃了。

[猫已经找不回了]

bool String::operator>(String &string2)
{String string1;
if (strcmp(string1.p,string2.p)>0)return true;
else return false;
}

楼主，你最好在构造函数里new一下，在析构里再delete掉。

string1.p现在是空指针，没有分配内存。

[hjjdebug]

bool String::operator>(String &string2)
{String string1(p);
if (strcmp(string1.p,string2.p)>0)return true;
else return false;
}
见红字，添加(p)

[Sou2012]

学习了