|
It works now. The semicolon was missing.
|
|
|
|
|
If you get an error, it helps if you tell us that, than just assuming we will have seen that on your screen!
You will be solving syntax errors often - we all do - so this may help you get started: How to Write Code to Solve a Problem, A Beginner's Guide Part 2: Syntax Errors[^]
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Please do not delete your original question, as the full thread could be helpful to other people. Just add "[Solved]" to the message's title.
|
|
|
|
|
My insert code is not working fine, bcz if i click radionbutton2 and checkbox 2 means it was inserting data automatically radionbutton1 and checkbox1 only, actually if which textbox is selected that data should enter null value radionbutton data should not saved in db
if (!string.IsNullOrEmpty(textBox11.Text) && !string.IsNullOrEmpty(textBox12.Text) && !string.IsNullOrEmpty(textBox13.Text) && !string.IsNullOrEmpty(textBox14.Text) && (radioButton1.Checked || radioButton2.Checked) && (checkBox1.Checked || checkBox2.Checked))
{
string connectionString;
MySqlConnection cnn;
connectionString = @"Data Source=localhost;Initial Catalog=testDB;User ID=root;Password=mysql";
cnn = new MySqlConnection(connectionString);
cnn.Open();
string id = textBox14.Text;
string name = textBox11.Text;
string year = textBox12.Text;
string quality = radioButton1.Text == "" ? radioButton2.Text : radioButton1.Text;
string taste = checkBox1.Text == "" ? checkBox2.Text : checkBox1.Text;
string sales = textBox13.Text;
textBox11.Text = "";
textBox12.Text = "";
textBox13.Text = "";
textBox14.Text = "";
radioButton1.Text = "";
radioButton2.Text = "";
checkBox1.Text = "";
checkBox2.Text = "";
string query = "INSERT INTO fruits VALUES(@fruitsid, @fruitsname, @fruitsyear, @quality, @taste, @sales)";
using (MySqlCommand cmd = new MySqlCommand(query))
{
cmd.Parameters.AddWithValue("@fruitsid", Convert.ToInt32(id));
cmd.Parameters.AddWithValue("@fruitsname", name);
cmd.Parameters.AddWithValue("@fruitsyear", year);
cmd.Parameters.AddWithValue("@quality", quality);
cmd.Parameters.AddWithValue("@taste", taste);
cmd.Parameters.AddWithValue("@sales", sales);
cmd.Connection = cnn;
cmd.ExecuteNonQuery();
MessageBox.Show("Record Inserted Successfully");
DisplayData();
cnn.Close();
}
}
else
{
MessageBox.Show("Please Fill Data");
}
|
|
|
|
|
Quote: My insert code is not working fine, bcz if i click radionbutton2 and checkbox 2 means it was inserting data automatically radionbutton1 and checkbox1 only, actually if which textbox is selected that data should enter null value radionbutton data should not saved in db
The question is meaningless, you still haven't learned how to use a debugger, and you are still trying to get others to fix problems instead of thinking about what is actually going on.
what have you tried to work out what the problem might be, given that we can't understand your "explanation" of the problem, run your code, see your inputs, or access your DB?
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
And you are still posting a success message even if it fails.
cmd.ExecuteNonQuery();
MessageBox.Show("Record Inserted Successfully");
|
|
|
|
|
Optimism is a moral duty...
Luc Pattyn [My Articles]
The Windows 11 "taskbar" is disgusting. It should be at the left of the screen, with real icons, with text, progress, etc. They downgraded my developer PC to a bloody iPhone.
|
|
|
|
|
I'm sorry but i don't understand your question. One thought I have - are you trying to do an 'exclusive or'? In other words, are you trying to see if button 1 is selected, or button 2 is selected, but not both at the same time?
As others have pointed out a few different times in response to your questions - you cannot assume a SQL command is successful just because control moves to the next statement. You need to check to see if you were able to perform the SQL statement. You may lose your connection, you may not be authorized to the database, the update may have failed to run. There are ways to see if the SQL statement worked before you proceed to posting a 'success' message.
|
|
|
|
|
yes sir need to select any one at a time. and i am using mysql not sql
|
|
|
|
|
You will need to research how to do an "exclusive or" to learn the syntax.
As for my SQL comment - it still applies to MySQL. You must learn how to tell if the SQL command ran successfully. You cannot assume that it ran as desired just because control flows to the next statement. This is why others are telling you to check for sucess on your SQL operation before you proceed to the statement that says it succeeded.
When you become a professional programmer, you need to worry about things like "Do I have authority to access the database? Is the database online? Did some other unexpected condition occur?" Much of programming is dealing with gracefully handling errors and unexpected conditions.
CodeProject members are professionals and their tips and suggestions are to help you achieve that skill level.
|
|
|
|
|
hi all
how i get the dpi scale factor is selected.
thanks.
|
|
|
|
|
Where are you going to get it from?
|
|
|
|
|
|
var graphics = control.CreateGraphics())
int dpi_scale_val= graphics.DpiX;
but it always return 96
|
|
|
|
|
Le@rner wrote: but it always return 96 That is correct.
|
|
|
|
|
i have 4k laptop its by default resolution is 200%
|
|
|
|
|
If you want help with a programming question, you really need to provide proper details, rather than vague statements.
|
|
|
|
|
|
|
"200%" is not a resolution. It's a scaling factor used to enlarge text when displayed.
The display dpi is still 96.
One has nothing to do with the other.
|
|
|
|
|
yes i want this scaling factor in c#
|
|
|
|
|
|
Name of the school/uni?
Bastard Programmer from Hell
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.
|
|
|
|
|
DPI scaling is not that simple.
Here's how I do it for WinForms apps:
First, your app needs to be DPI-aware. You set that by the manifest. You also set that with with the AutoScaleDimensions and AutoScaleMode properties on each form.
Second, if you're designing forms from within Visual Studio, you need to do it on a system that is using 100% as the scaling factor for the display. If you don't do this, step 3 doesn't work.
Third, you can now use
scaleFrom96 = this.DeviceDpi / 96.0 to calculate a multiplication factor for resizing those things manually that WinForms will not do for you. Be warned, it is very hit and miss what things will be scaled for you automatically and what things will not. Lots of testing required.
OR you punt it all and design on a system that matches the system the app will be running on.
Judy
Be wary of strong drink. It can make you shoot at tax collectors - and miss.
Lazarus Long, "Time Enough For Love" by Robert A. Heinlein
|
|
|
|
|
Judy,
Great answer, I hope to see you answering more C# related questions in the future.
The Windows API equivalent to what you are doing there would be calling the GetDpiForMonitor function[^] and obtaining the MDT_EFFECTIVE_DPI[^] value and dividing by the system default. Of course the operating system hides dpi/scaling values from threads unless it's DPI aware[^]. So the caller needs to set thread DPI awareness before performing the calculation.
I'd like to add that GetScaleFactorForMonitor[^] may return incorrect values. The manual calculation is preferred.
I'd also like to add that the process doesn't necessarily need to be DPI aware. Beginning with Win10 1607 you can create a DPI aware thread to get those values by using the SetThreadDpiAwarenessContext function[^].
Hope to see you answering more often!
Best Wishes,
-David Delaune
|
|
|
|