Ci sarebbero una marea di cose da sistemare.
Variabili, codice e tanto altro.

Per esempio la cosa che hai fatto qui è pessima:
codice:
for (x=4;x < strlen(s);x++)
perchè ogni volta che ripete il ciclo esegue anche la funzione strlen con altre iterazioni e quindi è un codice poco efficiente in termini di risparmio memoria e velocità.

Meglio fatto così:
codice:
int n=strlen(s);
for(x=4;x < n;x++)