如何解决移除WebDriver静态修饰符后closeBrowser方法出现的NullPointerException问题

嘿，我明白你遇到的问题了——去掉static修饰符后，closeBrowser()抛出了空指针异常对吧？这其实是因为各个类之间的WebDriver实例没有统一起来，导致调用关闭方法的时候driver还是null。下面我一步步给你讲清楚怎么解决：

先看看你现在的代码结构（我整理了一下，看起来更清晰）：

去掉static后的CommonFunctions类 #

public class CommonFunctions { 
    public WebDriver driver; // 已移除static修饰符
    public void openWebsite(String url) { 
        driver.get(url); 
    } 
    public void closeBrowser() { 
        driver.close(); 
    } 
}

你的TestCase类 #

public class TestCase { 
    CommonFunctions commonFunctions = new CommonFunctions(); 
    @BeforeMethod 
    public void openWeb() { 
        homePage.openWeb(); 
    } 
    @Test 
    public void navigateLoginPage() { 
        homePage.login(); 
    } 
    @AfterMethod 
    public void closeBrowser() { 
        commonFunctions.closeBrowser(); 
    } 
}

问题根源 #

你现在的CommonFunctions里的driver根本没被初始化！而且HomePage里应该也有自己的driver实例，和TestCase里commonFunctions对象的driver完全是两个独立的对象——一个在HomePage里被正常使用，另一个在CommonFunctions里始终是null，调用closeBrowser()的时候当然会抛空指针了。

修复方案：用依赖注入共享同一个WebDriver实例 #

核心思路是让TestCase、CommonFunctions、HomePage三个类共用同一个WebDriver对象，而不是靠static来强行共享。我们用构造方法注入的方式来实现：

1. 改造CommonFunctions类 #

给它添加一个带WebDriver参数的构造方法，把外部传入的driver赋值给类内的成员变量：

public class CommonFunctions { 
    private WebDriver driver;

    // 通过构造方法注入driver
    public CommonFunctions(WebDriver driver) {
        this.driver = driver;
    }

    public void openWebsite(String url) { 
        driver.get(url); 
    } 

    public void closeBrowser() { 
        driver.close();
        // 建议加上quit，避免残留浏览器进程
        driver.quit();
    } 
}

2. 改造HomePage类 #

同样给它添加构造方法注入driver，确保它使用的和其他类是同一个实例：

public class HomePage {
    private WebDriver driver;

    public HomePage(WebDriver driver) {
        this.driver = driver;
    }

    public void openWeb() {
        // 这里用注入的driver打开目标网站
        driver.get("https://your-target-url.com");
    }

    public void login() {
        // 这里编写登录逻辑，使用this.driver定位元素即可
    }
}

3. 改造TestCase类 #

在@BeforeMethod里初始化WebDriver，然后把这个实例传递给CommonFunctions和HomePage：

import org.openqa.selenium.WebDriver;
import org.openqa.selenium.chrome.ChromeDriver;
import org.testng.annotations.AfterMethod;
import org.testng.annotations.BeforeMethod;
import org.testng.annotations.Test;

public class TestCase { 
    private WebDriver driver;
    private CommonFunctions commonFunctions;
    private HomePage homePage;

    @BeforeMethod 
    public void openWeb() { 
        // 初始化WebDriver实例（这里用ChromeDriver，可替换为其他浏览器驱动）
        driver = new ChromeDriver();
        // 把同一个driver注入到两个类中
        commonFunctions = new CommonFunctions(driver);
        homePage = new HomePage(driver);
        // 打开网站
        homePage.openWeb();
    } 

    @Test 
    public void navigateLoginPage() { 
        homePage.login(); 
    } 

    @AfterMethod 
    public void closeBrowser() { 
        commonFunctions.closeBrowser(); 
    } 
}

为什么这样能解决问题？ #

现在三个类用的都是TestCase里初始化的那个WebDriver实例，再也不会出现CommonFunctions里的driver为null的情况了。而且这种写法比用static更符合面向对象的设计原则，耦合度更低，后续要做单元测试或者切换浏览器也更灵活。

额外小建议 #

如果你的测试用例比较多，可以把WebDriver的初始化逻辑抽成一个单独的基类，让所有TestCase继承它，这样就不用每个测试类都重复写初始化代码啦！

内容的提问来源于stack exchange，提问作者heaven