I need to save with a simple condition

Israel

Hi!

I have on my form one label (lblMsgSend) with the text "Sended". I need with these codes when there datas that already exist it cant show this label's message "Sended".

Right now when I click to my save's button (even it found datas already exist its show the label's message) . How can I do please.

conn.Open();

OleDbCommand comm = new OleDbCommand();

comm.Connection = conn;

connstr = "SELECT COUNT(*) FROM propTrans WHERE [nome] = @nome";

comm.CommandText = connstr;

comm.Parameters.Clear();

comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);

comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);

comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);

int numRecords = (int)comm.ExecuteScalar();

if (numRecords == 0)

{

connstr = "INSERT INTO propTrans([nome],[tipopagamento],[mes])VALUES(@nome,@tipopagamento,@mes)";

comm.CommandText = connstr;

comm.Parameters.Clear();

comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);

comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);

comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);

comm.ExecuteNonQuery();

conn.Close();

/////////////////////////////////////////////////////////////////////////////////

timer1.Start();

lblMsgSend.Visible = true;

7 Replies

Know the answer? Post it — somebody with the same question will find it here.

Thrish10y ago
Hi,
try like below:

conn.Open();
OleDbCommand comm = new OleDbCommand();
comm.Connection = conn;
connstr = "SELECT COUNT(*) FROM propTrans WHERE [nome] = @nome";
comm.CommandText = connstr;
comm.Parameters.Clear();
comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);
comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);
comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);
int numRecords = (int)comm.ExecuteScalar();

//Below condition says: if the no.of.records==0 then insert and display label as 'Sended'
if (numRecords == 0)
{
connstr = "INSERT INTO propTrans([nome],[tipopagamento],[mes])VALUES(@nome,@tipopagamento,@mes)";
comm.CommandText = connstr;
comm.Parameters.Clear();
comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);
comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);
comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);
comm.ExecuteNonQuery();
conn.Close();
MessageBox.Show("Datas saved ..");
lblMsgSend.Text="Sended";
lblMsgSend.Visible = true;
}
//If no.of.records!=0 then display a message saying 'data already exists' and don't display label
else
{
MessageBox.Show("Datas already existentes");
lblMsgSend.Visible = false;
}
Israel10y ago
But my big target was this: when the data is already in the database the message "sended" its should not appear. Until there it's fine!
Curiously when there is not data into database mean that the message "sended" should appear. At this stage its doesnt appear. There is a problem here.
In conclusion, if datas are already in even the last codes to send SMS cannot be read it.
Thrish10y ago
Hi,
In your code, there is extra else block. That is the reason you are getting Invalid expression term 'else'
Israel10y ago
Dearest Sirs, all your solution working well. Now I dont know which one I should choose now.
Anywhere, thanks to ali tuncer, Amit Kumar, Ravi Patel.

But I have a small logic problem. I add others codes but its sturk on this line and cant continue.
Error 1 Invalid expression term 'else'
Error 2 ; expected

conn.Open();
OleDbCommand comm = new OleDbCommand();
comm.Connection = conn;
connstr = "SELECT COUNT(*) FROM propTrans WHERE [nome] = @nome";
comm.CommandText = connstr;
comm.Parameters.Clear();
comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);
comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);
comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);
int numRecords = (int)comm.ExecuteScalar();
if (numRecords == 0)
{
connstr = "INSERT INTO propTrans([nome],[tipopagamento],[mes])VALUES(@nome,@tipopagamento,@mes)";
comm.CommandText = connstr;
comm.Parameters.Clear();
comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);
comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);
comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);
comm.ExecuteNonQuery();

MessageBox.Show("Datas saved ..");
}
else
{
MessageBox.Show("Datas already existentes");
}
conn.Close();
lblMsgSend.Text="Sended"; // here its OK
lblMsgSend.Visible = true; // here its OK
}
else
{
lblMsgSend.Visible = false;// here its OK
}
lblMsgEnviadaPropinas.Text = "Sended";
lblMsgEnviadaPropinas.Visible = true;
}
else //Its stop here giving that error messd
{
lblMsgEnviadaPropinas.Visible = false;
}
try
{
WebClient client = new WebClient();
string to, msg;
to = lblNumeroMovelPropinas.Text;
msg = txtReceive.Text;
string baseurl = "http://api.clickatell.com/http/sendmsg?user=Leader2000&password=LPYaGgMLAVNYFH&api_id=3497416&to='" + to + "'&text='" + msg + "'";
client.OpenRead(baseurl);
MessageBox.Show("Mensagem enviada com exito...");
}
catch (Exception ex)
{
MessageBox.Show(ex.Message);
}
Ravi Patel10y ago
Hi ,

try this

conn.Open();
OleDbCommand comm = new OleDbCommand();
comm.Connection = conn;
connstr = "SELECT COUNT(*) FROM propTrans WHERE [nome] = @nome";
comm.CommandText = connstr;
comm.Parameters.Clear();
comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);
comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);
comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);
int numRecords = (int)comm.ExecuteScalar();
if (numRecords == 0)
{
connstr = "INSERT INTO propTrans([nome],[tipopagamento],[mes])VALUES(@nome,@tipopagamento,@mes)";
comm.CommandText = connstr;
comm.Parameters.Clear();
comm.Parameters.AddWithValue("@nome", cbxNomePropinas.Text);
comm.Parameters.AddWithValue("@tipopagamento", cbxTipoPagamentoPropinas.Text);
comm.Parameters.AddWithValue("@mes", cbxMesPropinas.Text);
comm.ExecuteNonQuery();
conn.Close();
lblMsgSend.Text="Sended";
lblMsgSend.Visible = true;
}
else
{
lblMsgSend.Visible = false;
}
Amit Kumar10y ago
hi dear,
use it..

if (numRecords == 0) //if data does not exist.
{
lblMsgSend.Visible = false;
}
else // if data is exists
{
lblMsgSend.Visible = true;
}
ali tuncer10y ago
if (numRecords == 0) //data does not exist
{
//your code
lblMsgSend.Visible = true;
}
else //data exists
{
lblMsgSend.Visible = false;
}