Ошибка при попытке использовать объект класса в определении другого класса

[Josefina]

Прошу помощи.... есть созданный класс работы со строкой:
"MyString.h"

Код:

class MyString
{
 
public:
    MyString(const char*);
    MyString(int);
    ~MyString();
 
public:
    void Clear();
 
    char* MString;
    int length;
    char& operator[] (int i) {return MString[i];}
    friend MyString& operator + (MyString&, MyString&);
};
 

Хочу использовать объект этого класса в "ethernet.h", то есть:

Код:

class Ethernet
{
public:
    Ethernet();
    ~Ethernet();
 
    int Connect();
    int Disconnect();
 
    int SetIP(IP_st &newIP);
    int GetIP(IP_st &curIP);
    bool IsConnected()
        
private:
    SOCKET eth_socket;
    IP_st ip;
    bool on_connected;
    sockaddr_in eth_addr;                
    int socket_error;                    
    int shift;
 
    MyString buffer(2000);
    MyString out_buffer(2000);
    MyString log_buffer(1000);
    MyString sendlog_buffer(1000);
    
    HANDLE socket_handle;                
 
};

Выдает ошибку: 'MyString' : 'class' type redefinition

Если я понимаю правильно, то в заголовочный файл нужно вводить нечто подобное:
#ifndef MYSTRING_H
#define MYSTRING_H

А вот как это правильно сделать? .....   

[Алексей++]

в начало  заголовочного файла напиши

#pragma once

и всё

[Josefina]

Спасибо  эта ошибка исчезла, а еще одна осталась: syntax error : 'string'

Ругается на каждое объявление объекта, типа MyString. Если объект этот объявлять в CPP-ом файле, то ошибки не возникает, но мне нужно объявить именно в объявлении класса 

[Алексей++]

Josefina, а на что именно ругается, на какую строку ?

[Josefina]

Ругается на:

Код:

MyString buffer(2000);
    MyString out_buffer(2000);
    MyString log_buffer(1000);
    MyString sendlog_buffer(1000);

Еще добавлю: ошибка исчезает если обращаться к объекту следующим образом:

Код:

MyString buff();

Но у меня два конструктора с параметрами, без параметров конструкторов нет  

[Алексей++]

Код:

class MyString
{
    char* MString;
    int length;
    char& operator[] (int i) {return MString[i];}
    /*friend*/ MyString& operator + (MyString&, MyString&)
   {
      чивота забыла;
   }

 
public:
    MyString(const char*)
   {
      чивота забыла;
   }

    MyString(int)
   {
      чивота забыла;
   }

    ~MyString()
    {
      чивота забыла;
   }

public:
    void Clear()
    {
      чивота забыла;
   }

};

[Josefina]

Да, вроде бы ничего не забыла, вот код:

Код:

MyString::MyString(const char* Str)
{
    MString = new char [strlen(Str)];
    strcpy(MString,Str);
    length = strlen(Str);
}

MyString::MyString(int len)
{
    MString = new char [len];
    memset(MString,len,NULL);
    length = len;
}

MyString::~MyString()
{
    delete[] MString;
}

void MyString::Clear()
{
    memset(MString,NULL,strlen(MString));
}

MyString& operator + (MyString& Str1, MyString& Str2)
{
    int len = strlen(Str1.MString) + strlen(Str2.MString);
    MyString Str(len);
    strcpy(Str.MString,Str1.MString);
    strcat(Str.MString,Str2.MString);
    return Str;
}

Если использовать это объявление внутри CPP-шного кода, то все работает как надо, никаких ошибок.

У меня такое ощущение, что программа пытается, не создать объект типа MyString, а создать функцию типа MyString (воспринимает запись, как будто я функции создаю) и вот ругается, что это я вместо объявления параметров функции, непонятную константу пишу. Может такое быть?

[Алексей++]

Josefina, а заголовочник в cpp файл включить не забыла ?

[Josefina]

Алексей1153++, не забыла 

[Алексей++]

прицепи проект (только удали *.ncb, RELEASE и DEBUG)

Добавлено через 51 секунду:
кстати, чем стандартные классы работы со строкой не устроили ? )

[Josefina]

прицепи проект (только удали *.ncb, RELEASE и DEBUG)

можно по-подробнее, что значит прицепить проект? )))

кстати, чем стандартные классы работы со строкой не устроили ? )

Приходится много строк соединять, постоянные strcpy. strсat, memset + создание массовов строк, выделение памяти и удаление памяти раздувают код и делают его некрасивым, хочется немного "причесать", сделать интуитивно понятнее. Кроме того есть несколько нестандартных функций, которые парсят строку, хочу также их сюда добавить.

[Алексей++]

Josefina,
"прицепить" - это значить сделать архив и приаттачить к посту ))

Приходится много строк соединять, постоянные strcpy. strсat, memset + создание массовов строк,

#include <string>
класс std::string

[Джон]

Насколько я не забыл С++, резервировать место при декларации нельзя. Нужно делать так:

MyString m_buffer;

потом в cpp-файле:

Ethernet::Ethernet() : m_buffer(2000)
{

}

[Josefina]

Попробую, ночером отпишусь что получилось, что не получилось 

[Джон]

А ещё лучше (на всё оставшуюся жизнь) дополнительно таки почитать теорию, например Страуструпа.

[Алексей++]

резервировать место при декларации нельзя.

а я и не обратил внимания В уме скомпилировать не смог

[Антон (LogRus)]

Код:

MyString buff();

Так ты объявляешь функцию, удали скобки.

Немного поправил код
баги в неём еще есть
как минимум надо хранить размер строки и размер буфера
избегай вычисления длинный строки, это очень дорого

Код:

MyString::MyString(const char* Str)
{
    length = strlen(Str) + 1;
    MString = new char [length];
    strcpy(MString, Str);
    MString[length-1] = 0;
}

MyString::MyString(int len)
{
    MString = new char [len];
    MString[0] = 0;
    length = len;
}

MyString::~MyString()
{
    delete[] MString;
}

void MyString::Clear()
{
    MString[0] = 0;
}

MyString& operator + (MyString& Str1, MyString& Str2)
{
    int len = Str1.length +Str2.length;
    MyString Str(len + 1);
    strcpy(Str.MString,Str1.MString);
    strcpy(Str.MString+Str1.length,Str2.MString);
    return Str; // SegFault
}

[Josefina]

Антон (LogRus), Джон, Алексей1153++, спасибо, разобралась, все получилось 

Теорию безусловно читаю