改进ruby代码

时间:2016-10-15 12:23:58

标签: ruby-on-rails ruby

人们告诉我,此时我的代码错误。此方法的主要思想是,如果用户按下一次,则创建1个事件;如果每天或每周按下一个用户,则创建一行事件。一切都运作良好,但代码是如此笨重。

a = i = 0.05
b = 1.0
while a <= b:
  print a
  a += i

我无法找到其他解决方案。有任何想法吗?不要注意日期上的解析方法,它需要JS。

1 个答案:

答案 0 :(得分:2)

我认为问题在于您使用控制器做了太多工作而且很难阅读。通常控制器看起来像:

def create
  @client = Client.new(params[:client])
  if @client.save
    redirect_to @client
  else
    render "new"
  end
end

花一点时间考虑一下你的数据。你创建事件。如果每天或每周按下创建按钮,您希望事件出现在一行中。所以我们正在考虑如何显示它,这是一个css / html问题,而不是与控制器有关的问题。

人们说你的代码错误的原因是因为你做的事非常不典型。如果你为学校或项目提交了这个代码,你会得到一个低分,因为你让其他人都努力工作,而不是遵循每个人都接受过培训的基本风格。