Как реорганизовать этот код, чтобы сделать его более читабельным и эффективным?

Question

Мне нужна помощь в рефакторинге кода. Я попробовал лучший, получив следующий код.Есть ли что-нибудь, что я могу сделать

class OrdersController < ApplicationController
  before_action :get_cart
  before_action :set_credit_details, only: [:create]

  # process order
  def create
    @order = Order.new(order_params)

    # Add items from cart to order's ordered_items association
    @cart.ordered_items.each do |item|
      @order.ordered_items << item
    end

    # Add shipping and tax to order total
    @order.total = case params[:order][:shipping_method]
    when 'ground'
      (@order.taxed_total).round(2)
    when 'two-day'
      @order.taxed_total + (15.75).round(2)
    when "overnight"
      @order.taxed_total + (25).round(2)
    end

    # Process credit card

    # Check if card is valid
    if @credit_card.valid?

    billing_address = { 
      name: "#{params[:billing_first_name]} # . 
      {params[:billing_last_name]}",
      address1: params[:billing_address_line_1],
      city: params[:billing_city], state: params[:billing_state],
      country: 'US',zip: params[:billing_zip],
      phone: params[:billing_phone]
     }
    options = { address: {}, billing_address: billing_address }

   # Make the purchase through ActiveMerchant
    charge_amount = (@order.total.to_f * 100).to_i
  response = ActiveMerchant::Billing::AuthorizeNetGateway.new(
      login: ENV["AUTHORIZE_LOGIN"],
      password: ENV["AUTHORIZE_PASSWORD"]
    ).purchase(charge_amount, @credit_card, options)

  unless response.success?
    @order.errors.add(:error, "We couldn't process your credit 
  card")
  end
else
  @order.errors.add(:error, "Your credit card seems to be invalid")
  flash[:error] = "There was a problem processing your order. Please try again."
  render :new && return
end

@order.order_status = 'processed'

if @order.save
  # get rid of cart
  Cart.destroy(session[:cart_id])
  # send order confirmation email
  OrderMailer.order_confirmation(order_params[:billing_email], session[:order_id]).deliver
  flash[:success] = "You successfully ordered!"
  redirect_to confirmation_orders_path
else
  flash[:error] = "There was a problem processing your order. Please try again."
  render :new
end
end

private

def order_params
  params.require(:order).permit!
end

def get_cart
  @cart = Cart.find(session[:cart_id])
  rescue ActiveRecord::RecordNotFound
end

def set_credit_details
  # Get credit card object from ActiveMerchant
  @credit_card = ActiveMerchant::Billing::CreditCard.new(
    number: params[:card_info][:card_number],
    month: params[:card_info][:card_expiration_month],
    year: params[:card_info][:card_expiration_year],
    verification_value: params[:card_info][:cvv],
    first_name: params[:card_info][:card_first_name],
    last_name: params[:card_info][:card_last_name],
    type: get_card_type # Get the card type
  )
end

def get_card_type
  length, number = params[:card_info][:card_number].size, params[:card_info][:card_number]
  case
  when length == 15 && number =~ /^(34|37)/
    "AMEX"
  when length == 16 && number =~ /^6011/
    "Discover"
  when length == 16 && number =~ /^5[1-5]/
    "MasterCard"
  when (length == 13 || length == 16) && number =~ /^4/
    "Visa"
  else
    "Unknown"
  end
end

end

Продукты с ценовым атрибутом.У нас есть тележки для покупок, в которых много товаров через объединенную таблицу OrderedItems.OrderedItem принадлежит к корзине и продукту.У него есть количественный атрибут для отслеживания количества заказанных продуктов.

OrderedItem также принадлежит_ к заказу

Я хотел бы знать, можно ли провести дальнейший рефакторинг.

Answer 1

1. Прежде всего вы должны переместить всю эту бизнес-логику из контроллера в модели и сервисы (OrderProcessService, PaymentService).Все приватные методы контроллера принадлежат PaymentService.

2. Разделить код на более мелкие методы.

   Если делать это на уровне модели, некоторые вещи, которые приходят мне в голову при чтении вашего кода, следующие:

   @order.add_items_from_cart(@cart)

   @order.add_shipping_and_tax(shipping_method)

3. Заказы должны быть сначала сохранены (сохранены в БД), а затем обработаны (приобретены с изменением их статуса).

   • @order.save может завершиться неудачно после успешного платежа, поэтомуклиент потеряет деньги и не получит свой заказ.
   • покупка является важным и критическим процессом, поэтому вы должны убедиться, что все готово к нему (заказ действителен и сохранен)
   • клиент должен иметь возможность совершить покупку позже или после того, как страница оплаты была случайно перезагружена, без повторного заполнения формы
   • обычно после выполнения платежа вы должны отправить идентификатор заказа в платежную систему.Платежная система сохранит идентификатор, и вы всегда будете знать, к какому заказу относится платеж.

Есть много других вещей, которые следует учитывать.У вас много работы.