Discussion :

[Garrisonn]

Bonsoir,

Le but de mon programme est de mettre à zéro toutes les cases de ma matrice, mon programme s'exécute mais la matrice ne s'affiche pas dans mon terminal.

Voici mon main.cpp :

Code :

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

1
2
3
4
5
6
7
8
9
#include <iostream>
#include "graphe.h"
using namespace std;
 
int main()
{
	graphe g;
	init_graphe(g);
}

Voici mon fichier.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
#ifndef GRAPHE
#define GRAPHE
 
typedef int mat[52][52];
 
struct arc_arete
{
	char initial; //sommet initial	
	char finale; //sommet final
	float poids; //cout de l'arc ou arete
};
 
struct graphe
{
	int nature_du_graphe; //valeur 0 si graphe orienté et valeur 1 sinon
	int nb_sommets; //nb de sommets du graphe
	int nbarcs; //nb d'arc du graphe
	char liste_sommets[52]; //liste des sommets du graphe
	arc_arete arcs[100]; //liste des arcs ou aretes du graphe (100 max)
	mat matriceadjacence;
};
 
void init_graphe(graphe &g);
 
#endif

Voici ma fonction.cpp

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
#include <iostream>
#include "graphe.h"
using namespace std;
 
void init_graphe(graphe &g)
{
	int i,j;
	for (i=0; i<g.nb_sommets; i++)
	{
		for (j=0; j<g.nb_sommets; j++)
		{
			g.matriceadjacence[i][j]=0;
			cout<<g.matriceadjacence[i][j]<<endl;
		}
 
	}
 
}

Merci d'avance de me dire ce qui cloche, j'ai du mal à discerner mes erreurs. Merci.
Bonne soirée.

[bacelar]

Merci d'avance de me dire ce qui cloche

A peu près tout, ça fait au moins 20 ans qu'on ne code plus comme ça en C++.
Votre code, c'est du C avec des flux.

j'ai du mal à discerner mes erreurs

Normal, vous devez avoir un professeur fossile ou qui a obtenu son badge de professeur de C++ en une beuverie d'université en échange d'un badge de professeur de C.

Code :

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

typedef int mat[52][52];

C'est quoi ce typedef, WTF ???

Qui initialise "nb_sommets" ?

Les constructeurs, c'est pas pour les chiens, donc cette "init_graphe" est une aberration.

Si vous voulez faire du C++, utilisez des std::array à la place de ces tableaux C-styles (complètement inutile et casse-gueule en C++).

Pensez à poster votre code avec la base /[CODE/] (le bouton [#]).

Pensez aussi à indenter correctement votre code.

Et lisez un livre sur le C++11 ou le C++14, pas un vieux truc qui traine dans les bibliothèques universitaires depuis 20 ans.

[Garrisonn]

Merci pour tout vos conseils, je vais tâcher de mettre tout ça au propre.

[Invité]

Normal, vous devez avoir un professeur fossile ou qui a obtenu son badge de professeur de C++ en une beuverie d'université en échange d'un badge de professeur de C.

Bravo, avec des remarques comme çà il va tout de suite mieux comprendre le C++.
Et tu as oublié de dire que tous ces méchants professeurs d'université étaient aussi responsables des attentats terroristes et du réchauffement climatique...

@Garrisonn: tu utilises nb_sommets dans init_graphe mais il faut que tu l'initialises avant. Et c'est vrai que ton code est horrible et que tu devrais regarder un bouquin de C++ moderne...

[stendhal666]

@Garrisonn: console-toi en te disant qu'on apprend plus de ses erreurs que de ses réussites; maintenant tu sais ce qu'il en coûte de ne pas suivre les principes du C++ et surtout l'un des plus importants:
Ressource acquisition is initialization (RAII) ou en français: quand on acquiert une ressource, on l'initialise. Tu as réservé une ressource (en l'occurrence un int) sans l'initialiser et tu t'es fait mal. Pour faire face à ce problème, le C++ te donne un outil puissant: les objets et leurs constructeurs.

Pour un objet défini avec struct ou class, tu peux définir un constructeur qui en initialise les composants. Pour un exemple simple:

Code :

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

1
2
3
4
5
struct Point {
  int _x, _y;
  Point() : _x(0), _y(0) {} // un constructeur par défaut qui met x et y à 0
  Point(int x, int y) : _x(x), _y(y) {} // un constructeur pour déterminer les coordonnées du point
};

Pour reprendre ton exemple en lui donnant un tour plus C++ (mais il reste du travail à faire):

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
// C++
struct Graphe
{
	bool est_oriente; //valeur 1 si graphe orienté et valeur 0 sinon --> modif pour être plus expressif, aucun lien avec le C++
	int nb_sommets; //nb de sommets du graphe
	int nbarcs; //nb d'arc du graphe
 
        std::vector<char> liste_sommets; //utilise des vecteurs pour t'éviter des limites de taille artificielles et les parcourir aisément
	std::vector<arc_aretes> arcs; // idem
	std::vector<std::vector<int>> matrice; // et en prime tu t'évites les typedef 
 
        Graphe() : est_oriente(true), nb_sommets(8), nbarcs(7), matrice(std::vector<std::vector<int>>(8, std::vector<int>(8, 0))) {} // à moins que tu n'aies d'autres valeurs par défaut évidemment
                                                                                       //  l'initialisation à 0 est faite juste au-dessus ^
 
};

[ternel]

par contre, tu as tout intérêt à ne pas faire de vector de vector.
au pire, un array de array, et au mieux un vector ou un array.

Regarde dans la faq les deux entrées sur les matrices:
pourquoi et comment coder correctement une matrice.