383任务复盘

image.png

一、什么是383任务?

好牛逼的名字,有木有,哈哈,其实就是给chat和tag之间加个中间model,就像上面图片所说的,增加Chat::Tag模型

本来打算9.6~9.8号3天完成的,结果却是这样:

image.png

到9.25号才完成,花了20天才完成,7倍啊,我也是醉了。

二、分析一下原因吧?

总的来说,个人原因居多,自己水平不行啊,客观原因可能就是微信上问问题效率太慢,而且github上回复也慢呢

仔细分析下犯的错误吧。数了一下,至少30个以上的错误,这里选择31个比较有代表性的。

1、不小心把本地的Gemfile.lock给上传了

image.png

2、chat/customer.rb忘记加accepts_nested_attributes_for、json接口和customers_tags_attributes属性


image.png

首先,accepts_nested_attributes_for :customers_tags是为了让customers_tags的属性也能和customers属性一样被更新、删除

其次,json_options是序列化的参数,到时候h5页面可能会用到

最后,customers_tags_attributes应该是这样写:

[ customers_tags_attributes: %i[ id tag_id _destroy ] ]

如果不加的话,怎么传参数啊,id是customers_tags的id,_destroy可以删除本记录。

3、没有加validate

image.png

4、字母序

image.png

5、少了acts_as_paranoid(现在知道了这是软删除)

image.png

6、tag也要加上validate

image.png

当时没有认真想想tag添加的时候也要验证一下在未删除情况下唯一性、是否存在

7、这个忘记加dependent::restrict_with_error了(现在知道了,当tag被customer使用时是不能删除的,当出现有关联的tag被删除时,restrict_with_error会限制并报错)

image.png

8、字母序

image.png

9、本来抄系统里别的文件的代码,但是感觉enabled的默认值应该是true才对,于是将原来的false改成true,谁知道数据库的默认值都为false,由Ruby代码控制实际默认值的

image.png

10、lock_version是乐观锁,忘加了

image.png

11、不知道还能这么变(现在知道了,可以写成这样,代码更少,但是不知道为什么要要加deleted_at_and_customer_id_and_tag_id)

image.png

12、不知道Rails标准(现在知道了,如果要加索引,可以这样)

image.png

13、少了allow_destroy(加上这个就也能把关联的属性删除)

image.png

14、customer对应的是多个customers_tags,这里我直接写成了一个tag_id,明显不对嘛,在写这个的时候,还是20天前,那时候对这些关联啊懵懵懂懂的,现在基本都明白了

为嘛json_options不用写

image.png

15、写多了一行,太粗心了

image.png

16、这个也是对acts_as_paranoid不知道,所以犯了这么个错误

image.png

17、要加上validate

image.png

18、又是字母序

image.png

19、 又缺少acts_as_paranoid

![image.png](http://upload-im ages.jianshu.io/upload_images/1866870-eb68997de2bf304c.png?imageMogr2/auto-orient/strip%7CimageView2/2/w/1240)_

21、这样肯定不对啊,customer的view怎么会有tag_id的字段呢?当时完全是懵逼的状态。现在知道了customer不只有一个tag,所以应该是多维数组的形式来传参数

image.png

22、这也不对

zhe'ye

23、当时完全不会做,剑爸5分钟写出来的,后来还是需要修改一下。

因为当时是通过chat/cusomer修改bag/customer,有且只有这样的逻辑,所以没加上chat/cusomter本身的修改。

另外,chat/customer和bag/customer的关联是polymorphic的,如果用accepts_nested_attributes_for 需要手动指定type,前端就比较复杂。

image.png

后来改成这样了

    @customer = customers.find_by(user_name: params[:id]) || customers.find(params[:id])
    @saved = model.transaction do
      @customer.attributes = param.permit(model.admin_fields)
      @customer.customizable&.add_remark!(param[:remark_name])
      @customer.save!
    end

24、这句话啥意思

image.png

有很多疑问,比如什么是customizable,什么是polymorphic类型

customizable是customer的多态类型,那么,可以这样定义:

belongs_to :chat

为什么polymorphic类型的关联不能直接用accepts_nested_attributes_for

25、学习了

image.png

26、系统会自动读取chat/customers_tag的表名

image.png

27、学习了


image.png

28、如果换成t.格式的话就要写在creat_table里面

image.png

29、为什么要加上chatroom.find(user_name:params[:id])呢?

image.png

user_name是微信用户登录的识别码

30、method::get 可以省略 这是因为默认就是get方式

image.png

31、让我用transaction写,但是不会写啊,后来剑爸写的

image.png

第一次做这么完整一点的任务,包含三个model,多对多的关系。

收获:让我了解了很多Rails的功能和特性

1、关联的Model如果加上touch: true,那么和这个Mode关联的另外一个model在更新时,这个model的updated_at也会被更新

2、inverse_of可以避免重复查询

3、accepts_nested_attributes_for可以更新、删除关联的属性

4、transaction可以直接写成一个代码块

@saved = model.transaction do
 ....
end

5、acts_as_paranoid 是软删除的gem

6、polymorphic多态让model更简洁

7、validate的相关用法

比如uniqueness、scope、existence、presence

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,445评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 85,889评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,047评论 0 337
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,760评论 1 276
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,745评论 5 367
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,638评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,011评论 3 398
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,669评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 40,923评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,655评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,740评论 1 330
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,406评论 4 320
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 38,995评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,961评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,197评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,023评论 2 350
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,483评论 2 342

推荐阅读更多精彩内容