1- Questions en orienté objet

Version imprimable

11/05/2008, 17h34
dot-_-net

1- Questions en orienté objet
:salut:
Code:

1 2 3 4 5 6 7 class String{ protected: char * str; private: String copyleft(char * s, int cp); };
Code:

1 2 3 4 5 6 7 8 String String::copyleft(char *s,int cp){ int i; this->str=new char [strlen(s)]+1; for(i=0;i<cp;i++)this->str[cp]=s[cp]; this->str[i]='\0'; return *this; }
Je veux retourner un objet de type String
1- Est-ce que ce n’est pas nécessaire de l’alloue de la mémoire ?
2- Pour retourner un objet est ce qu’on fait : return objet ; ou bien : return * objet ???
:merci:

:salut:
Attention!!
Code:

1 2 3 4 5 6 7 String String::copyleft(char *s,int cp){ int i; this->str=new char [strlen(s)+1]; for(i=0;i<cp;i++)this->str[cp]=s[cp]; this->str[i]='\0'; return *this; }
(le +1, fallait le mettre dans les crochets)

Ca m'a l'air bon, mais fais attention à utiliser delete[] quelque part pour libérer la mémoire.

Donc, avec le delete[], ça ferait:
Code:

1 2 3 4 5 6 7 8 9 String String::copyleft(char *s,int cp){ delete [] str; int i; this->str=new char [strlen(s)+1]; for(i=0;i<cp;i++)this->str[cp]=s[cp]; this->str[i]='\0'; return *this; }
Mais là, lors du premier appel, on aura un problème de libération de mémoire sur une zone non allouée. Il faut donc initialiser le pointeur à NULL.
Ensuite, il faut aussi penser à libérer la mémoire quand la classe disparaît.

On a finalement:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 class String{ public: String() : str(NULL) { ; } ~String() { delete [] str; } protected: char * str; private: String copyleft(char * s, int cp); }; ---- String String::copyleft(char *s,int cp){ delete [] str; int i; this->str=new char [strlen(s)+1]; for(i=0;i<cp;i++)this->str[cp]=s[cp]; this->str[i]='\0'; return *this; }
Enfin, il subsiste un dernier problème.
Si tu veux retourner l'objet, et non un pointeur vers l'objet, il faut utiliser *this.

Mais là, il ne te retournera pas l'objet, seulement une copie créée. Quand cette copie sera détruite, le pointeur sera libéré, et donc ta classe d'origine pointera vers de la mémoire déjà libérée.

Il faut donc retourner l'objet lui-même, et non une copie:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 class String{ public: String() : str(NULL) { ; } ~String() { delete [] str; } protected: char * str; private: String &copyleft(char * s, int cp); }; ---- String& String::copyleft(char *s,int cp){ delete [] str; int i; this->str=new char [strlen(s)+1]; for(i=0;i<cp;i++)this->str[cp]=s[cp]; this->str[i]='\0'; return *this; }
Enfin, je ne vois pas trop l'intérêt pour l'objet de se retourner lui-même dans une méthode private, à laquelle lui seul peut accéder :?

11/05/2008, 18h12
Mat007
Salut,
Citation:
Envoyé par coyotte507

Code:

1 2 3 4 5 6 7 8 9 String& String::copyleft(char *s,int cp){ delete [] s; int i; this->str=new char [strlen(s)+1]; for(i=0;i<cp;i++)this->str[cp]=s[cp]; this->str[i]='\0'; return *this; }
Y'a comme un problème avec le delete de s...

Il faut aussi faire gaffe à la construction par copie et à l'assignation.
Et puis idéalement il faudrait faire du code exception safe, cf. Copy-and-swap...

Et si c'est pas pour un exercice il faut utiliser std::string bien sûr.

MAT.
11/05/2008, 18h18
coyotte507

oui, tu as raison bien sûr, delete [] str.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
date::date(String *s){
	String *p=new String;;
	p->copyleft(s->str,2);
	sscanf(p->str,"%d",&this->jour );	
	delete [] p->str;
	p->copymidl(s->str,3,5);
	sscanf(p->str,"%d",&this->mois);
	delete [] p->str;
	p->copyright(s->str,4);
	sscanf(p->str,"%d",&this->anne);
}

int main(){
	
String *s=new String;
s->str=new char [10];
strcpy(s->str,"12/34/5678");
date *k = new date;
k->date(s);

return 0;
}

Code:

1
2
3
D:\c++\projetC\date.cpp(37) : error C2273: 'function-style cast' : illegal as right side of '->' operator
main.cpp
Error executing cl.exe.

Pour quoi cette ligne k->date(s); me donne cette erreur ?

11/05/2008, 18h55
coyotte507
Le constructeur est appelé à la construction de l'objet, tu ne peux plus l'appeler après.

