Discussion :

[Niqolas]

Salut à tous !
Je suis entraine d'écrire un programme qui pourrait m'interroger sur du vocabulaire que j'ai précédemment rentré. Cependant je suis encore débutant, mon programme n'a pas l'air très stable et je ne comprend pas mes erreurs
Je précise avant de vous donner le code que le programme doit-être capable de recevoir du vocabulaire, de le sauvegarder et à l'avenir de mémoriser le nombre d'interrogation sur un mot ainsi que son nombre d'erreurs

Main :

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
int main(int argc, char** argv)
{
	char answer = 0;
	cout << " *** Bonjour ! " << endl;
	vector<Words*> list;
	do
	{
		cout << " *** Ecriver votre choix :" << endl;
		cout << " * 1  = Ajouter du vocabulaire" << endl;
		cout << " * 2  = S'entrainer" << endl;
		cout << " * 0  = Quitter" << endl;
		cout << " * 3  = tester la liste" << endl;
 
		cout << "Reponse : ";
		cin >> answer;
		if (answer == '1') list = new_vocabulary();
		if (answer == '2')test_vocabulary();
		if (answer == '3')test_list(list);
	} while (answer != '0' && answer != NULL && answer != ' ');
 
	for (int i = 0; i <= list.size(); i++)
	{
		delete list[i];
		list[i] = 0;
	}
}

New vocabulary :

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
 
vector<Words*> new_vocabulary()
{
	const string link("vocab.txt");
	vector<Words*> list = update_list();
	ofstream vocab_file;
	vocab_file.open(link.c_str(), ios::app);
	string engWord, frWord;
 
	cin.ignore();
 
	do
	{
		cout << " * Donner le mot en anglais : " << endl;
		getline(cin, engWord);
		if (engWord != "")
		{
			cout << " * Donner le mot en français : " << endl;
			getline(cin, frWord);
 
			cout << " * Votre mot est : " << engWord << endl;
			cout << " * Qui se traduit par : " << frWord << endl << endl;
 
			vocab_file << engWord << " | " << frWord << endl; 
 
		}
	} while (engWord != "");
	vocab_file.close();
	return list;
}
 
vector<Words*> update_list()
{
	ifstream vocab_file;
	vocab_file.open("vocab.txt");
	vector<Words*> list;
	string line, frWord, engWord, nextWord;
	int pos = 0;
	while (getline(vocab_file, line))
	{
		if (line.find(" | ") != string::npos)
		{
			pos = line.find(" | ");
			for (int i = 0; i < pos; i++) 
			{
				engWord += line[i];
			}
			for (int i = pos + 3; i < line.length(); i++)
			{
				frWord += line[i];
			}
			cout << "Mot anglais : " << engWord << endl;
			cout << "Mot français : " << frWord << endl << endl;
			list.push_back(new Words(engWord, frWord));
		}
		else cout << "Erreur séparation introuvable" << endl;
		frWord = "";
		engWord = "";
	}
	vocab_file.close();
	return list;
}

et test vocabulary est encore vide.
Je remercie d'avance ceux qui prendront le temps de se pencher sur mon code, qui je pense doit-être remplie d'erreurs

[LittleWhite]

Bonjour,

Lorsque je vois un vecteur de pointeur (ici std::vector<Words*>), je me dis que l'erreur est sûrement lié au fait que vous utilisez des pointeurs dans un vecteur. Pour simplifier le code de votre programme, faites simplement un std::vector<Words>. Ou si vous souhaitez absolument utiliser des pointeurs, alors utilisez des pointeurs "intelligents" (std::shared_ptr/std::unique_ptr).
Ensuite, si vous avez un plantage/crash, votre meilleur ami sera le debuggeur.
Le nom de la fonction new_vocabulary() est trompeur. Dedans vous faites un chargement de fichier (au travers de update_vocabulary() (nom aussi trompeur), une entrée utilisateur et une sauvegarde dans un fichier.

Accessoirement et afin de mieux pouvoir vous aider, je me rends compte que vous n'indiquez pas les erreurs (ni observation) que vous obtenez. Est-ce que ce sont des erreurs de compilation, ou des plantages ? Si ce sont des plantages, ils arrivent à quels moment (après quel suite d'opérations).

