Alarm Manager

[Pavlex4]

Directory "/sys/bus/usb/devices" is empty when otg cable is not attached !!!!

[Lost User]

Then use your debugger to find out what is happening. As we keep telling you.

[Afzaal Ahmad Zeeshan]

First of all, my recommendation is to consider using the [FileObserver](https://developer.android.com/reference/android/os/FileObserver.html) in Android, instead of triggering that each second to check if there are files or not. That is a good approach towards going through what you want to do. Secondly, as Richard mentioned, the reason is that the list object is not null, that doesn't mean its count or length or size is also zero. Just to show you what I really mean, here is a sample code,

File directory = new File("/path/to/folder");
File[] contents = directory.listFiles();

if(contents == null) {
// directory not found
} else if(contents.length == 0) {
// directory exists, but no files
} else {
// has further content, down the tree.
}

You can update your code, to meet this structure and then show the Toast notifications in Android. So kindly implement these two things, first of all that service will help you not throttle CPU every second. For an example of FileObserver please see: [java - How do you implement a FileObserver from an Android Service - Stack Overflow](http://stackoverflow.com/questions/7265906/how-do-you-implement-a-fileobserver-from-an-android-service). [file io - How to check if a directory is empty in Java - Stack Overflow](http://stackoverflow.com/questions/5930087/how-to-check-if-a-directory-is-empty-in-java)

The shit I complain about It's like there ain't a cloud in the sky and it's raining out - Eminem ~! Firewall !~

[Afzaal Ahmad Zeeshan]

What happens is that his directory is indeed there, but he does not check up against files or other stuff inside the directory. I tried to explain the process to him in the post below, as well as another suggestion that will make his application not throttle CPU every 1 second to check if the directory was modified or not.

The shit I complain about It's like there ain't a cloud in the sky and it's raining out - Eminem ~! Firewall !~

[Lost User]

Afzaal Ahmad Zeeshan wrote:

I tried to explain

Largely a waste of time it seems.

[Afzaal Ahmad Zeeshan]

Richard MacCutchan wrote:

Largely a waste of time it seems.

By the number of his interactions in the forum, indeed.

The shit I complain about It's like there ain't a cloud in the sky and it's raining out - Eminem ~! Firewall !~

[Pavlex4]

Now when I launch it shows otg not connected and it's repeating it many times and when I plug in otg cable it's still repeating "otg not connected" and after some time it shows "otg connected" !!!! When I start app it shouldn't show anything, it should only show when I plug in or when I plug out otg cable.And it shouldn't repeat same message many times !!!!

public class AlarmReceiver extends BroadcastReceiver
{
@Override
public void onReceive(Context context,Intent intent)
{

    File directory = new File("/sys/bus/usb/devices");
    File\[\] contents = directory.listFiles();

    if(contents.length == 0)
    {
        Toast.makeText(context,"otg not connected",Toast.LENGTH\_SHORT).show();
    }
    else
    {
        Toast.makeText(context,"otg connected",Toast.LENGTH\_SHORT).show();
    }
}

}

[Afzaal Ahmad Zeeshan]

That is because there are a few Toasts already in the queue, that need to be rendered. That is one of the backdrops of this timer trigger.

The shit I complain about It's like there ain't a cloud in the sky and it's raining out - Eminem ~! Firewall !~

[Pavlex4]

I don't understand!What should I do?

[Pavlex4]

What to do?

[David Crow]

Pavlex4 wrote:

I want to check every second if /sys/bus/usb/devices/ directory is empty or it contain files...I need to check every second if directory is empty or it contain files?

Bad idea. All that does is needlessly tie up the CPU.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Pavlex4]

Than what's the better way to do this?

[David Crow]

FileObserver[^]

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Pavlex4]

Why it's not working?

public class MainActivity extends AppCompatActivity
{
private Process suProcess;
private static int conn_length = -1;
File directory = new File("/sys/bus/usb/devices");
File[] contents = directory.listFiles();

@Override
protected void onCreate(Bundle savedInstanceState)
{
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity\_main);

    getRoot();



    FileObserver observer = new FileObserver("/sys/bus/usb/devices")
    {
        @Override
        public void onEvent(int event, String file)
        {
            if(contents.length == conn\_length){
                return;
            }
            else{
                conn\_length = contents.length;
            }

            if(conn\_length == 0)
            {

                Toast.makeText(MainActivity.this,"otg not connected",Toast.LENGTH\_SHORT).show();
            }
            else
            {
                Toast.makeText(MainActivity.this,"otg connected",Toast.LENGTH\_SHORT).show();
            }
        }
    };
    observer.startWatching();
}

private void getRoot()
{
    try
    {
        suProcess = Runtime.getRuntime().exec("su");
    }
    catch (IOException e)
    {

    }
}

}

[David Crow]

Define "not working." Since you asked for ALL_EVENTS, your event handler is potentially going to get called for a lot of events, yet you are not checking for any of them. Is that intentional? You are calling listFiles() once before startWatching(). I would think that you'd want to call listFiles() only after a CREATE event.

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles

[Pavlex4]

I have changed code to this but it still won't show message when cable is connected or disconnected !!!!

public class MainActivity extends AppCompatActivity
{
private Process suProcess;
private static int conn_length = -1;
File directory = new File("/sys/bus/usb/devices");
File[] contents;

@Override
protected void onCreate(Bundle savedInstanceState)
{
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity\_main);

    getRoot();



    FileObserver observer = new FileObserver("/sys/bus/usb/devices")
    {
        @Override
        public void onEvent(int event, String file)
        {
            if(event == FileObserver.CREATE)
            {
                contents = directory.listFiles();

                if (contents.length == conn\_length)
                {
                    return;
                }
                else
                {
                    conn\_length = contents.length;
                }

                if (conn\_length == 0)
                {

                    Toast.makeText(MainActivity.this, "otg disconnected", Toast.LENGTH\_SHORT).show();
                }
                else
                {
                    Toast.makeText(MainActivity.this, "otg connected", Toast.LENGTH\_SHORT).show();
                }
            }
        }
    };
    observer.startWatching();
}

private void getRoot()
{
    try
    {
        suProcess = Runtime.getRuntime().exec("su");
    }
    catch (IOException e)
    {

    }
}

}

[David Crow]

Did you set a breakpoint in the event handler? Using the debugger, what is the value of event, contents, and conn_length?

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"You can easily judge the character of a man by how he treats those who can do nothing for him." - James D. Miles