该帖已经被评为新手帖
|
|
---|---|
作者 | 正文 |
发表时间:2007-02-14
ruby学的差不多了,开始仔细阅读中国人自己写的第一个ror的open soruce Hamster。从中学到了一些东西。但看了两天还是决定放弃他了(没有任何贬低Hamster原作者的意思)。之所以决定放弃学习Hamster原因主要还是感觉其中的部分source还是优点不够严谨。下面举几个例子 1.存在没有防止sql注入的代码 tmp_roles = Role.find_by_sql("select roles.id from roles,departments where roles.department_id = departments.id and departments.id=#{department_id}") 2.在进行修改的时候用了下面的方法 def edit_agent if request.get? @agent = Agent.find(params[:id]) else agent = Agent.find(params[:id]) agent.name = params[:name] agent.part = params[:part] agent.person = params[:person] agent.sex = params[:sex] agent.phone = params[:phone] agent.mobile = params[:mobile] agent.lingtong = params[:lingtong] agent.fax = params[:fax] agent.address1 = params[:address1] agent.address2 = params[:address2] agent.qq = params[:qq] agent.msn = params[:msn] agent.email = params[:email] agent.favor = params[:favor] agent.second_person = params[:second_person] agent.second_phone = params[:second_phone] agent.memo = params[:memo] if agent.save flash[:notice] = "<img src='/images/ok.gif'><font color='green'>#代理商#{agent.name}已经成功修改!</font>" redirect_to(:action=>'list_agent') end end end 我觉的用下面的方法就可以了 def edit_agent if request.get? @agent = Agent.find(params[:id]) else agent = Agent.find(params[:id]) if agent.update_attributes(params[:agent]) flash[:notice] = "<img src='/images/ok.gif'><font color='green'>#代理商#{agent.name}已经成功修改!</font>" redirect_to(:action=>'list_agent') end end end 3.在control中存在类似@params这样直接使用实例变量的地方。我觉的还是使用访问子的方式params[:XXX]比较好 4.在前态页面最好使用rails提供的帮助方法,如果想自己写静态标签也应该和rails提供的帮助方法生成的静态标签相同的,不要出现下面的代码 <input type="text" value="<%=@family.name%>" name="name"> 这样到contorl里面只能大量的使用params[:XXX] 5.如果使用下面的代码 <%= form_tag %> <%= end_form_tag %> 在control/action里面如果还想重定向到当前的action里面,那么就不用在action里面再写redirect_to了。毕竟rails的哲学是Don't repeat yourself 再次重申一下,上面的一切内容都是从纯技术的角度看待hamster,也许在hamster的最新版里已经没有现在的情况了(自己比较懒没有跟踪最新版,我说的这五点可能也有不对的地方)同时要感谢hamster的原作者们在业余时间写了hamster,为我们学习ror提供了良师易友 我现在在学习Mephisto,不知道大家都在学什么 声明:ITeye文章版权属于作者,受法律保护。没有作者书面许可不得转载。
推荐链接
|
|
返回顶楼 | |
发表时间:2007-02-14
其实《Agile Web Development with rails》的Depot就足够了。
|
|
返回顶楼 | |
发表时间:2007-02-14
Depot的source小弟已经敲了一遍了,对编写rails代码的帮助是挺大的。可现在没有ror的项目可做,项目又不是很紧,就想找些好的 open source看看,学习一下别人好的动西:)
|
|
返回顶楼 | |
发表时间:2007-02-14
robbin 写道 其实《Agile Web Development with rails》的Depot就足够了。
同意 如果你没有项目做的话,就学习学习ruby。跑到这里来挑人家的刺就是你的不对了。毕竟人家这个项目是以一己之力,在ror这本书还没有出之前写的。有些瑕疵应当理解。而且,有足够的勇气开源出来,值得尊敬。至于里面的一些瑕疵,可以有读者自己去辨别,吸取自己需要的东西,不用全盘接受。 建议:下载一个开源的项目,在编写自己的项目的时候遇到问题了打开看看就行了。有时间陪陪女朋友岂不是乐哉。。。 如果不能接受,自己去写一个,也开源出来。不就有项目做了。 |
|
返回顶楼 | |
发表时间:2007-02-14
dogstar 写道 robbin 写道 其实《Agile Web Development with rails》的Depot就足够了。
同意 如果你没有项目做的话,就学习学习ruby。跑到这里来挑人家的刺就是你的不对了。毕竟人家这个项目是以一己之力,在ror这本书还没有出之前写的。有些瑕疵应当理解。而且,有足够的勇气开源出来,值得尊敬。至于里面的一些瑕疵,可以有读者自己去辨别,吸取自己需要的东西,不用全盘接受。 建议:下载一个开源的项目,在编写自己的项目的时候遇到问题了打开看看就行了。有时间陪陪女朋友岂不是乐哉。。。 如果不能接受,自己去写一个,也开源出来。不就有项目做了。 可能dogstar误解我的意思了,我强调了两遍我的立场,其实写这些东西也很担心有人象dogstar这样误解,我的初中就是想让大家的ror程序更合理,没有其他意思。如果大家都能把自己看到的程序的缺点写出来(当然不可能是全部),那大家起不是在日后写程序的时候都会有进步了。 |
|
返回顶楼 | |
发表时间:2007-02-14
在这里说句题外话,有时候指出别人存在的问题,很可能是出于一种善意的目得,而不是象其他人说的。。。
|
|
返回顶楼 | |
发表时间:2007-02-14
听说beast不错。不过我到是没有参考过什么rails的开源项目。
|
|
返回顶楼 | |
发表时间:2007-02-14
如果你用rest,beast就是最好的参考。里面可以发现很多不错的practice.
|
|
返回顶楼 | |
发表时间:2007-02-16
beast有种很恶劣的倾向,刻意减少程序行数,有时候到了很过分的程度。我现在看到的最长的一行有大约250个字符,不知道有没有更长的。
不过,尽管如此,还是可以从beast学到不少东西。 |
|
返回顶楼 | |
发表时间:2007-02-26
beast之前我發現到一個bug
不過好像都沒人回應我T_T |
|
返回顶楼 | |