[dalfab]

Bonjour,

Il y a de nombreux problèmes dans ton code, donc tu poses finalement de nombreuses questions.
Quelques cas que j'ai vu, liste non exhaustive.

Erreurs de syntaxe:
- les types de pos et i ne sont pas les bons. Il faut utiliser size_t. Le compilateur devrait avoir émis des warnings indiquant cela.

Problèmes d'exécution:
- main/ligne 21. Tu parcours un élément de trop dans liste, c'est un comportement indéfini qui devrait planter.
- main/ligne 16. Cette ligne va écraser la valeur précédente de list, d'où va perdre irrémédiablement de la mémoire allouée dynamiquement.

Problèmes sémantiques:
- Pourquoi mettre des pointeurs dans list. Gérer directement les allocations dynamiques c'est garantir d'avoir des tas de comportement indéfinis (cf pb précédent). En plus ici, un std::vector<Word> est tout à fait utilisable.
- les variables doivent être définies au plus près de leur utilisation. Cela t'éviterait de créer des lignes inutiles telles que NewVocabulary/lignes 57 et 58 qui mettent à zéro des variables qui ne servent plus à rien.
- la fonction new_vocabulary() lit list, mais n'y ajoute pas le nouveau vocabulaire!
- il faut placer le cin.ignore() proche du pourquoi tu fais ça (c'est du à la lecture d'un char et tu le fais dans une autre fonction!)

Problèmes fonctionnels:
- tu recharges list à chaque appel de new_vocabulary(). Il faut la charger au lancement de l'application et la compléter à chaque appel de new_vocabulary().
- la sauvegarde au fur et mesure des ajouts est une possibilité, c'est utilisable mais cela rend la modification quasi-impossible.

Une proposition de code:

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
#include <iostream>  // for std::cin std::cout
#include <string>    // for std::string
#include <vector>    // for std::vector<>
#include <fstream>   // for std::ifstream std::ofstream
#include <utility>   // for std::move()
#include <limits>    // for std::numeric_limits<>
 
void  test_vocabulary( std::vector<Words>const& list );
void  test_list( std::vector<Words>const& list );
 
std::vector<Words>  new_vocabulary( std::vector<Words> list ) {
	std::ofstream  vocab_file( "vocab.txt",  std::ios::app );
 
	std::string  engWord;
	do {
		std::cout << " * Donner le mot en anglais : " << std::flush;
		std::getline( std::cin, engWord );
		if ( engWord != "" ) {
			std::cout << " * Donner le mot en français : " << std::flush;
			std::string  frWord;
			std::getline( std::cin, frWord );
 
			std::cout << " * Votre mot est : " << engWord
					<< "\n * Qui se traduit par : " << frWord << '\n' << std::endl;
 
			vocab_file << engWord << " | " << frWord << std::endl;
 
			list.emplace_back( engWord, frWord );
		}
	} while ( !engWord.empty() );
	return  list;
}
 
std::vector<Words>  update_list() {
	std::ifstream  vocab_file( "vocab.txt" );
	std::vector<Words>  list;
	std::string  line;
	while ( std::getline( vocab_file, line ) ) {
		size_t  pos = 0;
		if ( ( pos = line.find( " | " ) ) != std::string::npos ) {
			std::string  engWord = line.substr( 0, pos );
			std::string  frWord = line.substr( pos+3 );
			list.emplace_back( engWord, frWord );
		}
		else {
			std::cerr << "Erreur séparation introuvable" << std::endl;
			break;
		}
	}
	return  list;
}
 
