Discussion :

[dot-_-net]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ???

[coyotte507]

Attention!!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Mat007]

Salut,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[coyotte507]

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

[dot-_-net]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[coyotte507]

Le constructeur est appelé à la construction de l'objet, tu ne peux plus l'appeler après.

Ca donne donc:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.