//PrimzahlenTest
#include <iostream>
using namespace std;

int iswhole(float number);
int isprime(float in);

int main()
{
//VAR
float x;

cout << "Primzahlentester\nGeben sie die Zahl ein!" << endl;
cin >> x;

cout << isprime(x) << endl;

int zz;
cout << "Geben sie eine belibige Zahl + ENTER ein" << endl;
cin >> zz;

return 0;
}
int iswhole(float number)
{
    bool truefalse;

    while(number >= 1)
    {
        number--;
    }
    if(number==0)
    {
    truefalse = 1;
    }
    else
    {
    truefalse = 0;
    }
    return truefalse;

}
int isprime(float input)
{
bool bprime=0;
int sum;
int loop = input-1;
float test;

while(loop > 1)
 {
    test = input/loop;
    cout <<""; //<--Wenn ich diese Zeile lösche funktioniert das Programm nicht mehr :(
    sum+=iswhole(test);
    loop--;
 }
if (sum==0)
    {
    bprime = 1;
    }
return bprime;
}

//PrimzahlenTest 
#include <iostream> 
using namespace std; 

int iswhole(float number); 
int isprime(float in); 

int main() 
{ 
	//VAR 
	float x; 

	cout << "Primzahlentester\nGeben sie die Zahl ein!" << endl; 
	cin >> x; 

	cout << isprime(x) << endl; 

	int zz; 
	cout << "Geben sie eine belibige Zahl + ENTER ein" << endl; 
	cin >> zz; 
}

int iswhole(float number) 
{ 
    bool truefalse; 

    while(number >= 1) 
        number--; 

    if(number==0) 
		truefalse = 1; 

    else 
		truefalse = 0; 

    return truefalse; 

}

int isprime(float input) 
{ 
	bool bprime=0; 
	int sum; 
	int loop = input-1; 
	float test; 

	while(loop > 1) 
	{ 
		test = input/loop; 
		cout <<""; //<--Wenn ich diese Zeile lösche funktioniert das Programm nicht mehr :( 
		sum+=iswhole(test); 
		loop--; 
	} 

	if (sum==0) 
		bprime = 1; 

	return bprime; 
}

Das ist nicht möglich. Dieses Statement hat praktisch überhaupt keine Auswirkung auf dein Programm.
Gib mal eine genaue Fehlerbeschreibung. Ich versuche gleich mal, das Programm zu starten und sehe es mir an.

Ein Problem bei deinem Code ist, dass du Fließkommazahlen verwendest, obwohl du das überhaupt nicht brauchst - und dadurch alles durcheinander bringst. Wieso verwendest du also float statt unsigned ?

"Funktioniert nicht" ist keine Fehlerbeschreibung. Genaue Fehlermeldung?

Das Programm giebt anstelle von entweder 0 oder 1 (Primzahl oder Keine Primzahl)
nur 0 aus egal ob es eine Primzahl ist oder nicht.

Dein Prog bewertet 7 bei mir im Debug-Modus als Primzahl und im Release-Modus als Nicht-Primzahl.

Das deutet darauf hin, daß Du was gemacht hast, was undefiniertes Verhalten erzeugt. Auf Anhieb sehe ich, daß sum nicht initialisiert wurde. Ob's das schon war?

cout <<""; //<--Wenn ich diese Zeile lösche funktioniert das Programm nicht mehr :(

Das ist nicht möglich. Dieses Statement hat praktisch überhaupt keine Auswirkung auf dein Programm.
Gib mal eine genaue Fehlerbeschreibung. Ich versuche gleich mal, das Programm zu starten und sehe es mir an.

Ein Problem bei deinem Code ist, dass du Fließkommazahlen verwendest, obwohl du das überhaupt nicht brauchst - und dadurch alles durcheinander bringst. Wieso verwendest du also float statt unsigned ?[/quote]
Das Programm giebt (egal ob Primzahl oder nicht) nur 0 aus anselle von 0 (Keine Primzahl) oder 1 (Primzahl)
Es funktioniert aber auch nicht im Debug modus.

Sone schrieb:

Ein Problem bei deinem Code ist, dass du Fließkommazahlen verwendest, obwohl du das überhaupt nicht brauchst - und dadurch alles durcheinander bringst. Wieso verwendest du also float statt unsigned ?

Ich habe (wie bereits gesagt) gerade erst angefangen C++ zu lernen.
Ich wusste bis gerade noch nicht einmal das es etwas wie unsigned giebt.

Auf Anhieb sehe ich, daß sum nicht initialisiert wurde. Ob's das schon war?

initialisieren heiß Wert zuweißen, oder ?
Jedenfalls funktioniert das Programm jetzt, habe einfach.

int sum;

zu

int sum=0;

geändert und jetzt Funktioniert es.
Danke für helfen.

EDIT: Ich brauch ne Brille oder n groesseren Bildschirm...

bool isPrime(unsigned number)
{
    if(number < 2)
         return false;

    for(unsigned i = 2; i < number; ++i)
        if(number % i == 0)
            return false;

    return true;
}

Sone schrieb:

cout <<""; //<--Wenn ich diese Zeile lösche funktioniert das Programm nicht mehr :(

Das ist nicht möglich. Dieses Statement hat praktisch überhaupt keine Auswirkung auf dein Programm.
Gib mal eine genaue Fehlerbeschreibung. Ich versuche gleich mal, das Programm zu starten und sehe es mir an.

Ein Problem bei deinem Code ist, dass du Fließkommazahlen verwendest, obwohl du das überhaupt nicht brauchst - und dadurch alles durcheinander bringst. Wieso verwendest du also float statt unsigned ?

Das Programm giebt (egal ob Primzahl oder nicht) nur 0 aus anselle von 0 (Keine Primzahl) oder 1 (Primzahl)
Es funktioniert aber auch nicht im Debug modus.

Sone schrieb:

Ein Problem bei deinem Code ist, dass du Fließkommazahlen verwendest, obwohl du das überhaupt nicht brauchst - und dadurch alles durcheinander bringst. Wieso verwendest du also float statt unsigned ?

Ich habe (wie bereits gesagt) gerade erst angefangen C++ zu lernen.
Ich wusste bis gerade noch nicht einmal das es etwas wie unsigned giebt.[/quote]
vergiss erstmal unsigned.
denk dir sones vorschlag mit int.
"wieso verwendest du float statt int" wäre dann die frage.

Warum ein rekursiver Ansatz fuer Primzahlen?

Da haste aber geschielt, alös Du den rekursiven Ansatz im Programm erkannt hast.

1 (Primzahl)

Ich glaube, ich habe dich missverstanden.

Edit: Ja, habe ich. Hätte ich dir auch nicht zugetraut, nie nie.