Discussion :

[hbx360]

Bonjours, je viens à vous car j'ai un core dumped et je n'arrive pas a voir ou est le problème.

je débute en C++, je suis les cours de CodeurPlusPlus sur youtube et j'ai fait exactement comme lui j'ai repris ces sources et je suis à l'dentique pour le code.

Le problème vient de l'héritage si je n'utilise pas les fonctions d'héritage donc si je fais comme ça :

Code :

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

intarray A=intarray();

ça marche, mais quand je fais :

Code :

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

extintarray A=extintarray();

Là ça me fait un core dumped.
J'ai un makefile mais je penses pas que ça vienne de là.
Je suis sous cygwin version3 et j'ai essayer cygwin version4 c'est pareille.

Je vous met tout le code le test_array.cc le intarray.cc et le intarray.h

intarray.h :

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
25
26
27
28
29
30
31
32
33
 
#ifndef __INTARRAY_H__
#define __INTARRAY_H__
 
const int INTARRAY_DEFAULT_ALLOC = 1024;
const int INTHEAP_DEFAULT_ALLOC = 1024;
 
class intarray{
 protected: 
  int len; 
  int *data;
 public:
  intarray();
  intarray(int len);
  intarray(const intarray &A);
  ~intarray();
  int get(int index);
  void set(int index, int n);
  int length();
  friend std::ostream & operator<<(std::ostream & O,intarray & A);
};
 
class extintarray : public intarray  {
 protected: 
  int alloc;
 public:
  extintarray();
  extintarray(int alloc);
  extintarray(int len,int alloc);
  ~extintarray();
};
 
#endif

Le intarray.cc :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
 
#include<iostream>
 
#include"intarray.h"
 
 
std::ostream &operator<<(std::ostream &O, intarray &A)
{
  for(int i=0;i<A.len;i++)
    O<<A.data[i]<<" ";
  return O;
}
 
intarray::intarray(const intarray &A)
{
  this->len=A.len;
  this->data=new int[A.len];
  for(int i=0;i<A.len;i++)
    this->data[i]=A.data[i];
}
 
int intarray::length()
{
  return this->len;
}
 
intarray::intarray()
{
  this->len=INTARRAY_DEFAULT_ALLOC;
  this->data=new int[INTARRAY_DEFAULT_ALLOC];
}
 
intarray::intarray(int len)
{
  this->len=len;
  this->data=new int[len];
}
 
intarray::~intarray()
{
  delete[] this->data;
}
 
int intarray::get(int index)
{
  return this->data[index];
}
 
void intarray::set(int index, int n)
{
  this->data[index]=n;
}
 
/**********class héritage***********************************/
 
extintarray::extintarray ()  : intarray () 
{
  this->alloc = INTARRAY_DEFAULT_ALLOC;
}
 
extintarray::extintarray (int alloc)
{
  this->len = 0;
  this->alloc = alloc;
  this->data = new int[alloc];
}
 
extintarray::extintarray (int len, int alloc)
{
  this->len = len;
  this->alloc = alloc;
  this->data = new int[alloc];
}
 
extintarray::~extintarray() 
{
  delete[] this->data;
}

Et le test_array.cc :

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
 
#include<iostream>
#include<string>
 
#include"intarray.h"
 
 
int main(int argc, std::string argv[])
{
  using namespace std;
  extintarray A=extintarray(); //marche pas. Par contre si je met : intarray A=intarray(); ça marche.
 
  /*for(int i=0;i<A.length();i++)
    {
      A.set(i,i*5);
      cout<<A.get(i)<<" ";
      }*/
  /*intarray B=intarray(A);
  cout<<A;
  cout<<endl;
  cout<<B;*/
  argc=0; argv=0;
  return 0;
}

Merci par avance pour votre aide.

[Bousk]

Salut,

tu as oublié le constructeur par copie de extintarray.
Il te manque aussi les opérateurs= de chacune des classes.

[hbx360]

Merci pour ta réponse.

C'est quoi les opérateurs = de chacune des class ou se positionne t-il ?

[Bousk]

http://cpp.developpez.com/faq/cpp/?p...-rule-of-three

[hbx360]

ok merci

[hbx360]

Et comment sa ce concrétise dans mon code ??
Je rapels je débute à peine ou dois-je mettre ce fameux opérateur dans la class ou dans la class héritage ??

Merci par avance.

[Bousk]

Ben..

les opérateurs= de chacune des classes.

les 2.

