O código sempre retorna 1, quando não está nenhum RadioButton Checked
, porque só há um controle dentro do controle c
. Se houvesse 2 retornaria 2, etc.
Repare que o i
é incrementado no ciclo for por cada controle em c
, terminado com um valor igual a cc.Count
.
Você deve usar uma variável auxiliar para guardar o índice do RadioButton Checked
.
Deve também garantir se o controle é do tipo RadioButton antes de verificar se ele está Checked
. Isso evitará o uso do bloco try/catch
, que não devem ser utilizados para este tipo de situações.
public static int getCheckedRadioButton(Control c)
{
int index = -1;
Control.ControlCollection cc = c.Controls;
for (i = 0; i < cc.Count; i++)
{
if(cc[i] is RadioButton)
{
index++;
RadioButton rb = cc[i] as RadioButton;
if (rb.Checked)
{
return index; //retorna o indice do RadioButton selecionado
}
}
}
return -1;
}
É possível simplificar o código de várias formas, entre elas usando foreach
:
public static int getCheckedRadioButton(Control c)
{
//índice do RadioButton selecionado, -1 indica que não há nenhum.
int index = -1;
//Controls implementa a interface IEnumerable, podemos usar foreach
foreach(var control in c.Controls)
{
if(control is RadioButton)//Verifica se é um RadioButton
{
//Foi encontrado um RadioButton, incrementa o índice
index++;
//faz o cast do control para RadioButton e verifica se está selecionado
if ((RadioButton)control.Checked)
{
return index; //retorna o índice do RadioButton selecionado
}
}
}
return -1;
}
Utilizando LINQ pode ser ainda mais simplificado e obter directamente uma referência ao RadioButton seleccionado:
var checkedRadioButton = container.Controls
.OfType<RadioButton>()
.FirstOrDefault(r => r.Checked);
Onde container
é o controle onde estão os RadioButton.
Se não houver nenhum seleccionado checkedRadioButton
será null
;