Duvida If / Else

27/08/2013

Pessoal estou fazendo uma tabela via string com concatena��o.
Mas percebi, que algumas condi��es, est�o sendo ignoradas, veja o trecho do c�digo.
Em uma parte do c�digo ele ignora meu if e vai direto para o else.

 else if (id_projeto == 2)
            {

                var drOcorrencias = clsLojas.retornaQuery("SELECT * FROM OCORRENCIAS where id_loja=" + id_loja + "");

                if (drOcorrencias == null) //ESSE IF E IGNORADO E VAI DIRETO PARA O ELSE, O QUE DEVO ESTAR FAZENDO DE ERRADO..
                {
                    tablestring2 = tablestring2 + "<table align='center' border='1' bordercolor='#666666' bordercolordark='#000000' width='100%' style='font-size:12px'>";
                    var drItens = clsLojas.retornaQuery("SELECT * FROM ITENS where id_projeto = 2 and id_canal = " + id_canal + " order by id_item");
                    while (drItens.Read())
                    {
                        tablestring2 = tablestring2 + "<tr bgcolor=\"\" onMouseOver=\"javascript:this.style.backgroundColor='#FFCC66';\" onMouseOut=\"javascript:this.style.backgroundColor='';\">";
                        tablestring2 = tablestring2 + "<td style='padding:25px;'> " + drItens["ds_item"] + "</td>";
                        tablestring2 = tablestring2 + "<td>";
                        tablestring2 = tablestring2 + "<table width='100%' border='0'>";
                        var drResp = clsLojas.retornaQuery("SELECT * FROM RESP_2 where id_item = " + drItens["id_item"].ToString().Trim() + " and id_subitem = 0 and id_loja = " + id_loja.ToString().Trim() + " ");
                        while (drResp.Read())
                        {
                            tablestring2 = tablestring2 + "<tr><td colspan='2' style='text-align:center;'>";
                            tablestring2 = tablestring2 + "<hr />";
                            tablestring2 = tablestring2 + "�" + drResp["subresp"] + "�";
                            tablestring2 = tablestring2 + "<hr />";
                            //obs = (string)drResp["obs"];
                        }
                        tablestring2 = tablestring2 + "</td></tr>";
                        var drSubItens = clsLojas.retornaQuery("SELECT * FROM subitens where id_item = " + drItens["id_item"].ToString().Trim() + " AND ID_SUBITEM IN (SELECT ID_SUBITEM FROM UNI_TB_RESP_2 where id_item = " + drItens["id_item"] + " and id_loja = " + id_loja + " AND SUBRESP <> '') order by id_subitem");
                        while (drSubItens.Read())
                        {
                            tablestring2 = tablestring2 + "<tr><td style='text-align:center;'><hr />";
                            if (drSubItens["tipo"].ToString().Trim() == "D" || drSubItens["tipo"].ToString().Trim() == "N" || drSubItens["tipo"].ToString().Trim() == "I" || drSubItens["tipo"].ToString().Trim() == "S" ||
                                drSubItens["tipo"].ToString().Trim() == "")
                            {
                                tablestring2 = tablestring2 + "�" + drSubItens["ds_subitem"] + ":";
                                
                            }

                            var drResp2 = clsLojas.retornaQuery("SELECT * FROM RESP_2 where id_item = " + drItens["id_item"].ToString().Trim() + " and id_subitem = " + drSubItens["id_subitem"].ToString().Trim() + " and id_loja = " + id_loja + " ");
                            while (drResp2.Read())
                            {

                                tablestring2 = tablestring2 + "�" + drResp2["subresp"] + "�";
                                //drResp2["obs"].ToString();
                                drResp2.Close();
                            }

                            tablestring2 = tablestring2 + " </td></tr>";
                            tablestring2 = tablestring2 + "<tr><td colspan='2'><hr /></td></tr>";
                        }
                        drItens.Close();
                            drResp.Close();
                            drSubItens.Close();
                            
                        tablestring2 = tablestring2 + "</td></tr>";
                        tablestring2 = tablestring2 + "</table>";
                        divRelatorio.InnerHtml = tablestring2;
                    }  
                }
                else
           
                {
                    while (drOcorrencias.Read())
                    {
                        tablestring2 = tablestring2 + "<table align='center' border='1' bordercolor='#666666' bordercolordark='#000000' width='100%' style='font-size:12px'>";
                        tablestring2 = tablestring2 + "<tr><td align='center'><font color='red'><strong>OCORRENCIA</strong></font></td></tr>";
                        tablestring2 = tablestring2 + "<tr><td align='center'>�<br />" + drOcorrencias["obs"] + "<br />�</td></tr>";
                        tablestring2 = tablestring2 + "<tr><td align='center'>Respons�vel: " + drOcorrencias["responsavel"] + "</td></tr>";
                        tablestring2 = tablestring2 + "</table>";
                        divRelatorio.InnerHtml = tablestring2;
                        
                    }
                }
                drOcorrencias.Close();
            }

