Корявое копирование строки. Помогите найти ошибку.

[GonZaleZ]

Вот недавно прочитал тему про классы в С++ и решил написать свою программу, использующую классы.
Для тестирования конструктора попробовал сначала просто ввести данные прямо из программы. Вот код

#include <cstdlib>
#include <iostream>
#include <string.h>
#include <windows.h>

using namespace std;

class tovar {
      public:
             tovar(char*, char*, int, double);
             void change_name(char*);
             void change_tipe(char*);
             void change_coast(double);
             void show_tovar();
             ~tovar();
      private:
              char name[];
              char tipe[];
              int art;
              double coast;
      };
      
      tovar::tovar(char* tname = "0", char* ttipe = "0", int tart = 0, double tcoast = 0) {
                        strcpy(name, tname);
                        strcpy(tipe, ttipe);
                        art = tart;
                        coast = tcoast;
                        };
      void tovar::change_name(char tname[]) {
           strcpy(name, tname);
           };
           
      void tovar::change_tipe(char ttipe[]) {
           strcpy(tipe, ttipe);
           };
           
      void tovar::change_coast(double tcoast) {
           coast = tcoast;
           };
      
      void tovar::show_tovar() {
           cout << "Артикул: " << art << endl
                << "Название: " << name << endl
                << "Тип: " << tipe << endl
                << "Стоимость: " << coast << endl;
           };
           
      tovar::~tovar() {
                      cout << "Объект с данными:\n";
                      show_tovar();
                      cout << "Был удалён из базы\n";
                      };

int main(int argc, char *argv[])
{
    ::SetConsoleCP(GetACP());
    ::SetConsoleOutputCP(GetACP());
    tovar hleb("Хлеб Бородинский", "Хлеб", 175835, 15.40);
    hleb.show_tovar();
    system("PAUSE");
    return EXIT_SUCCESS;
}

Программа спокойно компилируется, но при создании объекта hleb информация о названии и типе криво копируется при помощи strcpy().
Может я что-то намудрил с указателями?
Пишу в Dev-C++

[netrino]

name и tipe объявите как char* и выделяйте под них память в конструкторе

[GonZaleZ]

name и tipe как char* объявил, а как выделить под них память в конструкторе? а то я ещё не очень хорошо с этой темой знаком...

[GonZaleZ]

всё, вопрос решил сам. помогло предварительное определение размера массивов в самом классе

[forsaken66]

::SetConsoleCP(GetACP());
::SetConsoleOutputCP(GetACP());

а это что? объясни пожалуйста.

[netrino]

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

всё, вопрос решил сам. помогло предварительное определение размера массивов в самом классе

правильно, это один из способов. Из минусов - объект класса будет занимать много места и сложно предугадать размер строк, потому приходится брать с запасом, а не занятое место будет гулять )

      tovar::tovar(char* tname = "0", char* ttipe = "0", int tart = 0, double tcoast = 0) {
                        name = new char[strlen(tname) + 1];
                        tipe   = new char[strlen(ttipe) + 1];
                        strcpy(name, tname);
                        strcpy(tipe, ttipe);
                        art = tart;
                        coast = tcoast;
                        };

Также не следует забывать про освобождение памяти когда она уже больше не нужна с помощью delete[](в деструкторе и в методах change_name и change_tipe)

[GonZaleZ]

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

::SetConsoleCP(GetACP());
::SetConsoleOutputCP(GetACP());

а это что? объясни пожалуйста.

если честно, я сам не знаю, как это работает, но оно работает =)
эти 2 функции используются для нормального отображения русского текста: одна для ввода, другая - для вывода. давно ещё где-то вычитал, а теперь уже в привычку вошло в начале всегда эти 2 функции писать.
только это работает при установке в консоли шрифта Lucida Console

[GonZaleZ]

Вот блин же... от одной проблемы избавился, так появилась другая...
После того, как задал изначальный размер массивов в классе, всё шло хорошо. Потом я захотел немного изменить конструктор так, чтобы он предлагал пользователю вручную ввести данные о товаре.
Весь класс и методы (с конструктором) компилируется нормально, но вот ругается уже под конец на строку 68

    hleb.show_tovar();

