Talvez seja a questão não esteja bem elaborada, pois não entendo completamente qual o problema, mas o que entendo dos teus parâmetros é

    $sql - SQL que vai retornar os resultados para preencher o <select> com valores
    $idcampo - O nome do campo a dar ao <select> (pode ser qualquer coisa!)
    $selected - O valor que deve ficar seleccionado no render

Neste momento tens `<?php echo $objct->ComboBox('select id, nome from cliente', $objcli->getIdcli(), 0); ?>`, ou seja, estás a passar o `ID` do objecto `$objcli` como o campo a ser usado, presumo que, actualmente, o HTML na página seja algo tipo `<select name="21">` (se for o 21 o valor retornado por `$objcli->getIdcli()`), e deveria ser algo tipo `<select name="id">`.


Para isso, a tua chamada na verdade teria de ser algo tipo `<?php echo $objcli->ComboBox('select id, nome from cliente', 'id', $objcli->getIdcli()); ?>`

Entretanto, o formulário onde esse `<select>` está, quando for submetido e quiseres saber qual o valor que foi escolhido, basta usares o `$_POST` (ou `$_GET`), por exemplo, se passaste `$idcampo` como `'id'`, seria `$_POST['id']` e a partir dai fazes o que precisares com esse valor.

//
PDO::FETCH_ASSOC
----------------

`PDO::FETCH_ASSOC` retorna um array com os nomes dos campos como indice.

    // Exemplo
    Array
    (
        [id] => 1
        [nome] => Jão
    )

Na tua funcao `ComboBox`, tu usas `$row[1]`, que na verdade deveria estar a dar erro, **Undefined index**. Provavelmente deverias usar `PDO::FETCH_BOTH`, que te dá algo tipo

    Array
    (
        [id] => 1
        [0] => 1
        [nome] => Jão
        [1] => Jão
    )

E com este resultado, na função devias mudar o `$row[1]` para `$row[0]` e o `$row[2]` para `$row[1]` (visto que é um 0 based array).

O valor a passar em `$idcampo` pode ser só um string, podes por só `'id'` e deve funcionar correcto.