Ca donne donc:
Code:

1 2 3 4 5 6 7 8 9 int main(){ String *s=new String; s->str=new char [10+1]; strcpy(s->str,"12/34/5678"); date *k = new date(s); return 0; }
Mais pourquoi utiliser l'allocation dynamique ici?
Code:

1 2 3 4 5 6 7 8 9 int main(){ String s; s.str=new char [10+1]; strcpy(s.str,"12/34/5678"); date k(&s); return 0; }
il faut faire char[10+1] car on doit compter le caractère vide en plus.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
void String::devchaine(String l1,String l2){
int longe=this->Length();
int test=0;
int cp1=0;
l1.str=new char[longe];
l2.str=new char [longe];
for(int i=0;i<longe;i++){
	if(this->str[i]=' ')test=1;
	else{
		if(test==0){
			l1.str[i]=this->str[i];
			l1.str[i+1]='\0';
	}
		if(test==1){
			l2.str[cp1]=this->str[i];
			l2.str[cp1+1]=this->str[i];
			cp1++;
		}
 
}
 
}
 
}

Comment sortir avec un objet en paramètre ?

Jae,

Hum, je n'ai pas tout lu en détail, mais quand je vois ceci
Code:

1 2 3 4 5 6 7 8 9 10 11 String String::copyleft(char *s,int cp){ int i; this->str=new char [strlen(s)]+1; /* ======== la ligne encadrée par ceci ======= */ for(i=0;i<cp;i++)this->str[cp]=s[cp]; /* ======== la ligne encadrée par ceci ======= */ this->str[i]='\0'; return *this; }
La ligne encadrée ne fait pas du tout ce que tu penses.

Et je me demande si tu as testé ton code, car ça n'a pas l'air d'avoir été corrigé au fil des messages.

12/05/2008, 20h36
Médinoc

Un return *this qui n'est pas par référence, c'est suspect.

Cette fonction modifie la chaîne sur laquelle tu l'appelles, puis retourne une copie, modifiée également, de celle-ci...
12/05/2008, 22h04
droggo

Gia,

Citation:

Envoyé par Médinoc

Un return *this qui n'est pas par référence, c'est suspect.

Cette fonction modifie la chaîne sur laquelle tu l'appelles, puis retourne une copie, modifiée également, de celle-ci...

Même pas:

Code:

for(i=0;i<cp;i++)this->str[cp]=s[cp];

met dans tous les caractères de la nouvelle chaîne le contenu de l'indice [sp] de la chaîne d'origine, quel qu'il soit.

D'autre part, comme on ne veut copier que les sp premiers caractères de la chaîne, il faut remplacer

Code:

this->str=new char [strlen(s)]+1;

par

Code:

this->str=new char [sp+1];

:)
13/05/2008, 01h06
dot-_-net

Tous ça se sont des fautes d’inattention qui n’ont pas d’importance car le compilateur va me les corriger en les signalant pour , ce qui compte pour moi c’est de savoir est ce qu’ont peut sortir avec un objet en paramètre
13/05/2008, 03h57
droggo

Fao,

Citation:

Envoyé par dot-_-net

Tous ça se sont des fautes d’inattention qui n’ont pas d’importance car le compilateur va me les corriger en les signalant pour , ce qui compte pour moi c’est de savoir est ce qu’ont peut sortir avec un objet en paramètre

Pas celle que j'ai signalée, qui est une erreur que jamais un compilateur ne trouvera : tu copies tout simplement les mauvais caractères dans ta nouvelle chaîne, et le compilateur n'a pas la moindre possibilité de décider que ce n'est pas ça que tu veux effectivement faire.

Plutôt que compter sur le compilateur pour te signaler ce qui ne va pas, il vaudrait mieux apprendre ce que tu fais, et le faire proprement.
Comme vu ci-dessus, ton compilateur ne décidera jamais à ta place de ce qu'il faut faire si ton code est valide, mais faux logiquement, ce n'est pas une machine à lire la pensée.
13/05/2008, 04h04
droggo

Gia,

Citation:

Envoyé par droggo

Gia,

Même pas:

Code:

for(i=0;i<cp;i++)this->str[cp]=s[cp];

met dans tous les caractères de la nouvelle chaîne le contenu de l'indice [sp] de la chaîne d'origine, quel qu'il soit.

Correction de mon autre message, ce que j'ai mis ici en rouge est une petite erreur d'inattention, mais de toute manière, la fonction est fausse.

Le vrai comportement est :

met cp fois le caractère d'indice dans la chaîne d'origine dans le caractère de même indice cp dans la nouvelle chaîne.

D'autre part, tout le monde aura corrigé mon erreur d'écrire sp au lieu de cp. :)