0

私のアプリケーションでは、ステータスの読み取りに使用される 12 桁のバイト配列があります。この配列を書き込んだ後、さまざまなボタンのステータスを取得しています。この応答に基づいて、ボタンをON | OFFに設定しています。このタスクを繰り返し実行しています。このインターバル時間を 100 ミリ秒に設定しすぎて、これらのボタンを頻繁に押しすぎると、アプリケーションが応答しなくなります。

これがコードスニペットです。

@Override
protected void onCreate(Bundle savedInstanceState) 
{
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main_activity);         
    getSharedSettings();

    fanDimmer1=(ToggleButton)findViewById(R.id.button_fan1);
    fanDimmer2=(ToggleButton)findViewById(R.id.button_fan2);
    dimmerLight1=(ToggleButton)findViewById(R.id.button_light1);
    dimmerLight2=(ToggleButton)findViewById(R.id.button_light2);


    fanDimmer1.setOnClickListener(this);
    fanDimmer2.setOnClickListener(this);
    dimmerLight1.setOnClickListener(this);
    dimmerLight2.setOnClickListener(this);

    if(ip.equals("") || port.equals(""))
    {
        new AlertDialog.Builder(MainActivity.this)
        .setTitle("Warning !")
        .setMessage("Please set IP and PORT first")
        .setIcon(android.R.drawable.ic_dialog_alert)
        .setNeutralButton("ok", null)
        .show();
    }
    else
    {
        new Thread(new Runnable() 
        {   
            @Override
            public void run()
            {
                Log.v(TAG, "openconnection");
                openConnection();
            }
        }).start();
        m_handler = new Handler();
        startRepeatingTask();

    }

}

public void openConnection()
{
    // TODO Auto-generated method stub
    try 
    {
        s = new Socket(ip, Integer.parseInt(port));
        i = s.getInputStream();
        iD = new DataInputStream(i);
        o = s.getOutputStream();
        oD = new DataOutputStream(o);
        Log.v(TAG, "openconnection 2");

    }
    catch (UnknownHostException e) {
        // TODO: handle exception
        Log.v("UnknowHostException :::::", "In Catch Block");
        e.printStackTrace();
    }
    catch (IOException e) {
        // TODO: handle exception
        Log.v("IOException :::::", "In Catch Block");
        e.printStackTrace();
    }
}

Runnable m_statusChecker = new Runnable() 
{
    @Override
    public void run() 
    {
        updateStatus();
        m_handler.postDelayed(m_statusChecker,100);
    }

    private void updateStatus() 
    {
        // TODO Auto-generated method stub
        Log.v("test", "1");
        try {
            byte[] data1 = new byte[1024], packet1 = 
                { 
                    (byte) 0x00,(byte) 0x00,(byte) 0x00, 
                    (byte) 0x00,(byte) 0x00,(byte) 0x06, 
                    (byte) 0x01,(byte) 0x01,(byte) 0x00,
                    (byte) 0x00,(byte) 0x00,(byte) 0x19
                };

            o.write(packet1);
            i.read(data1, 0, 1024);

            byte_to_hex = ConversionMethods.bytesToHex(data1).substring(18, 26);
            char[] arr = byte_to_hex.toCharArray();
            for (int i = 0; i < arr.length - 1; i += 2) 
            {
                char temp = arr[i];
                arr[i] = arr[i + 1];
                arr[i + 1] = temp;
            }

            swapped_result=new String(arr);
            result = ConversionMethods.hexStringToNBitBinary(swapped_result, 32);

            int counter = 0;
            for( int i=0; i<result.length(); i++ ) 
            {
                if( result.charAt(i) == '1' )
                {
                    counter++;        
                }  
            }
            status=Integer.toString(counter);
            txt_status.setText(status);
            Log.v(TAG, "status is ::"+status);


            char[] c=result.toCharArray();
            int count=0;
            for (int i=0;i<result.length();i++)
            {
                count++;
                char j=c[i];
                //Log.v(TAG, count+"::"+j);
                if(count==1)
                    toggleButton=dimmerLight1;
                else if(count==2)
                    toggleButton=fanDimmer2;
                else if(count==3)
                    toggleButton=fanDimmer1;
                else if(count==4)
                    Log.v(TAG, "Count 4 is 0");

                if(j=='1')
                    toggleButton.setChecked(true);
                else
                    toggleButton.setChecked(false); 
            }

        } catch (UnknownHostException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }

    }
};