Obridado Galera

Rodrigolima

Curtir t�pico + 0

Responder

Posts

27/08/2013

Pjava

� bem prov�vel que ocorrencia n�o esteja null. Coloque um Break Point antes do IF e teste a vari�vel ocorr�ncia e veja se tem algo. Lembrando que vazia ("") n�o � NULL.

Responder

Gostei + 0

27/08/2013

Tiago

Recomendo utilizar o Debug e verifique o valor atual do objeto... bem lembrado pjava, acredito que resolve.

Responder

Gostei + 0

27/08/2013

Rodrigolima

Isso mesmo amigo, resolvi fazendo essa decis�o
if (drOcorrencias.HasRows == false)
{

}

obrigado amigos.

Responder

Gostei + 0

28/08/2013

Tiago

Por nada Rodrigo, boa sorte na programa��o ai!

Responder

Gostei + 0

28/08/2013

Clayton Silva

Isso mesmo amigo, resolvi fazendo essa decis�o
if (drOcorrencias.HasRows == false)
{

}

if j� procura o resultado boolean, n�o precisa fazer esta compara��o.

if (drOcorrencias.HasRows)

j� resolve, caso queria negativo use uma exclama��o antes.

if (!drOcorrencias.HasRows)

Responder

Gostei + 0

28/08/2013

Rodrigolima

Obrigado amigo mas uma vez, todo dia eu aprendendo, mais uma.
Fiz dessa maneira, uma pratica mais correta.

Obrigado.

Responder

Gostei + 0

28/08/2013

Clayton Silva

Obrigado amigo mas uma vez, todo dia eu aprendendo, mais uma.
Fiz dessa maneira, uma pratica mais correta.

Obrigado.

Tentamos ajudar sempre, mais umas dicas que vi em seu c�digo.

comece a usar par�metros, evite concatenar strings nas querys pois deixa o c�digo propenso a invas�o.

Evite concatenar strings usando +, � um procedimento que gasta muita mem�ria e usado v�rias vezes, como em seu c�digo, n�o � recomendado. Para isto use um stringbuilder.

Responder

Gostei + 0

28/08/2013

Tiago

Essa observa��o � muito importante, afinal usar par�metros � uma boa pr�tica e evita erros de SQLINJECTION... concatenar strings � uma m� pr�tica, se bem que ainda vemos em exemplos de faculdade ou coisa e tipo... mas de como n�o fazer hehe.

Responder

Gostei + 0

28/08/2013

Joel Rodrigues

Como a d�vida foi resolvida, estou marcando o t�pico como Conclu�do.

Responder

Gostei + 0

10/09/2013

Rodrigolima

Pessoal usei fiz update no meu codigo, usei paramentros e o StringBuilder sb = new StringBuider();
Obrigado Pessoal

Responder

Gostei + 0

10/09/2013

Clayton Silva

Pessoal usei fiz update no meu codigo, usei paramentros e o StringBuilder sb = new StringBuider();
Obrigado Pessoal

Meus parab�ns.

Responder

Gostei + 0

10/09/2013

Rodrigo Odasaki

else if (id_projeto == 2)

Coloca um enum, seu c�digo ficara mais entend�vel.

Responder

Gostei + 0