Tengo un modelo de rieles que valida la unicidad del valor de order_number, debe comenzar desde 1_000_000, así que agregué una variable que se usa como primer valor:
# order model STARTING_NUMBER = 1_000_000 validates :order_number, uniqueness: trueCuando revisé mi código por Rubocop tuve un error:
app/models/order.rb:3:3: C: Rails/UniqueValidationWithoutIndex: Uniqueness validation should be with a unique index. validates :order_number, uniqueness: true Lo arreglé agregando disable/enable comentarios de Rubocop:
STARTING_NUMBER = 1_000_000 # rubocop:disable Rails/UniqueValidationWithoutIndex validates :order_number, uniqueness: true # rubocop:enable Rails/UniqueValidationWithoutIndex¿Hay una solución mejor?
La solución adecuada es agregar un índice único a su base de datos con una migración:
def change add_index :orders, :order_number, unique: true endEso solucionará el problema subyacente y evitará que Rubocop se queje.
Del fino manual de Rubocop :
Cuando define una validación de unicidad en el modelo de Active Record, también debe agregar un índice único para la columna. Hay dos razones Primero, los registros duplicados pueden ocurrir incluso si se define la validación de Active Record. En segundo lugar, provocará consultas lentas.
Rubocop ve que tiene una validación de unicidad pero no encontró un índice único correspondiente en su db/schema.rb . Una validación de unicidad en un modelo está sujeta a condiciones de carrera, por lo que aún puede obtener valores duplicados.
Rubocop le dice que agregue un índice/restricción único en la base de datos para garantizar la unicidad. Lasguías de Rails dicen lo mismo:
No crea una restricción de unicidad en la base de datos, por lo que puede suceder que dos conexiones de bases de datos diferentes creen dos registros con el mismo valor para una columna que pretende que sea única. Para evitar eso, debe crear un índice único en esa columna en su base de datos.
La validación también realizará una consulta de base de datos potencialmente costosa, por lo que realmente desea indexar esa columna de todos modos, también podría convertirla en un índice único para garantizar la integridad de los datos mientras lo hace (el código roto es temporal, los datos rotos son para siempre).
No suprima la advertencia, encárguela.