VPF::Вывод фамилий в алфавитном порядке

akizelokro

Дата 23.2.2014, 00:11 (ссылка)

(нет голосов)

Загрузка ...

Крокодил

Профиль
Группа: Участник
Сообщений: 761
Регистрация: 30.7.2007

Репутация: 1
Всего: 5

А кто вообще такие переменные в функцию передаёт?
И кто вообще такие комментарии пишет?

Плюнь на этих кадров,
cсделай эту перестановку жёстче, и всё пройдёт

Это сообщение отредактировал(а) akizelokro - 23.2.2014, 00:18

--------------------

a = a + b; b = a - b; a = a - b;

ioManip

Дата 23.2.2014, 05:35 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

Убрал точку с запятой, но все равно нет.... smile

main.cpp

Код


#include <string>
#include <iostream>
#include "PatientInfo.h"

using namespace std;

int Menu()
{
    int choise = 0;
    
    cout << "(1) Список госпитализированных пациентов" << endl;
    cout << "(2) Список негоспитализированных пациентов" << endl;
    cout << "(3) Список госпитализированных детей" << endl;
    cout << "(4) Выход" << endl;
    cout << endl;
    cout << "-> ";
    
    cin >> choise;
    
    return choise;
}

int askCount()
{
    int count;
    
    for(;;)
    {
        cout << "Введите количество пациентов: ";
        cin >> count;
        
        if(cin.fail())
        {
            cout << "Это должно быть число" << endl;
            cin.clear();
            string invalid;
            cin >> invalid;
            continue;
        }
        
        if(count <= 0)
        {
            cout << "Пациентов должно быть > 0" << endl;
            continue;
        }
        
        return count;
    }
}

int main()
{
    string surname;
    string gender;
    string homeAdress;
    string diagnosis;
    string infoHos;
    string question;
    unsigned int yearBirth;
    //int i = 0;
    
    int fQuit = false;
    int count = askCount();
    
    cout << "\033c";
        
    PatientInfo **patients = new PatientInfo *[count];
    
    for(int i = 0; i < count; i++)
    {
        cout << "#" << i << endl;
        
        cout << "Фамилия: ";
        cin >> surname;
        
        cout << "Пациент является ребенком?[Да/Нет]: ";
        cin >> question;
        
        cout << "Пол: ";
        cin >> gender;
        
        cout << "Домашний адрес: ";
        cin >> homeAdress;
        
        cout << "Диагноз: ";
        cin >> diagnosis;
        
        cout << "Год рождения: ";
        cin >> yearBirth;
        
        cout << "Пациент Госпитализирован?[Да/Нет]: ";
        cin >> infoHos;

        cout << endl;
        
        patients[i] = new PatientInfo(surname);
        
        patients[i]->setQuestion(question);
        patients[i]->setGender(gender);
        patients[i]->setHomeAdress(homeAdress);
        patients[i]->setDiagnosis(diagnosis);
        patients[i]->setInfoHos(infoHos);
    }
    
    int choise = Menu();
    
    if (choise < 1 || choise > 4)
    {
        cout << "Выбран не верный пункт меню, поробуйте заново";
        cout << endl << endl;
        cout << "\033c";
       // continue;
    }
    
    switch (choise)
    {
        case 1:
        {
            cout << "Список госпитализированных пациентов: " << endl;
            cout << "======================================";
            for(int i = 0; i < count; i++)
            patients[i]->DisplayHosPatient(patients, count);
            cout << "======================================" << endl;   
        }
        break;
        
        case 2:
        {
            cout << "Список негоспитализированных пациентов: " << endl;
            cout << "======================================";
            for (int i = 0; i < count; i++)
            {
                patients[i]->DispalyNotHosPatient();
            }
            cout << "======================================" << endl; 
        }
        break;
        
        case 3:
        {
            cout << "Список госпитализированных детей: " << endl;
            cout << "======================================";
            for (int i = 0; i < count; i++)
            {
                patients[i]->DisplayHosChildren();
            }
            cout << "======================================" << endl;
        }
        break;
        
        case 4:
        {
            fQuit = true;
            cout << "Выход из программы, пожалуйста подождите..." << endl << endl;
            break;
        }
    }
    
    for(int i = 0; i < count; i++)
    {
        delete patients[i];
    }
    delete [] patients;
    
    return 0;
}

PatientInfo.cpp

Код


#include <iostream>
#include <cstring>
#include <stdlib.h>  
#include "PatientInfo.h"

using namespace std;

PatientInfo::PatientInfo(string _surname)
{
    surname = _surname;
}

void PatientInfo::setGender(string _gender)
{
    gender = _gender;
}

void PatientInfo::setYearBirth(unsigned int _yearBirth)
{
    yearBirth = _yearBirth;
}

void PatientInfo::setHomeAdress(string _homeAdress)
{
    homeAdress = _homeAdress;
}

void PatientInfo::setDiagnosis(string _diagnosis)
{
    diagnosis = _diagnosis;
}

void PatientInfo::setInfoHos(string _infoHos)
{
    infoHos = _infoHos;
}

