Discussion :

[tir0nik]

Bonjour,

J'ai un problème avec une fonction qui récupère des données sur un socket et qui fait une boucle. Tout marche parfaitement quand il y a std::cout (ou printf) dans la boucle mais ça plante une fois sur 2 quand il n'y est pas :s. Je n'ai pourtant aucun warning. Qu'est ce qui peut déclencher ceci ? 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
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
#include <iostream>
using namespace std;
#define order 8
void printall(int arr[]);
bool validity_check(int,int,int []);
int main()
{
        int nextjump[order];//Keeps track for every order what should be the pos. of next jump.
        for(int i=0;i<order;i++) nextjump[i]=0;
        int level=0;
        int position[order];
        mainloop:
        while(true)
        {
                //cout<<"Entered the while loop"<<endl; //This is the line which can change the whole output when uncommented
                for(int i=nextjump[level];i<order;i++)
                {
                        if(validity_check(level,i,position))
                        {
                                cout<<"Successful at level "<<level<<" using position "<<i<<endl;
                                position[level]=i;
                                nextjump[level++]=i+1;


                                if(level==order)
                                {

                                        printall(position);
                                        goto mainloop;
                                }

                                nextjump[level]=0;
                                goto mainloop;
                        }
                }

                level-- ;
                goto mainloop;
        }
        return 0;
}

bool validity_check(int level,int n, int current[])
{
        for(int i=0;i<level;i++)
        {
                if(current[i]==n) return false;
        }
        //Level 0 has always full validity.
        if((level>0)&&(level<order-1))
        {
                if((current[level-1]==n)|(current[level+1]==n)) return false;
        }
        if(level==order-1)
        {
                if(current[level-1]==n) return false;
        }

}

void printall(int current[])
{
        for(int i=0;i<order;i++)
        {
                cout<<current[i]<<endl;
        }
}

Merci

[Melem]

cout n'est pas vraiment le responsable. Dans ta fonction bool validity_check(int level,int n, int current[]), il y a juste des cas qui ne sont pas traités. Lorsqu'un de ces cas se produit, le code atteint la fin de la fonction or la fonction, dans un tel cas, ne spécifie pas de valeur de retour donc sa valeur de retour sera à priori imprévisible. Sur une architecture x86 32 bits, un return <valeur booléenne ou entière>; est traduit en :

Code :

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

1
2
mov eax, <la valeur à retourner>
ret

Si t'as bien compris, la valeur de retour d'une fonction, c'est juste la valeur courante de eax. Qui sait ce qu'il peut bien y avoir là-dedans si tu ne spécifies rien ...

[Lavock]

Dans ta fonction bool validity_check(int level,int n, int current[]), il y a juste des cas qui ne sont pas traités.

Si tel était le cas, il est pas censé avoir un warning ? Et puis, pourquoi le cout provoquerez un changement de comportement ?


Bon, je connais pas ton socket, ni comment les données arrive dessus, mais il se peut que se soit un problème de synchro. Essaye de remplacer le cout par une mini attente (genre 10µs), voir si tu as le même résultat. Si oui, va falloir que tu creuse la synchro.

[tir0nik]

Merci pour vos réponses.

Effectivement, quand je met un Sleep(10) juste après ma fonction recv tout marche.
C'est bizarre je n'utilise pourtant pas de thread, qu'est ce qui peut bien provoquer ça ?

[Lavock]

Bon, j'ai pas trouvé d'opération sur ton socket dans ton code. Comment les données arrives dessus si ce n'est pas un autre thread ?

[tir0nik]

En fait, ce n'est pas mon code. C'est juste que j'ai le même type de problème, c'est-à-dire une boucle avec un std::cout dedans et quand j'enlève le cout ça bug.

Voici, en gros, comment est mon 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
 
std::string Client::Receive()
{
    char buffer[BUFFERSIZE];
    std::string retour;
    int  lu = 0;
    do
    {
        memset(buffer,0,sizeof(buffer));
        lu = recv(sockreceive,buffer,sizeof(buffer),0);
        retour += buffer
       std::cout<<"hello"<<std::endl;//tout marche quand je laisse ce cout ou que je met un Sleep(10)
 
    } while (lu == BUFFERSIZE);
    return retour;
}