En ese método podrían haber varias cosas que mejorar y que me parecen raras, primero que todo veo que estas usando punteros, lo cual en C#, no es muy común, así que sí los vas a usar deberías estudiar este artículo de MSDN, y usar la instrucción unsafe
, pues es recomendable y una buena práctica en C#, para cuando usas punteros.
Ahora bien, me parece raro la forma en que mandas los parámetros, pues se parece mucho a los de C++, así que te aclaro, para mandar un vector por parámetro sería de la siguiente forma:
public void miMetodo(tipoDato [] nombreVariable);
public void FindClosest(T_PLANET[] v);
Ahora, veo que para devolver el planeta más cercano, lo asignas a un parámetro con un asterisco. Sí lo que quieres es asignarlo y que cuando termine el método, siga con el valor asignado, debes usar el ref
, para que se pase el parámetro por referencia
public void miMetodo(ref tipoDato nombreVariable);
public void FindClosest(ref T_PLANET p_closest);
Ahora bien, te recomiendo, que no utilices el devolver un parámetro por referencia, en un método void
, pues deberías usarlo sí ya el método devuelve un valor y ocupas obtener otro valor que se haya modificado en el método.
Finalmente, deberías revisar la condición del if
, pues creo que se encuentra incorrecta.
Aquí te dejo como quedaría el código, ya optimizado (teniendo en cuenta que no conozco el resto del código ni las especificaciones del programa).
public T_PLANTE FindClosest(T_PLANET [] v, int n)
{
int i;
int j;
j=0;
for (i=1; i<n; i++)
{
if (v[i].data.distance < v[j].data.distance)
{
j=i;
}
}
p_closest= &(v[j]);
return;
}
Postada:
Para recorrer un vector también puedes usar un foreach()
, con esto podrías enviar un parámetro menos y no tendrías que preocuparte por el contador de la instrucción cíclica, aquí te dejo información