Tarea Refactoring1

[Adrianlsq2000]

Tarea Refactoring

Beneficio de hacer la refactorizacion es que se reduce la complejidad del método original en AbstracSshShell y esto hace que el código sea más claro y fácil de entender.

También encapsulamos la lógica en una clase separada entonces cada clase tiene una responsabilidad más específica una única responsabilidad. La clase ExecutionMethodObject puede ser reutilizada en otra parte que se necesite sin duplicar código. Cualquier futura modificación se podría hacer en la clase ExecutionMetodObject sin afectar la estructura general de AbstractSshShell.

Codigo despues de la refactorizacion

• Clase exec

@Override
public int exec(final String command, final InputStream stdin,
    final OutputStream stdout, final OutputStream stderr)
    throws IOException {
    return new ExecutionMethodObject(this, command, stdin, stdout, stderr).execute();
}

• Nueva clase ExecutionMethodObject

class ExecutionMethodObject {
    private final AbstractSshShell sshShell;
    private final String command;
    private final InputStream stdin;
    private final OutputStream stdout;
    private final OutputStream stderr;

    ExecutionMethodObject(AbstractSshShell sshShell, String command, InputStream stdin,
        OutputStream stdout, OutputStream stderr) {
        this.sshShell = sshShell;
        this.command = command;
        this.stdin = stdin;
        this.stdout = stdout;
        this.stderr = stderr;
    }

    int execute() throws IOException {
        Execution execution = new Execution(
            command,
            stdin,
            stdout,
            stderr,
            sshShell.session()
        );
        return execution.exec();
    }
}