Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.70
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
#1

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

15.11.2013, 02:44. Просмотров 1404. Ответов 58
Метки нет (Все метки)

Не могу исправить ошибки можете помочь, тут в 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();
}
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
15.11.2013, 02:44
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Ошибка в функции (C++):

А почему ошибка, что функции print и read не определены в функции main()? - C++
Что-то не понимаю. Они определены в заголовочном файле. А подчеркивается красным что нет.... #include &lt;iostream&gt; #include...

Ошибка в шаблоне функции: "Отсутствуют экземпляры шаблон функции, соответствующие списку аргументов" - C++
#include &lt;iostream&gt; #include &lt;iterator&gt; #include &lt;map&gt; #include &lt;Windows.h&gt; #include &lt;functional&gt; #include &lt;string&gt; #include...

Ошибка в функции - C++
Помогите исправить ошибку мне надо Составить функцию countX, которая, получив c и b через параметры по значению, &quot;а&quot; через глобальную...

Ошибка в функции - C++
#include &lt;iostream&gt; #include &lt;conio.h&gt; #include &lt;Windows.h&gt; #include &lt;iomanip&gt; using namespace std; char bufUkr; char...

ошибка в функции - C++
помогите найти ошибку. Надо найти определитель матрицы 2х2, #include &lt;iostream.h&gt; #include &lt;conio.h&gt; #include &lt;stdio.h&gt; ...

Ошибка в функции - C++
Ситуация следующая, есть string строка, в которой надо найти и сохранить все цифры, написал небольшую функцию, но возвращает ошибку &quot;term...

58
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:19 #16
Так вы создали объект, используя конструктор по умолчанию. Этот конструктор только инициализирует элементы данных m, n и z. Но он не вызывает функцию create(), которая в свою очередь вызывает сервисную функцию newMemory() для выделения памяти под массив. Естественно, что вы ничего не можете вывести, так как вы ещё ничего не создали, и ваш указатель ни к чему не привязан и ни на что не указывает.
0
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 04:22  [ТС] #17
StackOverflow, а можете поподробней с кодом где ошибки обяснить, а то я уже с этим кодо 10й час вожусь
0
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() в тела всех конструкторов.
0
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);
}
0
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:49 #20
Массив сам ещё нужно заполнить. Он у вас заполняется при помощи рандома только при вызове конструктора с параметрами.
Так, попробуйте создать объект и передать ему через конструтор все необходимые параметры, не используя write()
0
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 04:53  [ТС] #21
StackOverflow, можете продемонстрировать пожайлуста как а то я уже не соображаю что делаю, а утром сдавать это а она еще не готова(
0
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:55 #22
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();
};
0
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 05:00  [ТС] #23
StackOverflow, а какая разница между тем что было и тем что вы скинули? оно же вроде одинаковое просто некоторые буквы вместо больших маленькие и наоборот
0
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 05:17 #24
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() я здесь не добавил.
И, да, названия классов пишутся с заглавной буквы, а названия функций - с маленькой. Так принято.
0
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 05:32  [ТС] #25
StackOverflow, Ошибка 3 error C2678: бинарный "~": не найден оператор, принимающий левый операнд типа "Dynamic" (или приемлемое преобразование отсутствует) c:\users\dima\documents\visual studio 2012\projects\лабараторные\лаба 2\dynamic.cpp 79

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

Добавлено через 9 минут
StackOverflow, я её немного подправил но он все ровно ту же ошибку что и была выдает
0
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 05:46 #26
Ага, тобишь он lvalue требует. Буду знать. Я как крот сейчас, сам проверить не могу. Вы мои глаза)
Тогда убираем из destroy() деструктор, а в её тело копируем тело деструктора.
Всё равно не работает? Вы пробовали сделать так, как я говорил, а именно, передать объекту параметры при создании?
0
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 05:51  [ТС] #27
StackOverflow, можете написать где и как да куда, а то я то я засыпаю сижу мне хотя бы одну эту сделать что бы работала да пойти хоть час поспать
0
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 05:55 #28
C++
1
2
3
4
5
6
7
int main()
{
    Dynamic object1( 5, 7, 0 );
 
//далее ваш код из main
 
}
0
tAruuuch
0 / 0 / 0
Регистрация: 24.02.2013
Сообщений: 144
15.11.2013, 05:58  [ТС] #29
StackOverflow,теперь вообще Debug error
0
StackOverflow
All rights reserved.
91 / 81 / 5
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 06:16 #30
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() кроме того, что выводит массив?
0
15.11.2013, 06:16
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
15.11.2013, 06:16
Привет! Вот еще темы с ответами:

Ошибка в функции - C++
Вот упражнение : Напишите программу, использующую структуру sterling, которая описана в упражнении 10 главы 4. Программа должна получать...

Ошибка функции - C++
Почему у меня при обработке наипростейшей функции появляется ошибка: 1&gt;------ Построение начато: проект: Fun2, Конфигурация: Debug...

Ошибка в функции - C++
Помогите пожалуйста исправить ошибку bool zero (double*Ar, int n) { bool flag = false; int j = 1; do{ if (Ar=0)...

Ошибка в функции - C++
я написал функцию для загрузки графа из файла, но при попытке открытия происходит ошибка ...что нужно...


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

Или воспользуйтесь поиском по форуму:
30
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru