Java problem 1: commandline check

package rose.mymedic2.connectiontoggle;

import java.io.DataInputStream;
import java.io.DataOutputStream;

import android.util.Log;

public class RunCommand {
/**
* @param Command
*            To be run as super user.
* @param ExpectOutput
*            Expects no output if false. If true, this method expects
*            output.
* @return
*/
public static String RunSuperUserCommand(String Command) {
return RunSuperUserCommand(Command, false);
}

public static String RunSuperUserCommand(String Command, boolean ExpectOutput) {
Process suProcess;
DataInputStream osRes;
DataOutputStream os;

ExpectOutput = false;
String Output = null;

try {
suProcess = Runtime.getRuntime().exec("su");
os = new DataOutputStream(suProcess.getOutputStream());
osRes = new DataInputStream(suProcess.getInputStream());

if (null != os && null != osRes) {
// running the command.
os.writeBytes(Command + "\n");
os.flush();

if (ExpectOutput) {
Output = osRes.readLine();
}

os.writeBytes("exit\n");
os.flush();

}
} catch (Exception e) {
Log.d( Command ,"Error [" + e.getClass().getName() + "] : "
+ e.getMessage());
}
return Output;
}

}

plazm (2) skrev:

Er det ikke bare at sætte et async read op, vente på at processen bliver færdig og når den gør det skrive det ud som din async read har fået og lukke den pænt ned igen?

arne_v (3) skrev:

Den bør returnere når processen er færdig.

Qw_freak (4) skrev:

hvad så hvis der aldrig kommer svar?

arne_v (3) skrev:

#1

readLine bør ikke vente forever. Den bør returnere når processen er færdig.

if (ExpectOutput && (osRes.available() >= 1)) {
Output = osRes.readLine();
}