Ошибка в функции

@tAruuuch · Регистрация: 24.02.2013

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

Не могу исправить ошибки можете помочь, тут в dynamic::dynamic(int n, int m, float l) ошибки:
1.error C2109: для индекса требуется массив или указатель
2.error C2660: dynamic::Create: функция не принимает 2 аргументов

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
#include "dynamic.h"
 
using namespace std;
 
dynamic::dynamic()
{
    m = 0;
    n = 0;
    z = NULL;
}
 
dynamic::dynamic(int n, int m, float l)
{
    Create((n,m) true);
 
    z[0][0] = l;
 
    for (int i = 1; i < n; i++)
        for(int j = 1; j < m; j++)
            z[i][j] = (rand() % 99 + 1);
}
 
void dynamic::Write()
{
    printf("Введите размер матрицы: ");
    cin >> n; 
    cin >> m;
}
 
void dynamic::NewMemory (int n, int m)
{
    this->n = n;
    z = new float [n];
 
    this->m = m;
    z = new float [m];
}
 
void dynamic::DelMemory ()
{
    delete []z;
    m = 0;
    n = 0;
    z = NULL;
}
 
void dynamic::Create ()
{
    if (m && n) DelMemory ();
        NewMemory (m,n);
}
 
void dynamic::Destroy ()
{
    if (m && n) DelMemory();
}
 
void dynamic::Napus()
{
    printf("Наша матрица\n");
 
    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++)
            cout << setw(4) << z[i][j] << "   ";
            cout << endl;
    }
    
    if(n%2==1 && m%2==1)
        printf("Середнiй елемент: z[%d][%d]=%d\n", n/2, m/2, z[n/2][m/2]); 
    else if(n%2==1 && m%2==0)
        printf("Середнiй елемент:\nz[%d][%d]=%d\nz[%d][%d]=%d\n", n/2, (m-1)/2, z[n/2][(m-1)/2], n/2, m/2, z[n/2][m/2]);
    else if(n%2==0 && m%2==1)
        printf("Середнiй елемент:\nz[%d][%d]=%d\nz[%d][%d]=%d\n", (n-1)/2, m/2, z[(n-1)/2][m/2], n/2, m/2, z[n/2][m/2]);
    else{
        printf("Середнiй елемент:\n");
        printf("z[%d][%d]=%d\n", (n-1)/2, (m-1)/2, z[(n-1)/2][(m-1)/2]);
        printf("z[%d][%d]=%d\n", (n-1)/2, m/2, z[(n-1)/2][m/2]);
        printf("z[%d][%d]=%d\n", n/2, (m-1)/2, z[n/2][(m-1)/2]);
        printf("z[%d][%d]=%d\n", n/2, m/2, z[n/2][m/2]);
    }
 
    system("pause");
}
 
dynamic::~dynamic()
{
    Destroy();
}

@tAruuuch · 15.11.2013, 04:53 **[ТС]**

