Fonction de tri C++

Question

J'essaie donc de faire fonctionner une fonction de tri. Il devrait trier par prénom, mais si les prénoms sont les mêmes, il devrait trier par nom de famille. Je continue d'obtenir une erreur dans mySort() qui dit "expression primaire inattendue avant") "jeton" sur la ligne où inOrder = arr. Qu'est-ce qui se passe à cela et comment puis-je le réparer? J'ai besoin de passer mes objets dans une fonction dans une classe séparée. Je les lierai dans pastebin.

Voici le pilote

#include <iostream> 
#include <fstream> 
#include <string> 
#include "phoneEntry.h" 
using namespace std; 

void mySort(PhoneEntry& arr, int size) 
{ 
    bool inOrder = false; 
    for (int i = size - 1; i > 0 && !inOrder; i--) 
    { 
     inOrder = true; 
     for (int j = 0; j < i; j++) 
     { 
      inOrder = arr.alphaGreater(arr&); 
     } 
    } 
}; 

int main() 
{ 
    const int MAXNUM = 500; 
    PhoneEntry entry[MAXNUM]; 
    ifstream filezilla; 
    filezilla.open("phone.txt"); 
    int count = 0; 

    if(filezilla) 
    { 
     while(count < MAXNUM && entry[count].readEntry(filezilla)) 
     { 
      count++; 
     } 
     mySort(entry&, count); 
     for(int i = 0; i < count; i++) 
     { 
      entry[i].writeEntry(cout) << endl; 
     } 
    } 
    else 
    { 
     cout << "Four Oh Four - File Not Found" << endl; 
    } 

    return 0; 
} 

Phone Entry Header

Phone Number Header

Tri Texte (http://pastebin.com/HE8Rsmbg)

Answer 1

Vous voulez passer une référence, de sorte que le l'argument doit être défini comme appel par référence dans la définition/déclaration de la fonction, ce qui est déjà fait:

bool PhoneEntry::alphaGreater(const PhoneEntry& item) const; 

Comme vous pouvez le voir, PhoneEntry::alphaGreater prend une PhoneEntryréférence constante. Donc, il suffit d'utiliser

inOrder = arr.alphaGreater(arr); 

Answer 2

vous pouvez passer comme pointeurs aussi

Answer 3

Il y a plusieurs choses mal avec votre code. Le premier, celui sur lequel le compilateur se plaint, est qu'il n'y a pas d'opérateur postfixe & en C++. Je ne sais pas exactement ce que vous essayez de faire avec, ou ce que vous pensez que cela signifie, mais il n'existe pas en C++. Le deuxième est que mySort prend une référence à un PhoneEntry unique; vous avez un tableau de PhoneEntry, donc vous devrez en sélectionner un pour passer le . Sauf que le nom de la fonction et le fait que vous réussissiez un nombre de suggèrent que vous voulez vraiment passer un tableau.

Et je ne peux pas comprendre ce que mySort est supposé faire. Il ne trie certainement rien; en fait, il semble fondamentalement un no-op coûteux, puisqu'il ne renvoie rien, il ne modifie pas arr, et il n'accède ni ne modifie aucun état global.

Pour le reste, vous avez besoin de plus de contrôle d'erreur sur l'entrée (le fait ouvert réussir, etc.), et vous devriez vraiment utiliser std::vector, plutôt que un tableau de style C. (Il y a des cas où les tableaux de style C sont appropriés, mais ce n'est pas l'un d'entre eux.)

Plus fondamentalement, je vous suggère d'obtenir un bon livre et de l'étudier. ( recommandons Principes de programmation de Stroustrup et de la pratique en utilisant C++.) Vous ne pouvez pas écrire un programme dans aucune langue sans connaître au moins les les bases les plus basiques et la syntaxe de base.