Динамический массив C++

[Maray]

Добрый день!

Помогите пожалуйста с программой. Программа с динамическим массивом. Вводится цепочка чисел. Из этой цепочки нужно удалить числа кратные трем.

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

Текст программы:

#include <iostream>
#include <algorithm>
using namespace std;
int proc(const int N)
{
    int* mas = new int[N];
    int k;
 
    for (int i = 0; i < N; i++)
    {
        mas[i] = new int[N]; // <----- значение типа int* нельзя присвоить сущности типа int 
    }
 
    for (int i = 0; i < N; i++)
    {
        cin >> mas[i];
    }
 
    cout << endl;
    for (int i = 0; i < N; i++)
    {
        cout << mas[i] << " ";
        cout << endl;
    }
    cout << endl;
    k = N;
    for (int i = 0; i < N; i++)
    {
        if (mas[i] % 3 = 0)   // <----выделяет mas и пишет: выражение должно быть допустимым для изменения левосторонним значением
        {
            delete[]mas[i]; // <--- выражение должно иметь тип указателя
            k = k - 1;
        }
        cout << endl;
    }
 
    for (int i = 0; i < k; i++)
    {
        cout << mas[i] << " ";
        cout << endl;
    }
 
    for (int i = 0; i < k; i++)
    {
        delete[] mas[i];  <--- выражение должно иметь тип указателя
    }
    delete[] mas;
    return 0;
 
}
int main() {
    int N;
    cout << "Введите количество элементов в строке: ";
    cin >> N;
    cout << endl;
    proc(N);
    system("pause");
}

[Stilet]

значение типа int* нельзя присвоить сущности типа int

Нужно так:

int** mas = new int*[N];

выделяет mas и пишет: выражение должно быть допустимым для изменения левосторонним значением

Не = а ==.
Сравнение в Си пишется двумя знаками "равно"

<--- выражение должно иметь тип указателя

Первое выполнишь - эта ошибка уйдет.

Из этой цепочки нужно удалить числа кратные трем.

Могу предложить такой вариант:http://www.programmersforum.ru/showp...9&postcount=24

[Smitt&Wesson]

    int* mas = new int[N];
    int k;
 
    for (int i = 0; i < N; i++)
    {
        mas[i] = new int[N]; // <----- значение типа int* нельзя присвоить сущности типа int 
    }

Я не понял, чего Вы этим хотели добиться? Если нужна матрица NxN, то как показал Stilet, нужно делать указатель на указатель, а потом формировать строки при помощи цикла. Но у Вас в задании,

Вводится цепочка чисел. Из этой цепочки нужно удалить числа кратные трем.

В этом случае, вышеприведённый цикл не имеет смысла. К тому-же всё остальные циклы работают просто с массивом N.

delete[]mas[i]; // <--- выражение должно иметь тип указателя

Так делать нельзя. Вернее можно, но для динамической матрицы.
Для массива делаем так: Находим удаляемый элемент и перемещаем на его место весь "хвост массива. При этом при помощи delete ничего не удаляется, просто декрементируется переменная, проверяющая массив на окончание. Если дальше предполагается использовать массив новой длины, можно создать новый динамический массив, а старый удалить. Или использовать новое значение N. К стати, при создании динамических массивов, N совсем не обязательно делать константой.

[Stilet]

Вернее можно, но для динамической матрицы.

Да у ТС и так динамика
Но ты прав. Я даже в примере (см. ссылку) предпочел не перестраивать массив, а просто его дефрагментировать.

[Maray]

Вроде всё исправила, но некоторые ошибки остались

#include <iostream>
#include <algorithm>
using namespace std;
int proc(const int N)
{
	int** mas = new int*[N];
	int k;

	for (int i = 0; i < N; i++)
	{
		mas[i] = new int[N];
	}

	for (int i = 0; i < N; i++)
	{
		cin >> mas[i];                  
<-------отсутствует оператор ">>", соответствующий этим операндам типы операндов: std::istream>>int*
	}

	cout << endl;
	for (int i = 0; i < N; i++)
	{
			cout << mas[i] << " ";
		cout << endl;
	}
	cout << endl;
	k=N;
	for (int i = 0; i < N; i++)
	{
		if (mas[i]%3==0)       
<------(здесь ошибку исправила, но вместо неё появилась другая) выражение должно относиться
 к целочисленному типу или типу перечисления без области видимости
		{delete[]mas[i];
		k=k-1;
		}
		cout << endl;
	}

	for (int i = 0; i < k; i++)
	{
			cout << mas[i] << " ";
		cout << endl;
	}

	for (int i = 0; i < k; i++)
	{
		delete[] mas[i];
	}
	delete[] mas;
	return 0;

}
int main() {
	int N;
	cout << "Введите количество элементов в строке: ";
	cin >> N;
    cout << endl;
	proc(N);
	system("pause");
}

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

Могу предложить такой вариант:http://www.programmersforum.ru/showp...9&postcount=24

А возможно ли доработать эту программу? Я с ней хоть более менее разобралась

[Stilet]

отсутствует оператор ">>", соответствующий этим операндам типы операндов: std::istream>>int*

У тебя mas - матрица, а не одномерный массив.
Тогда уж так:

int proc(const int N)
{
	int* mas = new int[N];
	int k;

	for (int i = 0; i < N; i++)
	{
		cin >> mas[i];
	}

И вот это:

for (int i = 0; i < k; i++)
{
delete[] mas[i];
}

Убирай.

[Maray]

Так у меня уже получается не динамический массив. Но и он не работает. А мне нужно с динамическим.

#include <iostream>
#include <algorithm>
using namespace std;
int proc(const int N)
{
	int* mas = new int[N];
	int k;

	for (int i = 0; i < N; i++)
	{
		cin >> mas[i];
	}

	cout << endl;
	for (int i = 0; i < N; i++)
	{
			cout << mas[i] << " ";
		cout << endl;
	}
	cout << endl;
	k=N;
	for (int i = 0; i < N; i++)
	{
		if (mas[i]%3==0)
		{delete[]&mas[i];
		k=k-1;
		}
		cout << endl;
	}

	for (int i = 0; i < k; i++)
	{
			cout << mas[i] << " ";
		cout << endl;
	}

	return 0;

}
int main() {
	int N;
	cout << "Введите количество элементов в строке: ";
	cin >> N;
    cout << endl;
	proc(N);
	system("pause");
}

[Stilet]

Изначально неверный подход, помимо того что ты не понимаешь что такое динамический массив...
Настаиваю на своем примере, что я выложил по ссылке, и более ничем помочь не могу. Удачи.

[Maray]

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

[Maray]

В этом коде пишет, что не определены идентификаторы getchar, printf, _TCHAR