Tanto per cominciare occhio agli indici degli array.

Questi vanno da 0 a n-1 e non da 1 a n.

Le for quindi dovranno essere

for(i=0; i<n; i++)

Detto questo, se devi memorizzare il nome di tante città allora un semplice vettore di char come

char citta[20];

non è sufficiente. Devi usare una matrice di char.