Questa riga

codice:
if(giocatori[j] == 'S' || giocatori[j] == 's')
e' sbagliata perche' stai confrontando una stringa con un carattere.

Per confrontare le stringhe devi usare la funzione strcmp

Anche questa riga

codice:
if(giocatori[j] <= 0 || giocatori[j] > MAX_NOME_GIOC)
e' sbagliata perche' per ottenere la lunghezza di una stringa si usa la funzione strlen

La return e' sbagliata ma non ho capito cosa deve restituire la funzione (che hai indicato restituire un singolo char ...).