How Do I Fix This Update Query? C#.Net Using Mysql Wb

Question

0.00/5 (No votes)

See more:

hello guys, how do i fix this kind of error, i used all kinds of insert query, but nothing is working, so i decided to coding again back to the beginning. and still getting this error, it's so very frustrating. anyone would like to help me?,

this message always prompt: "You have an error in your SQL syntax; check the manual that corresponds to your Mysql server version for the right syntax to use near 'Description = 'ink Cartridges', Item Code = '0012', Stock = 'System.Windows.Form.' at line 1.

C#

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using MySql.Data.MySqlClient;

namespace ToolManagementTest
{
    public partial class Form5 : Form
    {
        public Form5()
        {
            InitializeComponent();
        }

        public void btn1_Click(object sender, EventArgs e)
        {
            try
            {
            string modConnection = "datasource=localhost;port=3306;username=root;password=1234";
            string modQuery = "UPDATE toolsmanagement.itemlist SET Item Description = '" + this.modItemTxt.Text + "',Item Code = '" + this.modCodeTxt.Text + "',Stock = '" + this.modStockTxt + "',Unit = '" + this.modUnitTxt.Text + "',Item Type = '" + this.modTypeTxt.Text + "',Item Status = '" + this.modStatTxt.Text + "' WHERE Item Code = '" + this.modCodeTxt.Text + "'";

            MySqlConnection modData = new MySqlConnection(modConnection);
            MySqlCommand modCommandData = new MySqlCommand(modQuery, modData);
            MySqlDataReader MyReader;
         

            
                modData.Open();
       MyReader = modCommandData.ExecuteReader();                                  
                MessageBox.Show("Registered Data..");
                
                this.Hide();
                while (MyReader.Read())
                {

                }
                modCommandData.ExecuteNonQuery();
                modData.Close();

            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
        }

       
    }
}

Posted 4-Jun-15 17:11pm

Member 11707739

Updated 4-Jun-15 19:10pm

Gihan Liyanage

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

DamithSL · Accepted Answer · 2015-06-04T17:23:00

change this.modStockTxt to this.modStockTxt.Text
other than that you better use parameters instead of concatenating strings to build sql statement, your application is widely open for sql attacks now.
sample sql statement with parameters:

SQL

UPDATE toolsmanagement.itemlist SET 
     `Item Description` = @ItemDescription,
     `Stock` = @Stock,
     `Unit` = @Unit,
     `Item Type`= @ItemType,
     `Item Status` = @ItemStatus 
WHERE 
     `Item Code` = @ItemCode

Note that when you have space in column names, use ``
since you have update statement you don't need MySqlDataReader , create MySqlCommand using parameterized sql statement as above and then you can set each parameter values as below

C#

cmd.Parameters.AddWithValue("@ItemDescription", this.modItemTxt.Text);
// do the same for all other parameters

finally execute the command

C#

cmd.ExecuteNonQuery();