#define XTFREQ          7372800         //On-board Crystal frequency
#define PLLMODE         16             	//On-chip PLL setting
#define FCY             XTFREQ*PLLMODE/4        //Instruction Cycle Frequency
#define CYC		1000000000/FCY

double ergebnis= CYC;

kann mir das jemand erkären?

1000000000 / 7372800 * 16 / 4 = ?

Tipp: Klammersetzung und so

define quad(x) x*x

:p

Da du auch mit speziellen Geräten arbeitest, würde mich mal der Wertebereich des int auf dieser Plattform interessieren

cu André

#define CYC    1000000000/(FCY)

schreibe, gehts.

Genau den gleichen Streit habe ich auch immer mit meinem Taschenrechner..

#define FCY             XTFREQ*PLLMODE/4

, dann setzt der Compiler für FCY an den Zielcode auch nur das ein, und setzt nicht automatisch Klammern.

#define CYC        1000000000/FCY   // so hast du CYC definiert

// CYC hat den Wert 
1000000000/XTFREQ*PLLMODE/4    // keine Klammern!

Ich bin mir allerdings nicht ganz sicher, eventuell wird für CYC auch der bereits ausgerechnete Wert eingesetzt. In diesem Falle wird an der bereits erwähnten Integer-Division liegen, die du aber sowieso beseitigen musst, wenn du genaue Werte willst.

Noch was: Treffende Titel sind sehr nützlich; viele Leute, die hier posten haben wohl etwas, das sie nicht verstehen.

//---------------------------------------------------------------------------

#pragma hdrstop
#include <iostream.h>
//---------------------------------------------------------------------------

#pragma argsused
int main()
{

double zahl1, zahl2, ergebnis;
char coperator, auswahl='j';

while (auswahl=='j' || auswahl=='J')
{

cout<<"--------Taschenrechner-------\n";
cout<<"-----------------------------\n";

cout<<"zahl eingeben \n";
cin>>zahl1;

cout<<"operator eingeben \n";
cin>>coperator;

cout<<"zahl eingeben \n";
cin>>zahl2;

//--Vergleich der Operatoren mit Rechnung---------
while (coperator !='/' || coperator != '*' || coperator != '+'||coperator !='-')
{

switch(coperator)
{

case '': ergebnis=zahl1zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl;

break;

case '/': ergebnis=zahl1/zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl;

break;

case '+': ergebnis=zahl1+zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl;

break;

case '-': ergebnis=zahl1-zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl;

break;

default: cout<<"neues rechenzeichen eingeben\n";cin>>coperator;

}
}
cout<<"nochmal";
cin>>auswahl;

system ("Pause");
}

getchar();
return 0;
}

das problem ist das er prüfen soll ob wirklich ein richtiges rechenzeichen eingegeben wird

Nein, das Problem ist Klammersetzung. Wenn Du ein eigenes Problem haben möchtest, mach doch bitte Deinen eigenen Thread auf, danke.

???

//--Vergleich der Operatoren mit Rechnung---------
while (coperator !='/' || coperator != '*' || coperator != '+'||coperator !='-')

???

Da fehlt aber noch ein wenig was oder?
Lese mal bitte ein Buch über C++, und es gibt auch Leerzeichen, das ist ja furchtbar.

programmnoob schrieb:

//--------------------------------------------------------------------------- 

#pragma hdrstop 
#include <iostream.h>  // iostream.h ist veraltet. Benutze iostream
//--------------------------------------------------------------------------- 

#pragma argsused // bist du sicher, dass du diese Pragmas brauchst? Kennst du deren Auswirkungen überhaupt?
int main() 
{ 

double zahl1, zahl2, ergebnis; 
char coperator, auswahl='j'; 

while (auswahl=='j' || auswahl=='J') // sag mir einen Fall, in dem auswahl != 'j' ist.
{ 

cout<<"--------Taschenrechner-------\n"; 
cout<<"-----------------------------\n"; 

cout<<"zahl eingeben \n"; 
cin>>zahl1; 

cout<<"operator eingeben \n"; 
cin>>coperator; 

cout<<"zahl eingeben \n"; 
cin>>zahl2; 

//--Vergleich der Operatoren mit Rechnung--------- 
// Die while-Schleife wird so lange ausgeführt, wie coperator keines der vier Zeichen ist.
while (coperator !='/' || coperator != '*' || coperator != '+'||coperator !='-') { 

switch(coperator) 
{ 

case '*': ergebnis=zahl1*zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl; 

break; 

case '/': ergebnis=zahl1/zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl; 

break; 

case '+': ergebnis=zahl1+zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl; 

break; 

case '-': ergebnis=zahl1-zahl2; cout<<zahl1<<" "<<coperator<<" "<<zahl2<<" = " << ergebnis<<endl<<endl; 

break; 

default: cout<<"neues rechenzeichen eingeben\n";cin>>coperator; 

} 
} 
cout<<"nochmal"; 
cin>>auswahl; 

system ("Pause"); 
} 

getchar(); // was soll den das? Du hast ja bereits system("Pause").
return 0; // überflüssig, du befindest dich am Ende von main()
}

Zum schlechten Stil noch: Benutze in Zukunft [ cpp ]-Tags. Die Strukturierung deines Codes ist zudem grauenhaft (Einrückung, Zeilenabstände, Blöcke, etc.). Die cout-Anweisungen im switch-Block könntest du zu einer zusammenfassen. Benutze using namespace std; oder schreibe std:: vor alle cout und cin, sonst wird das nicht gehen.

BTW: Dein Name spricht für sich. Es bringt wohl am meisten, wenn du zuerst ordentlich C++ lernst und dann noch einmal neu mit deinem Programm beginnst. So kommst du nämlich nicht weit.