Una función de swap (intercambio) debe intercambiar dos valores, ni más ni menos. En tu caso estás desarrollando una función de intercambio mucho más compleja de lo necesario sin aplicar el principio de la mínima sorpresa (el resaltado es mío):
Este principio establece que cuando dos elementos de una interfaz entran en conflicto o son ambiguos, el comportamiento del programa ha de ser el que genere la mínima sorpresa por parte del usuario. En particular, un programador debería pensar en el comportamiento que menos sorprenda a quien use el programa en lugar del más natural para quien conozca su comportamiento interno.
Las decisiones que has tomado al redactar la función swap es el equivalente a diseñar un coche que tenga las puertas en el techo: es sorprendente que un coche tenga las puertas en esa posición, los conductores no sabrán qué esperar de ese vehículo.
Tampoco estás siguiendo el principio KISS (el resaltado es mío):
El principio KISS (del inglés Keep It Simple, Stupid!: «¡Mantenlo sencillo, estúpido!») es un acrónimo usado como principio de diseño.
El principio KISS establece que la mayoría de sistemas funcionan mejor si se mantienen simples que si se hacen complejos; por ello, la simplicidad debe ser mantenida como un objetivo clave del diseño, y cualquier complejidad innecesaria debe ser evitada.
Las decisiones que has tomado al redactar la función swap es el equivalente a diseñar un coche que en lugar de tres o cinco puertas tenga nueve, haciendo que su fabricación y mantenimiento sea más complejo.
Así que tienes un coche con nueve puertas en el techo. ¿Qué podemos hacer para resolver este embrollo?:
- Mínima sorpresa: Una función de swap (intercambio) debe intercambiar valores.
- KISS: Para intercambiar dos valores, sólo necesitas dos parámetros.
void swap(int *a, int *b)
{
int comodin = *a;
*a = *b;
*b = comodin;
}
Después, tal y como dice SuperG280, usando una nomenclatura adecuada, puedes hacer la llamada pertinente:
int x1, y1, x2, y2;
printf("\nintroduzca su coordenada x\n");
scanf("%d",&x1);
printf("\nintroduca su coordenada y\n");
scanf("%d",&y1);
printf("\nintroduzca otra coordenada x\n");
scanf("%d",&x2);
printf("\nintroduzca otra coordenada y\n");
scanf("%d",&y2);
swap(&q[x1][y1], q[x2][y2]);
Siguiendo esos dos principios, no habrías cometido el error que te llevó a escribir tu pregunta.