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

При изменении каких либо данных программа либо вылетает, либо просто не изменяет данные

21.03.2017, 22:50. Просмотров 813. Ответов 1
Метки нет (Все метки)

Добрый вечер. Только недавно начал заниматься С++. И вот возникли проблемы. При изменении каких либо данных. Программа либо вылетает(Qt), либо просто не изменяет данные (CodeBlock). Прошу прощения, если не та тема.
Вот код:

Кликните здесь для просмотра всего текста
#include <iostream>
#include <stdio.h>
#include <cmath>
#include <fstream>
#include <Windows.h>
#include <cstring>

using namespace std;

struct Employer {
char *name;
char *surname ;
double workedhours;
};



int InitStruct(Employer **&Employees) {
char tmp[100];
int count = 0;


cout<<"\n Set array size: "; cin>> count;
Employees = new struct Employer *[count];

for (int i = 0; i < count; i++) {
Employees[i] = new struct Employer;
cout << "\nType name: ";cin >> tmp;
Employees[i]->name = new char((strlen(tmp)) + 1);
strcpy(Employees[i]->name, tmp);

cout << "\nType surename: ";cin >> tmp;
Employees[i]->surname = new char((strlen(tmp)) + 1);
strcpy(Employees[i]->surname, tmp);

cout << "\nType work time: ";cin >> Employees[i]->workedhours;
}
return count;
}
int readFromFile(Employer **&Employees, int var) {
int count = 0;
char buffer[100];

ifstream fin("data2.txt");
if (!(fin.is_open())){
cout <<"not open\n";
fin.close();
return 0;
}


if(var == 0)
fin >> count;

if(var == 1){
ifstream fin_count("data.txt");
if(!(fin_count.is_open())){
cout<<"not open\n";
fin_count.close();
return 0;
}

for(;!(fin_count.eof()){
fin_count>>buffer;
fin_count>>buffer;
fin_count>>buffer;
fin_count>>buffer;
count++;
}

fin_count.close();
}

Employees = new Employer *[count];

for(int i = 0; i< count; i++){
Employees[i] = new struct Employer;
fin>>buffer;
Employees[i]->name = new char [(strlen(buffer))+1];
strcpy(Employees[i]->name, buffer);

fin>>buffer;
Employees[i]->surname = new char [(strlen(buffer))+1];
strcpy(Employees[i]->surname, buffer);

fin>>Employees[i]->workedhours;

}
fin.close();
return count;
}
void writeInFile(Employer **&Employees, int count){

char fileName[50];

cout<<"Type name file: ";cin>>fileName;

ofstream fout(fileName);
if(!(fout.is_open())) {
cout<<"not open\n";
fout.close();
}

for(int i = 0; i< count; i++) {
fout<<Employees[i]->name;
fout<<" ";
fout<<Employees[i]->surname;
fout<<" ";
fout<<Employees[i]->workedhours;
fout<<"\r\n";
}
fout.close();
}
void printStruct(Employer **&Employees, int count){

for(int i = 0;i< count; i++) {
cout << " \n";
cout << " \n";
cout << "-----------------------------\n";
cout << "Mark: " << Employees[i]->name << "\n";
cout << "Max resolution: " << Employees[i]->surname << " \n";
cout << "Price: " << Employees[i]->workedhours << " р \n";
}
}
void del(Employer **&Employees, int count) {
for(int i = 0; i< count; i++) {
delete (Employees[i]->name);
delete (Employees[i]->surname);
Employees[i]->workedhours = 0;
}
delete [] Employees;
}
void calculate(Employer **&Employees, int count){

double sallary = 0;
for(int i = 0; i < count; i++){
Employer *e = Employees[i];
cout<<"---------------------------------------------\n";
cout<<"Sallary is: " << (e->workedhours) * 125 / count << " p \n";
cout<<"Name is: " << e->name << "\n";
cout<<"Surmame is: " << e->surname << "\n";
cout<<"---------------------------------------------\n";
}
}
void changeParameters(Employer **&Employees, int count) {
int EmployyNumber;
int c;
char tmp[100];

cout<<"Type monitor number : ";cin>>tmp;
EmployyNumber = EmployyNumber - 1;

cout<<"1. Change Name\n"
<<"2. Change surname\n"
<<"3. Change workedhours\n";

cout<<"--------------------------------------\n"
<<"Type command number: "; cin>>c;

switch (c) {
case 1: {
cout << "\nType name: ";cin >> tmp;
Employees[EmployyNumber]->name = new char((strlen(tmp)) + 1);
strcpy(Employees[EmployyNumber]->name, tmp);
break;
};
case 2: {
cout << "\nType surname: ";cin >> tmp;
Employees[EmployyNumber]->surname = new char((strlen(tmp)) + 1);
strcpy(Employees[EmployyNumber]->surname, tmp);
break;
};
case 3: {
cout << "\nType workedhours: ";cin >> Employees[EmployyNumber]->workedhours;
break;
};
default: cout<<"error";
}

}




int main() {
int c;
int count = 0;
SetConsoleCP(1251);
struct Employer **Employees;

do {
cout<<"-------------------------------------\n"
<<"\Monitor\n"
<<"1. Init array\n"
<<"2. Read from file\n"
<<"3. Total salary\n"
<<"4. Write in file\n"
<<"5. Print on scree\n"
<<"6. Change parameters\n"
<<"7. Exit\n\n";

cout<<"--------------------------------------\n"
<<"Type command number: "; cin>>c;

switch (c) {
case 1: {
if(count != 0) del(Employees, count);
count = InitStruct(Employees);
break;
};
case 2: {
if(count != 0) del(Employees, count);
count = readFromFile(Employees, 1);
};
case 3: {
if(count != 0) calculate(Employees, count);
else{cout<<"\nThe structure is not initialized\n";}
break;
};
case 4: {
if(count != 0) writeInFile(Employees, count);
else{cout<<"\nThe structure is not initialized\n";}
break;
};
case 5: {
if(count != 0) printStruct(Employees, count);
else{cout<<"\nThe structure is not initialized\n";}
break;
};
case 6: {
if(count != 0) changeParameters(Employees, count);
else{cout<<"\nThe structure is not initialized\n";}
break;
};
case 7: {
if(count != 0) del(Employees, count);
return 1;
};
default: cout<<"error";
}


} while(1);

return 0;
}
0
Лучшие ответы (1)
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
21.03.2017, 22:50
Ответы с готовыми решениями:

две прямые либо паралельны либо совпадают либо не существуют
Д даны числа a1, b1, c1, a2, b2, c2. Напечатать координаты точки пересечения...

Что-то интересное. Или программа, которая принимает либо 1, либо 2 числа
Программа - консольное приложение, в качестве параметров при вызове принимает...

При потсроении пишет что конструктор либо недоступен либо объявлен как explicit
/*Все обьекты содержимые в контейнерах без проблем выводятся через потоковые...

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

Можно ли передать в функцию либо вектор, либо список, если да, то как?
Можно ли передать в функцию либо вектор, либо список, если да, то как?

1
DU3
281 / 233 / 115
Регистрация: 07.09.2016
Сообщений: 587
21.03.2017, 23:05 2
Лучший ответ Сообщение было отмечено guardian как решение

Решение

из увиденного:
C++
1
2
3
4
5
6
7
void changeParameters(Employer **&Employees, int count) {
int EmployyNumber;
int c;
char tmp[100];
 
cout<<"Type monitor number : ";cin>>tmp;
EmployyNumber = EmployyNumber - 1; // EmployyNumber - нет инициализации. студия падает тут.
C++
1
2
3
4
5
6
7
8
9
10
11
12
void del(Employer **&Employees, int count) {
for(int i = 0; i< count; i++) {
delete (Employees[i]->name); // неверно. массив надо удалять delete []
delete (Employees[i]->surname); // неверно. массив надо удалять delete []
Employees[i]->workedhours = 0; // это лишнее
 
а где
delete Employees[i]; // ??
 
}
delete [] Employees;
}

C++
1
2
3
4
5
6
7
8
9
Employees[i]->name = new char((strlen(tmp)) + 1);
...
Employees[i]->surname = new char((strlen(tmp)) + 1);
...
Employees[EmployyNumber]->name = new char((strlen(tmp)) + 1);
...
Employees[EmployyNumber]->surname = new char((strlen(tmp)) + 1);
 
надо выделять массив. т.е. new char[]. квадратные скобки, а не круглые.
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
21.03.2017, 23:05

Проверка открытия файла либо работает, либо уходит в бесконечность
string A=Enterway('t');//функция ввода адреса файла fstream str(A.c_str(),...

По условию вывести сумму либо чётных, либо нечётных разрядов заданного шестизначного числа
Пользователь вводит шестизначное число. Если сумма четных разрядов больше суммы...

Никак не могу найти ошибку, должен сосздать матрицу либо из звездочеку либо из пробелов
#include &lt;iostream&gt; #include &lt;cmath&gt; #include &lt;ctime&gt; using namespace std;...


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

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

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