我的测量执行时间的代码存在问题

发布于 2024-11-08 02:54:56 字数 3220 浏览 5 评论 0原文

所以,我试图测量一些排序方法的执行时间。

这是我的代码:

public static void main(String[] args)
{
    ...

    MeasureExecutionTime(new Runnable() { public void run() { insertionSort(C); } }, "insertionSort()");
}

=====================

private static void MeasureExecutionTime(Runnable r, String s)
{
    startTime = System.nanoTime();
    try
    {
        r.run();
    }
    finally
    {
        endTime = System.nanoTime();
    }
    elapsedTime = endTime - startTime;
    System.out.println(s + " takes " + elapsedTime + " nano-seconds which is " + formatTime(elapsedTime));
}

=====================

public static String formatTime(long nanoSeconds)
{
    long hours, minutes, remainder, totalSecondsNoFraction;
    double totalSeconds, seconds;

    totalSeconds = (double) nanoSeconds / 1000000000.0;
    String s = Double.toString(totalSeconds);
    String [] arr = s.split("\\.");
    totalSecondsNoFraction = Integer.parseInt(arr[0]);
    hours = totalSecondsNoFraction / 3600;
    remainder = totalSecondsNoFraction % 3600;
    minutes = remainder / 60;
    seconds = remainder % 60;
    seconds = Double.parseDouble(Long.toString((long)seconds) + Double.parseDouble("." + arr[1]));

    StringBuilder result = new StringBuilder(".");
    String sep = "", nextSep = " and ";
    if(seconds > 0)
    {
        if(seconds > 1) result.insert(0, " seconds").insert(0, seconds);
        else result.insert(0, " second").insert(0, seconds);
        sep = nextSep;
        nextSep = ", ";
    }
    if(minutes > 0)
    {
        if(minutes > 1) result.insert(0, sep).insert(0, " minutes").insert(0, minutes);
        else result.insert(0, sep).insert(0, " minute").insert(0, minutes);
        sep = nextSep;
        nextSep = ", ";
    }
    if(hours > 0)
    {
        if(hours > 1) result.insert(0, sep).insert(0, " hours").insert(0, hours);
        else result.insert(0, sep).insert(0, " hour").insert(0, hours);
    }
    return result.toString();
}

我的问题是:

运行此程序后,我输入 int[1000000] 作为输入,它会在大约 12-13 分钟内执行 insertionSort() 然后返回:

insertionSort() takes 767186856920 nano-seconds which is 12 minutes and 470.18685692 seconds.

为什么它给出 470秒?我的代码有什么问题?

=========================

编辑:

替换 seconds = Double.parseDouble(Long.toString(( long)Seconds) + Double.parseDouble("." + arr[1]));Seconds = Seconds + Double.parseDouble("." + arr[1]); ,上一个问题消失了,但又出现了一个问题:

insertionSort() takes 22864 nano-seconds which is 2.000002864 seconds.

应该是0.000022864秒。

========================== =

编辑2:

我可能会发现错误。当nanoSeconds很大时,arr[1]就可以了,但是当nanoSeconds很小时,arr[1]将转换为指数形式,即14931纳秒=> 4.931E-6 秒。。我该如何解决这个问题?

==========================

EDIT3:

好的,我找到了解决方案:

if(arr[1].contains("E")) seconds = Double.parseDouble("." + arr[1]);
else seconds += Double.parseDouble("." + arr[1]);

So, I am trying to measure the execution time of some sorting methods.

Here is my code:

public static void main(String[] args)
{
    ...

    MeasureExecutionTime(new Runnable() { public void run() { insertionSort(C); } }, "insertionSort()");
}

=====================

private static void MeasureExecutionTime(Runnable r, String s)
{
    startTime = System.nanoTime();
    try
    {
        r.run();
    }
    finally
    {
        endTime = System.nanoTime();
    }
    elapsedTime = endTime - startTime;
    System.out.println(s + " takes " + elapsedTime + " nano-seconds which is " + formatTime(elapsedTime));
}

=====================

public static String formatTime(long nanoSeconds)
{
    long hours, minutes, remainder, totalSecondsNoFraction;
    double totalSeconds, seconds;

    totalSeconds = (double) nanoSeconds / 1000000000.0;
    String s = Double.toString(totalSeconds);
    String [] arr = s.split("\\.");
    totalSecondsNoFraction = Integer.parseInt(arr[0]);
    hours = totalSecondsNoFraction / 3600;
    remainder = totalSecondsNoFraction % 3600;
    minutes = remainder / 60;
    seconds = remainder % 60;
    seconds = Double.parseDouble(Long.toString((long)seconds) + Double.parseDouble("." + arr[1]));

    StringBuilder result = new StringBuilder(".");
    String sep = "", nextSep = " and ";
    if(seconds > 0)
    {
        if(seconds > 1) result.insert(0, " seconds").insert(0, seconds);
        else result.insert(0, " second").insert(0, seconds);
        sep = nextSep;
        nextSep = ", ";
    }
    if(minutes > 0)
    {
        if(minutes > 1) result.insert(0, sep).insert(0, " minutes").insert(0, minutes);
        else result.insert(0, sep).insert(0, " minute").insert(0, minutes);
        sep = nextSep;
        nextSep = ", ";
    }
    if(hours > 0)
    {
        if(hours > 1) result.insert(0, sep).insert(0, " hours").insert(0, hours);
        else result.insert(0, sep).insert(0, " hour").insert(0, hours);
    }
    return result.toString();
}

