Si...corretto. Si, personalmente se posso utilizzo il ternario ma se non ti piace puoi benissimo mettere una if. 2 cose solo:

1) non vedo l'utilità di chiamare 2 volte la funzione glob. Io l'assegnerei ad una variabile e poi utilizzerei quella
2) Adesso che rivedo bene la documentazione il codice si potrebbe semplificare ancora in questo modo:
Codice PHP:
$files glob('upload/*.jpeg');
$fileCount 0;

/*
glob ritorna FALSE se va in errore o un array vuoto se non trova nessun elemento => controllo se la variabile NON è vuota
*/
if(!empty($files)){
   
$fileCount count($files);

In questo modo non c'è nemmeno necessità di utilizzare un operatore ternario