You have a loop going through the individual parts of what is a string.
What if the string isn't 10 characters long? The loop will try and read past the end of the string, which doesn't exist, and there you get an overflow error. Your best bet is to use:
And use a switch instead of a huge list of If statements, much cleaner.Code:list<byte> binaryValue = new List<byte>(); foreach(byte x in Convert.ToByte(yourText)) { binaryValue.add(x); }
He who asks is a fool for five minutes, but he who does not ask remains a fool forever.
The fool wonders, the wise man asks. - Benjamin Disraeli
There are no foolish questions and no man becomes a fool until he has stopped asking questions. Charles Steinmetz
Hi DanFraser
Thankyou very much for your reply. Will it be an optimal solution if i use switch cases?
Can i convert string to hexadecimal directly in c sharp?
Code:if (this.lvItem.SubItems[1].Text == "BINARY") { byte[] binaryValue = new byte[10]; for(int i = 0 ; i <10 ; i++) { binaryValue[i] = Convert.ToByte(editBox.Text)); } ManagedRegistry.Delete(nodeFullPath, selectedItem.Text); ManagedRegistry.WriteKeyBinary(nodeFullPath, keyName, binaryValue); }
Pls edit the code and reply me with the corrected code as i am unable to understand your logic. Pls help me as am a beginner to C# programming.
when i change the value of string of characters to 5, it still shows
OverFlow exception was Unhandled
Value was either too large or too small for an unsigned byte.
why don't you use try-catch to see what you have? Is the value in the string from 0-255?
