Discussion :

[Seabirds]

Je cherche à convertir ça en C++:

Code :

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

int *neigh = (int *)malloc(n * sizeof(int)), nn;

Je vois bien qu'il y a un std::vector<int> neigh(n);, mais je ne comprends pas ce que fait le nn, ni quel est son type. Ca ressemble à un integer, parce que dans la suite du code, on trouve des choses comme:

• nn = 0;
• if(j< nn)
• nn++

[foetus]

Ca ressemble à un integer

C'est la base du C la variable nn est de type int.

On utilise l'opérateur virgule pour chaîner les définitions des variables exemple : int x, y, z;.
Mais dans 1 définition est inclus l'initialisation int x=25, y, z=82;.

Les variables pointeurs sont assez "vicieuses" parce que l'étoile n'affecte que la variable concernée, mais le type affecte toute la chaîne des définitions.
C'est pour cela que certains préfèrent "coller" l'étoile à la variable concernée (int *x;) et non au type (int* x;).

[Bousk]

Vu que c'est assigné à un int*, nn est forcément un int*.
Quant au code, il est juste immonde et génère rien d'autre qu'une fuite mémoire.

[Seabirds]

Aaaah bah décidez-vous les copains! int ou int* ?
Oui oui, on est bien d'accord, c'est pas du joli code ahahah. Mais je fait le ménage et propage la bonne parole du C++ moderne, t'en fais pas

[Bousk]

Ce seul code est ambigu sans contexte.
Soit c'est l'opérateur , donc tout est exécuté et seule la dernière valeur est assignée, donc ça fait le malloc, puis assign nn, donc nn doit être un int* précédemment déclaré, et le malloc est une fuite mémoire.
Soit la, est juste un délimiteur et c'est une déclaration de neigh comme int* auquel on assigne le malloc et la déclaration de nn comme int non initialisé.

Code :

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

1
2
3
nn = 0;
if(j< nn)
nn++

Rien de tout ça n'est interdit sur des pointeurs.

[Seabirds]

Oh mon dieu, mais c'est vraiment vraiment immonde alors!
Donc de ce que je vois du code, nn n'a jamais été déclarée (ni vers un int, ni vers un *int. Donc c'est ta deuxième option qui l'emporte.
C'est vrai que ça coute super cher en encre de taper int nn = 0; !

[foetus]

C'est vrai que ça coute super cher en encre de taper int nn = 0; !

C'est 1 peu stupide ce que tu dis

Chaîner les définitions cela permet de réduire la taille des blocs des définitions. Alors certes en C++, on peut définir 1 variable n'importe quand.
Mais ce n'est pas le cas avant le C99 qui oblige de définir en début de bloc.

Après définir 1 pointeur et 1 variable normale en même temps, ce n'est pas vraiment approprié (parce qu'en définitive ce n'est pas le même type).
En tout cas, ton debugger/ compilateur (avec 1 printf par exemple) devrait te dire les types.