SFML community forums

General => SFML projects => Topic started by: paupav on August 20, 2014, 07:00:03 pm

Title: Bouncing Thingy v1.9
Post by: paupav on August 20, 2014, 07:00:03 pm

(http://s30.postimg.org/anograni9/screenshot82747794.png)
(http://s30.postimg.org/5av0zfa81/screenshot25658741.png)

(http://s30.postimg.org/87tzzfog1/image.png)
(http://s30.postimg.org/8w2uidn5t/2_2.png)

Fullres images:
http://s27.postimg.org/4b9qd19cx/image.png
http://s9.postimg.org/6upnx7dz1/image.png

Check it out and post feedback!
Source: http://www.mediafire.com/download/pjbedqke837l2hu/BouncingThingysourcev1.9.zip
Linux x64 precompiled: http://www.mediafire.com/download/1m1w6pwpc0v1513/BouncingThingyLinux64v1.9.tar.gz

Changelog:
-Escape shortcut to exit game
-ALt+ Enter to change from fullscreen to windowed and vice versa (options will be saved)
-Main menu added
-Options menu added
-new text that shows current ball speed
-take screenshots with F12

Please comment my coding!
Is my coding stupid, messy?

(click to show/hide)

#include <SFML/Graphics.hpp>
#include <iostream>
#include <fstream>
#include <string>
#include "menu.h"

//menus are bool, in case they return true, all the windows will close
bool MainMenu ()
{

    enum menuOptions {start, options, exit, last};
    int optionSelecter = 0;
    int textSize = sf::VideoMode::getDesktopMode().height/10;
    sf::ContextSettings settings;
    settings.antialiasingLevel = 16;

    sf::Vector2i res(sf::VideoMode::getDesktopMode().width, sf::VideoMode::getDesktopMode().height);


    bool fullscreen = checkIfFullscreen();
    bool changeMode = false;
    std::cout << "Fullscreen----: " << fullscreen << std::endl;

    sf::RenderWindow window(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Fullscreen, settings);
    if(!fullscreen)
        window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Default, settings);

    window.setFramerateLimit(7);

    sf::Event event;

    bool closeVariable;

    //menu options crap
    sf::Font menuFont;
    sf::Text startText, optionsText, exitText, arrow ;

    menuFont.loadFromFile("font/menu.ttf");

    startText.setCharacterSize(textSize);
    optionsText.setCharacterSize(textSize);
    exitText.setCharacterSize(textSize);
    arrow.setCharacterSize(textSize);

    startText.setFont(menuFont);
    optionsText.setFont(menuFont);
    exitText.setFont(menuFont);
    arrow.setFont(menuFont);

    startText.setString("Start game");
    optionsText.setString("Options");
    exitText.setString("Exit");
    arrow.setString("->");

    //stupid variable just so that the code is cleaner
    int optionsDis = startText.getGlobalBounds().height;

    startText.setPosition(res.x/2 - startText.getGlobalBounds().width/2, res.y/2 - optionsDis);
    optionsText.setPosition(res.x/2 - optionsText.getGlobalBounds().width/2, res.y/2 );
    exitText.setPosition(res.x/2 - exitText.getGlobalBounds().width/2, res.y/2 + optionsDis);



    while(window.isOpen())
    {

        changeMode = changeViewMode(fullscreen);
        arrow.setPosition(res.x/2 - startText.getGlobalBounds().width/2 - 2*textSize, res.y/2 - optionsDis + optionSelecter * optionsDis);

    //controlling selector

        if(sf::Keyboard::isKeyPressed(sf::Keyboard::Down))
            optionSelecter++;
        if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up))
            optionSelecter--;


            if(changeMode == true)
            {
                fullscreen = checkIfFullscreen();
                if(fullscreen)
                    window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Fullscreen, settings);
                else if (!fullscreen)
                    window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Default, settings);
            }

        if(optionSelecter < 0)
            optionSelecter = last-1;
        if(optionSelecter >= last)
            optionSelecter = 0;


        if(optionSelecter == start && sf::Keyboard::isKeyPressed(sf::Keyboard::Return) && !sf::Keyboard::isKeyPressed(sf::Keyboard::LAlt))
            return false;
        if(optionSelecter == options && sf::Keyboard::isKeyPressed(sf::Keyboard::Return) && !sf::Keyboard::isKeyPressed(sf::Keyboard::LAlt))
        {
            window.close();
            if(closeVariable = OptionsMenu())
                return true;
            MainMenu();
        }
        if(optionSelecter == exit && sf::Keyboard::isKeyPressed(sf::Keyboard::Return) && !sf::Keyboard::isKeyPressed(sf::Keyboard::LAlt))
            return true;

        while(window.pollEvent(event))
        {
            if(sf::Keyboard::isKeyPressed(sf::Keyboard::Escape) || event.type == sf::Event::Closed)
                return true;
        }


         if(sf::Keyboard::isKeyPressed(sf::Keyboard::F12))
        {
            sf::Image screen = window.capture();
            long int randomNum = 0;
            for(int i = 0; i < 5; i++)
                    randomNum = rand() % 125000000 + 1;
            std::string fileName = "screenshot" + std::to_string(randomNum) + ".png";
            screen.saveToFile(fileName);
        }

        //draw options
        window.clear();
        window.draw(startText);
        window.draw(optionsText);
        window.draw(exitText);
        if(optionSelecter >= 0 && optionSelecter < last)
            window.draw(arrow);
        window.display();
    }//while window is open


}//main





