我应该在重用方法之外或之内声明变量吗?

时间:2013-05-27 02:31:00

标签: java

(我使用String作为示例,但它可以替换为Object> MB的内存)

做了很多这样的事情:

private static String mTempString = "";

private static void SomeMethod()
{
    mTempString = "Whatever Result";
}

现在我的问题是,如果我这样写的话:

private static void SomeMethod()
{
    String mTempString = "Whatever Result";
}

并在循环中使用它(每秒执行数百次)作为示例,Java会知道如何管理内存吗?记忆会有效地存在。 (对不起,我现在不能自己测试一下)

哪种内存效率更高(忽略它们是小变量的事实)

- 编辑--- 在这里找到一篇很好的文章解释它http://www.cs.berkeley.edu/~jrs/4/lec/08

6 个答案:

答案 0 :(得分:9)

尽量缩小变量的范围。

这很重要有几个原因:

  1. 可读性。如果您在具有四个不同值的三个不同位置使用该变量(如图所示),那么您将很难辨别该变量的用途投放。

  2. 错误。如果将单个变量保留在单个明确指定的范围内,则可以减少应用程序中可能出现的错误数量。假设你有那个String,并且你期望它在两个方法中有一些价值,但它完全不同。

  3. 目的意图。我在可读性部分提到了这一点,但是如果你有一个static变量不断重新定义,那么它的预期目的就不清楚了。通常,静态方法和变量可以独立于对象的状态使用,因此如果对象的状态影响静态变量的值,则意图变得混乱。

  4. 此时我不会担心内存效率(除非你有大量的字符串,但我会说你有两个要担心的事情。)*。

    *:优化的第一条规则:不要这样做。
    第二条优化规则(仅限专家!):不要做

答案 1 :(得分:3)

通常,更喜欢尽可能地限制变量的范围。这使代码更容易理解,调试和重构。

在这种情况下,这两种情况下这些变量所需的内存非常小。每个变量都是对象的引用,而不是对象本身。当用作局部变量时,引用可能只需要四个字节。

答案 2 :(得分:1)

  

局部变量的范围应始终尽可能小。

除非您需要在范围之外的变量,否则最好在范围内声明变量。在这种情况下没有性能差异,但建议最佳编码实践在尽可能小的范围内声明变量。看到这个类似且受欢迎的SO question

另一种情况是使用不可变类型而不是字符串。哪里会有轻微的性能差异(非常微不足道)。由于您不需要在每个循环中初始化变量,因此在顶部声明会更好一点。但是在字符串中,因为它每次都在创建新对象,所以这不是你的问题。

答案 3 :(得分:1)

Jvm应该能够确定这是一个常量并在必要时对其进行优化,这样你就不必担心这两种风格。

对于算术表达式,JVM可以执行常量折叠优化。

如果您只对字符串感兴趣,则常量字符串存储在字符串池中。

您可能知道字符串在java中是不可变的。

因此,如果在java中有常量字符串,它们只会存储一次,所有引用都将指向该字符串对象。

e.g。

    String s = "a" + "bc";

    String t = "ab" + "c";

    System.out.println(s == t);

返回true,因为t和s指向同一个字符串对象。

此外,java类具有常量池,其中所有常量都移动到这些池中。

基本上JVM非常聪明,我认为这些玩具示例中的任何一个都不比其他任何一个更好,至少在效率方面。虽然其他答案提到,但您可能应该考虑软件设计注意事项。

答案 4 :(得分:1)

我对此进行了自己的测试,看看编译器是否实际上是在每次迭代时创建一个“新”对象。以下是以下代码的结果:

private static long mStartedTime;

public static void main(String args[])
{
    long TotalTime = 0;
    int NumberOfLoops = 7;

    for(int i = 0; i < NumberOfLoops; i++)
    {
        mStartedTime = System.currentTimeMillis();

        for(float Index = 0; Index < 10000000; Index++)
        {
            test1("wewgwgwegwegwegsd veweewfefw fwefwef wfwefdwvdw wefwe wevwev etbe tbebetbetb evberve");
        }

        System.out.println("Program took: " + String.valueOf(System.currentTimeMillis() - mStartedTime) + " to complete.");
        TotalTime += System.currentTimeMillis() - mStartedTime;
    }

    System.out.println("Average time taken: " + String.valueOf(TotalTime / NumberOfLoops));
}


public static void test1(String THisIsText)
{
    String Test = THisIsText;
    Test = Test.substring(1);
}

private static String mTempString;
public static void test2(String THisIsText)
{
    mTempString = THisIsText;
    mTempString = mTempString.substring(1);
}

他们得出了不同的结果,似乎把变量放在局部范围导致性能损失:

我只能猜测本地方法变量是否被删除,这需要时间,因此test1需要更长的时间? (任何人都可以证实这一点)

(经过多次测试)

本地范围平均值:平均所用时间:1183

类范围的可重用变量:平均所用时间:1043

答案 5 :(得分:0)

如果mTempString是仅在someMethod()中需要的临时变量,那么它必须在方法内声明为局部变量,而不是作为类的静态成员。原因是你要确保没有其他人对你的临时变量感到困惑。

如果在类中声明它,那么您的代码更容易出错,因为您无法确定谁可以修改变量。

在您的示例中,临时变量被声明为静态字段。这使得代码不是线程安全的。如果两个不同的线程几乎同时调用相同的方法,那么第二个方法调用将改变影响调用第一个调用的线程的临时变量的值。