Want to win a PS4? Go Premium and enter to win our High-Tech Treats giveaway. Enter to Win

x
?
Solved

Converting C code into VB.Net 2008

Posted on 2009-05-20
6
Medium Priority
?
350 Views
Last Modified: 2012-05-07
Hello,

following is the code in C used in Modbus CRC calculation. Can someone convert this into VB.Net code since I am not good at C?

Regards,
MI
void ModbusRTU_CalculateCrc(void)
{
	BYTE ucCount = RESET,ucNoOfBits = RESET;
	WORD iCrc;
	
	iCrc = 0xFFFF ^ (*g_pucCRC_Buffer);
	
	for(ucCount = 1; ucCount <= g_ucCRCBufferCount ; ucCount++)
	{
		for(ucNoOfBits = 0; ucNoOfBits <= 7; ucNoOfBits++)
		{
			if((iCrc & 0x01) == 0X01)
			{
				iCrc >>= 1;
	        	iCrc = iCrc^0xA001;
			}	
			else
			{
				iCrc >>= 1;
            }
	 	}
		
		if(ucCount != g_ucCRCBufferCount)
		{
			iCrc = iCrc^(*(++g_pucCRC_Buffer));
		}
		else
		{
		}
	 }
	 
	//Extract lower byte crc.
	g_ucCRCLowByte = iCrc&0x00FF;
	//Extract higher byte of crc.
	g_ucCRCHighByte = ((iCrc>>8)&0x00FF);
	//End of function ModbusRTU_CalculateCrc()
}

Open in new window

0
Comment
Question by:vj_mi
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 3
  • 2
6 Comments
 
LVL 15

Expert Comment

by:oobayly
ID: 24435665
Rather than reinventing the wheel, I did a quick google:
http://modbus.control.com/thread/1026164935

It appears to be VB6 code, but it's quite easy to translate into VB.Net
0
 
LVL 15

Expert Comment

by:oobayly
ID: 24435941
Saying that, I'm a masochist. Best test that it works properly though!
A couple of things: The C code uses DWORD, which is an unsigned 16bit integer. As VB.Net isn't too keen on UInt16, I've used a 32bit integer instead.
I've used a cross between the C code and the VB code in the link. I didn't like the C loop, and I didn't like the VB bitshifting (mainly because VB6 doesn't have a bitshift operator)
Private Shared Function ModbusCRC(ByVal buff As Byte()) As Integer
    Dim crc As Integer = &Hff
    For i As Integer = 0 To buff.Length - 1
        crc = crc Xor buff(i)
        For j As Integer = 0 To 7
            If (crc And &H1) = &H1 Then
                crc >>= 1
                crc = crc And &Ha001
            Else
                crc >>= 1
            End If
        Next
    Next
    
    ' Lower & Upper bytes (if you need them)
    Dim lower As Byte = CByte(crc And &Hff)
    Dim upper As Byte = CByte((crc >> 8) And &Hff00)
    
    Return crc
End Function

Open in new window

0
 
LVL 75

Accepted Solution

by:
käµfm³d   👽 earned 2000 total points
ID: 24437096
Translated from your code (mind the notes in the comments)
Public Sub ModbusRTU_CalculateCrc()
    'Assumes you have the next line declared globally elsewhere
    'Dim g_pucCRC_Buffer, g_ucCRCBufferCount, g_ucCRCLowByte, g_ucCRCHighByte As Integer
 
    'Assumes RESET equals 0
    Dim ucCount As Integer = 0, ucNoOfBits As Integer = 0
    Dim iCrc As Integer
 
    iCrc = &HFFFF Xor g_pucCRC_Buffer
 
    For ucCount = 1 To g_ucCRCBufferCount
        For ucNoOfBits = 0 To 7
            If iCrc And &H1 = &H1 Then
                iCrc >>= 1
                iCrc = iCrc Xor &HA001
            Else
                iCrc >>= 1
            End If
        Next
 
        If ucCount <> g_ucCRCBufferCount Then
            g_pucCRC_Buffer += 1
            iCrc = iCrc Xor g_pucCRC_Buffer
        End If
    Next
 
    g_ucCRCLowByte = iCrc And &HFF
    g_ucCRCHighByte = (iCrc >> 8) And &HFF
 
End Sub

Open in new window

0
What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

 
LVL 15

Expert Comment

by:oobayly
ID: 24439308
Just noticed my code calculates upper incorrectly, originally I had done the bitshift after the bitwise and.
Also, ignore my comments on DWORD, I was reading it as WORD by mistake.
Where was my mind when I posted the answer? <sigh>
Dim upper As Byte = CByte((crc >> 8) And &Hff)

Open in new window

0
 

Author Comment

by:vj_mi
ID: 24452937
Hi friends,

I am yet to test the above code snippets since we got bounced by the data that we send to serial port. Unless that code is fixed, I cannot run the above code snipper. Please bear with me for a day or two. Meantime, I will post the serial port issue as a new thread.

Regards,
MI
0
 

Author Comment

by:vj_mi
ID: 24503008
oobayly:
The code you have mentioned (along with the rectification you asked for returns both lower and upper bytes as 0.

kaufmed:
I had to modify the code slightly for it to work...

Regards,
MI
0

Featured Post

Technology Partners: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

I think the Typed DataTable and Typed DataSet are very good options when working with data, but I don't like auto-generated code. First, I create an Abstract Class for my DataTables Common Code.  This class Inherits from DataTable. Also, it can …
Introduction As chip makers focus on adding processor cores over increasing clock speed, developers need to utilize the features of modern CPUs.  One of the ways we can do this is by implementing parallel algorithms in our software.   One recent…
In this video, Percona Solution Engineer Dimitri Vanoverbeke discusses why you want to use at least three nodes in a database cluster. To discuss how Percona Consulting can help with your design and architecture needs for your database and infras…
In this video, Percona Director of Solution Engineering Jon Tobin discusses the function and features of Percona Server for MongoDB. How Percona can help Percona can help you determine if Percona Server for MongoDB is the right solution for …
Suggested Courses

610 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question