Discussion :

[tdumas]

Bonjour,

Je travaille depuis quelque temps sur un projet en c++, j'arrive en général à résoudre mes erreurs mais là je suis vraiment bloqué.

Voici mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
File* SpeedHandler::findFile(int nodeFrom)
{
	File* file = NULL;
	for (auto i = 0; i < m_lFiles->size(); i++) {
		if (m_lFiles->at(i).getNodeFrom() == nodeFrom) {
			file = &m_lFiles->at(i);
			break;
		}
	}
	return file;
}

File* est l'une de mes classes ainsi que SpeedHandler, et m_lFiles est un vector. Je travaille avec Visual Studio qui me relève une erreur de type std_out_of_range.

Si quelqu'un a une idée d'où pourrait venir le problème...

[Guesset]

Bonjour,

Peut être que remplacer file = &m_lFiles->at(i); par file = &(m_lFiles->at(i)); améliorerait la situation ?

Salutations

[koala01]

Salut, et bienvenue sur le forum

Déjà, tu ne devrais pas utiliser la fonction membre at() de vector, car c'est une monstruosité conceptionnelle : elle lance une exception alors que nous avons affaire à une erreur de logique de la part du développeur, et qu'une assertion serait largement préférable

Ensuite, en C++, nous utiliserons de préférence nullptr au lieu de NULL.

En outre, je sais que bon nombre de mes confrères ne sont pas d'accord sur ce point, mais un break au milieu d'une boucle, cela ne a jamais inspiré confiance, car je trouve que cela fait "quick fix" : on se rend compte "après coup" que l'on a oublié de prendre une condition en compte pour la sortie de la boucle.

C'est d'autant plus vrai que tu pourrais facilement le remplacer par l'instruction return file;, ou par une boucle while(i < m_file.size() && file ==nullptr), et que ces deux possibilités fourniraient le même résultat au final

Enfin, tu pourrais assez facilement remplacer ta boucle par une boucle basée sur les intervalles:

Code :

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

1
2
3
4
5
6
7
8
File* SpeedHandler::findFile(int nodeFrom)
{
    for(File & file : m_lFiles){
        if(file.getNodeFrom() == nodeFrom)
           return &file;
    }
    return nullptr;
}

C'est plus concis, et cela évite pas mal d'erreur