Cela dit, ceci est juste mauvais mais pas la source réelle du problème. Juste quelque chose qui te pètera à la tronche plus tard si c'est pas fait.
Mais tu libères la mémoire de ton tableau dans le destructeur de ta classe dérivée.
En fait tu fais n'importe quoi dans ta classe dérivée, elle ne devrait pas manipuler de la sorte le contenu de la classe de base.
Et je parle pas des problèmes de const-correctness ou des fuites mémoires que tu as déjà.

[hbx360]

Merci pour ta réponse en faite je suis les cours de CodeurPlusPlus sur Youtube donc je fais la même chose que lui.
Comme je débute et que je ne connais pas le C++, je suis bien obligé de faire comme je vois.

[hbx360]

Mais tu libères la mémoire de ton tableau dans le destructeur de ta classe dérivée.

OK merci à toi sa venait de ça j'ai supprimé le destructeur de ma class héritage et sa compile.

Merci beaucoup pour ton aide.

Cela dit, ceci est juste mauvais mais pas la source réelle du problème. Juste quelque chose qui te pètera à la tronche plus tard si c'est pas fait.

Désolé je ne comprends pas cette phrase vue que je débute.

En fait tu fais n'importe quoi dans ta classe dérivée, elle ne devrait pas manipuler de la sorte le contenu de la classe de base.

En faite c'est pas moi qui ai écrit ce code c'est le code de CodeurPlusPlus que j'ai recopié au fure et a mesure de ces cours vidéo d'apprentissage du C++ qu'il fait sur youtube.
Ok désolé encore mais je n'ai pas assez de recule pour bien comprendre ce que tu dit et ce que cela implique.

Et je parle pas des problèmes de const-correctness

ou des fuites mémoires que tu as déjà.

Désolé encore je débute en C++ donc pour les fuite de mémoire je sais pas je comprends pas si tu me parle du C les fuite de mémoire je comprends y pas de problème mais en C++ je sais pas.

[Bousk]

OK merci à toi sa venait de ça j'ai supprimé le destructeur de ma class héritage et sa compile.

Vu que tu n'as rien de spécial à faire en détruisant un ext, le destructeur par défaut suffira oui.
Btw, ton destructeur devrait être virtuel, ça va le plus souvent de pair avec l'héritage.

Désolé je ne comprends pas cette phrase vue que je débute.

Ca veut dire texto ce que c'est écrit : là tout de suite c'est pas (vraiment) grave, mais plus tard en manipulant tes objets tu auras les mêmes problèmes de core dump.
Et plus tard serait maintenant si ton compilo n'optimise pas cette initialisation par un simple appel au constructeur mais utilise l'opérateur=.

En faite c'est pas moi qui ai écrit ce code c'est le code de CodeurPlusPlus que j'ai recopié au fure et a mesure de ces cours vidéo d'apprentissage du C++ qu'il fait sur youtube.

Entre nous, si c'est vraiment ça qui est écrit dans son cours, vaudrait mieux en trouver un autre.
Un cours qui montre un core dumped, ça fait pas rêver

const-correctness : http://cpp.developpez.com/faq/cpp/?p...apres-leur-nom

Désolé encore je débute en C++ donc pour les fuite de mémoire je sais pas je comprends pas si tu me parle du C les fuite de mémoire je comprends y pas de problème mais en C++ je sais pas.

Que tu sois en C, C++, ou peu importe, une fuite mémoire c'est toujours une fuite mémoire.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
extintarray::extintarray (int alloc)
{
  this->len = 0;
  this->alloc = alloc;
  this->data = new int[alloc];
}
 
extintarray::extintarray (int len, int alloc)
{
  this->len = len;
  this->alloc = alloc;
  this->data = new int[alloc];
}

L'appel au constructeur par défaut de intarray est implicite. Un breakpoint ou trace te l'assurera.
Dans ce constructeur par défaut tu alloues déjà data, puis ici tu le réalloues, sans libérer le précédent : fuite.

[hbx360]

Merci d'avoir pris le temps de répondre.
Je comprends mieux et pour la fuite de mémoire aussi c'est plus claire.

Entre nous, si c'est vraiment ça qui est écrit dans son cours, vaudrait mieux en trouver un autre.
Un cours qui montre un core dumped, ça fait pas rêver

En faite dans son cours chez lui sa compile c'est ça que je trouve bizarre.

[Bousk]

Un core dumped c'est une erreur d'exécution, pas de compilation.

[hbx360]

Ha OK merci pour la l'info