текст ругательства таков: request for member `show_tovar' in `hleb', which is of non-class type `tovar ()()'
причём тут какой-то tovar ()()? уже всё перепробовал, не получается(( думал, может нечаянно что-то удалил или дописал лишнего, но вроде всё осталось, как было...
вот ещё раз привожу весь код с внесёнными изменениями:

#include <cstdlib>
#include <iostream>
#include <string.h>
#include <windows.h>

using namespace std;

class tovar {
      public:
             tovar();
             void change_name(char*);
             void change_tipe(char*);
             void change_coast(double);
             void show_tovar();
             ~tovar();
      private:
              char name[128];
              char tipe[64];
              int art;
              double coast;
      };
      
      tovar::tovar() {
                        char tname[128] = "0";
                        char ttipe[64] = "0";
                        int tart = 0;
                        double tcoast = 0;
                        cout << "\nВведите артикул товара: "; cin >> tart;
                        cout << "\nВведите название товара: "; cin.getline(tname, 128);
                        cout << "\nОпределите тип товара: "; cin.getline(ttipe, 64);
                        cout << "\nНазначте цену товара : "; cin >> tcoast;
                        strcpy(name, tname);
                        strcpy(tipe, ttipe);
                        art = tart;
                        coast = tcoast;
                        };
      void tovar::change_name(char tname[]) {
           strcpy(name, tname);
           };
           
      void tovar::change_tipe(char ttipe[]) {
           strcpy(tipe, ttipe);
           };
           
      void tovar::change_coast(double tcoast) {
           coast = tcoast;
           };
      
      void tovar::show_tovar() {
           cout << "Артикул: " << art << endl
                << "Название: " << name << endl
                << "Тип: " << tipe << endl
                << "Стоимость: " << coast << endl;
           };
           
      tovar::~tovar() {
                      cout << "Объект с данными:\n";
                      show_tovar();
                      cout << "Был удалён из базы\n";
                      };

int main(int argc, char *argv[])
{
    ::SetConsoleCP(GetACP());
    ::SetConsoleOutputCP(GetACP());
    tovar hleb();
    hleb.show_tovar();
    system("PAUSE");
    return EXIT_SUCCESS;
}

[Ozerich]

#include <cstdlib>
#include <iostream>
#include <string.h>
#include <windows.h>

using namespace std;

class tovar {
      public:
             tovar();
             void change_name(char*);
             void change_tipe(char*);
             void change_coast(double);
             void show_tovar();
             ~tovar();
      private:
              char name[128];
              char tipe[64];
              int art;
              double coast;
      };
      
      tovar::tovar() {
                        char tname[128] = "0";
                        char ttipe[64] = "0";
                        int tart = 0;
                        double tcoast = 0;
                        cout << "\nВведите артикул товара: "; cin >> tart;
                        cout << "\nВведите название товара: "; cin.getline(tname, 128);
                        cout << "\nОпределите тип товара: "; cin.getline(ttipe, 64);
                        cout << "\nНазначте цену товара : "; cin >> tcoast;
                        strcpy(name, tname);
                        strcpy(tipe, ttipe);
                        art = tart;
                        coast = tcoast;
                        };
      void tovar::change_name(char tname[]) {
           strcpy(name, tname);
           };
           
      void tovar::change_tipe(char ttipe[]) {
           strcpy(tipe, ttipe);
           };
           
      void tovar::change_coast(double tcoast) {
           coast = tcoast;
           };
      
      void tovar::show_tovar() {
           cout << "Артикул: " << art << endl
                << "Название: " << name << endl
                << "Тип: " << tipe << endl
                << "Стоимость: " << coast << endl;
           };
           
      tovar::~tovar() {
                      cout << "Объект с данными:\n";
                      show_tovar();
                      cout << "Был удалён из базы\n";
                      };

int main(int argc, char *argv[])
{
    ::SetConsoleCP(GetACP());
    ::SetConsoleOutputCP(GetACP());
    tovar hleb=tovar();
    hleb.show_tovar();
    system("PAUSE");
    return EXIT_SUCCESS;
}

[GonZaleZ]

Спасибо. Попробовал написать просто tovar hleb; - тоже работает.
Но, блин, теперь ещё одна проблема... Надеюсь, последняя.
При выполнении программы почему-то пропускается cin.getline(tname, 128);, т.е. на экран выводится

Введите название товара:

а затем сразу

Определите тип товара:

и предложение ввести тип.
почему может пропускаться предложение ввести имя?