Форум программистов, компьютерный форум, киберфорум
Наши страницы
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.50/4: Рейтинг темы: голосов - 4, средняя оценка - 4.50
qwertynor
6 / 6 / 11
Регистрация: 28.05.2014
Сообщений: 51
1

Программа крашится на другом компьютере

29.05.2016, 16:41. Просмотров 785. Ответов 2
Метки нет (Все метки)

Здравствуйте. Имеется код программы, на одном компьютере он работает так как положено, но при запуске на другом компьютере происходят ошибки. В чем проявляется ошибка: сначала загружаем список из файла(1), затем выводим в консоль список (3) (это все работает). Вот тут начинается самое интересное: выбираем пункт сортировки(5), жмем показать список (3) и тут программа крашится.. В чем проблема? . Может сама сортировка реализована криво?
Код программы:
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
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
#include <iostream>
#include <string>
#include <fstream>
#include <windows.h>
using namespace std;
 
struct Cars {
 
    string name;
    int engine;
    int price;
    Cars *next,*prev;
};
 
struct NewCars {
    Cars field;
};
 
Cars *head,*tail;
 
void AddRecord(string name, int engine, int price) {
    Cars *temp = new Cars;
    temp->next=NULL;
    temp->name=name;
    temp->engine=engine;
    temp->price=price;
 
    if (head!=NULL) {
        temp->prev=tail;
        tail->next=temp;
        tail=temp;
    } else {
        temp->prev=NULL;
        head=tail=temp;
    }
 
}
 
void ShowListEnd() {
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
 
    Cars *temp = tail;
 
    while (temp!=NULL) {
        cout << temp->name << "\n";
        cout << temp->engine << "\n";
        cout << temp->price << "\n";
        temp = temp->prev;
    }
 
    delete tail;
 
}
 
void ShowListBegining() {
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
    Cars *temp = head;
 
    while (temp!=NULL) {
        cout << temp->name << "\n";
        cout << temp->engine << "\n";
        cout << temp->price << "\n";
        temp = temp->next;
    }
 
    delete head;
}
 
void SaveInFile() {
    //Cars *temp = head;
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
 
    ofstream ofFile("Cars.txt",ios_base::out);
 
    while (head!=NULL) {
        ofFile << head->name << "\t";
        ofFile << head->engine << "\t";
        ofFile << head->price << "\t";
        head = head->next;
        ofFile << "\n";
    }
 
    ofFile.close();
 
    //delete head;
}
 
int CountLines() {
    ifstream ifsFile ("Cars.txt", ios_base::in);
    int count = 0;
    string s;
    while (!ifsFile.eof()) {
        getline(ifsFile, s);
        count++;
    }
 
    return count;
}
 
void LoadFromFile() {
 
    Cars *temp = new Cars;
    ifstream ifFile("Cars.txt",ios_base::in);
 
    for (int i = 0; i < CountLines()-1; i++) {
        getline(ifFile,temp->name,'\t');
        ifFile >> temp->engine;
        ifFile.ignore();
        ifFile >> temp->price;
        ifFile.ignore();
        AddRecord(temp->name,temp->engine,temp->price);
        //AddRecord(temp->name,temp->engine);
    }
 
    delete temp;
}
 
void SortingList() {
    Cars *LinkedList = head; // связанный список
    Cars *node, * node2;
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
    for(node = LinkedList; node; node = node->next )
        for( node2 = LinkedList; node2; node2 = node2->next )
            if( node->price > node2->price ) { // если число из node меньше числа из node2 то переставляем их
                int i = node->price;
                node->price = node2->price;
                node2->price = i;
            }
}
 
void DeleteRecord()
{
    Cars *LinkedList = head;
    if (head==NULL)
    {
        cout << "Create list first!!\n";
        return;
    }
    if (LinkedList->prev == NULL && LinkedList->next==NULL)
        head=tail=NULL;
    else{
        LinkedList->next->prev = NULL;
        head = LinkedList->next;
        delete LinkedList;
        }
}
 
void ClearLinkedList()  //реализация удаления всех элементов списка
{
    while(head != NULL) //пока не указываем на хвост
    {
    Cars *temp = head;  //создаем временный элемент
    head = head->next;  //присваиваем ему указатель на следующий
    delete temp;    // и удаляем его
    }
head = NULL;
}
 
 
 
