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

Копирую строчки двумерного массива и программа начинает вести себя странно - C++

Восстановить пароль Регистрация
 
Woody-krsk
1 / 1 / 0
Регистрация: 20.12.2010
Сообщений: 62
13.08.2011, 20:07     Копирую строчки двумерного массива и программа начинает вести себя странно #1
В общем, есть некоторый полузапиленный код.
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
118
119
120
121
122
123
124
125
#include <iostream>
#include <cstdlib>
#include <stdlib.h>
#include <conio.h>
#include <time.h> 
using namespace std;
 
void showmass(int**, int, int);
void showmass(int*, int);
void showmass(double*, int);
int fitness (int*, int);
void cross(int*&, int*&, int);
void cross(int*&, int*&, int*&, int);
int main()
{
    srand(time(NULL));
    setlocale(0,"");
    int dm1 = 0, dm2 = 10;
    cout << "Введите размер популяции" << endl;
    cin >> dm1;
    cout << endl;
    int** popul = new int*[dm1];
    for (int i = 0; i < dm1; i++){
        popul[i] = new int[dm2];
    }
    for (int i = 0; i < dm1; i++){
        for (int j = 0; j < dm2; j++){
            popul[i][j] = rand()%2;
        }   
    }
    double* fitn = new double[dm1];
    int sum_pop = 0;
    for (int i = 0; i < dm1; i++){
        sum_pop= sum_pop + fitness(popul[i], dm2);
        fitn[i] = fitness(popul[i], dm2);
    }
    for (int i = 0; i < dm1; i++){
        fitn[i] = fitn[i]/sum_pop;
    }
    showmass(popul, dm1, dm2);
    cout << endl;
    showmass(fitn, dm1);
    cout << endl;
    //------------------------------
    //Селекция
    //------------------------------
    
    int* intermid = new int[5];
    for (int j = 0; j < 5; j++){
        int pos = 0;
        double tmp = 0;
        for (int i = 0; i < dm1; i++){
            if (tmp < fitn[i]){
                pos = i;
                tmp = fitn[i];
            }                       
        }
        intermid[j] = pos;
        fitn[pos] = 0;      
    }   
    showmass(intermid, 5);
    //--------------------
    //Вот тут
    popul[0] = popul[3];
    //--------------------
    showmass(popul, dm1, dm2);
    _getch();
 
    for (int i = 0; i < dm1; i++){
        delete[] popul[i];  
    }
    delete[] popul;
    delete[] fitn;
    delete[] intermid;
    return 0;
}
int fitness (int* ind, int lng){
    int fitness = 0;
    for (int i = 0; i < lng; i++){
        fitness = fitness + ind[i];
    }
    return fitness;
}
void showmass(int** mass, int dm1, int dm2){
    for (int i = 0; i < dm1; i++){
        for (int j = 0; j < dm2; j++){
            cout << mass[i][j] << " ";
        }
    cout << endl;
    }
}
void showmass(int* mass, int dm1){
    for (int i = 0; i < dm1; i++){
        cout << mass[i]<< " ";
        
    }
    cout << endl;
}
void showmass(double* mass, int dm1){
    for (int i = 0; i < dm1; i++){
        cout << mass[i]<< " ";
        
    }
    cout << endl;
}
void cross(int*& gen1, int*& gen2, int len){
    int pos = rand()%len;
    cout << pos << endl;
    int buff = 0;
    for(int i = 0; i < (len-pos); i++){
       buff = gen1[i];
       gen1[i] = gen2[i];
       gen2[i] = buff;
   }
}
void cross(int*& gen1, int*& gen2, int*& gen3, int len){
    int pos = rand()%len;
    cout << pos << endl;
    for(int i = 0; i < (len-pos); i++){
       gen3[i] = gen1[i];
   }
    for(int i = pos; i < len; i++){
       gen3[i] = gen2[i];
   }
}
В чем проблема: место где написано "Вот тут", почему-то с добавлением этой строчки, окно консольки после всех вычислений перестает закрываться. Просто становится неактивным и все. Если закрыть по красному крестику, то выдается следующее сообщение:
Поток 'Поток Win32' (0xe10) завершился с кодом -1073741510 (0xc000013a).
Поток 'Поток Win32' (0x13b8) завершился с кодом -1073741510 (0xc000013a).
Программа "[3616] listen.exe: Машинный код" завершилась с кодом -1073741510 (0xc000013a).
Подскажите, что делаю неправильно?
И еще вдогонку вопрос, в функциях showmass надо ли удалять в конце динамические массивы? Насколько я понимаю, программа ведь создает для функции отдельный массив, получается надо это делать?

