Доработать класс String

guess

Занимаюсь по книжке Стивена Праты, никак не могу найти ошибку в программе(написана на Visual C++).Помогите пожалуйста найти что в ней не так
#ifndef CLSTRING_H_
#define CLSTRING_H_
class String
{
private:
    char *str;
    int len;
    static int num_strings;
    static const int CINLIM = 80;
public:
    String(const char *s);
    String();
    String(const String &st);
    ~String();
    int length () const {return len;}
    String & operator=(const String &st);
    String & operator=(const char *s);
    char & operator[] (int i);
    const char & operator[] (int i) const;
    int has(char let);
    void stringlow();
    void stringup();
    friend void operator+(const String &st1, const String &st2);
    friend bool operator<(const String &st1, const String &st2);
    friend bool operator>(const String &st1, const String &st2);
    friend bool operator==(const String &st1, const String &st2);
    friend ostream & operator<<(ostream &os, const String &st);
    friend istream & operator>>(istream &is, String &st);
    static int howMany();
};
#endif
#include <iostream>
#include <cstring>
#include <cctype>
using namespace std;
#include "clString.h"
int String::num_strings = 0;
void String::stringlow()
{
    tolower(*str);
}
void String::stringup()
{
    toupper(*str);
}
int String::howMany()
{
    return num_strings;
}
String::String(const char *s)
{
    len=strlen(s);
    str=new char [len+1];
    strcpy(str,s);
    num_strings++;
}
String::String()
{
    len=4;
    str=new char[1];
    str[0]='\0';
    num_strings++;
}
String::String(const String &st)
{
    num_strings++;
    len=st.len;
    str=new char[len+1];
    strcpy(str,st.str);
}
String::~String()
{
    --num_strings;
    delete [] str;
}
String & String::operator=(const String &st)
{
    if (this==&st)
        return *this;
    delete [] str;
    len = st.len;
    str=new char[len+1];
    strcpy(str,st.str);
    return *this;
}
String & String::operator=(const char *s)
{
    delete [] str;
    len=strlen(s);
    str=new char[len+1];
    strcpy(str,s);
    return *this;
}
void operator+(const char *st1, const String &st2)
{
    strcat(*st1,*st2);
}
int String::has(char let)
{
    int kol=0;
    len=strlen(str);
    for (int i=0;i<len;i++)
        if (str[i]==let)
            kol++;
    return kol;
}
char & String::operator[] (int i)
{
    return str[i];
}
const char & String::operator[] (int i) const
{
    return str[i];
}
bool operator<(const String &st1, const String &st2)
{
    return (strcmp(st1.str, st2.str)<0);
}
bool operator>(const String &st1, const String &st2)
{
    return st2.str<st1.str;
}
bool operator==(const String &st1, const String &st2)
{
    return (strcmp(st1.str, st2.str)==0);
}
ostream & operator<<(ostream &os, const String &st)
{
    os<<st.str;
    return os;
}
istream & operator>>(istream &is, String &st)
{
    char temp[String::CINLIM];
    is.get(temp,String::CINLIM);
    if (is)
        st=temp;
    while (is && is.get() !='\n')
        continue;
    return is;
}
 
#include <iostream>
using namespace std;
#include "clString.h"
int main()
{
    String s1(" and I am a C++ student.");
    String s2="Please enter your name: ";
    String s3;
    cout<<s2;
    cin>>s3;
    s2="My name is " +s3;
    cout <<s2<<".\n";
    s2=s2 + s1;
    s2.stringup();
    cout<<"The string\n"<<s2<<"\ncontains "<<s2.has('A')
        <<" 'A' characters in it.\n";
    s1="red";
    String rgb[3]={
        String (s1),String("green"),String("blue")};
        cout<<"Enter the name of a primary color for mixing light: ";
        String ans;
        bool success=false;
        while(cin>>ans)
        {
            ans.stringlow();
            for (int i=0;i<3;i++)
            {
                if(ans==rgb[i])
                {
                    cout<<"That's right!\n";
                    success=true;
                    break;
                }
            }
            if (success)
                break;
            else
                cout<<"Try again!\n";
        }
        cout<<"Bye\n";
        cin.get();
        cin.get();
        return 0;
}
3 ответа

guess

guess, в чем конкретно проблема, что работает не так?


guess

При компиляции выдает ошибки: Error C2100: illegal indirection Error C2664: 'strcat' : cannot convert parameter 1 from 'const char' to 'char *' Error C2679: binary '=' : no operator found which takes A right-hand operand of type 'void' (or there is no acceptable conversion) Could be 'string &String::operator = (const String &)' Or 'String &String ::operator = (const char *)' While trying to match the argument list '(String, void)'


guess