int  main() {
	char  answer;
	std::cout << " *** Bonjour !" << std::endl;
	std::vector<Words>  list = update_list();
	do {
		std::cout << " *** Écrivez votre choix :\n"
				" * 1  = Ajouter du vocabulaire\n"
				" * 2  = S'entraîner\n"
				" * 0  = Quitter\n"
				" * 3  = tester la liste\n";
 
		std::cout << "Réponse : " << std::flush;
		std::cin >> answer;
		std::cin.ignore( std::numeric_limits<std::streamsize>::max() );  // sauter le \n après le caractère entré
 
		if ( answer == '1' )
			list = new_vocabulary( std::move(list) );
		if ( answer == '2' )
			test_vocabulary( list );
		if ( answer == '3' )
			test_list( list );
	} while ( answer != '0'  &&  answer != ' ' );
}

[Niqolas]

Bonjour,
J'ai pu faire pas mal de modifications sur le code et continuer à avancer grâce à vos conseils.
J'ai supprimé la liste de pointeurs pour en faire une liste simple, mais le problème est que lorsque je modifie la liste dans une fonction, cette modification n'est pas prise en compte par les autres. Le seul moyen que vois sans utiliser de pointeurs est de faire un return de la liste modifiée à chaque modifications, cependant cela ne me parait pas optimal.

Si vous avez d'autres conseils je suis preneur

Main.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
19
20
21
22
23
int main(int argc, char** argv)
{
	char answer = 0;
	const string link = "vocab.txt";
	cout << " *** Bonjour ! " << endl;
	vector<Words> list = load_list(link);
	do
	{
		cout << " *** Ecriver votre choix :" << endl;
		cout << " * 1  = Ajouter du vocabulaire" << endl;
		cout << " * 2  = S'entrainer" << endl;
		cout << " * 0  = Quitter" << endl;
		cout << " * 3  = tester la liste" << endl;
 
		cout << "Reponse : ";
		cin >> answer;
		cin.ignore();
		if (answer == '1')new_vocabulary(list, link);
		if (answer == '2')test_vocabulary(list);
		if (answer == '3')test_list(list);
	} while (answer != '0' && answer != NULL && answer != ' ');
	save_vocab(list, link);
}

New vocabulary.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
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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
#include "new_vocabulary.hpp"
 
using namespace std;
 
vector<Words> new_vocabulary(vector<Words> list, const string link)
{
	ofstream file;
	file.open(link.c_str(), ios::app);
	string engWord, frWord;
 
 
	do
	{
		cout << " * Donner le mot en anglais : " << endl;
		getline(cin, engWord);
		if (engWord != "")
		{
			cout << " * Donner le mot en français : " << endl;
			getline(cin, frWord);
 
			cout << " * Votre mot est : " << engWord << endl;
			cout << " * Qui se traduit par : " << frWord << endl << endl;
 
			file << engWord << " | " << frWord << " | 0 | 0" << endl;
			list.push_back(Words(engWord, frWord));
 
		}
	} while (engWord != "");
	return list;
}
 
vector<Words> load_list(const string link)
{
	ifstream file;
	file.open(link);
 
	vector<Words> list;
	string line, frWord, engWord, nextWord;
	int fpos, spos, tpos;
	string err, pass;
 
	while (getline(file, line))
	{
		if (line.find(" | ") != string::npos)
		{
			fpos = line.find(" | ");
 
		}
		if (line.find(" | ", fpos + 1.6) != string::npos)
		{
			spos = line.find(" | ", fpos+1.6);
		}
		if (line.find(" | ", spos + 1.6) != string::npos)
		{
			tpos = line.find(" | ", spos + 1.6);
		}
 
		for (int i = 0; i < fpos; i++) 
		{
			engWord += line[i];
		}
		for (int i = fpos + 3; i < spos; i++)
		{
			frWord += line[i];
		}
		for (int i = spos + 3; i < tpos; i++)
		{
			pass += line[i];
		}
		for (int i = tpos + 3; i < line.size(); i++)
		{
			err += line[i];
		}
 
		/*
		cout << "Mot anglais : " << engWord << endl;
		cout << "Mot français : " << frWord << endl;
		cout << "Nombre de passages : " << pass << endl;
		cout << "Nombre d'erreurs : " << err << endl << endl;
		*/
		list.push_back(Words(engWord, frWord, stoi(pass), stoi(err), fpos, spos, tpos));
 
		frWord = "";
		engWord = "";
		err = "";
		pass = "";
	}
 
	return list;
}
 
