Вылетает из запущенной программы

@Testero · Регистрация: 07.09.2011

Author24 — интернет-сервис помощи студентам

Нужна помощь, при сборке, все работает, но потом после ввода второго-третьего числа, окно закрывается. Кто подскажет, что делать?
Вот сама функция кода, из-за которой вылетает.

C++ (Qt)

QString MainWindow::Binary(int number)
{
    int N=0;
    int *B = new int[N];
    while (number!=0)
    {
        N++; B[N-1]=number%2;  number/=2;
    }
    QString result;
    for (int i=N-1;i>=0;i--)
    {
        result.append(QString("%1").arg(B[i]));
    }
    return result;
}

C++ (Qt)

1	ui->lineEdit_2->setText(Binary(а));

@NoMasters · 21.10.2012, 15:09

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

C++

1 2	int N=0; int *B = new int[N];

А на что ты рассчитывал?

@Testero · 21.10.2012, 15:18 **[ТС]**

Если можно, то конкретнее, что не так?

Добавлено через 29 секунд
Если менять изначально N, то ничего не меняется.

@NoMasters · 21.10.2012, 15:34

Testero, то есть ты считаешь, что выделение массива нулевой длины — это нормально?
А потом ты инкрементируешь N и всё равно используешь её же в качестве размера.

@Testero · 21.10.2012, 15:46 **[ТС]**

Извини за мое не понимание, но не мог бы ты объяснить и если не сложно показать, как было бы правильно, а?

@Арсенал · 21.10.2012, 16:16

Testero,

Крашится из-за того, что у тебя массив нулевой длины, ибо элементов в массиве равно нулю. Хочешь динамически изменять количество элементов массива - используй QVector (функция push_back()). Для определения размера вектора - используй функцию вектора size(). И еще, кто будет память очищать? После динамического выделения памяти с помощью оператора new нужно очищать память с помощью оператора delete.

@Testero 0 / 0 / 0 Регистрация: 07.09.2011 Сообщений: 13
	21.10.2012, 15:18 [ТС]	3
	Если можно, то конкретнее, что не так? Добавлено через 29 секунд Если менять изначально N, то ничего не меняется. 0

@NoMasters Псевдослучайный 1946 / 1145 / 98 Регистрация: 13.09.2011 Сообщений: 3,215
	21.10.2012, 15:34	4
	Testero, то есть ты считаешь, что выделение массива нулевой длины — это нормально? А потом ты инкрементируешь N и всё равно используешь её же в качестве размера. 0

@Testero 0 / 0 / 0 Регистрация: 07.09.2011 Сообщений: 13
	21.10.2012, 15:46 [ТС]	5
	Извини за мое не понимание, но не мог бы ты объяснить и если не сложно показать, как было бы правильно, а? 0

@Арсенал 145 / 67 / 14 Регистрация: 30.12.2011 Сообщений: 137
	21.10.2012, 16:16	6
	Testero, Крашится из-за того, что у тебя массив нулевой длины, ибо элементов в массиве равно нулю. Хочешь динамически изменять количество элементов массива - используй QVector (функция push_back()). Для определения размера вектора - используй функцию вектора size(). И еще, кто будет память очищать? После динамического выделения памяти с помощью оператора new нужно очищать память с помощью оператора delete. 0