由一个投票算法引发的思考

博主的APP最近又新加了一个小功能，每个员工都可以上传自己的工作照，其他员工可以点赞，规则是：每张工作照每个员工（除上传者外）每日可点赞一次。举个例子：

现有注册员工ABCD四人，A上传工作照两张P1和P2，BCD三人每天可为P1、P2分别点赞一次

博主略加思考，写了下面一段代码来实现：

        /// <summary>
        /// 投票
        /// </summary>
        /// <param name="id">被投票对象ID</param>
        /// <param name="up">赞OR踩</param>
        /// <param name="voterName">投票人</param>
        /// <param name="canVote">是否允许投票</param>
        /// <returns></returns>
        public int Vote(int id, bool up, string voterName, out bool canVote)
        {
            canVote = repo.RecordVote(voterName, id, timeService.Now);
            if (canVote)
            {
                return repo.UpdateVoteCount(id, up ? 1 : -1);
            }
            return 0;
        }

注：

1.RecordVote()方法记录投票信息，UpdateVoteCount()方法执行投票，并返回当前被点赞数

2.再来看看RecordVote()方法的实现：

        private static object voteLocker = new Object();

        public bool RecordVote(string voterName, int staffId, DateTime voteTime)
        {
            lock (voteLocker)
            {
                for (; ; )
                {
                    var voter = db.Voters.FirstOrDefault(t => t.VoterName == voterName && t.StaffId == staffId);
                    if (voter != null)
                    {
                        if (voter.VoteTime.Date == voteTime.Date)
                        {
                            return false;
                        }
                        voter.VoteTime = voteTime;
                    }
                    else
                    {
                        voter = new FVoter() { VoterName = voterName, StaffId = staffId, VoteTime = voteTime };
                        db.Voters.Add(voter);
                    }
                    try
                    {
                        db.SaveChanges();
                        return true;
                    }
                    catch (Exception)
                    {
                    }
                }
            }
        }

3.OK，再看看UpdateVoteCount()方法实现

        private static object locker = new Object();

        public int UpdateVoteCount(int staffId, int voteCountChange)
        {
            lock (locker)
            {
                for (; ; )
                {
                    var staff = db.Staffs.FirstOrDefault(i => i.Id == staffId);
                    staff.VoteCount += voteCountChange;
                    try
                    {
                        db.SaveChanges();
                        return staff.VoteCount;
                    }
                    catch (DbUpdateException)
                    {
                        Thread.Sleep(random.Next(1, 10));
                    }
                }
            }
        }

好嘛，这下写完了开始使用，也没啥问题。APP呢也没上线，没啥用户，就几个开发和公司内部的人点来点去...

 

那么，问题来了。比如一个管理员录入员工信息的方法，基本需求是：管理员提前录入所有签约员工信息，员工注册系统时输入手机号即被识别为已签约员工，个人信息自动获取。那这个方法肯定也是需要加上lock的，因为我们会同时存在多个管理员录入信息，要保证不重复录入，只有加上lock...

我想问的是：

1.有木有哪位大神能简单的归纳一下什么样的上下文中需要加lock，需要做并发处理？是不是所有可能并发的方法都需要？

2.管理员的操作算不算并发操作？系统目前允许同一个管理员账户在多端登录操作，那是不是管理员操作的所有方法都需要做并发控制？

3.由2引申到员工用户与企业用户，如果我开放允许这些用户的账户在多端登录系统，是不是都算并发？

4.再来随便挑一个方法，比如注册方法，用户名为邮箱，要求不重复，然而我的数据库字段并没有做唯一约束，是不是也得加上lock？因为是有可能同时很多用户用同一个邮箱注册的，或者客户端没写好，多次重复点击了注册按钮...

5.那么我很忧心的是，是不是所有方法都成了可能被并发的方法？

...

博主编程水平有限，团队也很小，感觉知识很难扩充。如果上述内容很白很菜（自己都这么觉得，感觉进了死胡同又绕不出来），请一笑了之，也欢迎指点一二，感激不尽。

 

天天为项目忙碌，感觉自己的水平并没有提高，迷茫ing...请各位指点迷津