È valido "strlen()" in fase di compilazione?

Question

A volte è necessario confrontare la lunghezza di una stringa con una costante.
Per esempio:

if (line.length() > 2) 
{ 
    // Do something... 
} 

Ma sto cercando di evitare l'uso di costanti "magici" nel codice.
solito utilizzare tale codice:

if (line.length() > strlen("[]")) 
{ 
    // Do something... 
} 

è più leggibile, ma non efficiente a causa della chiamata di funzione.
Ho scritto funzioni template come segue:

template<size_t N> 
size_t _lenof(const char (&)[N]) 
{ 
    return N - 1; 
} 

template<size_t N> 
size_t _lenof(const wchar_t (&)[N]) 
{ 
    return N - 1; 
} 

// Using: 
if (line.length() > _lenof("[]")) 
{ 
    // Do something... 
} 

In una build di rilascio (VisualStudio 2008) produce codice abbastanza buono:

cmp dword ptr [esp+27Ch],2 
jbe 011D7FA5 

E la cosa buona è che il compilatore non include il "[]" stringa nell'output binario.

È un ottimizzazione specifica del compilatore o è un comportamento comune?

Answer 1

La capacità di inline di una chiamata di funzione è sia l'ottimizzazione specifica del compilatore e un comportamento comune. Cioè, molti compilatori possono farlo, ma non sono obbligati a farlo.

Answer 2

Perché non

 
sizeof "[]" - 1; 

(meno uno per il nulla di trascinamento Si potrebbe fare sizeof "[]" -. Sizeof '\ 0', ma sizeof '\ 0' è spesso sizeof (int) in C, e "- 1" è perfettamente leggibile)

Answer 3

#define TWO 2 
#define STRING_LENGTH 2 
/* ... etc ... */ 

Scherzi a parte, perché passare attraverso tutto questo fastidio solo per evitare digitando un 2.? Onestamente penso che stai rendendo il tuo codice meno leggibile, e altri programmatori ti fisseranno come se stessi sniffando il caffè usato dal filtro.

Answer 4

Penso che molti compilatori lo ottimizzeranno via quando le ottimizzazioni sono abilitate. Se sono disabilitati, potrebbe rallentare il programma molto più del necessario.

Preferirei le funzioni del modello, poiché sono garantite per non chiamare strlen in fase di esecuzione. Naturalmente, piuttosto che scrivere funzioni separate per char e wchar_t, si potrebbe aggiungere un altro argomento di un template, e ottenere una funzione che funziona per qualsiasi tipo:

template <typename Char_t, int len> 
int static_strlen(const Char_t (&)[N] array){ 
    return len/sizeof(Char_t) - 1; 
} 

(Come già accennato nei commenti, questo darà risultati divertenti se si passa un array di int, ma siete suscettibili di farlo? E 'pensato per le stringhe, dopo tutto)

una nota finale, il nome _strlen è male. Tutti i nomi nello spazio dei nomi che iniziano con un carattere di sottolineatura sono riservati all'implementazione. Rischia alcuni brutti conflitti di denominazione.

A proposito, perché "[]" meno di una costante magica di 2 è?

In entrambi i casi, è un letterale che deve essere cambiato se il formato della stringa è confrontato ai cambiamenti.