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

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

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

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

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

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

10
фрилансер
 Аватар для Алексей1153
6444 / 5639 / 1128
Регистрация: 11.10.2019
Сообщений: 15,003
22.11.2019, 19:54
Цитата Сообщение от dima4o Посмотреть сообщение
я не правильно выделяю память
наверное. Но в приведённом куске кода нет выделения памяти
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 19:58  [ТС]
Цитата Сообщение от Алексей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
фрилансер
 Аватар для Алексей1153
6444 / 5639 / 1128
Регистрация: 11.10.2019
Сообщений: 15,003
22.11.2019, 20:00
dima4o, float** W - это всего лишь указатель, к тому же - не инициализированный

Добавлено через 1 минуту
dima4o, приведи рабочий код, чтобы я мог взять и запустить его )
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 20:01  [ТС]
Спасибо за информацию! Тогда не могли бы вы объяснить как правильно выделить память.
0
фрилансер
 Аватар для Алексей1153
6444 / 5639 / 1128
Регистрация: 11.10.2019
Сообщений: 15,003
22.11.2019, 20:03
Цитата Сообщение от Алексей1153 Посмотреть сообщение
dima4o, приведи рабочий код, чтобы я мог взять и запустить его )
код в студию! ))
0
0 / 0 / 0
Регистрация: 24.02.2019
Сообщений: 9
22.11.2019, 20:04  [ТС]
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
фрилансер
 Аватар для Алексей1153
6444 / 5639 / 1128
Регистрация: 11.10.2019
Сообщений: 15,003
22.11.2019, 20:09
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  [ТС]
Алексей подскажите как правильно записать?
0
фрилансер
 Аватар для Алексей1153
6444 / 5639 / 1128
Регистрация: 11.10.2019
Сообщений: 15,003
22.11.2019, 20:29
Лучший ответ Сообщение было отмечено 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  [ТС]
Алексей большое вам спасибо!
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
22.11.2019, 20:38
Помогаю со студенческими работами здесь

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

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

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

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

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


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

Или воспользуйтесь поиском по форуму:
11
Ответ Создать тему
Новые блоги и статьи
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост.
Programma_Boinc 28.12.2025
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / **********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / **********/ gallery/ bKBkQFf Пост отсюда. . .
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
Programma_Boinc 26.12.2025
Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .
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
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru