1

Tengo un DataGridView que muestra nombre, celular desde una base de datos en SQLServer y que tiene CheckBoxes. El botón enviará de manera masiva mensajes a esos clientes, por eso es que quiero que se cada CheckBox tildado almacene la variable. El problema está en que no se mandan los mensajes en el ciclo for que hice.

Para enviar mensajes de manera masiva se utiliza este bloque en string:

string bloque = "";

bloque += bloque + "ID1\tNumeroCelular\tMensaje\n";//cliente A

bloque += bloque + "ID2\tNumeroCelular\tMensaje\n";//cliente B

Entonces pensé que podría hacerlo de otra manera haciendo un ciclo, el cual aumentaría el ID de manera automática por cada CheckBox tildado y asignaría el celular al ciclo para que cuando presione el botón este envíe el mensaje a todos los clientes seleccionados.

Actualmente, esto es todo lo que tengo de código:

public partial class Form1 : Form {
    int contador = 0;
    public Form1(){
        InitializeComponent();
    }
    private void Form1_Load(object sender, EventArgs e){
        llenarId();
        //dtgId.ReadOnly = true;
        DataGridViewCheckBoxColumn chk = new DataGridViewCheckBoxColumn();
        chk.HeaderText = "Seleccione";
        chk.Name = "check";
        dtgId.Columns.Add(chk);
        dtgId.AllowUserToAddRows = false;
    }

