If you want list like behavior for your x and y array then have to use acts_as_list declaration. Without knowing your application requirements, it is difficult to suggest anything.
The option for class name should be: :class_name. Except for that, it looks fine. You may also want to specify the :dependent attribute.