Cómo devolver una referencia QString const en caso de error?

Question

Consideremos el siguiente código:

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 
    } 
} 

¿Cómo puedo evitar tener un QString vacío sin cambiar el tipo de retorno del método? (Parece que devolver una QString vacía asignada en la pila es una mala idea)

Gracias.

Answer 1

No puede. O bien no devolver una referencia const o utilizar una variable estática local como esto:

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; 
    } 
} 

La ventaja de este método sobre los otros métodos propuestos es que esta solución no requiere un cambio a la interfaz de MyClass. Además, usar un parámetro predeterminado puede confundir a los usuarios de su clase y provocar un uso incorrecto de la clase. Esta solución es transparente para el usuario.

Por cierto, ¿realmente está usando una matriz de estilo C en su clase?

Answer 2

Dado que se espera que esto devuelva un valor de const no veo ningún problema con tener un QString vacío global (o estático const) que todas estas funciones utilicen para devolver una cadena vacía.

Aunque no me gusta el nombre. Esperaría que el QString "vacío" fuera un miembro estático de la clase QString. para que tu código se vea así en su lugar.

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

¿Cómo sobre el uso de un valor por defecto pre-inicializado:

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

si insiste en devolver una referencia, debe tener un objeto para referirse a; así que debes tener el objeto QString en algún lugar de tu ejemplo, no hay forma de evitarlo.

Sin embargo, una técnica que parece adecuado para su caso es cambiar su método para aceptar un ID por defecto para volver por si el índice está fuera del rango:

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

También podría lanzar una excepción si el índice está fuera de rango, entonces no necesitaría regresar al fallo, pero probablemente eso no sea lo que desea.

Answer 5

No puede evitarlo sin cambiar el tipo de devolución.

Si elige devolver una referencia, entonces tiene tiene alguna variable del tipo de devolución que supera el alcance de la función. Si no puede cambiar la API (por ejemplo, debido a las promesas de compatibilidad binaria), entonces está bloqueado en esto para siempre. Deberá perder memoria almacenando algún valor del tipo relevante, incluso si cambia el resto de la implementación de su clase a, p. genere los valores sobre la marcha o recupérelos de alguna fuente externa.

Es por eso que las guías de diseño de la API C++ que conocen los problemas de compatibilidad binaria recomiendan no devolver const& sin una consideración cuidadosa.

Answer 6

¿Sería suficiente QString :: null?

Answer 7

No puede evitar la necesidad de un QString vacío sin cambiar la forma en que getId() funciona. Pero hay dos enfoques que nos vienen a la mente:

• en lugar de devolver silenciosamente una cadena vacía, lanzar una excepción; o
• no te molestes en devolver una referencia, y simplemente devuelve un QString, dependiendo de return value optimization para eliminar el costo de copiar el objeto.