Форум программистов, компьютерный форум, киберфорум
C++
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.50/6: Рейтинг темы: голосов - 6, средняя оценка - 4.50
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
1

Неправильное выделение памяти в массиве

22.11.2019, 19:49. Показов 1098. Ответов 10

Прошу помощи в исправлении ошибки памяти.

Ошибка:
Вызвано исключение: нарушение доступа для чтения.
this->**W** было 0x1110112.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
float** W
...
void generate_weights()
    {
        for (int l = 0; l < neurons; l++) {
            for (int i = 0; i < inputs; i++)
            {
                int randNum = rand() % 2;
                if (randNum == 1)
                    W[l][i] = -1 * (double(rand()) / (double(RAND_MAX) + 1.0));
                else
                    W[l][i] = double(rand()) / (double(RAND_MAX) + 1.0);
 
            }
        }
    }
Также поясню, что за пределы массива я не выхожу. Насколько я понял я не правильно выделяю память. Заранее спасибо за помощь.
0

Помощь в написании контрольных, курсовых и дипломных работ здесь.

Лучшие ответы (1)
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
22.11.2019, 19:49
Ответы с готовыми решениями:

Выделение памяти (CodeGear RAD Studio 2009 - C++ builder). Выделение памяти - консоль vs SDIApp
Есть определенный класс - длинная арифметика. Не идеальный - хранение данных исполнено в виде...

Выделение памяти в динамическом массиве
Я испытываю проблемы с выделением памяти в динамическом массиве, ниже привожу кусок проблемного...

