|
string sql = "UPDATE employees SET Firstname = '" + employeeTextBox.Text + "' Where EmployeeID = '" + employeeID + "'";
I am assuming the textbox is on the second form, therefore that control is correct ? Also, employeeId is an int, why is it stored as a string in the database ? If it's not, then change to this:
string sql = "UPDATE employees SET Firstname = '" + employeeTextBox.Text + "' Where EmployeeID = " + employeeID;
Note, I made a typing error, and so I lost the name of the textbox, and just made something up, you may need to fix that, to make it all work. The brackets are superfluous in either case.
And again, this is open to injection attacks. That means, someone can enter an employee name which erases your entire database.
Is this an assignment, or is it paid work ? If it's an assignment, I apologise for being a bit harsh previously, but if you've been given this code and told to work on it, that sounds like a job task. And, if that's true, you need to make the code more secure ( an assignment obviously is not as important, at this stage of your learning curve )
Christian Graus - Microsoft MVP - C++
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
This is a project that I am doing to help me learn but will also be used by our department if it is good enough. Please note I get paid the worst wage in the world so don't worry about that! The guy next to me has been told to help me out but he doesn't really like helping out so he quickly showed me the overloaded constructor and when I told him it doesn't work he didn't really continue helping. he has had 10 years coding experience. I thought it was a little harsh considering I never liked programming and I'm really trying to give myself the confidence that I'm good enough, so that definitely shot me right back down again. I also read on the forum for that people aren't supposed to write abusive things like that to you people, but it was the other way around.
rEgardless, thanks for your help.
|
|
|
|
|
OK - two things disturb me in the industry right now
1 - people in places like India, who can't realy code, but come here for help when they take jobs at $2 an hour
2 - people who are not trained to code, being pushed into doing it by their bosses.
My responses were based in part on my assuming that you were in category 1. Again, I apologise. In the second case, it's the bosses who I am mad at, not the people who are stuck with the job.
falles01 wrote: The guy next to me has been told to help me out but he doesn't really like helping out so he quickly showed me the overloaded constructor and when I told him it doesn't work he didn't really continue helping.
Talk to your boss. If you're being expected to learn to code on the job, and they told someone to teach you, then they will hold *you* accountable if you don't complain now, while they expect that you're working on it quite happily.
falles01 wrote: I'm really trying to give myself the confidence that I'm good enough
Programming isn't that hard, I'm sure you are 'good enough', it's just a question of learning, and, more correctly, you would ideally learn without being under pressure to create a product that was going to be used. My first two years of programming, looking back at anything I had written 3 months prior would make me cringe. But, if this is what you're stuck with, we're certainly here to help. Just make sure the guy who is assigned to helping you is also doing what he's paid to do, instead of relying on a bunch of unpaid help, which, by definition, is less likely to be there absolutely when you need it.
Christian Graus - Microsoft MVP - C++
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
Thanks. NO I am a good looking girl from Australia who has not got a personality for programming . ha ha, but I want to become a Business Analyst as I have an IT degree but I am a people person, so this will help me in the long run.
I also don't want to dop the guy in because he helps me sometimes, but my bosses also told me to work things out on my own. They don't like many questions here which is why I am seeking for other help. It took me 8 hrs to work out a problem yesterday that should've been so easy.
Thanks
|
|
|
|
|
falles01 wrote: They don't like many questions here which is why I am seeking for other help.
It sounds to me like they are not good people to work for. There's NOTHING wrong with asking questions, and everything wrong with NOT asking, when people around you are assuming you're doing fine.
falles01 wrote: It took me 8 hrs to work out a problem yesterday that should've been so easy.
Yeah, I've spent three nights on a problem like that ( learning WPF, a new framework ). It happens. I am in Australia too, so there's always at least one person on the boards who can help you, when you're at work.
The world needs more BAs who have some idea how to code, I hope it works out for you.
Christian Graus - Microsoft MVP - C++
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
Also, I wrote all the code myself from scratch. It has taken me 4 weeks so far. They gave me no code. The guy next to me just told me to use that small portion I gave you in order to reference the other class.
|
|
|
|
|
I am using this
string sql = "UPDATE employees SET (Firstname) = ('" + newempTextBox.Text.ToString() + "') Where EmployeeID = ('" + empNamecomboBox.Text.ToString() + "')";
that could be completely incorrect.
So basically I want to set the firstname to the users entered text in the new form, where they have selected a name from the drop down in the previous form (class).
Thank you for such a quick response. 
Sianny
|
|
|
|
|
Yes, this is a disaster.
First of all, if you're in another method, then the string you want is going to be what you passed in, not the exact same code which references a control that doesn't exist on this page.
Second, your code is completely insecure, and your database is compromised. You need to read up on SQL injections. Except, you say you've been programming for 2 months, so unless this is a class assignment ( which doesn't sound likely ), you should not be working on this at all.
Christian Graus - Microsoft MVP - C++
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
One way of doing .
Make a property in your combobox class and access your selecteditem through that property
|
|
|
|
|
But, the combobox is in another form....
Christian Graus - Microsoft MVP - C++
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
private DataSet GetLoginDataSetXML()
{
string ConnectionString=GetConnectionString();
OleDbConnection OleDbconSource = new OleDbConnection(ConnectionString);
OleDbCommand OleDbcmdSource = new OleDbCommand();
OleDbcmdSource.Connection = OleDbconSource;
OleDbcmdSource.CommandType = CommandType.Text;
OleDbcmdSource.CommandText = "SELECT * FROM [Contacts$]";
OleDbadpSource.SelectCommand = OleDbcmdSource;
OleDbadpSource.UpdateCommand = OleDbcmdSource;
DataSet dsSource = new DataSet();
OleDbadpSource .Fill(dsSource);
//code to modify data in dataset here
OleDbadpSource.Update(dsSource);
dsSource.Tables[0].Rows[2].ItemArray[0] = "ns1:sinkName";
dsSource.Tables[0].Rows[2].ItemArray[0] = "CRRHistoricalForecast:sinkName";
TRYING TO CHANGE "ns1:sinkName" to "CRRHistorical:sinkName" BUT NOTHING IS HAPPENING
return dsSource;
|
|
|
|
|
Hi.
I have 2 webcams connected, they are from different providers, (a Genius NB and another cam).
My application must be able to capure both cams, so, I have 2 picture box and I'm using the API function capGetDriverDescription to get the descripions of the cams as the following code shows:
public static Device[] GetAllDevices()<br />
{<br />
String dName = "".PadRight(100);<br />
String dVersion = "".PadRight(100);<br />
<br />
for (short i = 0; i < 10; i++)<br />
{<br />
if (capGetDriverDescriptionA(i, ref dName, 100, ref dVersion, 100))<br />
{<br />
Device d = new Device(i);<br />
d.Name = dName.Trim();<br />
d.Version = dVersion.Trim();<br />
<br />
devices.Add(d); <br />
}<br />
<br />
dName = "".PadRight(100);<br />
dVersion = "".PadRight(100);<br />
}<br />
<br />
return (Device[])devices.ToArray(typeof(Device));<br />
}
This is code is from an article from here, CP.
The problem is, that this code only get the description of 1 cam, I don't know why but capGetDriverDescriptionA always return true one time.
Both cams works well, even there's a program called SKIN distributed by my Genius cam that can shows both devices and you can select which one to use. I don't how this program does that.
But I need to show the video of 2 web cams at the same time. Which API could help me?
Thank you.
Demian.
"I have always wished that my computer would be as easy to use as my
telephone. My wish has come true. I no longer know how to use my telephone."
-Bjarne Stroustrup, computer science professor, designer of C++
programming language (1950- )
|
|
|
|
|
Hi guys,
In my programme, I have a scenario where I have a particular image displayed through a picturebox. When I try to delete the picture through the code, it gives an exception saying that it is accessed by another programme.
I tried using the PictureBox.Dispose() method and PictureBox.Image=null methods before the deletion code runs, to solve this.
But, neither worked.
What's wrong here?
Regards,
A
|
|
|
|
|
The file is still being held open from when you opened the file. How did you open it ? Do you call Dispose on the image itself ? Disposing of the picturebox doesn't help, and setting the image to null does not call Dispose on it.
Christian Graus - Microsoft MVP - C++
"I am working on a project that will convert a FORTRAN code to corresponding C++ code.I am not aware of FORTRAN syntax" ( spotted in the C++/CLI forum )
|
|
|
|
|
Well, I tried using dispose() on the picturebox.
I am displaying the image in the following way.
pictureBox.Image = System.Drawing.Image.FromFile(path);
How can I remove it from being held open?
|
|
|
|
|
I have a datagridview that I can rows using the down error and enterkey. I have DataGridView.Focus() firing after my method to addrows. The focus is on the proper cell but the cell is not active to type in. Does any have any suggestions on how to make the cell active?
|
|
|
|
|
you can select a cell with set the Selected property of that cell to true
like
this.dataGridView1[0, 0].Selected=true;
hope the post would be useful
good luck
|
|
|
|
|
Hello everyone!
I'm working on an application that converts hex data to decimal. Essentially what I do is send a command to an attached device via serial port and it spits back data to me.
The attached device spits data back in hex format and what I need to do is do a bit by bit comparision and convert some specific data to decimal format so I can do some calculations.
I'm at the point where the application I wrote spits this out:
55 55 FF AA 91 30 43 30 30 30 30 36 33 38 37 36 45 42 35 45 34 38 34 39 34 FF
The above data is just a sample. What I need to do is convert the data to decimal. The device that sends the data sends it back in asci format and i pass the data from a byte array into a function that converts it to hex digits.
What classes would I need to use to convert the data into decimal format? I have the initial data read into a byte array.
Sorry wasn't brief but I wanted to be as specific as possible
|
|
|
|
|
Int32.Parse(String, NumberStyles) First parameter is the string you want to convert to int. Use second parameter to specify that the string can be representation of hex number
|
|
|
|
|
Oh great thanks. I have tried that and the application crashes at runtime. So I ran the debugger and it says input string was not in correct format
//here is the method I created for this action
public int processArray(string data)
{
return Int32.Parse(data, NumberStyles.Integer);
}
here is where I call the method
string convertedS
convertedS = System.Text.Encoding.ASCII.GetString(buffer);
convert.processArray(convertedS);
maybe I'm doing something wrong from the very begining... Using .net 2.0 is it possible to read the data in the buffer I created for the serial port straight into a char array?
|
|
|
|
|
David Amyotte wrote: return Int32.Parse(data, NumberStyles.Integer);
Yes, it will produce an error because you are telling the compiler that the string is representation of number but in reality it is representation of HEX number so you should specify NumberStyles.HexNumber
|
|
|
|
|
You could convert the bytes directly using System.BitConverter.ToIntXX. Using a BinaryReader and a MemoryStream might be more convenient as you dont need to keep track of the array index.
|
|
|
|
|
thanks for your help so far.
What I'm doing so far is writing the data in the buffer into a Byte array then converting it to stringbuilder and the output so far is:
55 55 FF AA 91 30 43 30 30 30 30 36 33 38 37 36 45 42 35 45 34 38 34 39 34 FF
what I need to do is find an easy way to extract this data:
36 33 38 37 36 45 42 35 45 34 and convert it to decimal so I can do some calculations on the data. I'm new to .net so I figured this would be a good training exercise for me
Regards
|
|
|
|
|
It depends on the format of the data, you will need to look it up or work it out.
36 could be a byte
36 33 could be short
36 33 38 37 could be an int/float
36 33 38 37 36 45 42 35 could be a long/double
...
...
Example if 36 33 38 37 36 45 42 35 45 34 consits of 5 shorts:
byte[] data = {0x36, 0x33, 0x38, 0x37, 0x36, 0x45, 0x42, 0x35, 0x45, 0x34};
BinaryReader r = new BinaryReader(new MemoryStream(data));
for (int i = 0; i < 5; i++)
Console.WriteLine(r.ReadInt16());
|
|
|
|
|
In my old c++ days, I would use my old function below;
BOOL IsValidFilename(const char *szFilename)<br />
{<br />
<br />
if ( (!szFilename) || (!strlen(szFilename)) )<br />
return (FALSE);<br />
<br />
if ( (strchr(szFilename, '.')) || (strchr(szFilename, '\\')) ||<br />
(strchr(szFilename, '/')) || (strchr(szFilename, ':')) ||<br />
(strchr(szFilename, '<')) || (strchr(szFilename, '>')) ||<br />
(strchr(szFilename, '*')) || (strchr(szFilename, '?')) ||<br />
(strchr(szFilename, '|')) || (strchr(szFilename, '\"')) )<br />
return (FALSE);<br />
<br />
return (TRUE);<br />
}
I thought there might be a cleaner way to test a string before I used it for a file name.
Programmer
Glenn Earl Graham
Austin, TX
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
