@Vincent178 没见到楼主调用 save 啊，还是没有贴出来

#1 楼 @shawnyu 在 model 里面吗？之前却是没有调用， ＝ ＝，我刚试了一下，会产生这样的错误

undefined method `save' for 1:Fixnum

我想应该是 viewed_count 的值为 1 了，没有 save 的方法～

#1 楼 @shawnyu 搞定啦，万分感谢。现在 view! 方法是这样的

def view!
  if self.viewed_count.nil?
    self.viewed_count = 1
    self.save
  else
    self.viewed_count += 1
    self.save
  end
end

提个小建议，楼主的代码风格应该 ruby 一点。估计看到帖子的人心里知道但是没说....

self.class.update_counters self.id, views_count: 1

#4 楼 @WilliamZhu 嗯，谢谢你的建议。你看这段代码可以怎样改进呢？

#5 楼 @as181920 cool, 大哥你太神了。终于看懂了，马上去用一下！

def view!
  self.viewed_count ||= 0
  self.viewed_count += 1
  self.save
end

#8 楼 @fresh_fish 比我原来的好看很多，||=用到这里好棒，谢谢

数据库创建的时候建议给个默认值 0，然后可以用

def show
  @patient = Patient.find(params[:id])
  @patient.increment!(:view_count)
end

#10 楼 @quakewang 正解. 除去代码写法外，计数类的一定要考虑到对数据库性能影响。 有两种方案：

1. 使用外部 (如 Redis) 进行计数，积累到一定次数后一次 update
2. 使用对象缓存 对缓存的 patient 进行计数更新。当缓存进行的同步的时候会自动将计数更新进数据库。

#10 楼 @quakewang Great, 已经用上了你的方案！thx～

#11 楼 @WilliamZhu 感觉有点理解不能，主要是对“性能"这个词完全没有感觉，我觉得我还要揣摩好久，不过谢谢

#11 楼 @WilliamZhu Good！如果因为这个用上 Redis 有点未免太重，简单的在内存里 cache 下，定时 increase 到数据库中就好了，不过这样有部分数据丢失的风险，比如服务重启。

Patient.update_counters @patient.id, :viewed_count => 1

http://apidock.com/rails/ActiveRecord/CounterCache/update_counters

#15 楼 @Rei Great, helps a lot! thx!

#15 楼 @Rei 这种方式也是不会缓存的。点击一次，update 一次数据库。 #14 楼 @Magic 不中。应该很少应用没有用到外置缓存吧。内存 Cache 无法做到负载均衡。一个简单浏览计数，丢掉就丢掉，你 1000000 次访问丢掉几十次访问，应该没什么问题吧。服务重启的话，连访问暂时都没搁置了，还要什么点击次数

没有考虑同一 IP 多次访问限制的问题？

#18 楼 @lewisgrady 嗯，这个没有考虑进去 如果加入同 IP 算一次访问，是不是用 impressionist？