Елки палки... Без 100 грамм не разберешься... Во первых strcat(st1+st2) неверно априори, ибо strcat работает только с char*. Ты же пытаешься присоединить два объекта... st1.str+st2.str это верно. Но это еще не все... Остальное ща попробую разобрать...Летит она где-то с памятью... Оператор сложения реализован полностью неправильно, длина строки остается такой же какой подавал похоже.Не могу найти ошибки. Фиг знает где они. Летит все либо на + либо на return из той функции
//clString.h
#ifndef CLSTRING_H_
#define CLSTRING_H_
class String
{
private:
        char *str;
        int len;
        static int num_strings;
        static const int CINLIM = 80;
public:
        String(const char *s);
        String();
        String(const String &st);
        ~String();
        int length () const {return len;}
        String & operator=(const String &st);
        String & operator=(const char *s);
        char & operator[] (int i);
        const char & operator[] (int i) const;
        int has(char let);
        void stringlow();
        void stringup();
        friend String operator+(String &st1, String &st2);
        friend bool operator<(const String &st1, const String &st2);
        friend bool operator>(const String &st1, const String &st2);
        friend bool operator==(const String &st1, const String &st2);
        friend ostream & operator<<(ostream &os, const String &st);
        friend istream & operator>>(istream &is, String &st);
        static int howMany();
};
#endif
//clString.cpp
#include <iostream>
#include <cstring>
#include <cctype>
using namespace std;
#include "clString.h"
int String::num_strings = 0;
void String::stringlow()
{
        tolower(*str);
}
void String::stringup()
{
        toupper(*str);
}
int String::howMany()
{
        return num_strings;
}
String::String(const char *s)
{
        len=strlen(s)+1;
        str=new char [len+1];
        strcpy(str,s);
        num_strings++;
}
String::String()
{
        len=0;
        str=new char[1];
        str[0]='\0';
        num_strings++;
}
String::String(const String &st)
{
        num_strings++;
        len=st.len;
        str=new char[len+1];
        strcpy(str,st.str);
}
String::~String()
{
        --num_strings;
        delete [] str;
}
String & String::operator=(const String &st)
{
        if (this==&st)
                return *this;
        delete [] str;
        len = st.len;
        str=new char[len+1];
        strcpy(str,st.str);
        return *this;
}
String & String::operator=(const char *s)
{
        delete [] str;
        len=strlen(s);
        str=new char[len+1];
        strcpy(str,s);
        return *this;
}
String operator+(String&st1, String&st2)
{
    String st3;
    st3.str=strcat(st1.str,st2.str);
    st3.len=st1.len+st2.len;
    //st3.num_strings=st1.num_strings+st2.num_strings;
    return st3;
}
int String::has(char let)
{
        int kol=0;
        len=strlen(str);
        for (int i=0;i<len;i++)
                if (str[i]==let)
                        kol++;
        return kol;
}
char & String::operator[] (int i)
{
        return str[i];
}
const char & String::operator[] (int i) const
{
        return str[i];
}
bool operator<(const String &st1, const String &st2)
{
        return (strcmp(st1.str, st2.str)<0);
}
bool operator>(const String &st1, const String &st2)
{
        return (strcmp(st1.str,st2.str)>0);
}
bool operator==(const String &st1, const String &st2)
{
        return (strcmp(st1.str, st2.str)==0);
}
ostream & operator<<(ostream &os, const String &st)
{
        os<<st.str;
        return os;
}
istream & operator>>(istream &is, String &st)
{
        char temp[String::CINLIM];
        is.get(temp,String::CINLIM);
        if (is)
                st=temp;
        while (is && is.get() !='\n')
                continue;
        return is;
}
//main.cpp
#include <iostream>
#include <string>
using namespace std;
#include "clString.h"
int main()
{
        String s1(" and I am a C++ student.");
        String s2("Please enter your name: ");
        String s3;
        String s4("My name is ");
        String s5;
        cout<<s2;
        cin>>s3;
        s2=s4+s3;
        cout<< s5 <<".\n";
        s2=s2+s1;
        s2.stringup();
        cout<<"The string\n"<<s2<<"\ncontains "<<s2.has('A')
                <<" 'A' characters in it.\n";
        s1="red";
        String rgb[3]={
                String (s1),String("green"),String("blue")};
                cout<<"Enter the name of a primary color for mixing light: ";
                String ans;
                bool success=false;
                while(cin>>ans)
                {
                        ans.stringlow();
                        for (int i=0;i<3;i++)
                        {
                                if(ans==rgb[i])
                                {
                                        cout<<"That's right!\n";
                                        success=true;
                                        break;
                                }
                        }
                        if (success)
                                break;
                        else
                                cout<<"Try again!\n";
                }
                cout<<"Bye\n";
                cin.get();
                cin.get();
                return 0;
}