算法题用static声明变量引发的血案

【题目】

给定一棵二叉树，你需要计算它的直径长度。一棵二叉树的直径长度是任意两个结点路径长度中的最大值。这条路径可能穿过根结点。

示例 :
给定二叉树

          1
         / \
        2   3
       / \     
      4   5    
返回 3, 它的长度是路径 [4,2,1,3] 或者 [5,2,1,3]。

注意：两结点之间的路径长度是以它们之间边的数目表示。

来源：力扣（LeetCode）
链接：https://leetcode-cn.com/problems/diameter-of-binary-tree
著作权归领扣网络所有。商业转载请联系官方授权，非商业转载请注明出处。

【解析】

题目主要是求树深度的变种，可以理解为，求节点中左右子树和的最大值。最远的路径很容易可以想到是左边最远的地方到右边最远的地方；但是不一定是从根节点为中心，也可能是其他任意节点，所以需要求出最大值。

【解法】

/**
 * Definition for a binary tree node.
 * public class TreeNode {
 *     int val;
 *     TreeNode left;
 *     TreeNode right;
 *     TreeNode(int x) { val = x; }
 * }
 */
class Solution {
    private int max = 0;
    public int diameterOfBinaryTree(TreeNode root) {
        deep(root);
        return max;
    }
    public int deep(TreeNode node) {
        if (node == null) {
            return 0;
        }
        int left = deep(node.left);
        int right = deep(node.right);
        max = Max(max, left + right);
        return 1 + Max (left, right);
    }
    public int Max(int a, int b) {
        return a > b ? a : b;
    }
}

【问题】

这里碰到一个问题， 一开始没注意，int max = 0;定义成了static int max = 0; 

所以虽然本地运行一直正确，但是后台在跑自动化的时候，max并没有重置，原来的结果还在，所以一直报错，如图，明明为空，算法直接返回，但是max之前计算的值还在，导致返回的结果错误：

【教训】 

在做算法的时候，不用用static定义变量！（应该也不存在需要用static定义变量的情况吧，至少我没有碰过或者想到有需要用static定义变量的场景）