void save_vocab(vector<Words> list, const string link)
{
	ofstream file;
	file.open(link.c_str());
	for (int i = 0; i < list.size(); i++)
	{
		file << list[i].getWord() << " | " << list[i].getTransl() << " | " << list[i].getNbrPass() << " | " << list[i].getNbrErr() << endl;
	}
}

Test vocabulary.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
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
79
80
81
82
83
84
85
86
87
88
89
90
#include "test_vocabulary.hpp"
using namespace std;
 
void test_vocabulary(vector<Words> list)
{
	system("cls");
	string answer, transl;
	Words word = sort_list(list);
	transl = word.getTransl();
	cout << "Donne moi la definition de  " << endl;
	cout << word.getWord() << endl;
	cout << "Reponse : ";
	getline(cin, answer);
	if (answer == transl)
	{
		cout << "Bien joue mon gars ! " << endl << endl;
	}
	else
	{
		cout << "Frerot t'abuse ... C'etait " << transl << endl<< endl;
	}
}
 
void test_list(vector<Words> list)
{
	for (int i = 0; i < list.size(); i++)
	{
		cout << "==================================================|"<< endl;
		cout << "Liste numero " << i << endl;
		cout << " * Mot : " << list[i].getWord() << endl;
		cout << " * Trad : " << list[i].getTransl() << endl;
		cout << " * Nombre de passages : " << list[i].getNbrPass() << endl;
		cout << " * Nombre d'erreurs : " << list[i].getNbrErr() << endl;
		cout << " * Pourcentage d'erreurs : " << list[i].getPercErr() << "%" << endl << endl;
 
	}
}
Words sort_list(vector<Words> list)
{
	int minPassSave = 10000;
	int maxErrSave = 0;
	vector<int> nbrPass, minPass;
	vector<int> nbrErr, maxErr;
	vector<Words> word_to_return;
 
	/*Stock les mots les moins interrogés*/
	for (int i = 0; i < list.size(); i++)
	{
		nbrPass.push_back(list[i].getNbrPass());
		if (nbrPass[i] <= minPassSave) // Récupère la valeur pass la plus faible
		{
			minPassSave = nbrPass[i] ;
		}
	}
	for (int i = 0; i < list.size(); i++)
	{
		if (nbrPass[i] == minPassSave) // Stock tous les index des mots valant le pass min
		{
			minPass.push_back(i);
		}
	}
	/*Cherche le mot avec le plus d'erreurs*/
	for (int i = 0; i < minPass.size(); i++)
	{
		nbrErr.push_back(list[minPass[i]].getNbrErr());
		if (nbrErr[i] >= maxErrSave) // Récupère la valeur err la plus forte
		{
			maxErrSave = nbrErr[i];
		}
	}
	cout << " * Le plus grand nombre d'erreurs est " << maxErrSave << " fautes" << endl;
	for (int i = 0; i < minPass.size(); i++)
	{
		if (list[minPass[i]].getNbrErr() >= maxErrSave)
		{
			word_to_return.push_back(list[minPass[i]]);
			cout << " * Le mot " << list[minPass[i]].getWord() << " : " << endl;
			cout << list[minPass[i]].getNbrErr() << " faute(s)" << endl << endl;
		}	
	}
	int randomNumber = rand_nbr(0, maxErr.size());
	return word_to_return[randomNumber];
}
 