    public void llenarId(){
        try{
            string cadena = "cadena";
            using (SqlConnection con = new SqlConnection(cadena)){
                con.Open();
                string query = "SELECT id, nombre, celular FROM clientes";
                SqlCommand cmd = new SqlCommand(query, con);
                SqlDataAdapter da = new SqlDataAdapter(cmd);
                DataSet ds = new DataSet();
                da.Fill(ds);
                dtgId.DataSource = ds.Tables[0];
                con.Close();
            }
        }
        catch (SqlException ex){
            MessageBox.Show("Error: " + ex.Message, "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
        }
    }

    private void button2_Click(object sender, EventArgs e){
        List<string> celulares = new List<string>();
        foreach (DataGridViewRow row in dtgId.Rows){
            bool variable = (bool)row.Cells[3].Value;
            if (variable){
                celulares.Add(row.Cells[2].Value.ToString());//columna donde estan los celulares
            }
        }

        //Codigo para enviar SMS
        string usuario = "usuario";
        string clave = "clave";
        string respuesta = ""; 
        string texto = txtTexto.Text;
        string bloque = "";
        //bloque += bloque + "ID1\tNumeroCelular\tMensaje\n"; trato de reemplazar esto por el ciclo for
        contador++;
        for (int i = 0; i < celulares.Count; i++){
            bloque = bloque + "ID" + Convert.ToInt32(contador) + "\t" + celulares[i] + "\t" + texto + "\n";
            Console.WriteLine(bloque);
        }

        Uri uri = new Uri("uri");

        HttpWebRequest requestFile = (HttpWebRequest)WebRequest.Create(uri);
        requestFile.Method = "POST";
        requestFile.ContentType = "application/x-www-form-urlencoded";
        StringBuilder postData = new StringBuilder();
        postData.Append("api=" + System.Web.HttpUtility.UrlEncode("1") + "&");
        postData.Append("usuario=" + System.Web.HttpUtility.UrlEncode(usuario) + "&");
        postData.Append("clave=" + System.Web.HttpUtility.UrlEncode(clave) + "&");
        postData.Append("separadorcampos=" + System.Web.HttpUtility.UrlEncode("tab") + "&");
        postData.Append("bloque=" + System.Web.HttpUtility.UrlEncode(bloque) + "&");

        //byte[] byteArray = Encoding.GetEncoding("iso-8859-1").GetBytes(postData.ToString());
        byte[] byteArray = Encoding.UTF8.GetBytes(postData.ToString());

        requestFile.ContentLength = byteArray.Length;

        Stream requestStream = requestFile.GetRequestStream();
        requestStream.Write(byteArray, 0, byteArray.Length);
        requestStream.Close();
        HttpWebResponse webResp = requestFile.GetResponse() as HttpWebResponse;
        if (requestFile.HaveResponse){
            if (webResp.StatusCode == HttpStatusCode.OK || webResp.StatusCode == HttpStatusCode.Accepted){
                StreamReader respReader = new StreamReader(webResp.GetResponseStream(), Encoding.GetEncoding("iso-8859-1"));
                respuesta = respReader.ReadToEnd();
                MessageBox.Show(respuesta);
            }
        }
    }

    private void button1_Click(object sender, EventArgs e)
    {//agregué este botón sin el ciclo para mostrarles que de este modo si funciona
        //Codigo para enviar SMS
        string usuario = "usuario";
        string clave = "clave";
        string bloque = "";
        string respuesta = ""; //en esta variable quedará la respuesta del llamado a la API de SMS MASIVOS

        bloque = bloque + "ID1\t1111111111\tPrueba \n";
        bloque = bloque + "ID2\t2222222222\tPrueba \n";

        Uri uri = new Uri("uri");

        HttpWebRequest requestFile = (HttpWebRequest)WebRequest.Create(uri);

        requestFile.Method = "POST";
        requestFile.ContentType = "application/x-www-form-urlencoded";

        StringBuilder postData = new StringBuilder();
        postData.Append("api=" + System.Web.HttpUtility.UrlEncode("1") + "&");
        postData.Append("usuario=" + System.Web.HttpUtility.UrlEncode(usuario) + "&");
        postData.Append("clave=" + System.Web.HttpUtility.UrlEncode(clave) + "&");
        postData.Append("separadorcampos=" + System.Web.HttpUtility.UrlEncode("tab") + "&");
        postData.Append("bloque=" + System.Web.HttpUtility.UrlEncode(bloque) + "&");
        byte[] byteArray = Encoding.UTF8.GetBytes(postData.ToString());

        requestFile.ContentLength = byteArray.Length;

        Stream requestStream = requestFile.GetRequestStream();
        requestStream.Write(byteArray, 0, byteArray.Length);
        requestStream.Close();

        HttpWebResponse webResp = requestFile.GetResponse() as HttpWebResponse;

        if (requestFile.HaveResponse)
        {
            if (webResp.StatusCode == HttpStatusCode.OK || webResp.StatusCode == HttpStatusCode.Accepted)
            {
                StreamReader respReader = new StreamReader(webResp.GetResponseStream(), Encoding.GetEncoding("iso-8859-1"));

                respuesta = respReader.ReadToEnd();
                MessageBox.Show(respuesta);
            }
        }
    }
}
12
  • 1
    Cual seria exactamente el problema? porque estas guardando los celulares en una lista...
    – gbianchi
    Commented el 23 oct. 2019 a las 13:41
  • 1
    Al presionar que boton? donde esta ese codigo que falla? tomate unos segundos, y escribe la pregunta como para que entendamos tu problema.. asi como esta, es una maraña de codigo y no se entiende que esta pasando y donde...
    – gbianchi
    Commented el 23 oct. 2019 a las 13:49
  • 1
    Tu codigo esta tirando un error no?
    – gbianchi
    Commented el 23 oct. 2019 a las 14:22
  • 1
    Y entonces cuando dices que no funciona, que es lo que no funciona?
    – gbianchi
    Commented el 23 oct. 2019 a las 14:25
  • 1
    Y el string se arma correctamente? o sea el problema es armar el string o enviar los mensajes?
    – gbianchi
    Commented el 23 oct. 2019 a las 14:32

2 respuestas 2

3

Tu problema no es lo que decis.

Tu problema es que el string que estas armando esta mal.

No es un problema al enviar mensajes.

Fijate este codigo:

List<string> celulares = new List<string>();
....
for (int i = 0; i < celulares.Count; i++)
{
    bloque += bloque + "ID" + Convert.ToInt32(contador) + "\t" + celulares + "\t" + texto + "\n";//no funciona
}

Si celulares es una lista, que esperas que ponga al concatenar la lista celulares en ese string? pues, pone cualquier cosa, y tu string esta mal formado (por eso te pregunte si tu string estaba bien, y si lo miras, no, no esta bien)...

Lo que necesitas es agregarle cual fila de celulares queres...

Y para eso, como es una lista, y la misma se puede manejar como si fuera un array, alcanza con hacer

celulares[i]

Entonces, tu linea deberia ser:

bloque += bloque + "ID" + Convert.ToInt32(contador) + "\t" + celulares[i] + "\t" + texto + "\n"

Edición

Acabo de detectar otro error.. estas sumando bloque dos veces por cada vuelta! la linea deberia ser

bloque += "ID" + Convert.ToInt32(contador) + "\t" + celulares[i] + "\t" + texto + "\n"
13
  • 1
    Pero miraste que el string este bien? y donde en todo tu codigo se mandan los mensajes?
    – gbianchi
    Commented el 23 oct. 2019 a las 15:03
  • 1
    Y si escribis a mano funciona? Me parece que mareaste totalmente tu problema... @FranqoBalsamo. Para dos numeros pasados a mano podes?
    – gbianchi
    Commented el 23 oct. 2019 a las 15:05
  • 1
    Perfecto, y ahora nos muestras que queda en bloque una vez que pasa el for? para ver si lo esta armando bien, que es lo que te pregunte varias veces ya... @FranqoBalsamo
    – gbianchi
    Commented el 23 oct. 2019 a las 15:33
  • 1
    @FranqoBalsamo fijate que tenias otro error al armar el string.. sumabas dos veces el string anterior
    – gbianchi
    Commented el 23 oct. 2019 a las 15:35
  • 1
    @FranqoBalsamo que queda dentro del string bloque. depura tu programa y fijate si queda un string que sea valido...
    – gbianchi
    Commented el 23 oct. 2019 a las 15:46
-2

Lo que debes hacer es añadir un escucha de evento a los las cajas de chequeo tal que al cambiar de estado puedas eliminar o añadir a la lista donde guardas los números seleccionados para enviar el mensaje

1
  • 2
    Estimado, eso es mas un comentario que una respuesta, para responder por favor lee cómo responder para que tu respuesta sea bien aceptada. Saludos!
    – harifo
    Commented el 23 oct. 2019 a las 13: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.