2016-02-11 56 views
1

有什么方法可以重构这段代码吗?有没有办法重构这段代码?

def my_method 
    response = HTTParty.post 'http://www.url.com', { 
    body: { 
     param1: "value1", 
     param2: "value2" 
     } 
    } 
    response.message.include?("OK") ? true : false 
end 

我想检查响应消息包括好的,但无需限定响应作为一个变量。有没有什么办法可以检索方法中前一步的当前方法回复?

+1

为什么你无法将响应定义为变量? – Caillou

回答

9
HTTParty.post('http://www.url.com', { 
    body: { 
    param1: "value1", 
    param2: "value2" 
    } 
}).message.include?("OK") 

请注意,那? true : false也是多余的。

1
def my_method 
    HTTParty.post('http://www.url.com', { 
    body: { 
     param1: "value1", 
     param2: "value2" 
     } 
    }).message.include?("OK") 
end 
+0

你正在''Hash'实例上调用'message'。 – mudasobwa

+0

是的。我太快了。 – Caillou

+0

那么,我们在修正之后有相同的答案:p – Caillou

相关问题