int rand_nbr(int a, int b)
{
	int nRand;
	nRand = a + (int)((float)rand() * (b - a + 1) / (RAND_MAX - 1));
	return nRand;
}

[bacelar]

Pour les paramètre modifiable, les passer par référence, pas par valeur.

[Niqolas]

Pour les paramètre modifiable, les passer par référence, pas par valeur.

Dans mon cas le passage par référence fonctionne pour la fonction new_vocabulary() mais je ne parviens pas à l'appliquer a test_vocabulary()

[bacelar]

Code source + message d'erreur, SVP.

[Niqolas]

Code source + message d'erreur, SVP.

Aucun message d'erreur, simplement lorsque je teste le mot avec test_vocabulary() la modification (à savoir word.addPass() et/ou word.addErr() ) ne s'applique pas dans les autres fonctions.

test_vocabulary() :

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
void test_vocabulary(vector<Words> &list)
{ // Demande un mot en fonction du nombre d'erreurs et de passages
	system("cls");
	string answer, transl;
	Words word = list[sort_list(list)];
	word.addPass();
	transl = word.getTransl();
	cout << "Donne moi la definition de  " << endl;
	cout << word.getWord() << endl;
	cout << "Reponse : ";
	getline(cin, answer);
	if (answer == transl)
	{
		cout << "Bien joue mon gars ! " << endl;
	}
	else
	{
		cout << "Frerot t'abuse ... C'etait " << transl << endl;
		word.addErr();
	}
	cout << "Ce mot est a " << word.getNbrErr() << " erreurs  pour "  << word.getNbrPass() << " passages" << endl << endl;
}

sort_list() :

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
int sort_list(vector<Words> list)
{ //Rechecrhe le mot avec le plus d'erreurs pour le moins de passages et retourne son index dans list
	int minPassSave = 10000;
	int maxErrSave = 0;
	vector<int> nbrPass, minPass;
	vector<int> nbrErr, maxErr;
	vector<int> word_to_return;
 
	/*Stock les mots les moins interrogés*/
	for (int i = 0; i < list.size(); i++)
	{
		nbrPass.push_back(list[i].getNbrPass());
		if (nbrPass[i] <= minPassSave) // Récupère la valeur pass la plus faible
		{
			minPassSave = nbrPass[i] ;
		}
	}
	for (int i = 0; i < list.size(); i++)
	{
		if (nbrPass[i] == minPassSave) // Stock tous les index des mots valant le pass min
		{
			minPass.push_back(i);
		}
	}
	/*Cherche le mot avec le plus d'erreurs*/
	for (int i = 0; i < minPass.size(); i++)
	{
		nbrErr.push_back(list[minPass[i]].getNbrErr());
		if (nbrErr[i] >= maxErrSave) // Récupère la valeur err la plus forte
		{
			maxErrSave = nbrErr[i];
		}
	}
	cout << " * Le plus grand nombre d'erreurs est " << maxErrSave << " fautes" << endl;
	for (int i = 0; i < minPass.size(); i++)
	{
		if (list[minPass[i]].getNbrErr() >= maxErrSave)
		{
			word_to_return.push_back(minPass[i]);
			cout << " * Le mot " << list[minPass[i]].getWord() << " : " << endl;
			cout << list[minPass[i]].getNbrErr() << " faute(s)" << endl << endl;
		}	
	}
	int randomNumber = rand_nbr(2, maxErr.size()+2);
	randomNumber-2;
	cout << "Tu me donnera le mot numero " << randomNumber << endl;
	return word_to_return[randomNumber];
}

[bacelar]

ligne 5 de "test_vocabulary", word est un Word, pas une référence sur un Word, donc word contient une copie de "list[sort_list(list)]", etc...

Pour la fonction "sort_list", la paramètre "list" n'est pas modifié, il serait judicieux de le passer en "référence const"

Révisez les bases de l'usage des références.