[SOLVED]Having lots of trouble with sf::Clock and sf::Time

[The Terminator]

I'm trying to measure a certain amount of time to be used by a recoil method with doesn't allow a gun to shoot again until the time is past. When I try to do this, it prints out odd numbers to the console and doesn't work at all.

bool Gun::recoil(float time)
{
    sf::Clock recoilTimeClock;
    sf::Time Timer;

    Timer = recoilTimeClock.getElapsedTime();

    std::cout << Timer.asSeconds() << std::endl;

    recoilTimeClock.restart();
}

When I do this, it prints out extremely weird numbers that don't correlate with the program in any way. A picture is attached below.

[attachment deleted by admin]

[Nexus]

The numbers are not weird, and they correlate perfectly with the program.

They are that small because you create a clock and immediately measure the elapsed time. recoilTimeClock should be a member variable.

[eXpl0it3r]

5e-006 is the scientific way of writing numbers, 5 times 10 with exponent -6 or 5*10^(-6) or 0.000005.
So everything is fine and works as it should. You could use

std::cout << std::fixed << Timer.sasSeconds() << std::endl;

to get the decimal representation.

As for your gun I'm not sure what exactly you're trying to do.

[The Terminator]

I'm extremely confused on how to measure this. What I pretty much want is if the user shoots, it triggers the method recoil which will not allow the user to shoot in a certain amount of time. Here's the code I have so far, but it doesn't work.

void Gun::shoot()
{
    if (currentBullets > 0)
    {
        if (recoil(recoilTime))
        {
            std::cout << "Reloading..." << std::endl;
            std::cout << std::fixed << recoilTime << std::endl;
            std::cout << recoil(recoilTime) << std::endl;
        }

        if (!recoil(recoilTime))
        {
            switch (gunType)
            {
                case UNKNOWN:
                    std::cout << "GunType = Unknown" << std::endl;

                    break;

                case PISTOL:
                    std::cout << "Shooting Weapon: Pistol" << std::endl;
                    currentBullets--;
                    std::cout << currentBullets << " bullets left!" << std::endl;

                    break;
            }
        }
    }

    else
    {
        std::cout << "Out of bullets!" << std::endl;
    }
}

bool Gun::recoil(float time)
{
    if (recoilTimeClock.restart().asSeconds() < time)
        return true;
    else
        return false;
}

[cire]

recoil is resetting the time elapsed every time you call it.  That's probably not the behavior you want.

[ichineko]

http://www.sfml-dev.org/documentation/2.0/classsf_1_1Clock.php#a123e2627f2943e5ecaa1db0c7df3231b

restart does 2 things.  It resets the internal counter to 0, and then returns the amount of time that has elapsed since your game started, which means your check is always going to return true until the total time that your game has been open exceeds your time variable (which I assume doesn't change), and then it will always return false.;

Look into:
http://www.sfml-dev.org/documentation/2.0/classsf_1_1Clock.php#a799feb6acb099b57b58d8d20984fce11

What you want to do, is when you shoot successfully, restart the clock, and then in your recoil function, check the elapsed time.

clock.getElapsedTime().AsSeconds() < time

[The Terminator]

http://www.sfml-dev.org/documentation/2.0/classsf_1_1Clock.php#a123e2627f2943e5ecaa1db0c7df3231b

restart does 2 things.  It resets the internal counter to 0, and then returns the amount of time that has elapsed since your game started, which means your check is always going to return true until the total time that your game has been open exceeds your time variable (which I assume doesn't change), and then it will always return false.;

Look into:
http://www.sfml-dev.org/documentation/2.0/classsf_1_1Clock.php#a799feb6acb099b57b58d8d20984fce11

What you want to do, is when you shoot successfully, restart the clock, and then in your recoil function, check the elapsed time.

clock.getElapsedTime().AsSeconds() < time

THANK YOU SO MUCH! I got it working perfectly and it's exactly what I want. Thanks once again!!