Добавлено через 26 минут
Забыл добавить, среда visual c++ 2008
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
13.08.2011, 20:07     Копирую строчки двумерного массива и программа начинает вести себя странно
Посмотрите здесь:

Странно себя ведёт массив в классе. C++
C++ cin странно себя ведет
C++ циклы ведут себя как то странно
C++ Программа, сравнивающая строчки массива
класс string. Программа странно себя ведет C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
silentnuke
Android Programmer
137 / 138 / 5
Регистрация: 08.12.2010
Сообщений: 421
13.08.2011, 20:08     Копирую строчки двумерного массива и программа начинает вести себя странно #2
Цитата Сообщение от Woody-krsk Посмотреть сообщение
В общем, есть некоторый полузапиленный код.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
    //--------------------
    /*не вот тут . 
        ps: когда будете удалять память получите exception, 
        ибо удалится popul[0], а когда дойдет до popul[3], удалять нечего будет.
        ну и ещё будет проблема, если dm1<=3 ибо popul[3] не будет существовать.
        */
    popul[0] = popul[3];
    //--------------------
    showmass(popul, dm1, dm2);
      // а вот тут, оно ждет вашего нажатия на любую клавишу.
    _getch();
 
}
И еще вдогонку вопрос, в функциях showmass надо ли удалять в конце динамические массивы? Насколько я понимаю, программа ведь создает для функции отдельный массив, получается надо это делать?
кто создает, где создает?
прочитайте про передачу массивов в функцию, вопросы отпадут.
Woody-krsk
1 / 1 / 0
Регистрация: 20.12.2010
Сообщений: 62
14.08.2011, 06:46  [ТС]     Копирую строчки двумерного массива и программа начинает вести себя странно #3
silentnuke, я понимаю зачем у меня в коде _getch(), я говорю о том, что после того, как я нажму любую клавишу, программа не завершается, если присутствует указанная строка. Причем вчера нашел, что если сделать так:
C++
1
2
3
for(int i = 0; i < dm2; i++){
        popul[0][i] = popul[3][i];
    }
то проблема исчезает, только я не понимаю почему так, поскольку строки и в первом случае копировались, только с неприятным эффектом. На 0 и 3 не смотрите, я просто пробовал, там потом будут переменные, за пределы не выйду.
OstapBender
 Аватар для OstapBender
581 / 519 / 35
Регистрация: 22.03.2011
Сообщений: 1,585
14.08.2011, 17:09     Копирую строчки двумерного массива и программа начинает вести себя странно #4
popul - указатель на указатель
popul[i] - указатель на целое

C++
1
2
3
4
5
6
popul[0] = popul[3]; // теперь popul[0] указывает туда же куда и popul[3] 
...
        for (int i = 0; i < dm1; i++){
                delete[] popul[i]; // при попытке удалить повторно popul[3] 
                                  // привет access violation
        }
вместо вашего решения можно также юзать memmove()
silentnuke
Android Programmer
137 / 138 / 5
Регистрация: 08.12.2010
Сообщений: 421
14.08.2011, 18:58     Копирую строчки двумерного массива и программа начинает вести себя странно #5
я об этом же и говорил.

насчет gecth (), только что специально запустил первый код, нажал клавишу, все прекрасно завершилось, правда access violation прилетел, как и говорил вначале.
Woody-krsk
1 / 1 / 0
Регистрация: 20.12.2010
Сообщений: 62
15.08.2011, 04:40  [ТС]     Копирую строчки двумерного массива и программа начинает вести себя странно #6
silentnuke, ааа, теперь понял, спасибо огромное!
Я просто совсем новичок, потому неразжеванный вариант ответа не понял.
Yandex
Объявления
15.08.2011, 04:40     Копирую строчки двумерного массива и программа начинает вести себя странно
Ответ Создать тему
Опции темы

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