Rails 求助一下

fsword · 2012年09月19日 · 最后由 poshboytl 回复于 2012年09月19日 · 2169 次阅读

最近写了段烂代码

if Rails.env=='test' || Rails.env='cucumber'

结果掉坑里了,痛定思痛后总结了一下,发现自己潜意识里是把 Rails.env 看作一个不变量的,既然如此,我觉得应该有个机制限制它的变化,所以提了一个pull request

有人写了 comment

That said, currently, this has no tests, which would be needed in order to get it accepted as well. Or documentation. But I wouldn't bother writing stuff until someone from core actually reviews this.

我补充了测试用例,不过我的 E 文太烂了,不知道有没有人可以补充一下文档之类的?

个人感觉这个用处不大,可能不会被采用。。。

完整看了一下 pull request 的讨论,对方的意思是说:测试与文档是被 merge 的必要条件,而非充分条件。目前看来恐怕连提议被采纳的机会都不大,因为提倡的用法是 Rails.env.test?... 所以在确认可能 merge 之前连文档都懒得写。- -

是的,我下午看到你提这个问题的时候还没反应过来,推荐的用法确实是 Rails.env.production? 这种形式…… 似乎是 Rails 3 版本加进来的。

同觉得这个 pull request 被 merge 的可能性不大。但锻炼英文写作能力仍然很有必要的,多读多听就是了。

应该用 Rails.env.test?这种形式这个当然没错,所以我才说自己写了个烂代码。 不过,我依然觉得这里需要一种机制锁死某些 Rails 的“全局属性”,这两个没冲突的

这个看起来不像是 Rails 应该避免的问题,所以我估计不会 accept...

我见过一些人 从来都把 string 的对比 调过来

"test" == Rails.env

我不习惯这样,觉得不好看.. 也还算个好习惯吧....至少避免你这样的情况...

需要 登录 后方可回复, 如果你还没有账号请 注册新账号