void PatientInfo::setQuestion(string _question)
{
    question = _question;
}

string &PatientInfo::getSurname()
{
    return surname;
}

void PatientInfo::DisplayHosPatient(PatientInfo **_patients, int _count)
{
    int i, j;
    PatientInfo *temp;
   
    if (infoHos == "Да" || infoHos == "да")
    {
       for (i = 0; i < _count - 1; i++)
         for (j = 0; j < _count - i; j++)
             if (_patients[j]->getSurname() > _patients[j+1]->getSurname())
             {
                 temp = _patients[j];
                 _patients[j] = _patients[j+1];
                 _patients[j+1] = temp;
             }
    }  
    
    for (i = 0; i < _count; i++)
    {
        cout << _patients[i] << endl;
    }
}

void PatientInfo::DispalyNotHosPatient()
{
    if (infoHos == "Нет" || infoHos == "нет")
    {
        cout << endl;
        cout << "Пациент: " << surname << endl;
        cout << "Домашний адрес: " << homeAdress << endl;
    }
}

void PatientInfo::DisplayHosChildren()
{
    if (question == "Да" || question == "да")
    {
        cout << endl;
        cout << "Фамилия: " << surname << endl;
        cout << "Диагноз: " << diagnosis << endl;
    }
}

PatientInfo.h

Код


#ifndef PATIENTINFO_H
#define    PATIENTINFO_H

#include <cstring>

using std::string;

class PatientInfo
{
public:
    PatientInfo(string _surname);
    ~PatientInfo() {};
    
    void setGender(string _gender);
    void setYearBirth(unsigned int _yearBirth);
    void setHomeAdress(string _homeAdress);
    void setDiagnosis(string _diagnosis);
    void setInfoHos(string _infoHos);
    void setQuestion(string _question);
    
    string &getSurname();
       
    void DisplayHosPatient(PatientInfo **_patients, int _count);
    void DisplayHosChildren();
    void DispalyNotHosPatient();
    
private:
    string surname;
    string gender;
    string homeAdress;
    string diagnosis;
    string infoHos;
    string question;
    unsigned int yearBirth;
};

#endif    /* PATIENTINFO_H */

Это сообщение отредактировал(а) ioManip - 23.2.2014, 05:37

--------------------

Мечты не работают, пока ты не работаешь!

feodorv

Дата 23.2.2014, 09:45 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 12
Всего: 45

Цитата(ioManip @ 23.2.2014, 06:35

)

for(int i = 0; i < count; i++)
patients[i]->DisplayHosPatient(patients, count);

Зачем в DisplayHosPatient вставлена сортировка клиентов? А ведь в DispalyNotHosPatient() этой сортировки нет.
Зачем сортировать весь список при печати одного клиента? Зачем для сортировки там проверка на

Цитата(ioManip @ 23.2.2014, 06:35

)

if (infoHos == "Да" || infoHos == "да")

У Вас ведь задача какая:

ввести список клиентов
отсортировать список клиентов
вывести список клиентов по условию infoHos

Цитата(ioManip @ 23.2.2014, 06:35

)

но все равно нет

Ну вот и подскажите, пожалуйста, почему в DispalyNotHosPatient() такой вывод:

Цитата(ioManip @ 23.2.2014, 06:35

)

cout << "Пациент: " << surname << endl;
cout << "Домашний адрес: " << homeAdress << endl;

а в DisplayHosPatient() такой:

Цитата(ioManip @ 23.2.2014, 06:35

)

cout << _patients[i] << endl;

???

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

ioManip

Дата 23.2.2014, 12:13 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

Цитата(feodorv @ 23.2.2014, 09:45

)

Ну вот и подскажите, пожалуйста

Изначально задача такая была:

По требованию пользователя организовать
1. Вывод на экран списка госпитализированных пациентов в алфавитном порядке (DisplayHosPatient)
2. Список негоспитализированных пациентов с указанием домашнего адреса (DisplayNotHosPatient)
3. Список детей с указанием диагноза. (DisplayHosChildren)

Цитата(feodorv @ 23.2.2014, 09:45

)

У Вас ведь задача какая:

Т.е. да, задача такая, но только это часть из того, что нужно сделать.
Как раз с алфавитным выводом у меня проблемы, поэтому к вам и обратился!

--------------------

Мечты не работают, пока ты не работаешь!

ioManip

Дата 26.2.2014, 11:02 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

Победа!

Вообщем сидел, сидел, дай думаю индексы поменяю( j на i ) и ооо чудо!! Все получилось! smile

Код


void PatientInfo::DisplayHosPatient(PatientInfo **_patients, int _count)
{
    int i, j;
    PatientInfo *temp;
   
    if (infoHos == "Да" || infoHos == "да")
    {
       for (i = 0; i < _count - 1; i++)
       {
         for (j = 0; j < _count - i; j++)
         {
             if (_patients[i]->getSurname() > _patients[i+1]->getSurname())
             {
                 temp = _patients[i];
                 _patients[i] = _patients[i+1];
                 _patients[i+1] = temp;
             }
         }
       }
    }  
    
    for (i = 0; i < _count; i++)
    {
        cout << "Пациент: " << _patients[i]->getSurname() << endl;
        cout << "Диагноз: " << _patients[i]->getDiagnosis() << endl;
    }
}

