How do I set up an alert that always appears?
Right now, I just want some kind of battery status indicator when the user accesses the apps. For example, if a device is connected, then the device should say the charging and the level it is at. If the device is disconnected from the network, nothing should be displayed. If the device is not charging and the battery is discharging, the low battery indication should be displayed.
I have installed most of the code, but the problem is that, for example, if I launch the application and the device is connected, the charging indication is displayed on the screen. If I unplug the device, the charging is no longer visible, but if I reconnect the device, the charging is still not visible. Battery warnings should always be displayed when the device is plugged in or unplugged, with high or low energy. Change information should always be displayed.
So here is my broadcast receiver:
private BroadcastReceiver mBatInfoReceiver = new BroadcastReceiver() {
@Override
public void onReceive(Context arg0, Intent intent) {
//Battery level
int level = intent.getIntExtra("level", 0);
//Plugged in Status
int plugged = intent.getIntExtra(BatteryManager.EXTRA_PLUGGED, -1);
//Battery Status
int status = intent.getIntExtra(BatteryManager.EXTRA_STATUS, -1);
//If the device is charging or contains a full status, it charging
boolean isCharging = status == BatteryManager.BATTERY_STATUS_CHARGING ||
status == BatteryManager.BATTERY_STATUS_FULL;
//If the device isCharging and plugged in, then show that the battery is charging
TextView batteryTextView = ((TextView) findViewById(R.id.charging));
TextView batteryLowTextView = ((TextView) findViewById(R.id.low_battery));
ImageView batteryLowImageView= ((ImageView) findViewById(R.id.low_battery_icon));
ImageView batteryImageView =((ImageView) findViewById(R.id.charging_battery_icon));
if (isCharging && plugged == BatteryManager.BATTERY_PLUGGED_AC || plugged == BatteryManager.BATTERY_PLUGGED_USB) {
//Gets the 'last synced' string and sets to datetime of the last sync
Resources resources = context.getResources();
String chargingString = String.format(resources.getString(R.string.charging), level);
//Dynamically sets the value of the battery level
batteryLowTextView.setVisibility(TextView.INVISIBLE);
batteryLowImageView.setVisibility(ImageView.INVISIBLE);
batteryTextView.setText(chargingString + "%");
} else if (level < LOW_BATTERY_LEVEL && !isCharging) {
Resources resources = context.getResources();
String lowBatteryString = String.format(resources.getString(R.string.low_battery));
batteryTextView.setVisibility(TextView.INVISIBLE);
batteryImageView.setVisibility(ImageView.INVISIBLE);
batteryLowTextView.setText(lowBatteryString);
} else if (!isCharging && level > LOW_BATTERY_LEVEL) {
//do nothing
batteryTextView.setVisibility(TextView.GONE);
batteryImageView.setVisibility(ImageView.GONE);
}
}
};
In my OnCreate method I am calling
//calls registerReceiver to receive the broadcast for the state of battery
this.registerReceiver(this.mBatInfoReceiver,new
IntentFilter(Intent.ACTION_BATTERY_CHANGED));
So, I am wondering what am I doing wrong? Are these bolts?
source to share
- There are
if-then-else
many holes in your logic (and the problem is with @Vikram). This can result in the recipient doing nothing. Try to simplify your logic to remove all holes. - When you update a text view, you also need to remember it (and its image) first.
Here's a replacement for your if-then-else logic:
if (isCharging) {
//Gets the 'last synced' string and sets to datetime of the last sync
Resources resources = context.getResources();
String chargingString = String.format(resources.getString(R.string.charging), level);
//Dynamically sets the value of the battery level
batteryLowTextView.setVisibility(TextView.INVISIBLE);
batteryLowImageView.setVisibility(ImageView.INVISIBLE);
batteryTextView.setText(chargingString + "%");
batteryTextView.setVisibility(TextView.VISIBLE);
batteryImageView.setVisibility(ImageView.VISIBLE);
} else if (level <= LOW_BATTERY_LEVEL) {
Resources resources = context.getResources();
String lowBatteryString = String.format(resources.getString(R.string.low_battery));
batteryTextView.setVisibility(TextView.INVISIBLE);
batteryImageView.setVisibility(ImageView.INVISIBLE);
batteryLowTextView.setText(lowBatteryString);
batteryLowTextView.setVisibility(TextView.VISIBLE);
batteryLowImageView.setVisibility(ImageView.VISIBLE);
} else {
//show nothing
batteryTextView.setVisibility(TextView.GONE);
batteryImageView.setVisibility(ImageView.GONE);
batteryLowTextView.setVisibility(TextView.GONE);
batteryLowImageView.setVisibility(ImageView.GONE);
}
source to share
You AND
isCharging
with BatteryManager.BATTERY_PLUGGED_AC
here:
if (isCharging && plugged == BatteryManager.BATTERY_PLUGGED_AC || plugged == BatteryManager.BATTERY_PLUGGED_USB)
You need OR
controls BatteryManager
and a AND
result with the isCharging
following:
if (isCharging &&
(plugged == BatteryManager.BATTERY_PLUGGED_AC || plugged == BatteryManager.BATTERY_PLUGGED_USB))
If you need to know if the device is actually connected to AC or USB , you can use this method. However, the device can be connected to BATTERY_PLUGGED_WIRELESS
. Another thing is not your variable isCharging
already covers what you are trying to figure out:
(plugged == BatteryManager.BATTERY_PLUGGED_AC || plugged == BatteryManager.BATTERY_PLUGGED_USB)
I highly recommend reading this page and debugging for manipulating the values โโof the variables you are working with.
source to share