Выделение динамической памяти в двумерном массиве
/* Программа создаёт двумерный массив 10х10. Заполняет его случайными числами из диапазона , ищет...

Задача на выделение памяти. В массиве c(16) все целые элементы разделить на 3, а дробные округлить
Приветствую. Нужно написать программу: В массиве c(16) все целые элементы разделить на 3, а...

10
фрилансер
3012 / 2251 / 563
Регистрация: 11.10.2019
Сообщений: 6,670
22.11.2019, 19:54 2
Цитата Сообщение от dima4o Посмотреть сообщение
я не правильно выделяю память
наверное. Но в приведённом куске кода нет выделения памяти
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 19:58  [ТС] 3
Цитата Сообщение от Алексей1153 Посмотреть сообщение
наверное. Но в приведённом куске кода нет выделения памяти
Цитата Сообщение от dima4o Посмотреть сообщение
float** W
Алексей тут насколько я знаю идёт выделение намяти на массив без размера.

Добавлено через 1 минуту
Цитата Сообщение от dima4o Посмотреть сообщение
Прошу помощи в исправлении ошибки памяти.

Ошибка:
Вызвано исключение: нарушение доступа для чтения.
this->**W** было 0x1110112.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
float** W
...
void generate_weights()
    {
        for (int l = 0; l < neurons; l++) {
            for (int i = 0; i < inputs; i++)
            {
                int randNum = rand() % 2;
                if (randNum == 1)
                    W[l][i] = -1 * (double(rand()) / (double(RAND_MAX) + 1.0));
                else
                    W[l][i] = double(rand()) / (double(RAND_MAX) + 1.0);
 
            }
        }
    }
Также поясню, что за пределы массива я не выхожу. Насколько я понял я не правильно выделяю память. Заранее спасибо за помощь.
Вставил не совсем тот кусок кода.
После
Цитата Сообщение от dima4o Посмотреть сообщение
void generate_weights()
    {
Идёт
C++
1
float W[neurons][inputs];
0
фрилансер
3012 / 2251 / 563
Регистрация: 11.10.2019
Сообщений: 6,670
22.11.2019, 20:00 4
dima4o, float** W - это всего лишь указатель, к тому же - не инициализированный

Добавлено через 1 минуту
dima4o, приведи рабочий код, чтобы я мог взять и запустить его )
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 20:01  [ТС] 5
Спасибо за информацию! Тогда не могли бы вы объяснить как правильно выделить память.
0
фрилансер
3012 / 2251 / 563
Регистрация: 11.10.2019
Сообщений: 6,670
22.11.2019, 20:03 6
Цитата Сообщение от Алексей1153 Посмотреть сообщение
dima4o, приведи рабочий код, чтобы я мог взять и запустить его )
код в студию! ))
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 20:04  [ТС] 7
network.cpp
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
#include <iostream>
#include <cmath>
#include "layer.h"
using namespace std;
 
float training_set_inputs[4][4] = { {1.0,0.0,0.0,1.0}, {0.0, 0.0, 1.0,0.0}, {1.0, 1.0, 1.0,0.0}, {0.0, 1.0, 0.0,1.0}};
float training_set_outputs[4][1] = { {0.0}, {0.0}, {1.0}, {1.0} };
int epoches = 1000;
 
int law_1_neurons = 4;
int law_2_neurons = 1;
 
Law law_1(4, law_1_neurons, false, 1, 1);
Law law_2(law_1_neurons, law_2_neurons, true, 0, 2);
 
void train(){
    for (int i = 1; i <= epoches;i++) {
        cout << endl << "Epoch: " << i << endl;
        for (int l = 0; l <= 4;l++) {
            float* input = {law_1.think(training_set_inputs[l])};
            float* output = law_2.think(input);
 
            float* error;
            error = new float[law_2.get_neurons()];
            for (int k = 0; k < law_2.get_neurons(); k++) {
                error[k] = training_set_outputs[l][k] - output[k] ;
            }
            
            law_2.calc_derivatives(error);
            float* delta = law_2.get_delta();
 
            law_1.forward(delta, law_2.get_W(), law_2.get_neurons());
            
            law_2.train(input, error);
            law_1.train(training_set_inputs[l],error);
            l+=1;
        }
        for (int l = 1; l <= law_1.get_neurons(); l++) {
            law_1.cout_W(l);
        }
        for (int l = 2; l <= law_1.get_neurons(); l++) {
            law_2.cout_W(l);
        }
    }
}
 
int main() {
    int a,b,c,d;
    train();
    cout <<"Ready" << endl;
    while (true){
        cout << endl << "Enter 1st: ";
        cin >> a;
        cout << "Enter 2nd: ";
        cin >> b;
        cout << "Enter 3rd: ";
        cin >> c;
        cout << "Enter 4th: ";
        cin >> d;
        float input[4] = {a,b,c,d};
        float* inputs = { law_1.think(input) };
        cout << law_2.think(inputs)<< endl;
    }
    return 0;
}
layer.h
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
116
117
#include <cmath>
#include <iostream>
using namespace std;
 
class Law
{
private:
    int inputs;
    float* sum;
    float* output;
    float** W = {};
    float* delta;
    int neurons;
    bool output_law;
    float** related_forward;
    float* derivatives_forward;
    int neurons_forward;
    int law_name;
 
    void generate_weights()
    {
        float W[neurons][inputs];
        for (int l = 0; l < neurons; l++) {
            for (int i = 0; i < inputs; i++)
            {
                int randNum = rand() % 2;
                if (randNum == 1)
                    W[l][i] = -1 * (double(rand()) / (double(RAND_MAX) + 1.0)); // generate number between -1.0 and 0.0
                else
                    W[l][i] = double(rand()) / (double(RAND_MAX) + 1.0); // generate number between 1.0 and 0.0
 
            }
        }
    }
public:
    Law(int input, int neurons_local, bool is_out, int neurons_forward_local, int name) { inputs = input; generate_weights(); neurons = neurons_local; output_law = is_out; neurons_forward = neurons_forward_local; law_name = name; }
    void cout_W(int neuron) {
        for (int i = 0; i < inputs; i++) {
            cout << "Neuron " << neuron <<" of lawer " << law_name << ": " << W[neuron - 1][i] << " ";
        }
        cout << endl << endl;
    }
    float** get_W() {
        return W;
    }
    int get_neurons() {
        return neurons;
    }
    int get_inputs() {
        return inputs;
    }
    void calc_derivatives(float* error) {
        float* derivatives;
        derivatives = new float[neurons];
        if (output_law == true) {
            for (int i = 0; i < neurons; i++) {
                derivatives[i] = -error[i] * (exp(sum[i]) / pow((1 + exp(sum[i])), 2));
            }
        }
        else {
            for (int i = 0; i < neurons; i++) {
                derivatives[i] = (exp(sum[i]) / pow((1 + exp(sum[i])), 2)) * calc_forward_derivatives(i);
            }
        }
        delta = derivatives;
        //return derivatives;
    }
    float* get_delta(){
        return delta;
    }
    void forward(float* derivatives,float** w_local,int neuron){
        neurons_forward = neuron;
        related_forward=w_local;
        derivatives_forward=derivatives;
    }
    float* think(float* input) {
        for (int l = 0; l < neurons; l++) {
            float res = 0;
            for (int i = 0; i < inputs; i++)
            {
                res += input[i] * W[l][i];
            }
            sum[l]=res;
            output[l] = 1 / (1 + exp(-res));
        }
        //cout << -res << endl;
        return output;
    }
    float get_output(int neuron) {
        return output[neuron - 1];
    }
    float calc_forward_derivatives(int neuron) {
        float sum_derivatives=0;
        for (int i = 0; i < int(neurons_forward);i++) {
            sum_derivatives += related_forward[neuron][i] * derivatives_forward[i];
        }
        return sum_derivatives;
    }
    void train(float* input, float* error) {
        for (int l = 0; l < neurons; l++) {
            if (output_law == true) {
                float delta_local = -error[l] * (exp(sum[l]) / pow((1 + exp(sum[l])), 2));
                for (int i = 0; i < inputs; i++)
                {
                    W[l][i] += input[i] * delta_local;
                }
            }
            else {
                float delta_local = (exp(sum[l]) / pow((1 + exp(sum[l])), 2)) * calc_forward_derivatives(l);
                for (int i = 0; i < inputs; i++)
                {
                    W[l][i] += input[i] * delta_local;
                }
            }
        }
    }
};
Вот
0
фрилансер
3012 / 2251 / 563
Регистрация: 11.10.2019
Сообщений: 6,670
22.11.2019, 20:09 8
dima4o, вот тут неправильно:

C++
1
2
3
4
5
    void generate_weights()
    {
        float W[neurons][inputs];
...
...
float** W - это поле класса Law, а в данной процедуре ты объявляешь локальный статический массив. this->W при этом остаётся нулевым указателем
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 20:11  [ТС] 9
Алексей подскажите как правильно записать?
0
фрилансер
3012 / 2251 / 563
Регистрация: 11.10.2019
Сообщений: 6,670
22.11.2019, 20:29 10
Лучший ответ Сообщение было отмечено dima4o как решение

Решение

dima4o, выделяем память для двух измерений

C++
1
2
3
4
5
6
7
8
9
    void generate_weights()
    {
        W=new float*[neurons];
        for (int n = 0; n < neurons; n++)
        {
            W[n]=new float[inputs];
        }
...
...
правда, там ещё много вопросов останется. Например, если память уже была выделена, старую бы освободить сначала (если размерности другие). В деструкторе класса тоже освободить память нужно.

Добавлено через 1 минуту
в своём проекте я бы с памятью не возился, а применил бы контейнер

std::vector<std::vector<float>>
1
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 20:38  [ТС] 11
Алексей большое вам спасибо!
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
22.11.2019, 20:38

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

Динамическое выделение памяти в массиве алгоритм первый подходищий
Написать программу, реализующую согласно алгоритму Первый подходящий заданную стратегию...

Неправильное выделение байтов
void *_upointer; int type:2; В классе присутствуют 2 переменные. Насколько я знаю, указатель...

Неправильное выделение слов с ошибками
Word выделяет слова как ошибочные, хотя проблем в написании нет. И это не похожие на русские...


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

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

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