Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Для новичков > Вывод фамилий в алфавитном порядке

Автор: ioManip 16.2.2014, 07:22

Привет!

Вообщем не выводит D Не пойму, что не так!

Алгоритм:

Код


void PatientInfo::DisplayHosPatient(string _surnam, int _count)
{
    int i, j;
    char temp;
   
    if (infoHos == "Да" || infoHos == "да")
    {
       for (i = 0; i < _count; i++)
       {
         for (j = i + 1; j < _count; j++);
         {
             int a = int(_surnam[j]);
             int b = int(_surnam[j+1]);
             
             if (a > b)
             {
                 temp = _surnam[j];
                 _surnam[j] = _surnam[j+1];                    
                 _surnam[j+1] = temp;
             } 
         }
       }
    }  
    
    for (i = 0; i < _count; i++)
    {
        cout << _surnam[i];
    }
}

Применение:

Код


 for (int i = 0; i < count; i++)
            {
               cout << endl;
               patients[i]->DisplayHosPatient(surname, count);
            }

Автор: rudolfninja 16.2.2014, 17:39

А почему бы не использовать strcmp для сравнения, а не приводить все к int?

Автор: feodorv 17.2.2014, 01:07

Цитата(ioManip @ 16.2.2014, 08:22

)

Не пойму, что не так!

http://forum.vingrad.ru/faq/topic-200441.html не так.

Цитата(ioManip @ 16.2.2014, 08:22

)

       for (i = 0; i < _count; i++)
       {
         for (j = i + 1; j < _count; j++);

Посмотрите, пожалуйста, внимательно диапазон изменения переменных этих двух циклов. И не допускайте малейшей возможности выхода за пределы массива.

Автор: ioManip 17.2.2014, 11:42

feodorv,
rudolfninja, Попробыл! Исправил! Но все равно не получается. Даже, елс ивот так делать

Код



    std::set<std::string> set;

    set.insert(_surnam);

    std::copy(set.begin(), set.end(), std::ostream_iterator<std::string>(std::cout, "\n"));

Дело в том, что мне кажется я не правильно данные передаю, потому что при выводе всегда последняя введенная фамилия выводится. Вообщем вот:(если кратко, то)

Заполнение фамилий

Код


 PatientInfo **patients = new PatientInfo *[count];
    
    for(int i = 0; i < count; i++)
    {
        cout << "#" << i << endl;
        
        cout << "Фамилия: ";
        cin >> surname;
        
        patients[i] = new PatientInfo(surname);

Сортировка

Код


void PatientInfo::DisplayHosPatient(string _surnam)
{
    std::set<std::string> set;

    set.insert(_surnam);

    std::copy(set.begin(), set.end(), std::ostream_iterator<std::string>(std::cout, "\n"));
}

Автор: xvr 17.2.2014, 13:58

Вы вместо того, что бы сортировать пациентов в массиве patients (по их полю _surnam), сортуруете само поле _surnam (строку в нем), причем неправильно, т.к. для определения размера используется не строка (которую вы пытаетесь сортировать), а количество пациентов, которое никакого отношения к строке не имеет.

Автор: ioManip 17.2.2014, 15:09

Ага, т.е. Мне нужно массив патиентс передавать. Так,а как это сделать, ведь там получается массив объектов, как передать именно поле с фамилией?

Автор: ioManip 18.2.2014, 02:11

Вообщем думал, думал...и пришел к этому.

Сортировка:

Код


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

Передача в метод:

Код


for (int i = 0; i < count; i++)
{
       cout << endl;
       patients[i]->DisplayHosPatient(patients[i]->getSurname(), count);
}

Но в сортировке, есть ошибка, компилятор ругается на переменную temp типа char. Я думаю она должна быть типа PatientInfo, но не представляю как это сделать.

Вот мои мысли smile

Автор: feodorv 18.2.2014, 05:48

Цитата(ioManip @ 18.2.2014, 03:11

)

if (_patients[j] > _patients[j+1])

Как-то Вы хитро хотите строки сравнивать. Ведь тип у _patients - аж string **!!!

Цитата(ioManip @ 18.2.2014, 03:11

)

Вот мои мысли

Вот Ваши мысли:

Цитата(ioManip @ 17.2.2014, 16:09

)

Мне нужно массив патиентс передавать.

Так чего же Вы делаете-то?

Цитата(ioManip @ 18.2.2014, 03:11

)

patients[i]->DisplayHosPatient(patients[i]->getSurname(), count);

Что Вы здесь сортируете? Ну разве так трудно написать

Код


void PatientSort( PatientInfo **patients, int count)
{
  ...
    if( patients[j]->getSurname() > patients[j+1]->getSurname() ) 
      /* swap */;
}

Только при этом getSurname() должен возвращать ссылку на string, а не неизвестно что...

Автор: ioManip 18.2.2014, 10:10

Не получается! Ничего не происходит! smile

Вот get - метод:

Код


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

Вот переписал сортировку:

Код


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]->getSurname();
                 _patients[j]->getSurname() = _patients[j+1]->getSurname();                    
                 _patients[j+1]->getSurname() = temp;
             }

Автор: xvr 18.2.2014, 13:34

Цитата(ioManip @ 18.2.2014, 10:10

)

Вот переписал сортировку:

Вы пытаетесь менять имена у пациентов, а надо самих пациентов (строки 5-7)

Автор: feodorv 18.2.2014, 17:36

Цитата(ioManip @ 18.2.2014, 11:10

)

string PatientInfo::getSurname()

Как минимум:

Код


string& ....

А то Вы каждый раз создаёте по новому экземпляру класса string.

Цитата(xvr @ 18.2.2014, 14:34

)

Вы пытаетесь менять имена у пациентов, а надо самих пациентов (строки 5-7)

Да, swap нужен для указателей на пациентов, а не для их имён...

Автор: ioManip 22.2.2014, 06:53

Оно?

Код


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;
    }
}

Только так, тоже не выходит

Автор: xvr 22.2.2014, 11:52

Цитата(ioManip @ 22.2.2014, 06:53

)

Только так, тоже не выходит

Если не обращать внимания на то, что печатать вы пытаетесь не имена а указатели на классы, должно работать smile

Покажите код целиком

Автор: feodorv 22.2.2014, 16:01

Цитата(ioManip @ 22.2.2014, 07:53

)

for (j = 0; j < _count - i; j++);

Точка с запятой здесь зачем???

Цитата(xvr @ 22.2.2014, 12:52

)

Покажите код целиком

Присоединяюсь)))

Автор: xvr 22.2.2014, 17:43

Цитата(feodorv @ 22.2.2014, 16:01

)

Точка с запятой здесь зачем???

О! Точно! А слона то я и не заметил. Так явно работать не будет smile

Автор: akizelokro 23.2.2014, 00:11

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

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

Автор: ioManip 23.2.2014, 05:35

Убрал точку с запятой, но все равно нет.... 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 */

Автор: feodorv 23.2.2014, 09:45