Помогите найти ошибку: классы в С++

[pu4koff]

Сообщение от Ok-ei

Спасибо за объяснение...
Только при вызове ++S, программа в UniSchet& UniSchet:: operator++() не заходит... Что не так?

Как это не заходит? Компилируется, код работает, а в этот метод не заходит? Странновато как-то.

Сообщение от Ok-ei

И еще вопрос: а как написать это

Создайте конструктор копии: UniSchet(const UniSchet &obj);
и в операторе что-то в этом роде:

UniSchet UniSchet::operator++(int) // Возвращать по значению уже надо
{
  UniSchet res(*this); // Создаём копию текущего объекта
  ++this; // префиксный инкремент, чтобы измежать дублирования кода
  return res; // возвращаем копию копии)

Может быть не самый оптимальный вариант, но достаточно легко реализовывать. Зато понятно сразу, почему рекомендуется везьде префиксный инкремент/декремент использовать

[Ok-ei]

Спасибо, стало понятнее...

Сообщение от pu4koff

Как это не заходит? Компилируется, код работает, а в этот метод не заходит? Странновато как-то.

Компелируется, но при попытке вызвать этот метод программа вылетает.... Что может быть не так?

[pu4koff]

Сообщение от Ok-ei

Компелируется, но при попытке вызвать этот метод программа вылетает.... Что может быть не так?

С указателями ничего не намудрили в конструкторе копии? Может там одно и то же 2 раза удаляется в деструкторе или еще чего... Код конструктора копии в общем показывайте и текст ошибки с которой вылетает

[Ok-ei]

class Schetchik
{ public:
    int a;   // номер цифры в массиве, которая в данный момент выбрана
    int n;  // количество цифр в типе
    char *A; // массив цифр типа
    Schetchik(int m, char *B, char b);
    ~Schetchik();
    char val(); // возвращает значение счетчика
};

 Schetchik::Schetchik(int m, char *B, char b)
    {bool f=0;
     if (m<2)
        ShowMessage("Ошибка: слишком мало цифр в типе. Счетчик создать не удалось.");
     else
        {for (int i=0; i<m; i++)
            for (int j=i+1; j<m; j++)
               if (B[i]==B[j])
                  {ShowMessage("Ошбка: значения цифр в типе совпадают. Счетчик создать не удалось.");
                   f=1;
                  }
         if (f==0)
            {int i=0;
             while ((B[i]!=b)&&(i<m))
                i++;
             if (i>m)
                ShowMessage("Ошбка: значения счетчика не совпадает с типом цифр. Счетчик создать не удалось.");
             else
                {a=i;
                 n=m;
                 A= new char[n];
                 for (int k=0; k<n; k++)
                    A[k]=B[k];
                };
            };
        };
    };

  Schetchik::~Schetchik()
    { delete [] A;
    };

 char Schetchik::val()
    { return A[a];
    };

class UniSchet
{private:
    char *B;
 public:
    Schetchik **S;
    int m; //разрядность
    UniSchet(int k, int *n, char **A, char *a);
    UniSchet(const UniSchet &Obj); // коструктор копии счетчика
    ~UniSchet();
    char* val(); // возвращает значение счетчика
    UniSchet& operator ++ (); 
    UniSchet operator ++ (int); 
    
};

 UniSchet::UniSchet(int k, int *n, char **A, char *a)
   { if (k>0)
       {m=k;
        S=new Schetchik*[m];
        for(int i=0; i<m; i++)
           S[i]=new Schetchik(n[i], A[i], a[i]);
        B=new char [m+1];
        B[m]='\0';
       }
     else
        ShowMessage("Слишком маленькая разрядность. Счетчик создать не удалось");
   };

 
 UniSchet::UniSchet(const UniSchet &Obj)
   {m=Obj.m;
    S=Obj.S;
    B=Obj.B;
   };

 UniSchet::~UniSchet()
   { for (int i=0; i<m; i++)
        delete [] S[i];
     delete [] S;
     delete [] B;
   };

 char*  UniSchet::val()
    {for (int i=0; i<m; i++)
         B[m-i-1]=S[i]->val();
     return B;
    };

 UniSchet& UniSchet::operator++()
    { int i=0;
      bool f=0;
      while ((i<m)&&(f==0))
         if ((S[i]->a)<(S[i]->n-1))
            {f=1;
             S[i]->a++;
            }
         else
            i++;
      if (f==0)
        ShowMessage("Счетчик заполнен. Значение счетчика увеличить не удалось");
      else
         for (int j=0; j<i; j++)
            S[j]->a=0;
      return *this;

    };

 UniSchet UniSchet::operator++(int)
   { UniSchet res(*this);
     ++this;
     return res;
   };

С постфиксным инкрементом даже не компелируется:
[C++ Error] UniSchet.h(101): E2025 Assignment to 'this' not allowed, use X:: operator new instead

[pu4koff]

Поправочка: ++(*this); забыл я что-то, что this - это указатель.
Конструктор копии неправильный. B и S нельзя тупо копировать из obj, потому как получается, что 2 объекта уже будут хранить указатели на одни и те же области данных и в деструкторе одно и то же уничтожать.
Попробуй как-то так:

Schetchik::Schetchik(const Schetchik &obj)
{
  a = obj.a;
  n = obj.n;
  A = new char[n];
  for (int k = 0; k < n; ++k)
    A[k] = obj.A[k];
}

UniSchet::UniSchet(const UniSchet &obj)
{
  m=Obj.m;
  S=new Schetchik*[m];
  for (int i = 0; i < m; ++m)
    S[i]=new Schetchik(*obj.S[i]);
  B=new char [m+1];
  B[m]='\0';  
};

Писал тут, так что мог где-нибудь оплошать

[Ok-ei]

Скомпелировался....
Вылетает при работе:
Debugger exception notification:
Project Project1.exe raised exception class EAccessViolation with message 'Access violation at address 00402554 in module ‘Project1.exe’. Read of address 00000001 '.Process stopped. Use Step or Run to continue.

[pu4koff]

Ну пошагово поттлаживайте на какой именно операции вылетает. Мне просто лень это всё собирать в проект и тестить. Где-то косяк с указателями полюбому. Точки останова поставьте в конструкторах копии и инкрементах и посмотрите на какой именно строке какого метода вылетает ошибка.

[Ok-ei]

Есть ощущение, что я просто не правильно вызов этих методов пишу:

UniSchet *S;
char **B;
char C[]={'a', '4', 's'};
int D[]={2,5,3};
B= new char *[3];
B[0]= new char[2];
B[1]= new char[5];
B[2]= new char[3];
B[0][0]='a';
B[0][1]='-';
B[1][0]='*';
B[1][1]='k';
B[1][2]='4';
B[1][3]='a';
B[1][4]='0';
B[2][0]='8';
B[2][1]='q';
B[2][2]='s';

S=new UniSchet(3,D,B,C);

Сам счетчик нормально создается. А вот

++S

не работает... что не так?

[Ok-ei]

 ++*S

Вот так работает...

[pu4koff]

Ну всё правильно. Ошибка такая же, что и я в постинкременте допустил сначала. ++S - это инкремент указателя. Правильно именно ++*S в данном случае. Ну или UniSchet создавайте на стеке, а не в куче:

...
UniSchet S(3,D,B,C);
++S;