Veo varias cosas en tu código:
- Los dos métodos que muestras son redundantes. Con un solo método puedes obtener el idioma y retornarlo.
- En el primer método estás escribiendo un código vulnerabable a ataques de inyección SQL. Cuando puedas lee sobre el tema y sobre consultas preparadas. Tu servidor podría quedar gravemente expuesto si escribes código vulnerable. Dado que usas una propiedad
$stmt
que no sé de dónde la sacas, no puedo escribirte un código seguro aquí, pero conviene que uses tu conexión para preparar la consulta.
- En tu primer método estás usando
execute()
dos veces. Realmente no necesitas execute()
, porque estás usando query()
, que es una modalidad de execute()
pero NO segura en este caso, por lo ya dicho en (2).
- Para controlar el resultado de tu consulta debes hacerlo fuera de clase. Conviene que tengas en la cabeza algo así como un contrato y que lo reflejes en tu código. Ejemplo de contrato: devolverá
false
cuando no haya idioma. Dicho eso, donde recibes los datos verificas si la respuesta es false
o no, y en la clase, devuelves false
cuando no haya idiomas. Este es el comportamiento de fetch()
cuando no hay datos. En un programa más maduro, puedes por ejemplo mandar un array con una clave error
y un mensaje, para controlar los errores. Pero ese ya es otro asunto...
Aplicando lo dicho, tu clase podría quedar así:
class Expediente {
private function getIdiomaById($ID)
{
$stm = query("SELECT * FROM Idioma WHERE ID = '$ID'");
$rs=$stm->fetch(PDO::FETCH_OBJ);
/*
Asumo que:
1. Hay una columna llamada Idioma en tu tabla Idioma
2. Tu clase tiene una propiedad llamada Idioma
Verificaremos si $rs no es false
para setear la propiedad idioma y devolverla
de lo contrario, devolveremos false
Y en la llamada (código al final de la respuesta)
verificaremos el estado de lo que el
método haya devuelto: el idioma, o false
*/
if ($rs)
{
$this->Idioma = $rs->Idioma;
return $this->Idioma;
} else {
return false;
}
}
}
Y desde donde llamas el método:
# $mExpediente es una instancia de la clase Expediente
# en un contexto cualquiera
$idioma=$mExpediente->getIdiomaById($ID);
if ($idioma)
{
#Usar $idioma
} else {
#Idioma es false, no hay idioma, actuar en consecuencia
}
P.D.
Considera que en PDO puedes mapear una clase a un resultado (ver la pregunta ¿Como se mapea el resultado de una consulta SQL a una clase en PHP usando PDO y cómo se usa?), quizá convenga hacer esto en tu contexto para simplificar el código. Sacando de dentro de tu clase Expediente
la conexión a PDO. Las clases de entidad deberían ser pensadas como moldes que usas para construir modelos de esos datos, y no deberían arrastar consigo la conexión.