void startRepeatingTask() {
    m_statusChecker.run();
}

void stopRepeatingTask() {
    m_handler.removeCallbacks(m_statusChecker);
}

ボタンをクリックすると、私はこれをやっています:

@Override
public void onClick(View v) 
{

    if(v.equals(fanDimmer1))
    {
        if (fanDimmer1.isChecked()) 
        {
            Toast.makeText(MainActivity.this, "Fan 1 is ON", Toast.LENGTH_SHORT).show();
            setByteArray((byte) 0x01, (byte) 0xff);
        } else 
        {
            Toast.makeText(MainActivity.this, "Fan 1 is OFF", Toast.LENGTH_SHORT).show();
            setByteArray((byte) 0x01, (byte) 0x00);
        }
    }

}

これがsetByteArray()メソッドです。このタイプのボタンは24個あります。

アドバイスや提案をいただければ幸いです。
ありがとう

4

1 に答える 1

1

startRepeatingTask()メインスレッドから呼び出しています:

void startRepeatingTask() {
   m_statusChecker.run();
}

これは、statusCheckerがメインスレッドでも実行されることを意味します。

  • 最初に行うことは、updateStatusメソッドのコードを更新し、それを使用mHandlerしてui-updateコードのみをメインUIスレッドに投稿することです。

  • 次に、別のスレッドでstatusCheckerを実行する必要があります

それを行うには、(少なくとも)2つのオプションがあります:

  1. [複雑な方法]独自のスレッド管理を行い、startRepeatingTaskに次のように記述します

    void startRepeatingTask() { new Thread(m_statusChecker).start(); }

    停止方法を適応させます(つまり、モニター、無限ループ、およびスリープを使用します)

  2. [賢い方法]AndroidAPIのTimerTimerTask(およびこれ)を見て、それらを使用して必要なものを実装します。

編集

このコードに焦点を当てます:

for (int i=0;i<result.length();i++)
        {
            count++;
            char j=c[i];
            //Log.v(TAG, count+"::"+j);
            if(count==1)
                toggleButton=dimmerLight1;
            else if(count==2)
                toggleButton=fanDimmer2;
            else if(count==3)
                toggleButton=fanDimmer1;
            else if(count==4)
                Log.v(TAG, "Count 4 is 0");

            if(j=='1')
                toggleButton.setChecked(true);
            else
                toggleButton.setChecked(false); 
        }

このループの効果は次のようになります。

  • 反復0(存在する場合):dimmerLight1がオンまたはオフになります
  • 反復1(存在する場合):fanDimmer2がチェックまたはオフになります
  • 反復2(存在する場合):fanDimmer1がオンまたはオフになります
  • 反復3(存在する場合):fanDimmer1がオンまたはオフになります
  • 反復4(存在する場合):fanDimmer1がオンまたはオフになります
  • 反復5(存在する場合):fanDimmer1がオンまたはオフになります
  • ...(fanDimmer1は、ループが終了するまでチェックまたはオフになります)

私はこれがあなたが必要とするものではないとかなり確信しています。(result.size()が常に3である場合を除きますが、その他の状況では、このコードは奇妙なものを生成します)

あなたはこのようなことをすることができます:

//assuming buttons is a ToggleButton[32] populated with all your buttons in the correct order
for (int i=0;i<result.length();i++)
    {
        buttons[i].setChecked(c[i]=='1'); 
    }
于 2013-01-22T11:19:22.940 に答える