Вопрос

У меня есть эта программа Java, которая вычисляет время, которое человек, который потратил, чтобы ответить на диалоговое окно подтверждения. Но кажется, что и Time1, и Time 2 получают одинаковое значение. Что я сделал здесь неправильно:

import javax.swing.JOptionPane;
    import java.util.*;

    public class DialogTimer{
        public static void main(String args[]){
            int time1, time2, milli1, milli2, sec1, sec2, timeDifference;
            final int MILLISECSINSECOND =1000;

            JOptionPane.showConfirmDialog(null, "Is stealing ever justified? ");

            GregorianCalendar before=new GregorianCalendar();
                GregorianCalendar after= new GregorianCalendar();


            milli1=before.get(GregorianCalendar.MILLISECOND);
            milli2=after.get(GregorianCalendar.MILLISECOND);


            sec1=before.get(GregorianCalendar.SECOND);
            sec2=after.get(GregorianCalendar.SECOND);

            time1=MILLISECSINSECOND * sec1 + milli1;
            time2=MILLISECSINSECOND * sec2 + milli2;


            //timeDifference=time1 - time2;
            System.out.println(time1);
            System.out.println(time2);


            //JOptionPane.showMessageDialog(null, "It took " + (time1 - time2) + " milliseconds for you to answer");


        }

    }
Это было полезно?

Решение

import javax.swing.JOptionPane;
import java.util.*;

public class DialogTimer{
    public static void main(String args[]){
        int time1, time2, milli1, milli2, sec1, sec2, timeDifference;
        final int MILLISECSINSECOND =1000;

        GregorianCalendar before=new GregorianCalendar();
        milli1=before.get(GregorianCalendar.MILLISECOND);

        JOptionPane.showConfirmDialog(null, "Is stealing ever justified? ");

        GregorianCalendar after= new GregorianCalendar();
        milli2=after.get(GregorianCalendar.MILLISECOND);


        sec1=before.get(GregorianCalendar.SECOND);
        sec2=after.get(GregorianCalendar.SECOND);

        time1=MILLISECSINSECOND * sec1 + milli1;
        time2=MILLISECSINSECOND * sec2 + milli2;


        //timeDifference=time1 - time2;
        System.out.println(time1);
        System.out.println(time2);


        //JOptionPane.showMessageDialog(null, "It took " + (time1 - time2) + " milliseconds for you to answer");


    }

}

Другие советы

Почему вы не используете непосредственно System.currentTimeMillis ()? например:

long startAt = System.currentTimeMillis();
... your code ...
long elapsed = System.currentTimeMillis() - startAt;

Codegistcrest прав. Но в любом случае в вашем случае вы получите те же или почти одинаковые значения. Современные компьютеры очень быстрые, но разрешение времени, которое вы получаете от календаря, и CurrentTimemiLlis () - миллисекунд. Компьютер вызывает всю вашу программу менее 1 мс.

В реальной жизни вам не нужно все это. Ты должен сказать:

long before = System.currentTimeMillis();

Когда вы показываете диалог и

long after = System.currentTimeMillis();

как часть действия, которое вызывается, когда пользователь нажимал кнопку. Тогда скажи

time = after - before;

Лицензировано под: CC-BY-SA с атрибуция
Не связан с StackOverflow
scroll top