【项目实战典型案例】07.在线人员列表逻辑混乱反例

桃扇骨 2023-10-11 21:09 45阅读 0赞

#### 目录 ####

*  一：背景介绍
 *  二：思路&方案
 *   *  公共变量导致数据混乱现象
     *   *  解决方案1 使用ThreadLocal
         *   *  ThreadLocal介绍
             *  ThreadLocal常见使用场景
             *  ThreadLocal使用
         *  解决方案2 将全局变量修改为成员变量
     *  业务逻辑问题
     *  面向对象继承问题
     *  索引失效的问题
 *  三：总结

## 一：背景介绍 ##

**该《案例07.在线人员列表逻辑混乱反例》中涉及到的问题：**  
1、类中写了公共变量导致数据混乱现象  
2、保存数据没有考虑业务的隔夜覆盖导致的逻辑漏洞  
3、 如果涉及到继承，对于this，父类有同样的成员你最终用的是那个呢？  
4、 参数传递问题，参数不一致导致的后续维护混乱  
5、 mysql由于关联字段类型不一致导致产生索引失效问题，进而产生慢sql  
6、sql不考虑业务导致的明确逻辑漏洞  
**上面的问题主要分为四类问题：**  
1、公共变量（1）  
2、业务逻辑问题（ 2 4 6 ）  
3、面向对象的问题（3）  
4、索引失效的问题（5）

## 二：思路&方案 ##

### 公共变量导致数据混乱现象 ###

由于线程共享全局变量，并在实际业务的过程中对全局变量进行了操作，所以在高并发、多线程的条件下会产生公共变量导致数据混乱的现象。  
![在这里插入图片描述][07cdbd3656e7455d899ab6f87db6a0d7.png]

#### 解决方案1 使用ThreadLocal ####

##### ThreadLocal介绍 #####

[ThreadLocal详解][ThreadLocal]  
1、ThreadLocal叫做线程变量，意思是ThreadLocal中填充的变量属于当前线程，该变量对其他线程而言是隔离的，也就是说该变量是当前线程独有的变量。  
2、ThreadLocal提供了线程本地的实例。它与普通变量的区别在于，每个使用该变量的线程都会初始化一个完全独立的实例副本。  
3、既然每个Thread有自己的实例副本，且其它Thread不可访问，那就不存在多线程间共享的问题。  
4、ThreadLocal变量通常被private static修饰，当一个线程结束时，它所使用的所有ThreadLocal相对的实例副本都可被回收。

##### ThreadLocal常见使用场景 #####

*  每个线程需要有自己单独的实例
 *  实例需要在多个方法中共享，但不希望被多线程共享

##### ThreadLocal使用 #####

**多线程环境下未使用ThreadLocal出现数据混乱的问题**

public class Client {
        
        public static void main(String[] args) {
        
            // 定义线程实现接口
            Runnable runnable = new Runnable(){
        
                Counter counter = new Counter();
    
                @Override
                public void run() {
        
                    counter.count();
                }
            };
            // 启动10个线程
            for( int i= 0;i< 10;i++) {
        
                new Thread(runnable).start();
            }
        }
    }

public class Counter{
        
     private int number;
         public void count(){
        
            for (int i = 0; i <=10 ; i++) {
        
                number=number+i;
            }
            System.out.println(Thread.currentThread().getName()+ "--"+number;
        }
    }

**实现结果**  
正确计算结果都应该是55  
![在这里插入图片描述][30f46b3ff8474247ac270303210fe236.png]  
**多线程环境下使用ThreadLocal解决数据混乱的问题**

public class Counter {
        
        private static ThreadLocal<Integer> number = new ThreadLocal<Integer>() {
        
            // 重写这个方法，可以修改“线程变量”的初始值，默认是null
            @Override
            protected Integer initialValue() {
        
                return 0;
            }
        };
    
        //计数方法
        public  void count() {
        
            for( int i= 0;i<= 10;i++) {
        
                number.set(number.get()+i);
            }
            System.out.println(Thread.currentThread().getName()+ "--"+number.get());
        }
    }

**实现结果**  
![在这里插入图片描述][20df80c19389435e85db95b74edbbb7a.png]

#### 解决方案2 将全局变量修改为成员变量 ####

public  void count(){
        
             int number = 0;
    
            for (int i = 0; i <=10 ; i++) {
        
                number=number+i;
            }
            System.out.println(Thread.currentThread().getName()+ "--"+number);
        }
    }

**实现效果**  
![在这里插入图片描述][dfb0d52e66c841adac78f8aa041a5b07.png]

### 业务逻辑问题 ###

![在这里插入图片描述][d2a37c6dc0364ae4bc0b5599c7fc7470.png]  
在查询逻辑中添加了createDate的字段，如果13号这天学生没有下课的话，当14号这天你按照createDate字段查询的话，是查不到该学生的数据的，系统默认为该学生是第一次登录软件，所以就会再次存入一条该学生的数据。  
![在这里插入图片描述][838539f6400a4970a0b7fa83ba8c56ac.png]  
上面的paramUser中已经有了isOnline参数，就不需要在进行赋值了，避免数据不一致。  
![在这里插入图片描述][ba76ff3232c54af4996689e0dfcd04cb.png]  
1、如果更新了update\_time，那么在线人员记录中的最近一次登录时间就修改为同一个了  
2、如果在where条件上添加了create\_date的话，那么过了12点结束课程后，所有在线的人就都无法下线了

### 面向对象继承问题 ###

java中继承关系的父子类，相同的方法会被重写  
当子类父类中的成员没有重名时，子类都可以通过this去调用。  
当成员方法 重名，子类就会将父类中的方法进行重写。如果还想调用父类中的方法只能通过 super去进行调用。  
当时成员变量不存在覆盖重写：在子类中只能通过super调用父类的  
![在这里插入图片描述][35f789c7923049619df20a398de304a9.png]

### 索引失效的问题 ###

![在这里插入图片描述][d5eb819fa27144d382dbeecae42860c7.png]

## 三：总结 ##

做事情要从全局观出发，要明确事情的边界

[07cdbd3656e7455d899ab6f87db6a0d7.png]: https://img-blog.csdnimg.cn/07cdbd3656e7455d899ab6f87db6a0d7.png
[ThreadLocal]: https://blog.csdn.net/u010445301/article/details/111322569
[30f46b3ff8474247ac270303210fe236.png]: https://img-blog.csdnimg.cn/30f46b3ff8474247ac270303210fe236.png
[20df80c19389435e85db95b74edbbb7a.png]: https://img-blog.csdnimg.cn/20df80c19389435e85db95b74edbbb7a.png
[dfb0d52e66c841adac78f8aa041a5b07.png]: https://img-blog.csdnimg.cn/dfb0d52e66c841adac78f8aa041a5b07.png
[d2a37c6dc0364ae4bc0b5599c7fc7470.png]: https://img-blog.csdnimg.cn/d2a37c6dc0364ae4bc0b5599c7fc7470.png
[838539f6400a4970a0b7fa83ba8c56ac.png]: https://img-blog.csdnimg.cn/838539f6400a4970a0b7fa83ba8c56ac.png
[ba76ff3232c54af4996689e0dfcd04cb.png]: https://img-blog.csdnimg.cn/ba76ff3232c54af4996689e0dfcd04cb.png
[35f789c7923049619df20a398de304a9.png]: https://img-blog.csdnimg.cn/35f789c7923049619df20a398de304a9.png
[d5eb819fa27144d382dbeecae42860c7.png]: https://img-blog.csdnimg.cn/d5eb819fa27144d382dbeecae42860c7.png