C++一道程序,没有与这些操作数匹配的"[]"运算符

#include <iostream>
#include <string>
using namespace std;
class String
{
public:
String() { p=NULL;}
String(char *str){ p=str; };
friend bool operator>(String &str1,String &str2);
friend bool operator<(String &str1,String &str2);
friend bool operator==(String &str1,String &str2);
String& operator=(const String &s){};
void display();
int getlen()const { return len;}

private:
char *p;
int len;

};
void String::display()
{ cout<<p;}
bool operator>(String &str1,String &str2)
{
if(strcmp(str1.p,str2.p)>0)
return true;
else
return false;
}
bool operator<(String &str1,String &str2)
{
if(strcmp(str1.p,str2.p)<0)
return true;
else
return false;
}
bool operator==(String &str1,String &str2)
{
if(strcmp(str1.p,str2.p)==0)
return true;
else
return false;
}

String& String::operator=(const String &s)
{
if(this==&s)
return *this;
else
{
delete []p;
len=s.getlen();
p=new char[len+1];
for(int i=0;i<len;i++)
{
p[i]=s[i];//这里就说没有与这些操作数匹配的"[]"运算符，可是赋值运算符不是重载了吗为什么还不行？
}
p[len]='\0';
}
return *this;

}

赋值运算符是编译器自动重载的, 但是下标运算符你没有重载.
你这个程序还不止那一处错误:
String& operator=(const String &s){};

这个明显是一个函数声明, 但是你加了一对大括号.

s是String类型，不能用s[i], 你可以用s.SubString(1,1)这样子的 另外你这里可以用　strcpy(p,s)也许是这样strcpy(p,s.c_str())

你没有做<<运算符重载啊！ 类中增加 friend ostream &operator<<(ostream &os,const string &s); 重载函数 在类外进行函数实现 ostream &operator<<(ostream &os,const string &s) { os << s.m_data ; return os; }