Nybörjare i c++

Orkar inte kolla igenom särskilt noga nu, men du använder dig av "forkotta" istället för "forkorta" inne i while-loopen.

Men vad blir fel då? Går det inte att kompilera? Är din algoritm fel?

while(namnare%forkorta !=0 && taljare%forkorta !=0)

ska vara

while(namnare%forkorta !=0 || taljare%forkorta !=0)

Det behöver ju inte vara en rest på båda för att det inte ska gå att förkorta med det talet.

Edit: >_> ok den kommer ju alltid bli delbar med 1. nvm
Förutom detta så kommer svaret bli skumt om det inte finns ett lämpligt tal att förkorta med. Loopen borde avslutas om forkorta är 1. (vill ju inte gärna dela med 0 heller)

Grejen är att din while loop inte kommer att köras så många gånger du vill, eftersom du sätter din variabel forkorta till taljare så kommer du alltid få 0 där och då kommer din loop inte att köras längre.

Jag skulle istället använda mig av en for-loop för att lösa det där.

#include <iostream>
using namespace std;

int main()
{
    int taljare;
    int namnare;

    cin >> taljare;
    cin >> namnare;

    for(int forkorta = taljare; forkorta > 1; forkorta--){
        if((taljare % forkorta) == 0 && (namnare % forkorta) == 0){
            taljare = taljare/forkorta;
            namnare = namnare/forkorta;
        }
    }

    cout << taljare << endl;
    cout << namnare<< endl;


    return 0;
}

Detta borde lösa problemet fint.

EDIT:
Jag måste sluta vara så snabb och inte kopiera all kod

TS lösning är ju helt klart likvärdig (om inte bättre), den innehåller bara ett par småfel. Din lösning kommer att fortsätta även efter att talen delats med sin GCD, och det är fullständigt onödigt.

Ja om man ska börja räkna operationer så kommer min lösning vara snäppet sämre, orkade inte riktigt titta så noga men while-loopen var iofs lätt att fixa bara använda || istället för &&