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.

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

[droggo]

Jae,

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

Code :

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

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.

[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...

[droggo]

Gia,

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 :

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

	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 :

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

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

par

Code :

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

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

[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

[droggo]

Fao,

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.

[droggo]

Gia,

Gia,

Même pas:

Code :

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

	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.