Модели направляющих должны касаться других моделей ради тощих контроллеров?

Когда вы объявляете ссылочную переменную (т. е. объект), вы действительно создаете указатель на объект. Рассмотрим следующий код, в котором вы объявляете переменную примитивного типа int:

int x;
x = 10;

В этом примере переменная x является int, и Java инициализирует ее для 0. Когда вы назначаете его 10 во второй строке, ваше значение 10 записывается в ячейку памяти, на которую указывает x.

Но когда вы пытаетесь объявить ссылочный тип, произойдет что-то другое. Возьмите следующий код:

Integer num;
num = new Integer(10);

Первая строка объявляет переменную с именем num, но она не содержит примитивного значения. Вместо этого он содержит указатель (потому что тип Integer является ссылочным типом). Поскольку вы еще не указали, что указать на Java, он устанавливает значение null, что означает «Я ничего не указываю».

Во второй строке ключевое слово new используется для создания экземпляра (или создания ) объекту типа Integer и переменной указателя num присваивается этот объект. Теперь вы можете ссылаться на объект, используя оператор разыменования . (точка).

Exception, о котором вы просили, возникает, когда вы объявляете переменную, но не создавали объект. Если вы попытаетесь разыменовать num. Перед созданием объекта вы получите NullPointerException. В самых тривиальных случаях компилятор поймает проблему и сообщит вам, что «num не может быть инициализирован», но иногда вы пишете код, который непосредственно не создает объект.

Например, вы можете имеют следующий метод:

public void doSomething(SomeObject obj) {
   //do something to obj
}

В этом случае вы не создаете объект obj, скорее предполагая, что он был создан до вызова метода doSomething. К сожалению, этот метод можно вызвать следующим образом:

doSomething(null);

В этом случае obj имеет значение null. Если метод предназначен для того, чтобы что-то сделать для переданного объекта, целесообразно бросить NullPointerException, потому что это ошибка программиста, и программисту понадобится эта информация для целей отладки.

Альтернативно, там могут быть случаи, когда цель метода заключается не только в том, чтобы работать с переданным в объекте, и поэтому нулевой параметр может быть приемлемым. В этом случае вам нужно будет проверить нулевой параметр и вести себя по-другому. Вы также должны объяснить это в документации. Например, doSomething может быть записано как:

/**
  * @param obj An optional foo for ____. May be null, in which case 
  *  the result will be ____.
  */
public void doSomething(SomeObject obj) {
    if(obj != null) {
       //do something
    } else {
       //do something else
    }
}

Наконец, Как определить исключение & amp; причина использования Трассировки стека

5
задан 3 revs, 2 users 100% 15 September 2008 в 20:17
поделиться

5 ответов

но затем модель создаст и сохранит экземпляры других моделей, который является, где моя проблема.

Что не так с этим?

Если Ваша 'бизнес-логика' указывает, что Операция должна иметь допустимый набор Записей, то, конечно, нет ничего неправильно, чтобы Операционный класс знал о, и соглашение с Вашими объектами Записи.

Вы только получите проблемы, если Вы возьмете это слишком далеко и будете иметь Ваши модели, управляющие вещами, они не должны знать о, как EntryHtmlFormBuilder или безотносительно :-)

6
ответ дан 14 December 2019 в 04:51
поделиться

Легче думать с точки зрения каждого объекта, проверяющего себя и объекты, которые зависят друг от друга делегирующего их состояние к состоянию их связанных записей. В Вашем случае, например:

class Operation < ActiveRecord::Base
  has_many :entries
  validates_associated :entries
end

validates_associated проверит, допустим ли каждый связанный объект (в этом случае, все записи должны, если операция должна быть допустимой).

Очень заманчиво попытаться проверить все иерархии моделей в целом, но поскольку Вы сказали, место, где это будет наиболее легко сделано, является контроллером, который должен действовать больше как маршрутизатор запросов и ответов, чем имея дело с бизнес-логикой.

0
ответ дан 14 December 2019 в 04:51
поделиться

Путем я смотрю на него, то, что контроллер должен отразить, что конечный пользователь просматривает и переводит запросы в образцовые операции и reponses, также делая форматирование. В Вашем случае существует 2 вида операций, которые представляют простые операции с учетной записью/записью по умолчанию, и более сложные операции, которые имеют пользователя, выбрали записи и учетные записи. Формы должны отразить пользовательское представление (2 формы с различными полями), и должно быть 2 действия в контроллере для соответствия. Контроллер однако не должен иметь никакой логики, касающейся, как данными управляют, только как получить и ответить. У меня были бы методы класса для Операционного класса, которые берут в надлежащих данных из форм, и создает, один или несколько возражает по мере необходимости или помещает те методы класса для класса поддержки, который не является моделью AR, но имеет бизнес-логику, которая пересекает образцовые границы. Преимущество отдельного служебного класса состоит в том, что это сохраняет каждую модель сфокусированной на одной цели, вниз, сторона - то, что служебные классы не имеют никакого определенного места для проживания. Я поместил их в lib/, но направляющие не указывают место для образцовых помощников как таковых.

0
ответ дан 14 December 2019 в 04:51
поделиться

Виртуальные Атрибуты (больше информации здесь и здесь) помогут с этим значительно. Пасование назад целых параметрических усилителей к модели сохраняет вещи простыми в контроллере. Это позволит Вам динамично создавать свою форму и легко создавать объекты записей.

class Operation
  has_many :entries

  def entry_attributes=(entry_attributes)
    entry_attributes.each do |entry|
      entries.build(entry)
    end
  end

end

class OperationController < ApplicationController
  def create
    @operation = Operation.new(params[:opertaion])
    if @operation.save
      flash[:notice] = "Successfully saved operation."
      redirect_to operations_path
    else
      render :action => 'new'
    end
  end
end

Сохранение перестанет работать, если все не будет допустимо. Который приносит нам к проверке. Поскольку каждая Запись стоит одна, и необходимо проверить все записи при "создании", которое необходимо, вероятно, переопределить, проверяют в Операции:

class Operation
  # methods from above
  protected
    def validate
      total = 0
      entries.each { |e| t += e.amount }
      errors.add("entries", "unbalanced transfers") unless total == 0
    end
end

Теперь Вы получите сообщение об ошибке, говоря пользователю, что суммы выключены, и они должны решить проблему. Можно стать действительно необычными здесь и добавить, что много значения, будучи конкретным относительно проблемы, как говорит им, насколько они выключены.

2
ответ дан 14 December 2019 в 04:51
поделиться

Если Вы обеспокоены встраиванием этой логики в какую-либо конкретную модель, почему бы не поместить их в класс наблюдателя, который сохранит логику для Вашего создания связанных объектов отдельной от классов наблюдаемый.

0
ответ дан 14 December 2019 в 04:51
поделиться
Другие вопросы по тегам:

Похожие вопросы: