Discussion :

[atum25]

Bonjour a tous,

Voila j'ai un tableau de string, contenant des numéros de ports.

Je cherche a prendre a chaque numéro, et à rechercher si ce numéro existe déja dans mon tableau, car les numéros de ports qui reviennent.

J'utilise la fonction find() mais cela ne fonctionne pas .

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 AnsiString bj = TabPort[y].c_str();
                if(bj !=' ')
                {
                        IndexPort = find(TabPort[256].begin(),TabPort[256].end(),bj);
                        if(IndexPort)
                        {
                                cout<<"trouver";
                        }
                        else
                        {
                                cout<<"Non trouver";
                        }

[Neckara]

RTFM

http://www.cplusplus.com/reference/algorithm/find/

find ne renvois pas un booléen.

find renvois : un itérateur sur l'élément trouvé ou un itérateur sur le dernier élément + 1 (= end() )

Tu dois donc faire :

Code :

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

1
2
3
4
5
 
if(IndexPort != TabPort[256].end() )
          //trouvé
else
           //non trouvé

Ici TabPort n'est pas un conteneur donc tu dois faire :

Code :

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

1
2
3
4
5
 
if(IndexPort != TabPort+256 )
          //trouvé
else
           //non trouvé

Ensuite que veux-tu faire exactement? Ta méthode ne me semble pas très adaptée.

Ensuite, je ne pense que tu veux chercher ta chaîne de caractère parmi l'ensemble des strings contenu dans ton tableau :

Code :

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

1
2
 
IndexPort = find(TabPort,TabPort+256,bj);

Et met bien comment tu défini les variables que tu utilises.
TabPort est-il bien un tableau de string ou un vector de string?

[atum25]

En gros, je veux vérifier qu'un élement du tableau n'apparait qu'une fois dans le tableau.

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
 
        int cpt = TabPort[256].size();
 
        for(int y=0;y<=cpt;y++)
        {
                string::iterator IndexPort;
                string::iterator Debut = TabPort[256].begin();
 
                AnsiString bj = TabPort[y].c_str();
                if(bj !=' ')
                {
                        IndexPort = find(Debut,TabPort[256].end(),bj);
                        if(IndexPort!=TabPort[256].end())
                        {
                                cout<<"trouver";
                        }
                        else
                        {
                                cout<<"Non trouver";
                        }

[gbdivers]

Vu avec lui sur le chat, gros problème de compréhension de ce qu'est un itérateur et algo pas correct
En cours de correction

[gbdivers]

Pour ceux qui poserait la même question, voici le code corrigé et compilable sur des données de test

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
#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
 
using namespace std;
 
int main()
{
    const char* init[] =  { "1", "2", "3", "2", "4", "3" };
    vector<string> TabPort(init, init+6);
 
    vector<string>::iterator it;
 
    vector<string>::iterator start;
 
    start = TabPort.begin();
    for (it = TabPort.begin();it != TabPort.end();++it)
    {
        ++start;
        // itérateur faisant référence au string trouvé ou à end()
        vector<string>::iterator trouve;
 
        trouve = find (start, // on commence la recherche à partir de start
                       TabPort.end(), // on arrête à la fin du tableau
                       *it); // on cherche la string à laquelle fait référence it
 
        if (trouve != TabPort.end())
        {
            cout << *it << " trouvé:" << *trouve << endl;
        }
        else
        {
            cout << *it << " non trouvé:" << endl;
        }
    }
}

[oodini]

Si les éléments du tableau sont des nombres, c'est une hérésie de les stocker et de les rechercher sous forme de string.

Utilie un std::set<unsigned int>.