添加 docker 版对 user.conf 配置文件的支持

[shenmishajing]

将原版的 user.conf 文件从 git 仓库中重命名为 user.sample.conf 文件，git 仓库中不再具有 conf/user.conf 文件，使得 docker 版可以像 docker-compose.sample.yml 文件中通过挂载外部的 user.conf 文件提供配置。 顺便添加了对时区的支持和对随机跳过舰长礼物以减小被封概率的功能。

[Billyzou0741326]

概率性跳过舰长 而且还是不可调的... 目前没有这个需求的话不建议加

[shenmishajing]

自动送礼物的功能现在不再仅向佩戴的勋章送礼，而是向所有已有勋章送礼，送礼顺序是当前佩戴勋章-user.conf文件中定义的顺序-剩余勋章。

[shoucandanghehe]

自动送礼物的功能现在不再仅向佩戴的勋章送礼，而是向所有已有勋章送礼，送礼顺序是当前佩戴勋章-user.conf文件中定义的顺序-剩余勋章。

建议改成即将过期的礼物再向其它勋章投喂，有时候要囤点辣条留着黑屋应急

[yawwwwwn]

问题很多：

1. 代码里修改格式加空格这个问题。我知道我们代码写得很乱，几个 contributors 的习惯也不一样。所以不需要大家来帮我们改，这不重要。
2. 个人觉得一定概率放弃抢这个机制跟掩耳盗铃没区别，不予采纳。除非你在控制变量的情况下证明不概率放弃与进黑屋有统计学上的关联。
3. 你 README 里加的那一段只适合加在你的 fork 项目，不适合 PR 过来。且你学其他项目将 conf/user.conf 改成了 conf/user.sample.conf，这个属于开发者个人喜好吧，没必要。我们的用户不是所有人都用 docker，不应把过多操作放在 docker 文件中。
4. 自动送礼这部分。
   • 首先你在自己的 user.conf 加了['auto-gift']['sort_list']，但由于你将仓库代码改成了 user.sample.conf，所以这个修改没推上来。
   • 然后用 sort 这个词命名来表示“用户自主选择的”这个意思是不恰当的，别的 contributor 已经指出。
   • temp.reverse() 然后 range() 的 step 参数又设为 -1 是想干什么？
   • 获取的礼物列表我目前看到已经是按照要过期时间排序的，额外排序的必要性待考证。
   • 你的 fetch_medal() 返回值把用户没指定的送礼直播间又加在了后面，不还是返回所有直播间吗？所以你的 auto_send_gift() 是送满所有勋章？
   • 最后就是送礼这里的逻辑问题。你本意是想每天给多个勋章刷满，但是从其他用户的回复就能看出来，有些人不想过多透支之后的礼物，想在无收入时仍然能刷第一目标勋章。所以你这样修改是不行的。我自己目前的策略是，当天要过期的如果送满第一目标还有多的礼物，再送给其他指定的直播间。有空会更新，先鸽置。

[shenmishajing]

@yawwwwwn 我逐个回应一下吧…… 1、代码修改格式的问题，是pycharm自动帮我修改的，你喜欢原来的格式可以在merge的时候使用原来的。 2、我只是见过其他项目里有人讨论概率性跳过会降低被封机率，没有相应数据，同样的，你不喜欢可以在merge的时候使用你原来的代码，不必跟我商量。 3、README里的那一段同理，我只是提一个pr，你才是项目所有者，你不喜欢的都可以在merge的时候改用自己的，不需要跟我说……至于user.sample.conf那个不是个人喜好，如果项目里的是user.conf文件那么如果通过挂载的方式向docker容器提供user.conf文件会挂在 git pull 那一步，这也是我提这个pr的初衷。 4、自动送礼这部份

• 这个是我的失误，如果之后有时间加一个这部份的说明
• sort 那个词就是排序的意思，不是用户指定的意思……
• 先reserve再倒序遍历是为了可以删除列表中的元素，我即希望遍历顺序由原来的列表顺序指定，又希望可以通过删除列表项来减少不必要的迭代次数，先反序再倒序遍历是Python中常用的需要删除时的遍历方法。此处，我多插一句，不知道是不是开发者们都习惯使用c语言，项目的编码方式上更像一个c语言的项目，而非一个Python项目，又很多地方可以通过Python语言的特性提高很多代码的可复用性。
• 按照过期时间排序的原因很简单——我不想浪费任何一个礼物，如果没有这个排序就会存在这样一种情况，有一个礼物马上要过期了，但是送礼的时候选择了其他礼物，第二天这个礼物就浪费了。
• 是的，这个函数本来的意图就是送满所有的勋章。
• 还是想我在上面一个回复中说的，如果想要其他的送礼逻辑可以去项目里提fr或者pr，我提这个pr只是分享出我在用的代码，这个逻辑也是我希望的，如果我没有其他想法的话，我这边这个项目应该也不会再做改动了。最后，还是那句话，我只是提一个pr分享出我的代码，你才是项目的所有者，你不喜欢的东西大可以在merge的时候去掉，不需要跟我商量。

[yawwwwwn]

如果项目里的是user.conf文件那么如果通过挂载的方式向docker容器提供user.conf文件会挂在 git pull 那一步 @Dawnnnnnn

先 reserve 再倒序遍历的我懂了。

就我的理解啊，pull request 这个功能就是以 merge 为目的的，向项目中贡献自己的纠错与实现。你这种分享展示个人喜好的操作我不是很理解。我解释这么多只是怕你接受不了，你能接受那就好办了。可以自用但是不适合更新给所有用户的，直接关。

[shenmishajing]

每个项目的持有者想要实现不同的功能是很正常的，不然 fork 这个功能的存在也就没有意义了。 我提这个pr确实是以merge为目的的，不过最近没又太多的时间处理这个项目pr时的细节，就干脆把merge时调整的任务丢给你们了，提这个pr的分享的目的就只是说不定你们会喜欢这儿的一部分feature，仅此而已，既然你们不喜欢，那也无所谓。