Cứu bài MyString khi dùng toán tử

#include "iostream"
using namespace std;
#include "MyString.h"
#include "string.h"

// Hàm hủy
String::~String()
{    delete s;
}

// Hàm thiết lập 1 tham số
String::String(char *s1)
{    length = strlen(s1);
    s = new char[length + 1];
    strcpy(s, s1);
}

// Hàm thiết lập sao chép
String::String(String &s1)
{    length = s1.length;
    s = new char[length + 1];
    strcpy(s, s1.s);
}

// Nạp chồng toán tử
// Toán tử gán
String & String::operator =(String &s1)
{    if(this != &s1)
    {    delete s;
        length = s1.length;
        s = new char[length + 1];
        strcpy(s, s1.s);
    }
    else
        cout<<" Hai chuoi la 1, nen khong sao chep"<<endl;
    return *this;
}

// Toán tử Nhập - hàm Nhập
istream & operator >> (istream &d, String &s1)
{    cout<<"Nhap chuoi: ";
    d>>s1.s;
    return d;
}

// Toán tử Xuất
ostream & operator << (ostream &h, String &s1)
{    h<<"Hien thi chuoi: "<<endl;
    h<<s1.s<<endl;
    return h;
}

//MyString.h

#pragma once
#include "string.h"

class String
{
    private:
        char *s;
        int length; // các thuộc tính
    public:
        String(char *s1 = ""); // Hàm thiết lập 1 tham số
        String (String &s1); // hàm thiết lập sao chép
        ~String();             // Hàm hủy
        String & operator  = (String &s1); // Gán chuỗi A = B
        friend ostream & operator <<(ostream &, String &s);
        friend istream & operator >> (istream &, string &s);
};