4

Tengo este problema, ya estuve checando mi código pero no encuentro donde puede estar el problema.

Esto esta ubicado en un modelo:

public string Actualizar_Red(string usuario, int idRed, string urlRed)
    {
        string msj = "";
        try
        {
            DateTime fecha = DateTime.Now;
            SqlCommand cmd = new SqlCommand("LANDINGPAGE.ACTUALIZAR_REDES", con);
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.Parameters.Clear();
            con.Open();
            cmd.Parameters.AddWithValue("@P_ID_RED", idRed);
            cmd.Parameters.Add("@P_FEC_MODI", SqlDbType.Date).Value = fecha.ToShortDateString();
            cmd.Parameters.AddWithValue("@P_USU_MODI", usuario);
            cmd.Parameters.AddWithValue("@P_URL_RED", urlRed);
            if (Convert.ToInt32(cmd.ExecuteScalar()) != 0)
            {
                msj = "Red modificado correctamente";
            }
            else
            {
                msj = "Ningún cambio realizado";
            }
            con.Close();
            return msj;
        }
        catch
        {

            return "Error de conexión, intente de nuevo";
        }
    }

Esto, lo tengo en un controlador:

public ActionResult Actualizar_Red(string urlRed, int idRed)
    {

        ViewBag.Message = MD.Actualizar_Red("usuario", idRed, urlRed);
        return Json("");
    }

Este es mi procedimiento almacenado:

    ALTER PROCEDURE [LANDINGPAGE].[ACTUALIZAR_REDES]
    @P_ID_RED INTEGER,
    @P_FEC_MODI DATE,
    @P_USU_MODI VARCHAR(200),
    @P_URL_RED VARCHAR(200)
    AS BEGIN SELECT * FROM LANDINGPAGE.REDES
    IF EXISTS (SELECT * FROM LANDINGPAGE.REDES WHERE NUMB_ID_RED = @P_ID_RED)
    BEGIN 
    UPDATE LANDINGPAGE.REDES
        SET FECHA_MODIFICACION = @P_FEC_MODI,
            USUARIO_MODIFICACION = @P_USU_MODI,
            URL_RED = @P_URL_RED
    WHERE NUMB_ID_RED = @P_ID_RED
    SELECT @@ROWCOUNT as afectadas
    END
END
10
  • Hola Fer, has debugeado ? sabes hasta donde el valor es correcto, y donde se pierde, o cual es el lugar donde se produce el error ?
    – FederHico
    Commented el 1 jun. 2017 a las 14:34
  • Hola @FederHico, si ya lo hice, todos los campos los agarra pero en el idRed me lo marca como un cero. Cuando esta situado aquí if (Convert.ToInt32(cmd.ExecuteScalar()) != 0) me manda al catch y es cuando me sale el error. Commented el 1 jun. 2017 a las 14:38
  • @Pikoh lo tengo de esa manera porque donde trabajo así me dijeron que dejara ese campo. Commented el 1 jun. 2017 a las 14:38
  • creo que necesitas esto dateDate.Ticks }
    – ByGroxD
    Commented el 1 jun. 2017 a las 14:39
  • eso lo utilizan para el formato de la fecha, y lo hace bien. Commented el 1 jun. 2017 a las 14:41

2 respuestas 2

4

El problema con este código está aqui:

if (Convert.ToInt32(cmd.ExecuteScalar()) != 0)

ExecuteScalar devuelve el valor de la primera columna y la primera fila de los resultados devueltos, lo que en tu caso debe ser un DateTime. Si no necesitas recoger datos del procedimiento almacenado, debes usar ExecuteNonQuery que devuelve el número de filas afectadas:

if (Convert.ToInt32(cmd.ExecuteNonQuery()) != 0)
3

Te recomendaria modificar primero tu Stored Procedured y agregarle una variable de salida

@afectadas int OUTPUT

En tu procedimiento almacenado

SELECT @afectadas = @@ROWCOUNT

En tu Funcion c# Actualizar_Red

SqlCommand cmd = new SqlCommand("LANDINGPAGE.ACTUALIZAR_REDES", con);
SqlParameter afectadas = new SqlParameter("@afectadas", SqlDbType.Int)
{
    Direction = ParameterDirection.Output
};
cmd.CommandType = CommandType.StoredProcedure;
cmd.Parameters.Add(afectadas);
//Tu codigo...
conn.Open();
cmd.Parameters.AddWithValue("@P_ID_RED", idRed);
cmd.Parameters.AddWithValue("@P_USU_MODI", usuario);
cmd.Parameters.AddWithValue("@P_URL_RED", urlRed);
cmd.ExecuteNonQuery();
int numeroAfectadas = int.Parse(afectadas.Value.ToString());
if (numeroAfectadas > 0 )
{
    msj = "Red modificado correctamente";
}
else 
{
    msj = "Ningún cambio realizado";
}

De esta manera puedes capturar un valor de salida que ejecuta tu SP.

No se cual es la finalidad de la Fecha, pero si necesitas que siempre sea la actual, podrias dejarle ese trabajo al mismo SP y utilizar GETDATE()

UPDATE LANDINGPAGE.REDES
    SET FECHA_MODIFICACION = GETDATE(),
8
  • donde le pusiste tu codigo es donde ira lo que yo tengo que es lo de el llamado del procedimiento almacenado y sus variables? Commented el 1 jun. 2017 a las 15:29
  • claro, puse solo los cambios que agregue a tu codigo, pero editare y pondre todo
    – sioesi
    Commented el 1 jun. 2017 a las 15:29
  • ah ok ok :D, deja lo pruebo Commented el 1 jun. 2017 a las 15:32
  • cambie el código a esto, pero ahora me dice que La cadena de entrada no tiene el formato correcto Commented el 1 jun. 2017 a las 15:38
  • En que linea especificamente?
    – sioesi
    Commented el 1 jun. 2017 a las 15:45

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.