Question

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

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

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

Answer 1

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

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

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

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