c++ problem

if(test1 == "ett" || test1 == "två")

Du har glömt att kolla i om test1 är lika med "två"

 if (test1 == "ett" || test1 == "två")

Du borde ej ha test1 som en global variabel.

Håller med. Det är väldigt lätt hänt att man skriver felaktig kod när man använder globala variabler. De är bra att använda när man t.ex. vill göra konstanter. Fast då brukar man skriva const framför så att de inte går att ändra variabelns värde på ett lätt sätt. I fallet med din kod så är det absolut inte ens nödvändigt. Där är det bättre att skapa variabeln i scopet där std::cin anropas. d.v.s. i själva funktionen.

När jag vill göra lokala konstanter i en .cpp fil så brukar jag använda mig av annonymt namespace så slipper man deklarera konstanten static.

namespace
{
const double g_Pi = 3.14;
}

Så din kod skulle kunna skrivas såhär:

#include<iostream>

void test()
{
    std::string test1;
    std::cin >> test1;
    if ((test1 == "ett") || (test1 == "två"))
    {
        std::cout << "blev 1";
    }
    else
    {
        std::cout << "hahaah";
    }
}

int main()
{
    test();
    return 0;
}