StackOverflow, можете продемонстрировать пожайлуста как а то я уже не соображаю что делаю, а утром сдавать это а она еще не готова(

@StOverflow · 15.11.2013, 04:55

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
class Dynamic
{
    int m;
    int n;
    float **z;
 
    void newMemory();
    void deleteMemory();
public:
    Dynamic();
    Dynamic( int , int , float );
    ~Dynamic();
 
    void create( int n, int m );
    void destroy( int n, int m );
 
    void write();
    void napus();
};

@tAruuuch · 15.11.2013, 05:00 **[ТС]**

StackOverflow, а какая разница между тем что было и тем что вы скинули? оно же вроде одинаковое просто некоторые буквы вместо больших маленькие и наоборот

@StOverflow · 15.11.2013, 05:17

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
Dynamic::Dynamic() : z( NULL )
{
    m = 0;
    n = 0;
 
    newMemory();
}
 
Dynamic::Dynamic( int n, int m, float l) : z( NULL )
{
    newMemory();
 
    z[0][0] = l;
 
    for (int i = 1; i < n; i++)
    {
        for(int j = 1; j < m; j++)
            z[i][j] = (rand() % 99 + 1);
    }
}
 
void Dynamic::write()
{
    cout << "Введите n и m: ";
    cin >> n >> m;
}
 
void Dynamic::newMemory()
{
    z = new float * [ this->n ];
 
    for (int i = 0; i < n; i++)
        z [ i ] = new float [ this->m ];
}
 
void Dynamic::destroy()
{
    ~Dynamic();
}
 
Dynamic::~Dynamic()
{
    for ( int j = 0; j < m; j++ )
    {
        delete [] z[ j ];
    }
    delete [] z;
 
    z = 0;
}

Убрал некоторые на мой взгляд бесполезные функции. Интерфейс подкорректировать снова придётся. Ну и функцию nopus() я здесь не добавил.
И, да, названия классов пишутся с заглавной буквы, а названия функций - с маленькой. Так принято.

@tAruuuch · 15.11.2013, 05:32 **[ТС]**

StackOverflow, Ошибка 3 error C2678: бинарный "~": не найден оператор, принимающий левый операнд типа "Dynamic" (или приемлемое преобразование отсутствует) c:\users\dima\documents\visual studio 2012\projects\лабараторные\лаба 2\dynamic.cpp 79

вот здесь void Dynamic::destroy()
{
~Dynamic();
}

Добавлено через 9 минут
StackOverflow, я её немного подправил но он все ровно ту же ошибку что и была выдает

@StOverflow · 15.11.2013, 05:46

Ага, тобишь он lvalue требует. Буду знать. Я как крот сейчас, сам проверить не могу. Вы мои глаза)
Тогда убираем из destroy() деструктор, а в её тело копируем тело деструктора.
Всё равно не работает? Вы пробовали сделать так, как я говорил, а именно, передать объекту параметры при создании?

@tAruuuch · 15.11.2013, 05:51 **[ТС]**

StackOverflow, можете написать где и как да куда, а то я то я засыпаю сижу мне хотя бы одну эту сделать что бы работала да пойти хоть час поспать

@StOverflow · 15.11.2013, 05:55

C++
1
2
3
4
5
6
7
int main()
{
    Dynamic object1( 5, 7, 0 );
 
//далее ваш код из main
 
}

@tAruuuch · 15.11.2013, 05:58 **[ТС]**

StackOverflow,теперь вообще Debug error

@StOverflow · 15.11.2013, 06:16

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class Dynamic
{
public:
    Dynamic();
    Dynamic( int , int );
    ~Dynamic();
 
    void write();
    void napus();
private
    int m;
    int n;
    float **z;
 
    void newMemory();
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
Dynamic::Dynamic() : z( NULL )
{
    m = 0;
    n = 0;
 
    newMemory();
}
 
Dynamic::Dynamic( int n, int m ) : z( NULL )
{
    newMemory();
 
    for (int i = 1; i < n; i++)
    {
        for(int j = 1; j < m; j++)
            z[i][j] = (rand() % 99 + 1);
    }
}
 
void Dynamic::write()
{
    cout << "Введите n и m: ";
    cin >> n >> m;
}
 
void Dynamic::newMemory()
{
    z = new float * [ this->n ];
 
    for (int i = 0; i < n; i++)
        z [ i ] = new float [ this->m ];
}
 
 
Dynamic::~Dynamic()
{
    for ( int j = 0; j < m; j++ )
    {
        delete [] z[ j ];
    }
    delete [] z;
 
    z = 0;
}
 
void Dynamic::napus()
{
    cout << "Наша матрица: " << endl;
    
    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++)
            cout << setw(4) << z[i][j];
        cout << endl;
    }
 
    if(n%2==1 && m%2==1)
        printf("Середнiй елемент: z[%d][%d]=%d\n", n/2, m/2, z[n/2][m/2]); 
    else 
        {
            if(n%2==1 && m%2==0)
            
                printf("Середнiй елемент:\nz[%d][%d]=%d\nz[%d][%d]=%d\n", n/2, (m-1)/2, z[n/2][(m-1)/2], n/2, m/2, z[n/2][m/2]);
            else
            {
                if(n%2==0 && m%2==1)
                    printf("Середнiй елемент:\nz[%d][%d]=%d\nz[%d][%d]=%d\n", (n-1)/2, m/2, z[(n-1)/2][m/2], n/2, m/2, z[n/2][m/2]);
                else{
                    printf("Середнiй елемент:\n");
                    printf("z[%d][%d]=%d\n", (n-1)/2, (m-1)/2, z[(n-1)/2][(m-1)/2]);
                    printf("z[%d][%d]=%d\n", (n-1)/2, m/2, z[(n-1)/2][m/2]);
                    printf("z[%d][%d]=%d\n", n/2, (m-1)/2, z[n/2][(m-1)/2]);
                    printf("z[%d][%d]=%d\n", n/2, m/2, z[n/2][m/2]);
                }
            }
        }
    }
}

