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

[Ok-ei]

Задача: Требуется реализовать класс (набор классов) для универсального "счетчика".
Счетчик характеризуется "разрядностью" (количеством "цифр") и типом "цифр".
Примеры типов "цифр": обычная десятичная (0123...9), шестнадцатеричная (012...F), алфавитная (ABCD...Z), пользовательская (ваши варианты настройки для этого типа цифр).
Счетчик должен уметь:
- инициализироваться шаблоном задающим разрядность и тип цифр и текущим значением счетчика
- инкрементироваться, декрементироваться
- выводить текущее состояние в строку
Дополнительно:
тип каждой "цифры" в счетчике может быть своим, а не один и тот же для всего счетчика.

Для этого создаю два класса: Schetchik и UniSchet - массив Schetchik-ов. Проблема в конструкторах и деструкторах...

  class Schetchik
{ public:
    int a;   // номер цифры в массиве, которая в данный момент выбрана
    int n;  // количество цифр в типе
    char *A; // массив цифр типа
    Schetchik() {};
    Schetchik(int m, char *B, char b);
     ~Schetchik();
 };
Schetchik::Schetchik(int m, char *B, char b)
       { n=m;
         A= new char[n];
         for (int i=0; i<n; i++)
           A[i]=B[i];
         int i=0;
         while ((A[i]!=b)&&(i<n))
           i++;
         a=i;
       };

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

class UniSchet
{public:
    Schetchik *S;
    int m; //разрядность
    UniSchet(int k, int *n, char **A, char *a);
    ~UniSchet();
};

UniSchet::UniSchet(int k, int *n, char **A, char *a)
   { m=k;
     S=new Schetchik[m];
     for(int i=0; i<m; i++)
         S[i]=Schetchik(n[i], A[i], a[i]);
   };
UniSchet::~UniSchet()
   { delete [] S;
   };

Помогите, пожалуйста, найти ошибки...

[pu4koff]

S=new Schetchik[m];
Создание массива элементов типа Schetchik с вызовом конструктора по умолчанию для каждого объекта, а у вас конструктора такого нет. Вру. Есть, но он пустой.
Для этой строчки:
S[i]=Schetchik(n[i], A[i], a[i]);
не реализован оператор присваивания.

Либо пишите:

Schetchik **S;
...
S = new Schetchik*[m];
for (int i = 0; i < m; ++i)
  S[i] = new Schetchik(n[i], A[i], a[i]);

Ну и деструктор соответствующим образом исправить нужно будет

[Ok-ei]

Спасибо за помощь...

[Ok-ei]

Помогите, пожалуйста, еще...
Для вывода текущего состояния создаю функцию val. Она работает странно: кроме нужного значения выводит еще и пару странных символов...
Например: должна вывести s4a, а выводит s4a
|...

В чем ошибка?

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

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];
       }
     else
        ShowMessage("Слишком маленькая разрядность. Счетчик создать не удалось");
   };

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;
    };

[pu4koff]

Выделяйте в конструкторе под B m+1 символ и B[m] = '\0';

[Ok-ei]

Спасибо, помогло....
Теперь вопрос про переопределение операций...

class UniSchet
{.......
 public:
    ......
    UniSchet operator++();
  .......
};

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;
    };

В чем тут ошибки?

[pu4koff]

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

В чем тут ошибки?

Что именно не так? Может по ссылке возвращать

UniSchet& operator++();

? А то ведь нет соответствующего конструктора копии у UniSchet для работы

return *this

[Ok-ei]

Спасибо, & уже поставила....
Как сделать так, чтобы эта операция работала при таком вызове: S++, а не при S->operator ++()?

[netrino]

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

Спасибо, & уже поставила....
Как сделать так, чтобы эта операция работала при таком вызове: S++, а не при S->operator ++()?

Дело в том, что вы перегрузили префиксный ++, то есть использовать можно как ++S. Чтобы перегрузить постфиксный ++, необходимо определить метод operator++ (int);

class UniSchet
{.......
 public:
    ......
    UniSchet& operator++(int);    // Обратите внимание на тип в скобках, имя переменной не требуется, он необходим для указания, что оператор постфиксный
  .......
};

UniSchet& UniSchet::operator++(int)
    { 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;
    };

Ну и желательно придерживаться семантики: постфиксный ++ возвращает значение, а затем его увеличивает, а префиксный сначала увеличивает значение, а затем возвращает его

[Ok-ei]

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

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

постфиксный ++ возвращает значение, а затем его увеличивает