嘿，我一眼就看出问题所在啦！你肯定是把ArrayList ips声明在sendStaticResource方法内部了对吧？每次请求进来，方法都会创建一个全新的ArrayList，处理完请求后这个列表就被销毁了，所以每次长度都是1，旧IP根本留不住。

正确的声明位置和方式 #

要让IP列表在多次请求间留存，你需要把它声明为请求处理类的成员变量（比如你的Servlet类、自定义的请求处理器类），而不是方法里的局部变量。另外还要注意线程安全——Java服务器是多线程处理请求的，多个请求同时读写普通ArrayList会出并发问题，所以推荐用线程安全的集合类。

具体实现步骤

• 把IP列表声明为类的成员变量，使用CopyOnWriteArrayList（属于java.util.concurrent包），它天生支持安全的并发读写，非常适合这种读多写少的场景。
• 在sendStaticResource方法里，每次请求先获取客户端IP，检查列表中是否存在，再做对应的逻辑处理。

代码示例 #

import java.util.concurrent.CopyOnWriteArrayList;
// 假设这是你的请求处理类（比如Servlet或者自定义处理器）
public class ServerRequestHandler {
    // 声明为类的成员变量，全局复用，线程安全
    private final CopyOnWriteArrayList<String> visitedIps = new CopyOnWriteArrayList<>();

    public void sendStaticResource(HttpServletRequest request, HttpServletResponse response) throws IOException {
        // 获取客户端IP地址
        String clientIp = request.getRemoteAddr();
        
        if (visitedIps.contains(clientIp)) {
            // 已访问过，发送对应消息
            response.setContentType("text/html;charset=utf-8");
            response.getWriter().write("<h3>你已经访问过我们的服务器啦！</h3>");
        } else {
            // 首次访问，添加IP到列表并发送欢迎消息
            visitedIps.add(clientIp);
            response.setContentType("text/html;charset=utf-8");
            response.getWriter().write("<h3>欢迎第一次访问！</h3>");
        }
    }
}

额外注意事项 #

• Servlet场景的特殊说明：如果你的处理类是Servlet，默认Servlet是单例模式，所以这个成员变量会在整个Web应用生命周期内存在，所有请求共享这个列表，完全符合你的需求。
• 持久化需求（可选）：如果希望服务器重启后IP记录不丢失，你还需要把列表中的数据定期写入文件、数据库或者Redis等存储介质，重启时再加载回来。
• 替代方案：如果不想用CopyOnWriteArrayList，也可以用普通的ArrayList，但必须在读写操作时加synchronized锁，比如：

// 用普通ArrayList加锁的方式
private final ArrayList<String> visitedIps = new ArrayList<>();

// 在方法里的读写逻辑要加锁
synchronized (visitedIps) {
    if (visitedIps.contains(clientIp)) {
        // 已访问逻辑
    } else {
        visitedIps.add(clientIp);
        // 首次访问逻辑
    }
}

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