|
This[^] thread may help.
------------------------------------
I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
CCC League Table Link
CCC Link[ ^]
|
|
|
|
|
I have receieved this error , what is the wrong ?
Parameterized Query '(@StartDate nvarchar(6),@EndDate nvarchar(6),@StartTime nvarchar' expects parameter @MarkedListNumber, which was not supplied
SqlCommand cmd = new SqlCommand();
cmd.CommandText = "update VopMessage_tbl set StartDate=@StartDate , EndDate=@EndDate , StartTime=@StartTime , EndTime=@EndTime where MarkedListNumber=@MarkedListNumber";
cmd.Connection = con;
cmd.Parameters.Add("@StartDate", SqlDbType.NVarChar);
cmd.Parameters.Add("@EndDate", SqlDbType.NVarChar);
cmd.Parameters.Add("@StartTime", SqlDbType.NVarChar);
cmd.Parameters.Add("@EndTime", SqlDbType.NVarChar);
cmd.Parameters.Add("@MarkedListNumber", SqlDbType.Int);
con.Open();
for (int t = 0; t < messageSequence.Count; t++)
{
MessageInfo minfo = (MessageInfo)messageSequence[t];
for (int f = 0; f < Set30bytesMessage.Count; f++)
{
VopMessage vms = (VopMessage)Set30bytesMessage[f];
if (minfo.ListNumber == vms.ListNumber)
{
cmd.Parameters["@StartDate"].Value = minfo.StartDate;
cmd.Parameters["@EndDate"].Value = minfo.EndDate;
cmd.Parameters["@StartTime"].Value = minfo.StartTime;
cmd.Parameters["@EndTime"].Value = minfo.EndTime;
}
}
}
cmd.ExecuteNonQuery();
con.Close();
|
|
|
|
|
You're adding a parameter here:
cmd.Parameters.Add("@MarkedListNumber", SqlDbType.Int);
But you're not giving it a value within your for loop.
The error message itself pretty much tells you what is wrong.
The difficult we do right away...
...the impossible takes slightly longer.
|
|
|
|
|
|
Furthermore you seem to be storing dates and times as strings, which is a bad idea for several reasons:
- they don't compare and sort well;
- they are prone to formatting issues;
- they possibly would not be understood under a different language;
- they take up more space.
It is much better to use the actual date, time, or datetime field types your database supports.
|
|
|
|
|
Well spotted Luc.
Luc Pattyn wrote: they possibly would not be understood under a different language;
It's a new thing for me. Thanks 
BTW Congrats for the CP MVP ![Rose | [Rose]](https://codeproject.global.ssl.fastly.net/script/Forums/Images/rose.gif)
|
|
|
|
|
Thanks Raja. And to you too!
|
|
|
|
|
You are also calling cmd.ExecuteNonQuery(); outside of your loop so only the last values entered will be updated in the database. You were advised of this before.
Further more, as you were advised before, you should be using the using statement. The database connection is not being closed properly because the exception is being thrown before your conn.Close() call. The using statement will prevent this.
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
Hi All,
I am working on a designer which I wish to use for various task, I have however come to a problem which does not allow me to move forward.
according to this link http://msdn.microsoft.com/en-us/library/ms754193.aspx the XAML writer does not preserve event handling during serialization, however I need to use this feature or something very similar. It would be helpful if someone could please provide me with some resources, or links where I can learn how to do this. I am not necessarily looking for codes or pseudo-codes, just some gearing towards the steps or road to take. Thank you.
Yan
|
|
|
|
|
What exactly do you think that a serialized event handler would represent?
What do you expect it to do when you deserialize an event handler?
|
|
|
|
|
Using Windows Form, you cannot directly serialize Buttons or TextBox and other controls, but in XAML you can do this. however if once I have created the button and attached a "click" event per say, then when serializing the button, the xaml serializer will produce and error because of that click event.
My main goal is able to create a user interface, attach event to different controls, and being able to saved (serialize) them to a file, and the same way the other way around.
I hope I was clear enough.
Yanick
|
|
|
|
|
How about writing a pre-processing method that reflects all controls, unregistering each event and saving it to a textfile? You could then generate a new assembly based on the list of events in your textfile that re-attaches the eventhandlers
I are Troll 
|
|
|
|
|
Hi,
Thanks a lot for your input, by using your suggestion I guess I will have to use reflection. I was hoping not to, and maybe try another maybe simpler avenue to achieve the same result because I have not worked with reflections as of yet.
Regards
Yanick
|
|
|
|
|
Serval Soft wrote: Using Windows Form, you cannot directly serialize Buttons or TextBox and other controls, but in XAML you can do this
No.
The XAML contains a description which maps to a control. It doesn't contain the control itself.
http://blogs.msdn.com/b/mhendersblog/archive/2005/10/07/478284.aspx[^]
If you want some arbitrary event handler to do the same then you need to provide sufficient information to provide a description (not the event handler) so it can be mapped as well.
I would suspect that there might even be some fairly easy way to do that if one wrapped the event handler in a class and derived from the correct object. However depending on the layout I would suspect it would be easier and more correct to create your own controls which have event handlers and then do the appropriate work to map those controls into XAML.
|
|
|
|
|
Hi. Newest way to create a modern dictionary. For example, what bank is used or what is interface design?
Thanks! 
|
|
|
|
|
I'm sorry, but that doesn't make any sense as a question. I am sure that English is not your native language, but please try to give more detail on what the problem is, so that we can help. It may be worth describing it is small words in your native language and then using Google Translate to get it into English.
Real men don't use instructions. They are only the manufacturers opinion on how to put the thing together.
|
|
|
|
|
Sorry - but what are you talking about?
The funniest thing about this particular signature is that by the time you realise it doesn't say anything it's too late to stop reading it.
My latest tip/trick
|
|
|
|
|
Try here[^]
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
you question is not clear. Please clarify.
|
|
|
|
|
do you mean, how to implement IDictionary?
|
|
|
|
|
I want to update Some Value to Marked Number Value in MarkedNumber Column
I Want to know where's the error in this code
public void UpdateMessageToIncludeMarkedNumber()
{
#region Compare to Insert Marked Number With Matching Message into Table in Database.
SqlCommand cmd = new SqlCommand();
cmd.CommandText = "update VopMessage_tbl set MarkedNumber=@MarkedNumber where Byte24=@Byte24 and Byte4=@Byte4";
cmd.Connection = con;
for (int t = 0; t < Set10bytesMessage.Count; t++)
{
MarkedNumber mNumber = (MarkedNumber)Set10bytesMessage[t];
for (int f = 0; f < Set30bytesMessage.Count; f++)
{
VopMessage vms = (VopMessage)Set30bytesMessage[f];
if (mNumber.Byte4 == vms.Byte24 & mNumber.Byte5 == vms.Byte4)
{
cmd.Parameters.AddWithValue("@Byte24", vms.Byte24);
cmd.Parameters.AddWithValue("@Byte4", vms.Byte4);
string All2BytesOfTwoMarkedNumber = mNumber.Byte4 + mNumber.Byte5;
cmd.Parameters.AddWithValue("@MarkedNumber", All2BytesOfTwoMarkedNumber);
}
else
{
}
}
}
con.Open();
cmd.ExecuteNonQuery();
con.Close();
#endregion
}
|
|
|
|
|
You are only processing the last item in in Set30bytesMessage for the last item in Set10bytesMessage, if at all???
You repeatedly add the parameters in the the loop, so each parameter is potentially added more than once. Without knowing exactly what you are doing it is hard to tell but I'd hazard a guess and say you need to do this:
- The you should add the parameters before the loop
- The values of the parameters should be set inside the loop
- The
ExecuteNonQuery should happen inside the loop for each item
Is that any help? Some clue about what is going wrong/ what you expect would be good!
|
|
|
|
|
Here's the Error
The variable name '@Byte24' has already been declared. Variable names must be unique within a query batch or stored procedure.
If I put : cmd.Parameters.AddWithValue("@Byte24", vms.Byte24);
cmd.Parameters.AddWithValue("@Byte4", vms.Byte4);
out side loop , from where can I get the value of vms.Byte24
|
|
|
|
|
See my previous reply, my guess was correct!
public void UpdateMessageToIncludeMarkedNumber()
{ SqlCommand cmd = new SqlCommand();
for (int t = 0; t < Set10bytesMessage.Count; t++)
{
MarkedNumber mNumber = (MarkedNumber)Set10bytesMessage[t];
for (int f = 0; f < Set30bytesMessage.Count; f++)
{
VopMessage vms = (VopMessage)Set30bytesMessage[f];
if (mNumber.Byte4 == vms.Byte24 & mNumber.Byte5 == vms.Byte4)
{
<big>
}
}
}
con.Open();
<big>
cmd.ExecuteNonQuery();
con.Close();
}
}
|
|
|
|
|
+5 on the original answer, but wouldn't clearing the Parameters collection more efficient? He'd be reusing them for the same task, which seems logical.
Just curious 
I are Troll 
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