bool OptionsMenu()
{
    enum menuOptions{viewMode, exit, last};
    int optionSelecter = 0;
    sf::ContextSettings settings;
    settings.antialiasingLevel = 16;

    bool fullscreen = checkIfFullscreen();
    bool changeMode;

    sf::Vector2i res(sf::VideoMode::getDesktopMode().width, sf::VideoMode::getDesktopMode().height);

    sf::RenderWindow window(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Fullscreen, settings);

    if(!fullscreen)
        window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Default, settings);

    window.setFramerateLimit(7);

    sf::Event event;
    sf::Font menuFont;
    sf::Text changeRes, viewModeText, exitText, arrow, playerName;
    menuFont.loadFromFile("font/menu.ttf");

    int textSize = sf::VideoMode::getDesktopMode().height/10;

    viewModeText.setCharacterSize(textSize);
    exitText.setCharacterSize(textSize);
    arrow.setCharacterSize(textSize);

    viewModeText.setFont(menuFont);
    exitText.setFont(menuFont);
    arrow.setFont(menuFont);

    viewModeText.setString("Change View Mode");
    exitText.setString("Exit");
    arrow.setString("->");

    //stupid variable just so that the code is cleaner
    int optionsDis = viewModeText.getGlobalBounds().height;

    viewModeText.setPosition(res.x/2 - viewModeText.getGlobalBounds().width/2, res.y/2 - optionsDis);
    exitText.setPosition(res.x/2 - exitText.getGlobalBounds().width/2, res.y/2 );

    while(window.isOpen())
    {
        changeMode = changeViewMode(fullscreen);
        arrow.setPosition(res.x/2 - viewModeText.getGlobalBounds().width/2 - 2*textSize, res.y/2 - optionsDis + optionSelecter * optionsDis);


        //window controls
        while(window.pollEvent(event))
        {
            if(sf::Keyboard::isKeyPressed(sf::Keyboard::Escape) || event.type == sf::Event::Closed)
                return true;
        }

        if(changeMode == true)
        {
            fullscreen = checkIfFullscreen();
            if(fullscreen)
                window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Fullscreen, settings);
            else if (!fullscreen)
                window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Default, settings);
        }


        if(optionSelecter == viewMode && sf::Keyboard::isKeyPressed(sf::Keyboard::Return) && !sf::Keyboard::isKeyPressed(sf::Keyboard::LAlt))
        {
            changeViewModeInText(fullscreen);
            fullscreen = checkIfFullscreen();
            if(fullscreen)
                window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Fullscreen, settings);
            else if (!fullscreen)
                window.create(sf::VideoMode(res.x, res.y), "Bouncing ball thingy", sf::Style::Default, settings);

        }
        if(optionSelecter == exit && sf::Keyboard::isKeyPressed(sf::Keyboard::Return) && !sf::Keyboard::isKeyPressed(sf::Keyboard::LAlt))
            window.close();

        while(window.pollEvent(event))
        {
            switch (event.type)
            {
                case sf::Event::Closed:
                    return true;
            }
        }

        if(sf::Keyboard::isKeyPressed(sf::Keyboard::F12))
        {
            sf::Image screen = window.capture();
            long int randomNum = 0;
            for(int i = 0; i < 5; i++)
                    randomNum = rand() % 125000000 + 1;
            std::string fileName = "screenshot" + std::to_string(randomNum) + ".png";
            screen.saveToFile(fileName);
        }
        //controlling selector

        if(sf::Keyboard::isKeyPressed(sf::Keyboard::Down))
            optionSelecter++;
        if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up))
            optionSelecter--;

        if(optionSelecter < 0)
            optionSelecter = last-1;
        if(optionSelecter >= last)
            optionSelecter = 0;



        window.clear();
        window.draw(viewModeText);
        window.draw(exitText);
        window.draw(arrow);
        window.display();
    }
}



bool changeViewMode(bool fullscreen)
{

    if(sf::Keyboard::isKeyPressed(sf::Keyboard::LAlt) && sf::Keyboard::isKeyPressed(sf::Keyboard::Return))
    {
            changeViewModeInText(fullscreen);
            return true;
    }
    return false;
}

void changeViewModeInText(bool fullscreen)
{
    std::ofstream videoSettings;
    videoSettings.open("settings/display.txt");

        if(fullscreen)
        {
            videoSettings << "Fullscreen = No";
            return;
        }
        if(!fullscreen)
            videoSettings << "Fullscreen = Yes";

}

bool checkIfFullscreen ()
{
    std::ifstream videoSettingsCheck;
    std::string videoSettingsString;
    videoSettingsCheck.open("settings/display.txt");
    getline(videoSettingsCheck, videoSettingsString);
    if(videoSettingsString != "Fullscreen = No" && videoSettingsString !="Fullscreen = Yes")
    {
        std::ofstream videoSettings;
        videoSettings.open("settings/display.txt");
        videoSettings << "Fullscreen = Yes";
    }
    if(videoSettingsString == "Fullscreen = Yes")
        return true;
    return false;
}

Title: Re: Bouncing Thingy v1.9
Post by: Gobbles on August 21, 2014, 07:14:33 am

Congrats on getting a working game, it's the first step of many to greatness.

As for your questions, is your code messy? sure, but is it stupid? Nah, I don't believe any functioning code is stupid, maybe less optimal, but never stupid.

Take a look into some proper design and read up on using classes and Object Oriented Programming. I believe proper design is the key to making a great game without wanting to throw your computer out the window. It'll save you hours of frustrations in the future.

best of luck! and also I like your menu text, looks awesome.

Title: Re: Bouncing Thingy v1.9
Post by: achpile on August 21, 2014, 08:19:39 am

Just an advice.

        if(sf::Keyboard::isKeyPressed(sf::Keyboard::Down))
            optionSelecter++;
        if(sf::Keyboard::isKeyPressed(sf::Keyboard::Up))
            optionSelecter--;
 

I think it's better to use 'KeyReleased' event to handle situation like this. (to avoid skipping menu entries)