Prośba o sprawdzenie kodu

Moje starcie z c++ ciąg dalszy. Program ma za zadanie dodać do podanej liczby całkowitej 2 i wypisać wynik na ekranie .Należy zastosować własną funkcję. Napisałem coś takiego:

#include <iostream>
using namespace std;
int liczba=0;
int wartosc=0;
int dodawanie(int liczba);
int main ()
{
	cout<<"podaj liczbe\n";
	cin>>liczba;
	dodawanie(liczba);
	cout<<wartosc;

return 0;
}
int dodawanie (int liczba)
{
	wartosc=liczba+2;
}

Program kompiluje się w dev++ i wykonuje działanie stawiane przed nim.
Czy jest to dobre rozwiązanie, czy istnieje prostszy sposób rozwiązania problemu. Jeśli jak to jaki?
Z góry dziękuję za każdą konstruktywną krytykę. Pozdrawiam Krzysztof :)

to teraz zastanów się co oznacza int na początku deklaracji tej funkcji

funkcja wołana by zmieniać wartość globalną. Straszne

Zgadzam się z @Miang pewnie od ostrzeżeń to masz żółto

Lepszy sposób z pewnością istnieje.
Trzecią linię teleportuj przed obecną dziewiątą, a czwartą przed obecną siedemnastą, natomiast jedenastą - za obecną siedemnastą. Aby nie komplikować niepotrzebnie sprawy, w prototypie funkcji i w jej deklaracji zdematerializuj int, materializując tam void. Naturalnie chodzi o typ zwracany przez funkcję, a nie o typ przyjmowanego argumentu.
Wtedy chyba będzie nieco lepiej. A na pewno bezpieczniej. Nie wiem tylko czy prościej.

Przenalizuj taki kod:

#include <iostream>

using namespace std;

int dodawanie(int liczba);

int main()
{
    int liczba;
    cout << "podaj liczbe\n";
    cin >> liczba;
    auto wartosc = dodawanie(liczba);
    cout << wartosc << '\n';

}
int dodawanie(int liczba)
{
    return liczba + 2;
}