Как уменьшить избыточность обработки полей обработчиком фида

Question

Я являюсь подписчиком в фиде сообщения для нескольких полей, мне нужно установить значения из фида для объекта домена и иметь код, как показано ниже:

if (map.contains(quoteBidPriceAcronym)) { 
    quote.bid.price = Some(map.get(quoteBidPriceAcronym).get.asInstanceOf[Number].doubleValue()); 
    quote.changed = true; 
} 
if (map.contains(quoteBidSizeAcronym)) { 
    quote.bid.size = Some(sizeMultipler() * map.get(quoteBidSizeAcronym).get.asInstanceOf[Number].intValue()); 
    quote.changed = true; 
} 
if (map.contains(quoteBidNumAcronym)) { 
    quote.bid.num = Some(map.get(quoteBidNumAcronym).get.asInstanceOf[Number].shortValue()); 
    quote.changed = true; 
} 
if (map.contains(quoteAskPriceAcronym)) { 
    quote.ask.price = Some(map.get(quoteAskPriceAcronym).get.asInstanceOf[Number].doubleValue()); 
    quote.changed = true; 
} 
if (map.contains(quoteAskSizeAcronym)) { 
    quote.ask.size = Some(sizeMultipler() * map.get(quoteAskSizeAcronym).get.asInstanceOf[Number].intValue()); 
    quote.changed = true; 
} 
if (map.contains(quoteAskNumAcronym)) { 
    quote.ask.num = Some(map.get(quoteAskNumAcronym).get.asInstanceOf[Number].shortValue()); 
    quote.changed = true; 
} 
if (map.contains(quoteExchTimeAcronym)) { 
    quote.exchtime = getExchTime(String.valueOf(map.get(quoteExchTimeAcronym).get)); 
} 

Это выглядит довольно излишним, любые предложения по его улучшению?

Answer 1

Вы можете сделать что-то вроде:

map.get(quoteBidPriceAcronym).map { item => 
    quote.bid.price = item.map(_.asInstanceOf[Number].doubleValue()) 
    quote.changed = true 
} 

Другие вопросы могут быть лучше зафиксировать снаружи. Например. почему map [quoteBidPriceAcronym] хранит параметр, если ваш код предполагает, что он не будет None?

Answer 2

Лично мне не нравится код, изменяющий состояние объекта (quote), но это вопрос о Scala, а не о функциональном программировании.

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

например (при условии, map имеет тип Map[String, Any]):.

val actions: Map[String, PartialFunction[Any, Unit]] = Map(
    (quoteBidPriceAcronym, {case n: Number => quote.bid.price = Some(n.doubleValue())}), 
    (quoteBidSizeAcronym, {case n: Number => quote.bid.size = Some(sizeMultipler() * n.doubleValue())}), 
    ... 
    ... 
) 

for((k,v) <- map; action <- actions.get(k); _ <- action.lift(v)) 
    quote.changed = true; 

for конструкт здесь перебирает map ключевых значений, то (следующий уровень итерации, за возможных действий, доступных для ключа Если action обнаруживается, что частичная функция, она будет поднята, чтобы сделать его функцию от Any до Option[Unit]. таким образом, вы можете перебирать в дополнительном внутреннем уровне, так quote.changed = true только запускается, когда действие определяются для v.

Answer 3

Что-то вроде этого возможно?

val handlers = Map[String, Number => Unit] (
    quoteBidPriceAcronym -> { n => quote.bid.price = Some(n.doubleValue) }, 
    quoteBidSizeAcronym -> { n => quote.bid.size = Some(sizeMultipler() * n.intValue }, 
etc. ... 
) 

for { 
    (k,handler) <- handlers 
    values <- map.get(k).toSeq 
    quote.chanded = true 
    _ = handler(n.asInstanceof[Number]) 
}