This Program Calculate Checksum....Now I Want To Calculate Checksum Even I Enter Space , Comma Or Any Special Symbol Into The Textbox

Question

2.00/5 (3 votes)

See more:

C#

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace checksum
{
   public partial class Form1 : Form
   {
      public Form1()
      {
         InitializeComponent();
      }

      public static byte[] FromHex(string hex)
      {
         hex = hex.Replace("-", "");
         byte[] raw = new byte[hex.Length / 2];
         for (int i = 0; i < raw.Length; i++)
         {
            raw[i] = Convert.ToByte(hex.Substring(i * 2, 2), 16);
         }
         return raw;
      }

      private string CalculateChecksum(byte[] str)
      {
         int check = 0;
         foreach (byte chData in str)
         {
            check += chData;
         }
         check &= 0xff;
         return check.ToString("X2");
      }

      private void textBox1_TextChanged(object sender, EventArgs e)
      {

      }

      private void button1_Click(object sender, EventArgs e)
      {
         string hex;
         byte[] str;
         hex = textBox1.Text;
         str = FromHex(hex);

         textBox2.Text = CalculateChecksum(str);
      }

      private void textBox2_TextChanged(object sender, EventArgs e)
      {

      }
   }
}

Posted 15-Oct-14 20:30pm

Member 11145129

Updated 15-Oct-14 23:00pm

Nelek

v3

Add a Solution

Comments

BillWoodruff 16-Oct-14 3:15am

"Now I Need To Add When I Input Any Whitespace Or Comma" What does this mean ? Add what ? How does white-space of comma become part of a byte[] ?

Richard MacCutchan 16-Oct-14 4:25am

Just take the text as is rather than using the conversion from hex to byte.

Member 11145129 16-Oct-14 4:29am

it's not working ....showing 8 errors

Richard MacCutchan 16-Oct-14 4:36am

What is not working, what errors, what code?

Member 11145129 16-Oct-14 4:46am

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace checksum
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();

}

public static byte[] FromHex(string hex)
{

byte[] raw = new byte[hex.Length / 2];
for (int i = 0; i < raw.Length; i++)
{
raw[i] =(hex.Substring(i * 2, 2), 16);
}
return raw;
}

private string CalculateChecksum(byte[] str)
{
int check = 0;
foreach (byte chData in str)
{
check += chData;
}
check &= 0xff;
return check.ToString("X2");
}

private void button1_Click(object sender, EventArgs e)
{
string hex= textBox1.Text;

byte[] str;
str = FromHex(hex);
textBox2.Text = CalculateChecksum(str);
}

private void textBox2_TextChanged(object sender, EventArgs e)
{

}
}
}

Richard MacCutchan 16-Oct-14 4:49am

Please do not just dump code with no explanation. Update your question with all details of the problem.

Member 11145129 16-Oct-14 4:47am

i submitted after removing coversion ....

Member 11145129 16-Oct-14 4:47am

Error 2 Only assignment, call, increment, decrement, and new object expressions can be used as a statement C:\checksum\checksum\Form1.cs 29 48 checksum

Richard MacCutchan 16-Oct-14 4:51am

and please do not expect us to guess where line 29 is.

Member 11145129 16-Oct-14 5:03am

about my program :
when i enter value in textbox "abcd1234"
and i click on my button ...it takes )xab,0xcd,0x12,0x34 and store them in a byte aaray and add these values and give result in textbox2.
now,, i need the same thing even i give the input in text box as : "ab cd @ 12 # 34"..
means it must work after any special sybol in the same manner...plz help

Richard MacCutchan 16-Oct-14 7:29am

Then you need to write a string parsing routine which will remove all character that do not fall in the range 0-9, A-F. The String class has many methods that will help you.

2 solutions

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)

BillWoodruff · Answer 1 · 2014-10-16T01:41:00

Why not save yourself a lot of trouble and only allow the user to enter valid characters in the TextBox ? Like this:

C#

private string legalHex = "0123456789abcdefABCDEF";

private Keys legalKeys = Keys.Space | Keys.Back | Keys.Delete;

private void HexTextBox_KeyDown(object sender, KeyEventArgs e)
{
    e.SuppressKeyPress = ! (legalHex.Contains((char) e.KeyValue) || legalKeys.HasFlag(e.KeyCode));
}

Note that this allows space characters to be entered, so you'd need to clean-up the string before processing it by removing all space characters:

C#

private void btnProcessHexString_Click(object sender, EventArgs e)
{
    string currentHexString = HexTextBox.Text.Replace(" ", String.Empty);

    if (currentHexString.Length%2 != 0)
    {
        MessageBox.Show("you have not entered an even number of characters);

       return;
    }

    // process legal hex string
}

OriginalGriff · Answer 2 · 2014-10-15T22:40:00

Solution 1

I don't think you have quite understood what the difference is between Hex and Byte:
You probably don't want to do have anything to do with Hex as far as your input side is concerned: you want to use the entered data directly as Byte values.
So try just converting the input string to a byte array, and use that instead:

C#

string s = ...
byte[] bytes = System.Text.Encoding.Unicode.GetBytes(s);

Posted 15-Oct-14 22:40pm

OriginalGriff

Comments

BillWoodruff 16-Oct-14 6:25am

I think the OP wants to parse a string entered in hexadecimal format in a TextBox to create a byte[], treating each two-char pair as a hex expression of a byte. Why the OP wants to tolerate entering any old character into the TextBox ... a mystery :)