¿Y qué tienes hecho? Porque si lo que esperas es que te lo hagan, esperarás muuuuuucho tiempo...
string nombre;No existe tal string en C.
} cliente;Usa un identificador que te permita distinguir entre tipos y variables, por ejemplo tCliente. Así queda más legible.
char sn[1];¿Un vector de un elemento? Para eso pones char sn y es más sencillo...
int x;Pon nombres de variables que describan la función que hace la variable.
sn[0]='s';do...while te viene mejor en ese caso. Además, con el cambio comentado anteriormente te quedaría
while ((sn[0]=='s')||(sn[0]=='S'))