Sujet Précédent Sujet Suivant

Erreur sur le return des accesseurs

Résolu
gedfinos - Modifié par cptpingu le 2/02/2016 à 22:58
cptpingu Messages postés 3837 Date d'inscription dimanche 12 décembre 2004 Statut Modérateur Dernière intervention 28 mars 2023 - 2 févr. 2016 à 22:58
Bonjour, voici mon code 

#include <string.h>
#include <stdlib.h>
#include <iostream>
using namespace std;
#include <conio.h>
 
 class BaseMailingListEntry
 {
  private:
   std::string sfirstName;
   std::string sLastName;
   std::string sAdressLine1;
   std::string sAdressLine2;
   std::string sCity;
   std::string sZipcode;
   public:
    BaseMailingListEntry(void)
    {
    }
    BaseMailingListEntry(const BaseMailingListEntry& aCopy)
    {
    
    sfirstName = aCopy.sfirstName;
    sLastName=aCopy.sLastName;
    sAdressLine1 = aCopy.sAdressline1;
    sadressLine2 = aCopy.sAdressLine2;
    scity= aCopy.sCity;   
    sZipCode =aCopy.szipCode;
    
    
  }
    
    
     virtual bool First(void) = 0;
virtual bool Next(void) = 0;

 std::string getfirstName() (  return sfirstName:);=>error expected 'or'before return
  
  
   std::string getLastName() ( return sLastName:);
   std::string getAdress1() ( return sAdressLine1:);
   
   std::string getAdress2()  (return sAdressLine2: );
   
   std::string getCity() ( return sCity:)
   std::string getZipcode() ( return sZipCode:);
   void setfirstName(const char *strfirstName) (sfirstName = strfirstName:);
   void setLastName(const char *strLastName)  (sLastName = strLastName:);
   void setAdress1(const char *strAdress1 ) (sAdress1Line1 = strAdress1:);
   void setadress2(const char *stradress2) (sAdressLine2 = stradress2:);
          void setCity(const char *strCity) (sCity = strCity;);
          void setZipCode(sZipCode=const char *strZipCode) (sZipCode=strZipCode:);
      };

1 réponse

Réponse 1 / 1
cptpingu Messages postés 3837 Date d'inscription dimanche 12 décembre 2004 Statut Modérateur Dernière intervention 28 mars 2023 124
Modifié par cptpingu le 2/02/2016 à 23:00
Bonjour.

Tout d'abord, pense à mettre des balises de code ! Je viens de te les rajouter, mais la prochaine fois ton message ne sera pas accepté.
Ensuite, il serait bien que tu expliques un peu ton souci plutôt que de copier coller du code en mode "ok voici mon code, démerdez-vous, au revoir"...

Pour répondre à ta question:
Il semble que tu es mis des ":" (deux petits points) au lieu de ";" (point virgule) partout à la fin de tes return, et tu mets des parenthèses au lieu d'accolades pour le corps de celle-ci.

Quelques conseils:
- Evite les using namespace, c'est très sale. Voir: http://0217021.free.fr/portfolio/axel.berardino/articles/bon-usage-using-namespace
- Si tu ne fais rien de particulier avec tes accesseurs, autant laisser le champs en public et laisser l'attribut être modifié.
- En C++, contrairement au C, une méthode ou une fonction qui ne prend pas d'argument s'écrit plutôt "func()" et non "func(void)".
- conio.h, string.h et stdlib.h n'ont rien à faire dans tes "include", tu peux les retirer.
- Utilise une liste d'initialisation dans tes constructeurs.

Ex: 
BaseMailingListEntry(const BaseMailingListEntry& aCopy)
 : sfirstName(aCopy.sfirstName),
   sLastName(aCopy.sLastName),
   sAdressLine1(aCopy.sAdressline1),
   sadressLine2(aCopy.sAdressLine2),
   scity(aCopy.sCity),
   sZipCode(aCopy.szipCode)
{
}



Améliorer votre expérience CodeS-SourceS avec ce plugin:
http://codes-sources.commentcamarche.net/forum/affich-10000111-plugin-better-cs-2#cptpingu-signature
0
Rejoignez-nous