все директивы сами подключите пожалуйста и проверьте, заработает или нет. И скажите мне, что делает ваша функция napus() кроме того, что выводит массив?

@tAruuuch · 15.11.2013, 06:22 **[ТС]**

StackOverflow, она еще выводит цетральные элементы матрицы

Добавлено через 4 минуты
StackOverflow, опять не может читать память

@StOverflow · 15.11.2013, 06:30

Эм, тогда попробуйте сделать следующее: в newMemory() при создании массива уберите this->n и this->m и оставьте просто n и m.

@tAruuuch · 15.11.2013, 06:32 **[ТС]**

StackOverflow, все ровно не работает(

@StOverflow · 15.11.2013, 06:37

Та же ошибка?
поправил write():

C++
1
2
3
4
5
6
7
8
9
10
void Dynamic::write()
{
    cout << "Введите n и m: ";
    cin >> n >> m;
    for (int i = 1; i < n; i++)
    {
        for(int j = 1; j < m; j++)
            z[i][j] = (rand() % 99 + 1);
    }
}

Croessmah · 15.11.2013, 06:39

C++
1
2
3
Dynamic::Dynamic( int n, int m ) : z( NULL )
{
    newMemory();

Перед вызовом newMemory необходимо инициализировать данные класса n и m

C++
1
2
3
4
Dynamic::Dynamic( int n_, int m_ ) : z( NULL ), n(n_), m(m_)
{
    newMemory();
//...

@tAruuuch · 15.11.2013, 06:39 **[ТС]**

StackOverflow, та же ошибка(

Croessmah · 15.11.2013, 06:40

а тут получается

C++
1
2
3
4
    m = 0;
    n = 0;
 
    newMemory();

что в newMemory будет new [ 0 ]

@StOverflow · 15.11.2013, 06:42

Croessmah, ну наконец-то, ещё одна Голова в этой теме

Блин, не заметил свой же косяк, спасибо!
Отныне только через инициализатор, чтобы таких неприятностей не возникало.
tAruuuch, проверяйте скорее, работает?)

@tAruuuch · 15.11.2013, 06:43 **[ТС]**

Croessmah, у меня после любих изменений все ровно одна и та же ошибка, я уже и код 3 раза новый писал ,а толку 0((
и мне его через 3 часа сдавать а у меня можно сказать ничего нету(

Croessmah · 15.11.2013, 06:44

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

Croessmah, у меня после любих изменений все ровно одна и та же ошибка, я уже и код 3 раза новый писал ,а толку 0((
и мне его через 3 часа сдавать а у меня можно сказать ничего нету(

Давайте полный код, который у Вас есть на текущий момент

Новые блоги и статьи Все статьи Все блоги /
Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .	Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .
интеграция AnyLogic с самописным REST API и переход на Odoo anaschu 03.07.2026 Успешная интеграция AnyLogic с самописным REST API и переход на промышленную Odoo WMS Сегодня проделал огромный путь от простой симуляции физических процессов до построения полноценной. . .	Поиск всех путей на ориентированном графе. Linux dcc0 02.07.2026 Переработка старого кода из моей статьи. Через несколько переработок от PHP кода к C89 (надеюсь, 89). Но довольно запутанно получилось. Код для Linux. Но если убрать time и то, что с ним. . .	Сам себя обучал rest api anaschu 02.07.2026 Педагогический лайфхак: Почему чистый REST API для ученика намного круче, чем готовые библиотеки Когда мы отказались от капризного JAR-файла AnyLogic и переписали код на стандартный HttpClient,. . .	rest api anylogic - выполнение модели на своём русском сайте anaschu 02.07.2026 Как подружиться с AnyLogic Cloud API, победить провайдеров и развернуться Java-бэкенд в Docker на бесплатном хостинге: Двухдневный лог борьбы Всем привет! Хочу поделиться свежим (и довольно. . .

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 04:53 [ТС]
	StackOverflow, можете продемонстрировать пожайлуста как а то я уже не соображаю что делаю, а утром сдавать это а она еще не готова( 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 05:00 [ТС]
	StackOverflow, а какая разница между тем что было и тем что вы скинули? оно же вроде одинаковое просто некоторые буквы вместо больших маленькие и наоборот 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 05:32 [ТС]
	StackOverflow, Ошибка 3 error C2678: бинарный "~": не найден оператор, принимающий левый операнд типа "Dynamic" (или приемлемое преобразование отсутствует) c:\users\dima\documents\visual studio 2012\projects\лабараторные\лаба 2\dynamic.cpp 79 вот здесь void Dynamic::destroy() { ~Dynamic(); } Добавлено через 9 минут StackOverflow, я её немного подправил но он все ровно ту же ошибку что и была выдает 0

@StOverflow All rights reserved. 93 / 83 / 24 Регистрация: 03.08.2013 Сообщений: 258
	15.11.2013, 05:46
	Ага, тобишь он lvalue требует. Буду знать. Я как крот сейчас, сам проверить не могу. Вы мои глаза) Тогда убираем из destroy() деструктор, а в её тело копируем тело деструктора. Всё равно не работает? Вы пробовали сделать так, как я говорил, а именно, передать объекту параметры при создании? 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 05:51 [ТС]
	StackOverflow, можете написать где и как да куда, а то я то я засыпаю сижу мне хотя бы одну эту сделать что бы работала да пойти хоть час поспать 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 05:58 [ТС]
	StackOverflow,теперь вообще Debug error 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 06:22 [ТС]
	StackOverflow, она еще выводит цетральные элементы матрицы Добавлено через 4 минуты StackOverflow, опять не может читать память 0

@StOverflow All rights reserved. 93 / 83 / 24 Регистрация: 03.08.2013 Сообщений: 258
	15.11.2013, 06:30
	Эм, тогда попробуйте сделать следующее: в newMemory() при создании массива уберите this->n и this->m и оставьте просто n и m. 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 06:32 [ТС]
	StackOverflow, все ровно не работает( 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 06:39 [ТС]
	StackOverflow, та же ошибка( 0

@StOverflow All rights reserved. 93 / 83 / 24 Регистрация: 03.08.2013 Сообщений: 258
	15.11.2013, 06:42
	Croessmah, ну наконец-то, ещё одна Голова в этой теме Блин, не заметил свой же косяк, спасибо! Отныне только через инициализатор, чтобы таких неприятностей не возникало. tAruuuch, проверяйте скорее, работает?) 0

@tAruuuch 0 / 0 / 1 Регистрация: 24.02.2013 Сообщений: 146
	15.11.2013, 06:43 [ТС]
	Croessmah, у меня после любих изменений все ровно одна и та же ошибка, я уже и код 3 раза новый писал ,а толку 0(( и мне его через 3 часа сдавать а у меня можно сказать ничего нету( 0