0

这是代码

@Override
  public void onSensorChanged(SensorEvent event) {
    if (event.sensor.getType() == Sensor.TYPE_ACCELEROMETER) {
      getAccelerometer(event);
    }

  }

  private void getAccelerometer(SensorEvent event) {
    float[] values = event.values;
    //Movement
    float x = values[0];
    float y = values[1];
    float z = values[2];

    float accelationSquareRoot = (x * x + y * y + z * z)
        / (SensorManager.GRAVITY_EARTH * SensorManager.GRAVITY_EARTH);
    long actualTime = System.currentTimeMillis();
    if (accelationSquareRoot >= 2) //
    {
      if (actualTime - lastUpdate < 200) {
        return;
      }


      if (distance <= 50 && distance > 40) {
        view.setBackgroundColor(Color.GREEN);
        while (distance > 0) {
              distance--;
              Integer.toString(distance);
              lastUpdate = actualTime;
              Toast.makeText(this, distance, Toast.LENGTH_SHORT)
                  .show();
              int distance = Integer.parseInt(distance);


        }

        if (distance < 40){
            view.setBackgroundColor(Color.YELLOW);
        }



      } else {
          view.setBackgroundColor(Color.RED);

      } 
      color = !color;
    }
  }

每次摇晃设备时，我都会尝试使用距离变量来减少和查看。

这就是代码出错的地方。

  int distance = Integer.parseInt(distance);

它要我更改变量的名称。如果我这样做，我的循环将无法工作。

提前致谢！

4

3 回答 3

5

为什么不将距离保留为 int 值并String.format()在您的中使用 aToast呢？IE

Toast.makeText(this, String.format("%d", distance), Toast.LENGTH_SHORT).show();

从 UI 的角度来看，我不确定我是否愿意在循环的每次迭代中向 Toast 消息发送垃圾邮件，也许更新TextView小部件可能会更好。

于 2013-09-06T02:13:07.260 回答
0

错误是因为您声明了两次相同的变量。此外，您不需要转换distance为String以在 中显示它Toast，只需"" +在distance. 因此，通过这样做，您可以删除这两行：

Integer.toString(distance); 
int distance = Integer.parseInt(distance);
于 2013-09-06T02:13:47.833 回答
0

因此，您要将距离从 int 转换为字符串，然后再转换回来？您可以为您的字符串使用另一个变量，以节省这种类型转换的麻烦吗？

编辑：被 Karl 和 jbihan 打败。他们的回答相得益彰。您可以丢失两行并解决您的问题。

于 2013-09-06T02:14:44.253 回答