Запись нарушения памяти при использовании этого - PullRequest
Новая
       5

Запись нарушения памяти при использовании этого

0 голосов
/ 31 мая 2018 
void main()
{
    shape *a[200];
    int user_choice = 0, pilihan = 0, count = 0;
    while (user_choice != 3)
    {
        cout << "---===Menu===---" << endl;
        cout << "(1). Add Shape" << endl;
        cout << "(2). Show All" << endl;
        cout << "(3). Exit" << endl;
        cout << "Masukkan Pilihan Anda : ";
        cin >> user_choice;
        if (user_choice == 1)
        {
            cout << "---===Menu===---" << endl;
            cout << "(1). Lingkaran" << endl;
            cout << "(2). Segitiga" << endl;
            cout << "(3). Back" << endl;
            cout << "Masukkan Pilihan Anda : ";
            cin >> pilihan;
            if (pilihan == 1)
            {
                count++;
                int radius;
                cout << "Masukkan Radius : ";
                cin >> radius;
                a[count] = new lingkaran(radius);
            }
            else if (pilihan == 2)
            {
                count++;
                int alas, tinggi;
                cout << "Masukkan Alas : ";
                cin >> alas;
                cout << "Masukkan Tinggi : ";
                cin >> tinggi;
                a[count] = new segitiga(alas, tinggi);
                /*  for (int x = 0; x < count; x++)
                {
                    a[x]->set2(alas, tinggi);
                }
                count++;*/
            }
            else if (pilihan == 3)
            {
                break;
            }
        }
        else if (user_choice == 2)
        {
            for (int x = 0; x < count; x++)
            {
                cout << "test1" << endl;
                a[x]->show();
                cout << "test2" << endl;
                cout << endl;
            }
        }
    }
}

так что, в принципе, я думаю, что я делаю это правильно, но мое пустое шоу (), они просто открываются, а затем закрываются сами собой ... любое предложение помощи ?, * segitiga = треугольник, * lingkaran = круг 

for (int x = 0; x < count; x++)
{
    cout << "test1" << endl;
    a[x]->show();
    cout << "test2" << endl;
    cout << endl;
}

1 Ответ

0 голосов
/ 31 мая 2018

Вы создаете первый объект по индексу 1, а не 0.
Таким образом, первый элемент массива имеет неопределенное значение, и вам просто повезло, что вы вылетаете.

Если count это ваше количество элементов, это также индекс, в котором вы хотите создать следующий объект.

Увеличение count после добавление нового элемента: 

if (pilihan == 1)
{
    int radius;
    cout << "Masukkan Radius : ";
    cin >> radius;
    a[count] = new lingkaran(radius);
    count++;
}

или напишите в count-1: th элемент: 

if (pilihan == 1)
{
    count++;
    int radius;
    cout << "Masukkan Radius : ";
    cin >> radius;
    a[count-1] = new lingkaran(radius);
}

но, на мой взгляд, этот вариант немного сложнее понять.

...