Come restituire un riferimento const QString in caso di errore?

Question

considerare il seguente codice:

const QString& MyClass::getID(int index) const 
{ 
    if (i < myArraySize && myArray[i]) { 
     return myArray[i]->id; // id is a QString 
    } else { 
     return my_global_empty_qstring; // is a global empty QString 
    } 
} 

Come posso evitare di avere un vuoto QString senza cambiare il tipo di ritorno del metodo? (Sembra che restituire una QString vuota allocata nello stack sia una cattiva idea)

Grazie.

Answer 1

Non è possibile. O non restituire un riferimento const o utilizzare una variabile statica locale come questo:

const QString& MyClass::getID(int index) const { 
    static const QString emptyString; 

    if (i < myArraySize && (myArray[i] != 0)) { 
     return myArray[i]->id; // id is a QString 
    } else { 
     return emptyString; 
    } 
} 

Il vantaggio di questo metodo rispetto agli altri metodi proposti è che questa soluzione non richiede una modifica all'interfaccia di MyClass. Inoltre, l'utilizzo di un parametro predefinito potrebbe confondere gli utenti della tua classe e portare a un uso errato della classe. Questa soluzione è trasparente per l'utente.

A proposito, stai davvero usando un array in stile C nella tua classe?

Answer 2

Poiché si prevede che restituisca un valore const, non vedo alcun problema con un QString globale (o statico) vuoto che viene utilizzato da tutte queste funzioni per restituire una stringa vuota.

Non sono affatto arrabbiato per il nome. Mi aspetto che il QString "vuoto" sia un membro const const della classe QString. quindi il tuo codice sarebbe simile a questo.

const QString& MyClass::getID(int index) const 
{ 
    if (i < myArraySize && myArray[i]) { 
     return myArray[i]->id; // id is a QString 
    } else { 
     return QString::EmptyString; // is a global empty QString 
    } 
} 

Answer 3

Come sull'utilizzo di un valore di default pre-inizializzato:

const QString& MyClass::getID(int index, const QString& def = QString()) const 
{ 
    if (i < myArraySize && myArray[index]) { 
     return myArray[index]->id; // id is a QString 
    } else { 
     return def; 
    } 
} 

Answer 4

se ti ostini a restituire un riferimento, è necessario disporre di un oggetto per riferirsi a; quindi devi avere l'oggetto QString da qualche parte nel tuo esempio, non c'è modo di aggirarlo.

Tuttavia una tecnica che sembra adatto per il vostro caso è quello di modificare il metodo di accettare un ID predefinito per tornare nel caso in cui l'indice è fuori portata:

const QString& MyClass::getID(int i, const QString& default) const 
{ 
    if(i < myArraySize && myArray[i]) 
    return myArray[i]->id; 
    else 
    return default; 
} 

Si potrebbe anche un'eccezione se l'indice è fuori portata, quindi non è necessario tornare in caso di errore, ma probabilmente non è ciò che si desidera.

Answer 5

Non è possibile evitarlo senza modificare il tipo di ritorno.

Se si sceglie di restituire un riferimento, è necessario che disponga di una variabile del tipo restituito che sopravvive all'ambito della funzione. Se non riesci a modificare l'API (ad esempio a causa di promesse di compatibilità binaria), sei bloccato a questo per sempre. Dovrai sprecare memoria memorizzando un valore del tipo pertinente, anche se modifichi il resto dell'implementazione di classe ad es. generare i valori al volo o recuperarli da una fonte esterna.

Ecco perché le guide di progettazione API C++ che sono a conoscenza dei problemi di compatibilità binaria raccomandano di non restituire un const& senza un'attenta considerazione.

Answer 6

QString: null è sufficiente?

Answer 7

Non è possibile evitare la necessità di una QString vuota senza modificare il modo in cui funziona getId(). Ma ci sono due approcci che mi vengono in mente:

• invece di restituire una stringa vuota, lanciare un'eccezione; oppure
• non preoccupatevi di restituire un riferimento e restituite semplicemente un QString, facendo affidamento su return value optimization per eliminare il costo della copia dell'oggetto.