フォームのエレメントを作る部分とそれに必要な`$genre_ids_names`や`$genre_ids_names`を生成する部分が離れている（別ブロックになってる）と、他で使ってるような印象を受けて、結果的に読みづらいソースになります。基本変数は使う直前で生成が原則です。

あと、`$genre_ids_names`と`$genre_ids_names`は`Sdx_Db_Select`の`fetchCol`を使ってもう少し簡潔に作れます。`$genre_ids_names[""] = "何も選択しない";`も専用のメソッド`$elems->setDefulatEmptyChilde('何も選択しない');`が有ります。`setDefulatEmptyChilde`は「何も選択しない」などの文字列をテンプレート側でコントロールしたいのでテンプレ側でやります。

```php
$elems = new Sdx_Form_Element_Group_Radio();
$elems->setName('genre_id')->addChildren(
  Bd_Orm_Main_Genre::createTable()->getSelect()->setColumns(array('id', 'name'))->fetchCol()
);
$form->setElement($elems);
```