problème : attention : comparaison entre des expressions entières signées et non signées

Version imprimable

Bonjour,

voici ma fonction :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
__device__ __host__ inline float cost(string* algSeq, int s1, int s2){
	float score=0;
	string seq1=algSeq[s1];
	string seq2=algSeq[s2];
	int gap=0;
	//cout<<"s1 ; "<<s1<<endl;
	//cout<<"s2 : "<<s2<<endl;
	for(int i=0; i<strlen(const_cast<char*>(seq1.c_str())) || i<strlen(const_cast<char*>(seq2.c_str())); i++){
		if(seq1[i]!='-' && seq2[i]!='-'){
			score+=costMatrix[i][i];//mettre indice correspondant à la lettre
		}
		else{
			gap++;
		}
	}
	return score;
 
}

et donc j'ai le warning suivant : attention : comparaison entre des expressions entières signée et non signée!

je vois pas comment réglé ce problème.

Alaninho

04/10/2012, 14h29
ternel

je crois que strlen renvoie un unsigned int, il suffirait donc de changer i en unsigned int.
04/10/2012, 14h30
cob59

C'est extrêmement rigolo ce strlen(const_cast<char*>(seq1.c_str())).
Tu sais qu'il existe std::string::size() ? :mrgreen:
04/10/2012, 14h34
alaninho

Merci, pour vos réponse rapide, et je sais pas pourquoi j'utilisais pas str.size() qui est bien plus pratique.lol.

encore meric.

Alaninho
04/10/2012, 15h24
fenkys

strlen renvoie un size_t.

Sur un système 32 bits , size_t et unsigned int sont équivalents. Sur un système 64 bits ce n'est pas toujours vrai. Il faut mieux utiliser les bons types plutôt que des interprétations douteuses.