Всем спасибо! smile

--------------------

Мечты не работают, пока ты не работаешь!

xvr

Дата 26.2.2014, 11:44 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Не получится у вас чуда - введите штук 5 пациентов, с фамилиями строго в обратном алфавитном порядке smile

ioManip

Дата 27.2.2014, 08:53 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

xvr, Да, Вы правы! :(

--------------------

Мечты не работают, пока ты не работаешь!

ioManip

Дата 27.2.2014, 09:14 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

Оххх....уже столько вожусь с этой сортировкой!!! smile

Ну вообщем, поробовал я запустить эту программу при помощи отладчика, на этапе вывода отсортированных фамилий, отладчик мне выдает: "Сигнал перехвачен - SIGSEGV" , погуглив, узнал, что SIGSEGV - это в POSIX-системах сигнал, посылаемый процессу при попытке обращения к несуществующей памяти или обращения с нарушением прав доступа.

При этом у меня открывается файл basig_string.h и открывается на строчкке с таким содержанием

Код


return __lhs.compare(__rhs) > 0;

У меня 3 вопроса: у меня не работает, потому что linux ? Или я действительно обращаюсь к несуществующей памяти?
Или я уже не в те дебри полез?

--------------------

Мечты не работают, пока ты не работаешь!

xvr

Дата 27.2.2014, 10:16 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Цитата(ioManip @ 27.2.2014, 09:14

)

У меня 3 вопроса: у меня не работает, потому что linux ? Или я действительно обращаюсь к несуществующей памяти?

Linux тут не при чем, а памяти действительно нет. Судя по всему у вас сломалось на сравнении строк (стр 12), из за выхода за пределы массива.
Вам feodorv совершенно правильно заметил, что сортировка не должна быть внутри PatientInfo, она должна быть снаружи и вызываться 1 раз, а не на каждого пациента

ioManip

Дата 27.2.2014, 12:55 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

Ну вот смотрите! Теперь сортировка в main.
main.cpp

Код


//Заполнение
...........................

PatientInfo *temp;
            
                for (i = 0; i < count - 1; i++)
                {
                   for (j = 0; j < count - i; j++)
                   {
                      if (patients[j]->getSurname() > patients[j+1]->getSurname())
                      {
                          temp = patients[j];
                          patients[j] = patients[j+1];
                          patients[j+1] = temp;
                      }
                   }
                }
                
                for (i = 0; i < count; i++)
                {
                     patients[i]->DisplayHosPatient(patients, count);
                }

DisplayHosPatient()

Код


if (infoHos == "Да" || infoHos == "да")
    {
        cout << endl;
        for (int i = 0; i < _count; i++)
        {
            cout << "Фамилия: " << _patients[i]->getSurname() << endl;
            cout << "Диагноз: " << _patients[i]->getDiagnosis() << endl;
        }
    }

Т.е. идет заполнение массива клиентами, затем сортировка, затем уже отсортированный массив передается в DisplayHosPatient, а там уже по условию infoHos производится вывод результата....ну разве не так??!!

Но только все равно крашится! :(

--------------------

Мечты не работают, пока ты не работаешь!

feodorv

Дата 28.2.2014, 17:13 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 12
Всего: 45

Цитата(ioManip @ 27.2.2014, 13:55

)

Но только все равно крашится!

Гм. По ссылке, что я Вам дал, опечатка:

Цитата(ioManip @ 27.2.2014, 13:55

)

for (j = 0; j < count - 1; j++)

Хотя, быть может, имелось в виду:

Код


for (j = 0; j < count - i - 1; j++)

В любом случае правильный вариант здесь smile

Это сообщение отредактировал(а) feodorv - 28.2.2014, 17:36

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

xvr

Дата 28.2.2014, 17:39 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Цитата(ioManip @ 27.2.2014, 12:55

)

ну разве не так??!!

Цикл внутри DisplayHosPatient не нужен - у вас уже есть цикл снаружи. Но крашится не должно. Видимо еще где то бага. Что отладчик показывает? Где ломается?

feodorv

Дата 28.2.2014, 17:43 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2214
Регистрация: 30.7.2011

Репутация: 12
Всего: 45

Цитата(xvr @ 28.2.2014, 18:39

)

Но крашится не должно

Должно))) Выход за пределы массива)))

--------------------

Напильник, велосипед, грабли и костыли - основные инструменты программиста...

ioManip

Дата 2.3.2014, 04:46 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 162
Регистрация: 12.8.2011
Где: Благовещенск

Репутация: нет
Всего: нет

Цитата(feodorv @ 28.2.2014, 17:13

)

Хотя, быть может, имелось в виду

Слушайте! Скорей всего именно это и имелось в виду! Все заработало! smile

Уххх, спасибо, всем! smile

Хороший форум!

--------------------

Мечты не работают, пока ты не работаешь!

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Для новичков \| Следующая тема »