Ma in questo modo arrivi al caso in cui hai 7 segmenti accesi tutti alimentati da un unico pin dell'Arduino e questo non va bene: dal pin andresti a prelevare troppa corrente.
Devi anteporre 4 transistor, uno per ogni linea del display, le cui basi poi colleghi ai pin dell'Arduino. In questo modo lo piloti in sicurezza.
Sulla frequenza devi fare dei test. Sopra ai 30 Hz l'occhio umano non riesce più a percepire la transizione per cui dovresti essere a posto con tempi dell'ordine di 1/30=33 ms max, meglio 20 ms (50 Hz).
EDIT:
inoltre anche l'idea di mettere 1 unica resistenza per tutti i led non mi pare corretta. Meglio sempre 1 resistenza per segmento, magari di valore più basso rispetto a quello teorico ma sempre e comunque presente.