Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.54/13: Рейтинг темы: голосов - 13, средняя оценка - 4.54
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146

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

15.11.2013, 02:44. Показов 2866. Ответов 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
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
15.11.2013, 02:44
Ответы с готовыми решениями:

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

Ошибка формирования отчета в СКД: ошибка при попытке вычисления значения агрегатной функции
Решая свою задачку, нарыла формулу, в которой вылазит ошибка. И не могу понять.. Вот формула ВЫБОР КОГДА...

Ошибка в шаблоне функции: "Отсутствуют экземпляры шаблон функции, соответствующие списку аргументов"
#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...

58
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:09
А можно прикрепить ещё и "Dynamic.h"?

У вас в 47 строчке элемент-функция create определена как функция без параметров, то есть не принимающая никаких аргументов. А в 14 строке вы пытаетесь ей каким-то непонятным (может быть только для меня, но не для вас или кого-нибудь ещё) образом что-то передать. И что такое z?
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 03:13  [ТС]
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 не могу
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:26
Вы в newMemory привязываете к одному и тому же указателю два динамических массива? Как я понял, вы хотели создать двумерный динамический массив? Так это не так делается.
C++
1
float ** z = new float * [ size ];
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 03:31  [ТС]
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 ];
 
}
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:33
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 - это, соответственно, размерности строк и столбцов.
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 03:36  [ТС]
StackOverflow, понял, а что же с z делать, я просто так и не нашел в чем проблема(
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:42
А зачем вам в 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 ];
        }
}
Так вы переделали, и всё равно та же ошибка?
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 03:45  [ТС]
StackOverflow, из ошибок осталась только
error C2109: для индекса требуется массив или указатель
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:49
Остальные моменты, связанные с указателем, поправили (синтаксис)?
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 03:52  [ТС]
StackOverflow, да, осталось вот это, из-за него не хочет работать а найти в чем там дело не могу, уже раз 100 перерыл весь код
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 03:57
В интерфейсе объявите указатель на двумерный динамический массив:
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 привязываете массив.
И освобождение памяти поправить не забудьте.
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 04:04  [ТС]
StackOverflow, теперь выбивает на выводе массива на экран, там пишет чтение памяти невозможно
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:10
Теперь кидайте ваш main().
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 04:13  [ТС]
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ю
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:19
Так вы создали объект, используя конструктор по умолчанию. Этот конструктор только инициализирует элементы данных m, n и z. Но он не вызывает функцию create(), которая в свою очередь вызывает сервисную функцию newMemory() для выделения памяти под массив. Естественно, что вы ничего не можете вывести, так как вы ещё ничего не создали, и ваш указатель ни к чему не привязан и ни на что не указывает.
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 04:22  [ТС]
StackOverflow, а можете поподробней с кодом где ошибки обяснить, а то я уже с этим кодо 10й час вожусь
0
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:33
Без проблем. Смотрите, у вас есть несколько конструкторов в интерфейсе класса. В своей программе вы создаёте объект класса без каких-либо параметров. Следовательно, срабатывает конструктор по умолчанию, который делает ровно три вещи (в вашем коде): он инициализирует переменную m и n нулём, а также создаёт нулевой указатель z. Дале в вашей программе пользователь вводит число, чтобы выбрать тот или иной пункт меню. Если он выбирает пункт 1, то программа попросит его ввести размерность матрицы (вызывается элемент-функция write( m, n ) класса Dynnamic). Эта функция, в свою очередь, лишь присвает новые значения элементам данных m и n. Но указатель по прежнему ни на что не указывает, память до сих пор не выделена, двумерный динамический массив до сих пор не создан и не привязан к указателю. Поэтому, когды вы используете элемент-функцию nopus(), получается так, что вы пытаетесб через разменование указателя z обратиться к тем или иным ячейкам памяти. Но указатель ни на что не указывает, поэтому и память не может быть "read". Просто добавьте функцию create() в тела всех конструкторов.
0
0 / 0 / 1
Регистрация: 24.02.2013
Сообщений: 146
15.11.2013, 04:45  [ТС]
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
All rights reserved.
93 / 83 / 24
Регистрация: 03.08.2013
Сообщений: 258
15.11.2013, 04:49
Массив сам ещё нужно заполнить. Он у вас заполняется при помощи рандома только при вызове конструктора с параметрами.
Так, попробуйте создать объект и передать ему через конструтор все необходимые параметры, не используя write()
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
15.11.2013, 04:49
Помогаю со студенческими работами здесь

Ошибка "Присваивание имени функции вне тела этой функции" при наличии двух одноименных функций в одном контексте
Данный код: type T = class public function F: integer; begin F := 2; end; function F(x:...

График функции и ошибка "Неверное число параметров функции"
Program aaa; Uses GraphABC; Var x,y:integer; Begin For x:=0 to 360 do Begin Y:=Round( sin(x*3,14/180)*100); ...

Ошибка функции
выбивает ошибку функции

Ошибка в функции!
Доброго времени суток! Написал функцию которая возращает True если строка Sample (это строка из шестнадцеричных чисел) длиной Leng...

Ошибка в функции
Добрый день. Пожалуйста подскажите почему не работает функция, вроде с синтаксисом все нормально: function av_pereplata_func...


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

Или воспользуйтесь поиском по форуму:
20
Ответ Создать тему
Новые блоги и статьи
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Access
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru