Question

c#里说use of unassigned local variables b1,b2怎么改？

private void SConvert_Click(object sender, EventArgs e)
{
string[] coords = textBox1.Text.Split('\n');
byte[] b1,b2;
for (int i = 0; i <= coords.Length - 1; i++)
{
if (coords[i].Length > 0)
{
GetValue(coords[i]);
}
if (serialPort.IsOpen)
{
serialPort.Write(b1, 0, 4);
serialPort.Write(b2, 0, 4);
}
}
}
这段是接在上面下面的
private void GetValue(string strValue)
{ byte[] b1, b2;
float f1,f2;
。
。
。
b1 = System.BitConverter.GetBytes(f1);
b2 = System.BitConverter.GetBytes(f2);}

Answer 1

您这里使用的是局部变量byte[]类型的变量，而您在把此变量的值发送到串口的时候没有初始化，因此需要先初始化，要不然数组是可变的了!

当然这是个人经验，你可以试一下！但是在C#编程中变量都是要先初始化在使用的，而您在这里只是申明而没有初始化变量。
望采纳！！

追问

所以是怎么初始化呢？谢谢！

追答

因为此方法System.BitConverter.GetBytes(f1);返回的是4个字节的数组，因此在定义的时候可以
byte[] b1 = new byte[4];
byte[] b2 = new byte[4];
这样再调用应该就可以了

追问

谢谢你的回答！你的代码也很好用，只是看楼上那位大哥写了那么多，就用了他的。见谅！

追答

没事 能帮到你我的目的就达到了

Answer 2

你的b1和b2均未赋值也没有初始化，因为不清楚你的意图所以改不了，一般来说通常可能有这样的代码，比如
byte[] b1=new byte[4];
b1[0]=0x10;
...

追问

完整的发了，请指教！

追答

你不能这样写，你两处定义的b1和b2都是局部变量，作用域仅限于{}之内，比如说GetValue里的b1和b2，

private void GetValue(string strValue)

        { byte[] b1, b2;    //GetValue运行完之后，这些变量就无效了

           float f1,f2;

要简单的改动，可以这样

byte[] b1=null,b2=null;
private void SConvert_Click(object sender, EventArgs e)
        {
            string[] coords = textBox1.Text.Split('\n');
            for (int i = 0; i <= coords.Length - 1; i++)
            {
                if (coords[i].Length > 0)
                {
                    GetValue(coords[i]);
                }
                if (serialPort.IsOpen && b1!=null && b2!=null)
                {
                    serialPort.Write(b1, 0, 4);
                    serialPort.Write(b2, 0, 4);
                }
            }
        }
private void GetValue(string strValue)
  {
           float f1,f2;
            。
            。
            。
            b1 = System.BitConverter.GetBytes(f1);
            b2 = System.BitConverter.GetBytes(f2);
}

追问

我开始写的就是
private byte【】 b1,b2;
结果被导师说要写local的才可以

追答

这样也不是不行，只要GetValue传出值就行了，在上面代码的基础上，将byte[] b1=null,b2=null;这一行重新移回到SConvert_Click里，然后将GetValue的定义改为

private void GetValue(string strValue,out byte[] b1,out byte[] b2)

然后在SConvert_Click里调用的时候改为

 GetValue(coords[i],out b1,out b2);

 

当然你也可以考虑用List，比如说

private List<byte[]> GetValue(string strValue)
  {
           float f1,f2;
            List<byte[]> list=new List<byte[]>();
            list.Add(System.BitConverter.GetBytes(f1));
            list.Add(System.BitConverter.GetBytes(f2));
            return list;
}

然后调用的时候

private void SConvert_Click(object sender, EventArgs e)
        {
          List<byte[]> list=null;
           。。。。
          list=GetValue(coords[i]);
          。。。。
                if (serialPort.IsOpen && list!=null && list.Count>=2)
                {
                    serialPort.Write(list[0], 0, 4);
                    serialPort.Write(list[1], 0, 4);
                }
            }
        }