28个案例问题分析---007---在线人员逻辑反例--ThreadLocal、继承、索引失效、

深藏阁楼爱情的钟 2023-10-09 15:57 34阅读 0赞

#### ThreadLocal、继承、索引失效、 ####

*  一：背景介绍
 *   *  1.1 具体问题
     *  1.2 问题实际到的知识点
 *  二：解决方案
 *   *  2.1 类中公共变量由于线程共享，导致的数据混乱（公共变量）
     *   *  2.1.1使用ThreadLocal变量
         *   *  2.1.1.1 代码复现
             *  2.1.1.2 执行结果
     *  2.2 在线人员逻辑（逻辑漏洞）
     *   *  2.2.1 问题二：保存数据没有考虑业务隔夜
         *  2.2.2 参数不一致，导致维护混乱
         *  2.2.3 sql不考虑业务，出现明显逻辑漏洞
     *  2.3：继承（this）
     *  2.4 慢sql问题（索引失效）
 *  三：总结升华

## 一：背景介绍 ##

本文是对项目实战中出现的6个问题进行的总结，希望通过总结的方式，提升自己的技术水平，避免类似的错误。

### 1.1 具体问题 ###

1.  类中写了公共变量最后导致数据混乱现象
2.  保存数据没有考虑业务的隔夜覆盖导致的逻辑漏洞
3.  涉及到继承，对于this，如果父类有同样的成员最终使用哪一个？
4.  参数不一致导致后续维护混乱
5.  mysql由于关联字段类型不一致导致产生索引失效问题，进而产生慢sql
6.  sql不考虑业务导致有明显的逻辑漏洞

### 1.2 问题实际到的知识点 ###

上文涉及到的知识点主要有

*  公共变量（1）、
 *  业务逻辑（2，4，6）、
 *  继承中的this（3）、
 *  mysql索引失效（5）。

## 二：解决方案 ##

### 2.1 类中公共变量由于线程共享，导致的数据混乱（公共变量） ###

![在这里插入图片描述][9023a999ccc3438ca244e1f4a34cd6d2.png]  
线程是共享全局变量的，如果在业务过程中，对全局变量进行修改操作。在多线程的情况下，会产生全局变量数据混乱问题。

#### 2.1.1使用ThreadLocal变量 ####

ThreadLocal是Java中的一个类，用于实现线程本地变量。线程本地变量指的是每个线程拥有自己独立的变量副本，不同线程之间互不干扰。可以理解为在多线程环境下，ThreadLocal变量是每个线程私有的变量。

##### 2.1.1.1 代码复现 #####

直接上代码

package org.example;
    
    public class Main {
        
    
        static ThreadLocal<String> localVar = ThreadLocal.withInitial(() -> "默认localVar");
    
        static void  print(String str){
        
            //打印当前线程内存储的本地变量的值
            System.out.println( str +"   de~~~   " + localVar.get());
        }
    
        public static void main(String[] args) {
        
    
            //创建线程一
            Thread thread1 = new Thread(new Runnable() {
        
                @Override
                public void run() {
        
                    //打印一下localvar的值
                    System.out.println("thread1线程" + localVar.get());
                    //修改一下localvar的值
                    localVar.set("thread1的local");
                    //打印
                    print("thread1");
                }
            });
    
            thread1.start();
    
            //创建线程二
            Thread thread2 = new Thread(new Runnable() {
        
                @Override
                public void run() {
        
                    System.out.println("线程二不共享localVar变量，值为默认值" + localVar.get());
                }
            });
    
            thread2.start();
    
        }
    }

上述代码

*  创建了一个 ThreadLocal类型名为localVar的变量，并且赋初值为 默认**localVar**
 *  一个打印 localVar值的print（）方法
 *  main方法 创建了两个线程，第一个线程操作修改了localVar的值，第二个线程直接打印了localVar的值

##### 2.1.1.2 执行结果 #####

![在这里插入图片描述][24d2f046997640dba5cb231cc3d3b2f6.png]  
从结果上我们可以看出 **第一个线程修改了我们定义的localVar的值之后，第二个线程打印的仍是我们一开始初始化设置的默认值**  
结论：**ThreadLocal 定义的变量是线程安全的**

### 2.2 在线人员逻辑（逻辑漏洞） ###

#### 2.2.1 问题二：保存数据没有考虑业务隔夜 ####

![在这里插入图片描述][80c40daf29cd492fad67650a9b6d0b1a.png]  
在查询的逻辑里添加了 createDate的字段，这里的createDate是按照天取得年月日。如果跨天就会出现问题。

#### 2.2.2 参数不一致，导致维护混乱 ####

![在这里插入图片描述][6fdcfe8aabeb4124a9aadc3a03ea1c7b.png]  
登录状态问题，这里的状态在之前的业务里，已经传入了，但是后续却用了一个魔法值表示，虽然功能上没有出现问题，但是却多了一个写死的值，不利与复用。

#### 2.2.3 sql不考虑业务，出现明显逻辑漏洞 ####

![在这里插入图片描述][4e89f5bb541a4ea88d5f352f03f98514.png]  
此sql对应的是教师进行结课之后，将学生的最近登录时间修改。这里存在两个问题。

1.  update\_time字段的使用，这个是我们数据库建表规范必备三字段（id，create\_time,update\_time）之一,业务需要应该新建字段，而不是使用update\_time字段
2.  额外添加了createDate字段，用当天时间做为where条件，如果跨天的话，就会出现下线状态无法更新问题。

### 2.3：继承（this） ###

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

*  Java中继承关系的父子类，相同的方法会被重写
 *  当子类父类中的成员没有重名时，子类都可以通过this去调用。
 *  当成员方法 重名，子类就会将父类中的方法进行重写。如果还想调用父类中的方法只能通过 super去进行调用。
 *  当时成员变量不存在覆盖重写：在子类中只能通过super调用父类的

### 2.4 慢sql问题（索引失效） ###

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

*  索引会因为 隐示数据转换造成 索引失效的问题。在设置外键的时候，一定要保持与相对应的主键的数据类型一致，否则查询的时候，就会出现不走索引的慢sql。  
    具体原因可以参考：[mysql索引博客][mysql]

## 三：总结升华 ##

以上为我对开发中出现的6个问题的总结。这个问题高所我们使用一项技术的时候，一定要清楚使用边界。并且出现问题要及时总结，保证不出同样的问题。

[9023a999ccc3438ca244e1f4a34cd6d2.png]: https://img-blog.csdnimg.cn/9023a999ccc3438ca244e1f4a34cd6d2.png
[24d2f046997640dba5cb231cc3d3b2f6.png]: https://img-blog.csdnimg.cn/24d2f046997640dba5cb231cc3d3b2f6.png
[80c40daf29cd492fad67650a9b6d0b1a.png]: https://img-blog.csdnimg.cn/80c40daf29cd492fad67650a9b6d0b1a.png
[6fdcfe8aabeb4124a9aadc3a03ea1c7b.png]: https://img-blog.csdnimg.cn/6fdcfe8aabeb4124a9aadc3a03ea1c7b.png
[4e89f5bb541a4ea88d5f352f03f98514.png]: https://img-blog.csdnimg.cn/4e89f5bb541a4ea88d5f352f03f98514.png
[8881047b5c8d47cc8c98d05d6f845d29.png]: https://img-blog.csdnimg.cn/8881047b5c8d47cc8c98d05d6f845d29.png
[8d616104545d48d5b06432d69f4b0ece.png]: https://img-blog.csdnimg.cn/8d616104545d48d5b06432d69f4b0ece.png
[mysql]: https://blog.csdn.net/hlzdbk/article/details/129319278