int main() {
 
    setlocale(0,"russian");
    string name;
    int engine;
    int price;
    /*
    //AddRecord("test",46,98);
    //AddRecord("test64",11,100);
 
 
    for (int i = 0; i < 2; i++)
    {
    cin.ignore();
    cout << "Name of car "; getline(cin, name);
    cout << "Engine: "; cin >> engine;
    cout << "Price: "; cin >> price;
    AddRecord(name,engine,price);
    }
 
    cout << "From end" << "\n";
    ShowListEnd();
    */
    //cout << "Save" << "\n";
    //SaveInFile();
 
    string str;
 
    int ch,n;
 
    do {
        cout << "\t||Программа учета автомобилей||";
        cout << endl << endl;
        cout << "1. Загрузка из файла;\n2. Сохранить в файл;\n3. Показать список;\n4. Добавление элементов;;\n5. Сортировка списка;\n6. Удаление элемента;\n";
        cout << "0. Выход из программы.\n";
        cout << endl << "Выбор: ";
        cin >> ch;
        cout << endl;
        switch (ch) {
            case 1:
                LoadFromFile();
                system("cls");
                cout << "Загрузка из фаила завершена!";
                cout << endl << endl;
                break;
            case 2:
                SaveInFile();
                system("cls");
                cout << "Сохранение завершено успешно!\n";
                cout << endl << endl;
                break;
            case 3:
                system("cls");
                cout << "Текущий список: \n";
                ShowListBegining();
                cout << endl << endl;
                break;
            case 4:
                system("cls");
                cout << "Количество новых записей: ";
                cin >> n;
                system("cls");
                for (int i = 0; i < n; i++) {
                    cin.ignore();
                    cout << "Добавление " << i+1 <<" из "<< n << " записей;";
                    cout << endl << endl;
                    cout << "Название автомобиля: ";
                    getline(cin, name);
                    cout << "Мощность (л.с.): ";
                    cin >> engine;
                    cout << "Стоимость: ";
                    cin >> price;
                    AddRecord(name,engine,price);
                    system("cls");
                }
                break;
            case 5:
                SortingList();
                break;
            case 6:
                DeleteRecord();
                break;
            default:
                break;
        }
 
 
    }
 
    while(ch!=0);
    cout << "Завершение программы, нажмите любую клавишу...\n";
    ClearLinkedList();
 
    return 0;
}
0
Лучшие ответы (1)
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.05.2016, 16:41
Ответы с готовыми решениями:

Программа на другом компьютере
Знаю эта тема уже была также пользовался поиском находи но ничего не понял можете подробней...

Крашится программа
Вот такой вот код. Как только вводишь размерность массива, сразу выдает ошибку виндоус. С чем может...

Крашится программа!
Суть проблемы: когда прога читает из файла на определенном количестве слов в файле она падает, если...

Программа с двусвязным списком крашится
Здравствуйте! написал код для двусвязного списка, проблема следующая: все компилируется,...

Release. Работа на другом компьютере
Написал программу в Visual Studio 2010. Вчера скинул её другу посмотреть, она не запустилась. Ему...

2
hoggy
Нарушитель
Эксперт С++
7081 / 3125 / 647
Регистрация: 15.11.2014
Сообщений: 7,165
Завершенные тесты: 1
29.05.2016, 17:02 2
Лучший ответ Сообщение было отмечено qwertynor как решение

Решение

Цитата Сообщение от qwertynor Посмотреть сообщение
Может сама сортировка реализована криво?
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
void SortingList() {
    Cars *LinkedList = head; // связанный список
    Cars *node, * node2;
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
    for(node = LinkedList; node; node = node->next )
        for( node2 = LinkedList; node2; node2 = node2->next )
        {
            if( node->price > node2->price ) { // если число из node меньше числа из node2 то переставляем их
                int i = node->price;
                node->price = node2->price;
                node2->price = i;
            }
        }
}
краш в этом коде может быть только по одной причине:
битые указатели.

это может произойти, если:

1.
в какой то момент времени,
нечто караптит память,
и попадает прямиком на ваши указатели.

2.
возможно, ваша структура изначально неверно заполняется.
и указатели изначально невалидные.
в коде вы нигде их не проверяете.

поскольку файл с исходными данными вы не предоставили,
то запустить у себя и воспроизвести ошибку возможности не предоставляется.
помогальщики могут только гадать.

а телепаты как назло все в отпусках.


попробуйте сделать так:
замените:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void ShowListBegining() {
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
    Cars *temp = head;
 
    while (temp!=NULL) {
        cout << temp->name << "\n";
        cout << temp->engine << "\n";
        cout << temp->price << "\n";
        temp = temp->next;
    }
 
    delete head;
}
на :

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void ShowListBegining() {
 
    if (head==NULL)
    {
        cout << "Create list first!\n";
        return;
    }
    Cars *temp = head;
 
    while (temp!=NULL) {
        cout << temp->name << "\n";
        cout << temp->engine << "\n";
        cout << temp->price << "\n";
        temp = temp->next;
    }
 
   // delete head;  //<--- не понятно, зачем убивать данные
}
если это поможет, значит вам повезло:
подключение к астралу сработало.
1
qwertynor
6 / 6 / 11
Регистрация: 28.05.2014
Сообщений: 51
29.05.2016, 17:04  [ТС] 3
Без проблем.
Вот файл с исходными данными.
Сейчас попробую сделать как вы предложили.

upd: Все помогло. Спасибо!
0
Вложения
Тип файла: txt Cars.txt (53 байт, 0 просмотров)
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
29.05.2016, 17:04

Запуск программы на другом компьютере
Если я написал, построил прогу и скинул её на другой комп, то она просит .dll файл, есть...

Ошибка компиляции на другом компьютере
Снова проблема такого радо что скопировал код с одного компьютера и при запуске на своем компиляция...

Программа крашится после вызова метода
Суть программы в том, чтобы выводить поздравление, используя имена, праздники и пожелания, которые...


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

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

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