return valor select c#

[kristian159]

Hola a todos, tengo una duda,


He creado esta función, que recibe un id numerico, y lo que hace es devolver el valor de la columna, pero solo puedo imprimirlo, si pongo el return fuera del switch no me funciona porque no existe la variable, y si lo pongo dentro no me compila... no sé como puedo conseguir eso.. lo suyo seria guardar los valores de la bbdd (que son rutas y demas) en unas variables globales pero no sé muy bien el como hacerlo..

saludos

Código C:

public void leer()
        {
 
            String cadena_de_conexion = @"Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\status\bbdd\status.mdb;";
            OleDbConnection conexion = new OleDbConnection(cadena_de_conexion);
            conexion.Open();
 
            OleDbCommand query = new OleDbCommand("SELECT * FROM config ", conexion);
            OleDbDataReader reader = query.ExecuteReader();
 
            while(reader.Read()){
                string ruta_bbdd = reader.GetString(1);
 
                Console.WriteLine(ruta_bbdd);
 
                      
        }
 

[Malenko]

No hace falta que sean variables globales, simplemente variables locales en la funcion. El problema es que usas un "while", lo que indica más de un resultado y no tendría que ser así, no?

Yo lo pondría de esta forma (dejando el "bug" del while):

Código C:

public string leer()
{
    String ruta_bbdd = String.Empty;
    String cadena_de_conexion = @"Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\status\bbdd\status.mdb;";
    OleDbConnection conexion = new OleDbConnection(cadena_de_conexion);
    conexion.Open();
         
    OleDbCommand query = new OleDbCommand("SELECT * FROM config ", conexion);
    OleDbDataReader reader = query.ExecuteReader();
         
    while(reader.Read()){
        ruta_bbdd = reader.GetString(1);
        Console.WriteLine(ruta_bbdd);
    }
    
    return ruta_bbdd;
}
 

Lo dicho, el código aún tiene cosas bastante mejorables, como controlar las excepciones, pero supongo que con esto se resuelve tu duda.

[kristian159]

Buenas, ya lo intenté pero no me compila me da este error:

Error 2 The name 'ruta_bbdd' does not exist in the current context

Yo he puesto un while porque es la única forma que he conseguido para extraerlo de alguna forma, pero si hay otra forma mejor lo podría cambiar, me es indiferente tener o no el while...

saludos!

[Malenko]

El código que te he puesto te da ese error al compilar?

[kristian159]

fallo mio por no copiar y machacar lo que tenía, funciona muy bien por la primera línea.

pero no me funciona cuando añado un campo y un switch case...

Código C:

public string leer(int id) // el id es para devolver la ruta que quiero en ese momento
       {
 
 
    String ruta_bbdd = String.Empty;
    String ruta_backup = String.Empty;
    String cadena_de_conexion = @"Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\status\bbdd\status.mdb;";
    OleDbConnection conexion = new OleDbConnection(cadena_de_conexion);
    conexion.Open();
         
    OleDbCommand query = new OleDbCommand("SELECT * FROM config ", conexion);
    OleDbDataReader reader = query.ExecuteReader();
         
    while(reader.Read()){
        ruta_bbdd = reader.GetString(1);
        ruta_backup = reader.GetString(2);
 
        Console.WriteLine(ruta_bbdd);
    }
    switch (id)
    {
        case 1:
 
            return ruta_bbdd;
 
        case 2:
            return ruta_backup;
 
        default:
            Console.WriteLine("Default case");
            break;
    }
 

Error: Error 1 'status.config.leer(int)': not all code paths return a value

[Malenko]

Pues eso, el error es bastante claro: hay casos en que no devuelves nada, es decir, cuando entras en el default del switch. La solución es sencilla, pon un "return String.Empty;" dentro del default.

[kristian159]

Genial, tenía sentido...soy bastante novato como puedes ver...

gracias por todo!!

lo que no termino de conseguir es que.. si un campo es entero, como hago para filtrarlo o hacer que lo devuelva, no consigo controlar eso...

un saludo!

[Malenko]

Si quieres obtener un campo entero de la base de datos puedes usar el GetByte, GetInt16, GetInt32 , ... y usarlo de la misma manera que haces con el GetString.

[kristian159]

El problema es que devuelvo un string, el return es sobre un string no un int... y por eso me peta

[Malenko]

Usa el ToString() para pasar el tipo entero (int) a cadena.