Pull to refresh

Comments 3

а нельзя название класса CreateAddPersonAndGroceryListsAndFoodItemsTables сделать немного короче?
Можно, это никак не влияет на миграцию. Я просто так придумал :)
Несколько замечаний по статье:

«Затем в файле app/views/grocery_list/show.rhtml мы создадим соответствующее представление:» — представления нету, исправьте, что ли…

Миграции лучше писать так:

class InitialMigrate < ActiveRecord:: Migration
def self.up
create_table: people do |t|
t.string: name
t.timestamps
end
create_table: grocery_lists do |t|
t.string: name
t.references: person
t.timestamps
end
create_table: food_items do |t|
t.string: name
t.integer: quantity
t.integer: position
t.references: grocery_list
t.timestamps
end
end
def self.down
drop_table: people
drop_table: grocery_lists
drop_table: food_items
end
end

Вместо создания «standard.rhtml» и включения его через layout, лучше создать «application.html.erb», который будет подключатся автоматически по-умолчанию.

Ну и вообще, используйте, всё таки, <code>, пожалуста, а то этот грёбанный чёртов Типограф уродует код.

А в целом, статья отличная, добавил в избранное.
Спасибо.
Sign up to leave a comment.

Articles