Форум программистов, компьютерный форум CyberForum.ru

Ошибка в функции - C++

Восстановить пароль Регистрация
 
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.70
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 02:44     Ошибка в функции #1
Не могу исправить ошибки можете помочь, тут в 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();
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
15.11.2013, 02:44     Ошибка в функции
Посмотрите здесь:

C++ Функции. Ошибка(
Ошибка в функции C++
C++ Ошибка в Функции
Ошибка в функции C++
Ошибка в функции C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:09     Ошибка в функции #2
А можно прикрепить ещё и "Dynamic.h"?

У вас в 47 строчке элемент-функция create определена как функция без параметров, то есть не принимающая никаких аргументов. А в 14 строке вы пытаетесь ей каким-то непонятным (может быть только для меня, но не для вас или кого-нибудь ещё) образом что-то передать. И что такое z?
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 03:13  [ТС]     Ошибка в функции #3
StackOverflow,

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
#pragma once
 
#include <string.h>
#include <iostream>
#include <cstdlib>
#include <clocale>
#include <windows.h>
#include <stdio.h>
#include <conio.h>
#include <ctime>
#include <iomanip>
 
class dynamic
{
    int         m,
                n;
 
    float       *z;
 
    void NewMemory(int n, int m);
    void DelMemory();
public:
    dynamic();
    dynamic(int,int, float*);
    dynamic(int,int, float);
    dynamic(dynamic&);
    ~dynamic();
 
    void Create();
    void Destroy();
 
    void Write();
    void Napus();
};
Добавлено через 1 минуту
StackOverflow, с create ошибку исправил но вот с z не могу
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:26     Ошибка в функции #4
Вы в newMemory привязываете к одному и тому же указателю два динамических массива? Как я понял, вы хотели создать двумерный динамический массив? Так это не так делается.
C++
1
float ** z = new float * [ size ];
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 03:31  [ТС]     Ошибка в функции #5
StackOverflow, да мне нужно двумерный динамический массив

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

C++
1
2
3
4
5
6
7
8
9
void dynamic::NewMemory (int n, int m)
{
    this->m = m;
    this->n = n;
    float ** z = new float * [ n ];
        for (int i = 0; i < n; i++)
            z[i] = new float [ m ];
 
}
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:33     Ошибка в функции #6
C++
1
2
3
4
5
float ** z = new float * [ size1 ];
for ( int i = 0; i < size; i++ )
{
    z[ i ] = new float [ size2 ];
}
size1 и size2 - это, соответственно, размерности строк и столбцов.
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 03:36  [ТС]     Ошибка в функции #7
StackOverflow, понял, а что же с z делать, я просто так и не нашел в чем проблема(
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:42     Ошибка в функции #8
А зачем вам в newMemory что-то передавать? Сервисная функция, пусть она сама с элементами данных и взаимодействует в открытую. Я бы сделал так:
C++
1
2
3
4
5
6
7
8
void Dynamic::newMemory()
{
    float ** z = new float * [ this->m ];
        for ( int i = 0; i < size; i++ )
        {
            z[ i ] = new float [ this->n ];
        }
}
Так вы переделали, и всё равно та же ошибка?
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 03:45  [ТС]     Ошибка в функции #9
StackOverflow, из ошибок осталась только
error C2109: для индекса требуется массив или указатель
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:49     Ошибка в функции #10
Остальные моменты, связанные с указателем, поправили (синтаксис)?
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 03:52  [ТС]     Ошибка в функции #11
StackOverflow, да, осталось вот это, из-за него не хочет работать а найти в чем там дело не могу, уже раз 100 перерыл весь код
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:57     Ошибка в функции #12
В интерфейсе объявите указатель на двумерный динамический массив:
C++
1
float ** z;
Далее в конструкторах его инициализируйте через инициализатор переменных:
C++
1
2
3
4
5
Dynamic::Dynamic() : z( NULL )
{....}
 
Dynamic::Dynamic( int n, int m, int l ) : z( NULL )
{....}
Ну а в newMemory теперь просто к z привязываете массив.
И освобождение памяти поправить не забудьте.
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 04:04  [ТС]     Ошибка в функции #13
StackOverflow, теперь выбивает на выводе массива на экран, там пишет чтение памяти невозможно
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:10     Ошибка в функции #14
Теперь кидайте ваш main().
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 04:13  [ТС]     Ошибка в функции #15
StackOverflow, у меня выведение в ф-ции Napus, там есть наша матрица
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
#include "dynamic.h"
 
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "Rus");
 
    dynamic Dynamic;
 
    do
    {
        cout << "Виберiть один з запропонованих варiантiв" << endl;
        cout << "1. Ввести даннi" << endl;
        cout << "2. Вивести даннi" << endl;
        cout << "e. Вихiд" << endl << endl;
 
        int key = _getch ();
 
        switch (key)
        {
            case '1':   Dynamic.Write();                break;
            case '2':   Dynamic.Napus();                break;
 
            case 'e':
            {return 0;}
            default:
                cout << "Невiрний вибiр! Натиснiть будь-яку клавiшу для продовження..." << endl;
                _getch ();
        }
 
    } while (true);
}
Добавлено через 2 минуты
StackOverflow, + у меня еще буде 2 конструктора, один с 3мя аргументами и еще один с 6ю
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:19     Ошибка в функции #16
Так вы создали объект, используя конструктор по умолчанию. Этот конструктор только инициализирует элементы данных m, n и z. Но он не вызывает функцию create(), которая в свою очередь вызывает сервисную функцию newMemory() для выделения памяти под массив. Естественно, что вы ничего не можете вывести, так как вы ещё ничего не создали, и ваш указатель ни к чему не привязан и ни на что не указывает.
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 04:22  [ТС]     Ошибка в функции #17
StackOverflow, а можете поподробней с кодом где ошибки обяснить, а то я уже с этим кодо 10й час вожусь
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:33     Ошибка в функции #18
Без проблем. Смотрите, у вас есть несколько конструкторов в интерфейсе класса. В своей программе вы создаёте объект класса без каких-либо параметров. Следовательно, срабатывает конструктор по умолчанию, который делает ровно три вещи (в вашем коде): он инициализирует переменную m и n нулём, а также создаёт нулевой указатель z. Дале в вашей программе пользователь вводит число, чтобы выбрать тот или иной пункт меню. Если он выбирает пункт 1, то программа попросит его ввести размерность матрицы (вызывается элемент-функция write( m, n ) класса Dynnamic). Эта функция, в свою очередь, лишь присвает новые значения элементам данных m и n. Но указатель по прежнему ни на что не указывает, память до сих пор не выделена, двумерный динамический массив до сих пор не создан и не привязан к указателю. Поэтому, когды вы используете элемент-функцию nopus(), получается так, что вы пытаетесб через разменование указателя z обратиться к тем или иным ячейкам памяти. Но указатель ни на что не указывает, поэтому и память не может быть "read". Просто добавьте функцию create() в тела всех конструкторов.
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 04:45  [ТС]     Ошибка в функции #19
StackOverflow, понял, но я добавил и все ровно тоже самое(
и еще у меня вопрос возник как сделать свои main так что показывало одновременно роботу всех моих конструкторов кроме конструктора по умолчанию?

Добавлено через 4 минуты
StackOverflow, вот все что у меня получилось
Кликните здесь для просмотра всего текста
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
#pragma once
 
#include <string.h>
#include <iostream>
#include <cstdlib>
#include <clocale>
#include <windows.h>
#include <stdio.h>
#include <conio.h>
#include <ctime>
#include <iomanip>
 
class dynamic
{
    int         m,
                n;
 
    float       **z;
 
    void NewMemory(int n, int m);
    void DelMemory(int m, int n);
public:
    dynamic();
    dynamic(int,int,float);
    dynamic(dynamic&);
    ~dynamic();
 
    void Create(int n, int m);
    void Destroy(int n, int m);
 
    void Write();
    void Napus();
};


Кликните здесь для просмотра всего текста
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
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
#include "dynamic.h"
 
using namespace std;
 
dynamic::dynamic() : z ( NULL )
{
    m = 0;
    n = 0;
    z = NULL;
 
    Create((n,m), true);
}
 
dynamic::dynamic(int n, int m, float l) : z ( NULL )
{
    srand(time(NULL));
 
    Create((n,m), true);
 
    for (int i = 1; i < n; i++)
        for(int j = 1; j < m; j++)
            z[ i ][ j ] = (rand() % 99 + 1);    
}
 
/*dynamic::dynamic(int n, int m, int l) : z ( NULL )
{
    srand(time(NULL));
 
    Create((n,m), true);
 
    for (int i = 1; i < n; i++)
        for(int j = 1; j < m; j++)
            z[ i ][ j ] = (rand() % 10 + 1);
 
}*/
 
/*dynamic::dynamic(int n, int m, int l, int g, int b, int a) : z ( NULL )
{
    srand(time(NULL));
 
    Create((n,m), true);
 
    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)
{
    float ** z = new float * [ this->n ];
        for (int i = 0; i < n; i++)
            z [ i ] = new float [ this->m ];
}
 
void dynamic::DelMemory (int m, int n)
{
    delete []z;
    m = 0;
    n = 0;
    z = NULL;
}
 
void dynamic::Create (int n, int m)
{
    if (m && n) DelMemory (m,n);
        NewMemory (m,n);
}
 
void dynamic::Destroy (int n, int m)
{
    if (m && n) DelMemory(m,n);
}
 
void dynamic::Napus()
{
    printf("Наша матрица\n");
 
    for (int i = 0; i < n; i++)
    {
        for (int j = 0; j < m; j++)
            cout << 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]);
        }
 
    dynamic B ();
 
    system("pause");
}
 
dynamic::~dynamic()
{
    Destroy(n,m);
}


Кликните здесь для просмотра всего текста
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
#include "dynamic.h"
 
using namespace std;
 
int main()
{
    setlocale(LC_ALL, "Rus");
 
    dynamic Dynamic;
 
    do
    {
        cout << "Виберiть один з запропонованих варiантiв" << endl;
        cout << "1. Ввести даннi" << endl;
        cout << "2. Вивести даннi" << endl;
        cout << "e. Вихiд" << endl << endl;
 
        int key = _getch ();
 
        switch (key)
        {
            case '1':   Dynamic.Write();                break;
            case '2':   Dynamic.Napus();                break;
 
            case 'e':
            {return 0;}
            default:
                cout << "Невiрний вибiр! Натиснiть будь-яку клавiшу для продовження..." << endl;
                _getch ();
        }
 
    } while (true);
}
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.11.2013, 04:49     Ошибка в функции
Еще ссылки по теме:

C++ Ошибка в функции!
C++ Ошибка в функции
C++ Ошибка в функции

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:49     Ошибка в функции #20
Массив сам ещё нужно заполнить. Он у вас заполняется при помощи рандома только при вызове конструктора с параметрами.
Так, попробуйте создать объект и передать ему через конструтор все необходимые параметры, не используя write()
Yandex
Объявления
15.11.2013, 04:49     Ошибка в функции
Ответ Создать тему
Опции темы

Текущее время: 05:53. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru