Действия контроллера, вызывающие ошибку

Question

У меня есть имя метода метода process_message, и его целью является подписание людей до определенных групп в зависимости от того, какое сообщение оно получает. Сейчас все работает отлично, за исключением того, что он снова и снова будет регистрировать человека в той же группе. Я пытаюсь добавить некоторый быстрый код, чтобы сделать это, чтобы они могли только подписываться на определенную группу один раз. Я подумал, что я понял это, но я получаю сообщение об ошибке, когда пытаюсь выполнить то, что я написал. Ill показать некоторые скриншоты и некоторый код для ясности.

Вот мой метод контроллера

def process_message 
if message_params[:action] == 'subscribe' 
    # result = SubscribeToGroup.call(message_params) 
    # results.success? ? result.message : result.error 
    message_params[:id] = "visitor" if message_params[:id] == "tulip" # hack for now 
    @group = Group.find_by(name: message_params[:id])  
    if @group && @subscriber.groups << @group unless groups.include? (@group)            ^
    @group.response 
    else 
    'You cannot subscribe to unknown group' 
    end 
elsif message_params[:action] == 'stop' 
    # result = UnsubscribeToGroup.call(message_params) 
    # results.success? ? result.message : result.error 
    group_name = message_params[:id] 
    @group = @subscriber.groups.find_by(name: group_name) 
    @subscriber.groups.destroy(@group) if @group 
    "You are now unsubscribed from #{group_name.upcase} notifications. Text '#{group_name.upcase}' to start receieving updates again." 
else 
    "Sorry, we don't recognize that command." 
end 
rescue Exception => e 
"Something went wrong. Try again. #{e.message}" 
end 

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

Я надеюсь, что моя проблема ясна, дайте мне знать, если вам нужно больше кода.

Answer 1

Я бы избежать положить if и unless на одной и той же линии

Это то, что вы пытаетесь сделать?

# You may have meant to provide `@subscriber.groups` instead of `groups` here. I'm not sure from information provided. 
unless groups.include?(@group) 
    @subscriber.groups << @group 
end 

if @group && @subscriber.groups.present? 
# ... 

Answer 2

У вас есть if и unless в одной строке. Это дает вам синтаксическую ошибку.

Логика здесь не ясна. Возможно, что-то, что вы хотите, просто можно сделать, добавив новую строку, так что оператор if находится внутри if-блока, а не if-condition.

if @group.present? 
    @subscriber.groups << @group unless groups.include? (@group)             
    @group.response 
else 
    'You cannot subscribe to unknown group' 
end