Массив назначения недостаточно длинный

Question

Я пытаюсь установить последовательную связь от Arduino до C # в Visual Studio.Вот мой кодЯ использую таймер для запроса и получения данных.

private void timer1_Tick(object sender, EventArgs e)
    {
        isConnected = true;
        port.Write("A");

        int bytes = port.BytesToRead;
        byte[] buffer = new byte[bytes];
        port.Read(buffer, 0, bytes);

        for (int i = 0; i < bytes; i++)
        {
            int potensio = BitConverter.ToInt32(buffer, 0);
            string potString = Convert.ToString(potensio);
            label1.Text = potString;
        }
    }

Ошибка в этой строке int potensio = BitConverter.ToInt32(buffer, 0);

    System.ArgumentException
      HResult = 0x80070057
      Message=Destination array is not long enough to copy all the items in the collection.Check array index and length.
      Source= mscorlib
      StackTrace:
       at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
       at System.BitConverter.ToInt32(Byte[] value, Int32 startIndex)
       at SerialTest.Form1.timer1_Tick(Object sender, EventArgs e) in C:\Users\ThinkPad\source\repos\SerialTest\SerialTest\Form1.cs:line 73
       at System.Windows.Forms.Timer.OnTick(EventArgs e)
       at System.Windows.Forms.Timer.TimerNativeWindow.WndProc(Message& m)
       at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
       at System.Windows.Forms.UnsafeNativeMethods.DispatchMessageW(MSG& msg)
       at System.Windows.Forms.Application.ComponentManager.System.Windows.Forms.UnsafeNativeMethods.IMsoComponentManager.FPushMessageLoop(IntPtr dwComponentID, Int32 reason, Int32 pvLoopData)
       at System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context)
       at System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context)
       at System.Windows.Forms.Application.Run(Form mainForm)
       at SerialTest.Program.Main() in C:\Users\ThinkPad\source\repos\SerialTest\SerialTest\Program.cs:line 19

Answer 1

к;давайте попробуем исправить это;проблема в том, что данные не всегда поступают аккуратно и аккуратно.В вашем случае вы хотите обрабатывать 4 байта за раз, поэтому: давайте попробуем это:

byte[] buffer = new byte[4];
int bytesBuffered = 0;
bool inProgress = false;
private void timer1_Tick(object sender, EventArgs e)
{
    if (!inProgress) {
        // presumably this means "send me the current data?"
        inProgress = true;
        port.Write("A");            
    }

    // read whatever we still need to make 4, if available
    int bytes = Math.Min(port.BytesToRead, 4 - bytesBuffered);
    if (bytes <= 0) return; // nothing to do right now

    // read the next few bytes, noting the offset
    bytes = port.Read(buffer, bytesBuffered, bytes);
    // TODO: check if bytes is <= 0 - if so, the port may have disconnected
    bytesBuffered += bytes;

    // check whether we have enough to update the UI
    if (bytesBuffered == 4)
    {
        // we now have 4 bytes; update the UI, and reset
        int potensio = BitConverter.ToInt32(buffer, 0);
        string potString = Convert.ToString(potensio);
        label1.Text = potString;

        // and issue a new A next time
        bytesBuffered = 0;
        inProgress = false;
    }
}

Примечание: если вы получите странные результаты, возможно, вы не учли "порядковый номер байтов"- 4-байтовое целое число может быть закодировано как «big endian» или «little endian», а ваш процессор может быть «big endian» или «little endian» - эти два значения должны совпадать.Если результаты выглядят безумно большими (или безумно отрицательными), попробуйте добавить Array.Reverse(buffer) перед вызовом ToInt32, предпочтительно после проверки BitConverter.IsLittleEndian;то есть

// we now have 4 bytes; update the UI, and reset
if (BitConverter.IsLittleEndian) Array.Reverse(buffer); // fix endianness
int potensio = BitConverter.ToInt32(buffer, 0);

, и если это дает неправильный результат: поменяйте местами!

// we now have 4 bytes; update the UI, and reset
if (!BitConverter.IsLittleEndian) Array.Reverse(buffer); // fix endianness
int potensio = BitConverter.ToInt32(buffer, 0);