My problem is:

After run this program, and I enter int[1000000] as an input, it executes insertionSort() in about 12-13 minutes and then returns:

insertionSort() takes 767186856920 nano-seconds which is 12 minutes and 470.18685692 seconds.

why it gives 470 seconds? what is wrong in my code?

=========================

EDIT:

After replacing seconds = Double.parseDouble(Long.toString((long)seconds) + Double.parseDouble("." + arr[1])); with seconds = seconds + Double.parseDouble("." + arr[1]);, the previous issue is gone, but another issue showed up:

insertionSort() takes 22864 nano-seconds which is 2.000002864 seconds.

It should be 0.000022864 seconds.

=========================

EDIT2:

I might discover the error. when nanoSeconds is large, arr[1] will be ok but when nanoSeconds is small, arr[1] will convert to exponential form, i.e 14931 nano-seconds => 4.931E-6 seconds.. How can I solve this issue?

==========================

EDIT3:

Ok, I found the solution:

if(arr[1].contains("E")) seconds = Double.parseDouble("." + arr[1]);
else seconds += Double.parseDouble("." + arr[1]);

如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。

扫码二维码加入Web技术交流群

发布评论

需要 登录 才能够评论, 你可以免费 注册 一个本站的账号。

评论(4

花心好男孩 2024-11-15 02:54:56

问题就在这里:

seconds = Double.parseDouble(Long.toString((long)seconds) + 
                             Double.parseDouble("0." + arr[1]));

假设进入这一行的12,而arr[1]“456”。那么

seconds = Double.parseDouble("12" + Double.parseDouble("0.456"));
seconds = Double.parseDouble("12" + 0.456);
seconds = Double.parseDouble("12" + "0.456");
seconds = Double.parseDouble("120.456");
seconds = 120.456.

为什么不这样做:

seconds = seconds + Double.parseDouble("0." + arr[1]);

The problem is here:

seconds = Double.parseDouble(Long.toString((long)seconds) + 
                             Double.parseDouble("0." + arr[1]));

Say seconds is 12 entering this line and arr[1] is "456". Then

seconds = Double.parseDouble("12" + Double.parseDouble("0.456"));
seconds = Double.parseDouble("12" + 0.456);
seconds = Double.parseDouble("12" + "0.456");
seconds = Double.parseDouble("120.456");
seconds = 120.456.

Why not just do:

seconds = seconds + Double.parseDouble("0." + arr[1]);
酒绊 2024-11-15 02:54:56

我认为您正在连接两个字符串“47”和“0.186...”。

I think you are concatenating two string "47" and "0.186...".

海螺姑娘 2024-11-15 02:54:56

问题是这一行:

seconds = Double.parseDouble(Long.toString((long)seconds) + Double.parseDouble("0." + arr[1]));

第二个 parseDouble 返回“0.18685692”。由于 arr[1] 已经是小数点右侧的字符串,只需使用:

seconds = Double.parseDouble(Long.toString((long)seconds) + "." + arr[1]);

The problem is this line:

seconds = Double.parseDouble(Long.toString((long)seconds) + Double.parseDouble("0." + arr[1]));

The second parseDouble is returning "0.18685692". Since arr[1] is already the string to the right of the decimal point, just use:

seconds = Double.parseDouble(Long.toString((long)seconds) + "." + arr[1]);
旧话新听 2024-11-15 02:54:56

连接字符串时会添加一个额外的零。为了避免这种问题,最好使用格式化程序来创建字符串。请参阅此处的详细信息 http://download.oracle。 com/javase/1.5.0/docs/api/java/util/Formatter.html

此外,手动计算正确答案总是有帮助的。这使得问题更容易找到。

You are adding an extra zero when you concatenate the strings. To avoid this kind of problem it would be better to use a formatter to create you strings. See details here http://download.oracle.com/javase/1.5.0/docs/api/java/util/Formatter.html.

Also, it always helps to calculate the correct answer by hand. This made the problem much easier to find.

~没有更多了~
我们使用 Cookies 和其他技术来定制您的体验包括您的登录状态等。通过阅读我们的 隐私政策 了解更多相关信息。 单击 接受 或继续使用网站,即表示您同意使用 Cookies 和您的相关数据。
原文