cin User Eingabe (Anfänger Frage) c++

_matze

Nexus schrieb:

Die Schreibweise if (Literal == Variable) finde ich unschön und unlogischer, da man eigentlich den Wert der Variable prüft und nicht den Wert des Literals (das ist zumindest meine Assoziation bei einem Vergleich).

Da gebe ich dir schon Recht, aber die umgekehrte Schreibweise hat noch den Vorteil, dass beim Vergleich mit sehr langen Ausdrücken das Literal direkt sichtbar ist, ohne Scrolling oder sonstwas. Daher nutze ich es vereinzelt, wenn es der Lesbarkeit dient.

if(IDYES==MessageBox( /* absurd lange Parameterliste ;) */ )

Nexus

cooler78 schrieb:

Könntest du mir bitte ein Beispiel geben wie man es (deiner Meinung nach)
besser schreiben koennte?^^

Naja, ich würde halt einfach

if (Mantel == 'm')

schreiben. Aber dann riskierst du eben, dass ein Vertipper trotzdem kompiliert. Was aber ohnehin nur etwas bringt, wenn skalare RValues (oder const -Variablen) im Spiel sind.

Bei

if (Mantel == meinDritterMantel)

hast du wieder dasselbe Problem und kannst es nicht durch Vertauschen lösen. Aber ich finde es inkonsistent, hier den "abgefragten Wert" links und den "Vergleichswert" rechts zu schreiben, in Spezialfällen wegen der =-Gefahr jedoch umgekehrt. Aber das hat auch viel mit Geschmack zu tun. Mit der Zeit wirst du dir ohnehin deinen eigenen Stil bilden und merken, was für dich besser passt.

_matze schrieb:

Da gebe ich dir schon Recht, aber die umgekehrte Schreibweise hat noch den Vorteil, dass beim Vergleich mit sehr langen Ausdrücken das Literal direkt sichtbar ist, ohne Scrolling oder sonstwas. Daher nutze ich es vereinzelt, wenn es der Lesbarkeit dient.

Guter Punkt, daran habe ich jetzt gar nicht gedacht. Allerdings habe ich das Gefühl, bei mir haben solche längeren Funktionsaufrufe (oder arithmetische Ausdrücke) entweder oft bool -Rückgabetypen, oder es steht auf beiden Seiten des Vergleichsoperators etwas Langes. Zumindest ist es mir bisher noch nie sehr unübersichtlich vorgekommen...

cooler78

So noch ne Frage ich lese zur Zeit mein erstes c++ Buch es ist das Buch:

Einstieg in c++ von Arnold Willemer (vllt. kennt es ja jemand)

und ich wollte euch fragen welches Buch ihr empfehlen koennt
und ob es evtl. auch ein Buch gibt das auf den erworbenen Kentnissen des erwähnten Buches aufbaut.

und danke fuer die rege Beteiligung hier ^^

drakon

Hier:
http://www.c-plusplus.net/forum/viewtopic-var-t-is-251551.html

werden viele gute Bücher gennant (vom User "Mitleid").

Da hat es gute Bücher für Fortgeschrittene.

cooler78

wie geht der syntax für

if (cin=="y")
{

.....

}

Also wie geht der Code fuer eine Abfrage ob cin irgendeinen Buchstabe ist

ipsec

cin ist nie ein Buchstabe, es ist ein Stream-Objekt, mit dem von der Standardeingabe gelesen werden kann. Was du sicher willst, ist:

char c;
cin >> c;
if(c == 'y')
{
//...
}

Beachte dabei, dass Buchstabenliterale mit einfachen Anführungszeichen geschrieben werden, im Gegensatz zu den doppelten Anführungszeichen bei Strings.

cooler78

DANKE DANKE
Genau das habe ich gebraucht^^

cooler78

hat sich geklaert
zu dumm zum LEsen

cooler78

Tschuldigun fuer die dummen Fragen
aber warum geht das nicht

double VolumenKugel=float(4/3)*3,14159265*radius^3;

cooler78

Bitte muss dringend weiterkommen^^

unskilled

2 fehler:

double VolumenKugel=float(4/3)*3,14159265*radius^3;
//                              ^               ^

1. 3.14159265
2. radius*radius*radius

der ^-operator macht nicht das, was du dir erhoffst sondern ist ein XOR.

bb

cooler78

Danke an euch alle das ihr mir so hilft
das mit ^3 hab ich mir schon gedacht und hatte es auch umgeschrieben
aber das der Compiler 3,.. anders ließt als 3. .. (was ja eigtl.logisch ist)
darauf bin ich nicht gekommen ^^

ipsec

außerdem wird float(4/3) noch Probleme bereiten, korrekter wäre float(4)/3. Wobei in deinem Fall aber double günstiger wäre, da das Ergebnis auch double ist.
Und weil du double-Literale auch in den Quelltext direkt schreiben kannst, ist 4.0/3 am günstigsten.

cooler78

#include <cstdlib>
#include <iostream>
#include <cmath>
#include <windows.h>
using namespace std;

int main(int argc, char *argv[])
{
//Formeln

//V=G*H

 //Bei Pyramide , Kegel etc (Volumen)

//V=1/3*G*H   
//O=G+M 

//Bei Kugel (Volumen, Oberflaeche)

//4/3 * PI*r3

//o=4*PI*r^2

//Pi= 3,14159265

    //char und int Variabeln

                          //Beliebige Eingabe
    char iwas;
    char iwas2;
//--------------------------------------------------------------------------------- 

                       //Variabeln(chars floats doubles)   
    char Kugel;
    char Pyramide;
    char Kegel;
    char Zylinder;
    char Volumen;
    float VolumenKegel;
    float KegelOberflaeche;
    float PyramideOberflaeche;
    float VolumenKugel;
    float OberflaecheKugel;
    float VolumenPyramide;
    double Hoehe;
    double grundflaeche;
    double radius;
    double Mantel;
    //Headder Strings

string einfuehrung="Dies ist ein Programm zur Berecnung von Volumen , Oberflaeche , Mantel etc.. von einem beliebigen Objekt (Kugel , Pyramide ..)";
string gemachtvon="Diese Programm wurde von ***** erstellt";
string linie="--------------------------";    
string Eingabe1="Geben sie das Objekt an , dass sie berechnen wollen (z.b Kugel)";
string Eingabe2="Geben sie nun ein was sie berechnen wollen (z.b Volumen)";  
string EingabeVolumen="Geben sie nun die Grundflaeche des Objekts an ";
string EingabeVolumen2="Geben sie nun die Hoehe des Objekts an";
string kugel= "Wenn das Objekt eine Kugel ist geben sie /m ein";
string pyramide="Wenn das Objekt eine Pyramide ist geben sie /p ein";
string kegel= "Wenn das Objekt ein Kegel ist geben sie /k ein";
string zylinder="Wenn das Objekt ein Zylinder ist geben sie /z ein"; 

  cout<<gemachtvon<<endl<<linie<<endl<<einfuehrung<<endl<<linie<<endl; 

 cout<<kugel<<endl<<pyramide<<endl<<kegel<<endl<<zylinder<<endl<<endl;   

  cout<<Eingabe1<<endl;   
  cin>>iwas;

if (iwas =='m')
{
    cout<<endl<<endl;          
    cout<<"Sie haben sich fuer eine Kugel entschieden"<<endl<<endl<<"geben sie nun ein ob sie Volumen oder Oberflaeche etc... berechnen moechten"<<endl<<endl;          
    cout<<"Fuer Volumen /v fuer die Oberflaeche /o ..etc"<<endl;
    cin>>iwas2;

    if (iwas2=='v')
    {

        cout<<"Geben sie den Radius des Objekts ein"<<endl;
        cin>>radius;

        cout<<"Berechne das Volumen!!"<<endl; 
        //Berechnung
        float  VolumenKugel=float(4/3) * 3.14159265 * radius*radius*radius;
        Sleep(2000); 
        cout<<"Das Volumen der Kugel betraegt:"<<endl<<VolumenKugel<<"kubik(hier MASSEINHEIT)"<<endl<<endl;

     }          

              if(iwas2=='o')
              {

                 cout<<"Geben sie die Grundflaeche ein"<<endl;
                 cin>>grundflaeche;                             
                 cout<<"Geben sie die Hoehe ein "<<endl;
                 cin>>Hoehe;    
                 //Berechnung     
                 float OberflaecheKugel=float(4)*3.14159265*radius*radius;
                 Sleep(2000);
                 cout<<"Die Oberflaeche der Kugel betraegt:"<<endl<<OberflaecheKugel<<" quadrat(hier MASSEINHEIT)"<<endl<<endl;
                 cin.clear();         

                 }           

                 }

        if (iwas='p')    

    {
    cout<<endl<<endl;          
    cout<<"Sie haben sich fuer eine Pyramide entschieden"<<endl<<endl<<"geben sie nun ein ob sie Volumen oder Oberflaeche etc... berechnen moechten"<<endl<<endl;          
    cout<<"Fuer Volumen /v fuer die Oberflaeche /o ..etc"<<endl;
    cin>>iwas2;

    if (iwas2=='v')
    {
         cout<<"Geben sie die  Grundlaeche  des Objekts ein"<<endl;
        cin>>grundflaeche;
        cout<<"Geben sie die Hoehe des Objekts ein"<<endl;
        cin>>Hoehe;
        cout<<"Berechne das Volumen!!"<<endl; 
        //Berechnung
        float  VolumenPyramide=float(1/3) * grundflaeche * Hoehe ;
        Sleep(2000); 
        cout<<"Das Volumen der Pyramide betraegt:"<<endl<<VolumenPyramide<<"    kubik(hier MASSEINHEIT)"<<endl<<endl;

      }    

     if (iwas2=='o')
     {

      cout<<"Geben sie nun (evtl. nochmal) die Grundlflaeche  an"<<endl;
      cin>>grundflaeche;
      cout<<"Geben sie nun (evtl nochmal ) den Mantel ein"<<endl;
      cin>>Mantel;
      cout<<"Berechne die Oberflache!!"<<endl;
      //Berechnung            
      PyramideOberflaeche = float(grundflaeche)+Mantel;             
      cout<<"Die Oberflaeche der Pyramide betraegt:"<<endl<<PyramideOberflaeche<<"    quadrat(hier MASSEINHEiT)"<<endl<<endl;               
      cin.clear();              

     }               

    }                

        if (iwas='k')    

    {
    cout<<endl<<endl;          
    cout<<"Sie haben sich fuer einen Kegel entschieden"<<endl<<endl<<"geben sie nun ein ob sie Volumen oder Oberflaeche etc... berechnen moechten"<<endl<<endl;          
    cout<<"Fuer Volumen /v fuer die Oberflaeche /o ..etc"<<endl;
    cin>>iwas2;

    if (iwas2=='v')
    {
         cout<<"Geben sie die  Grundlaeche  des Objekts ein"<<endl;
        cin>>grundflaeche;
        cout<<"Geben sie die Hoehe des Objekts ein"<<endl;
        cin>>Hoehe;
        cout<<"Berechne das Volumen!!"<<endl; 
        //Berechnung
        float  VolumenKegel=float(1/3) * grundflaeche * Hoehe ;
        Sleep(2000); 
        cout<<"Das Volumen des Kegels betraegt:"<<endl<<VolumenKegel<<"    kubik(hier MASSEINHEIT)"<<endl<<endl;

      }    

     if (iwas2=='o')
     {

      cout<<"Geben sie nun (evtl. nochmal) die Grundlflaeche  an"<<endl;
      cin>>grundflaeche;
      cout<<"Geben sie nun (evtl nochmal ) den Mantel ein"<<endl;
      cin>>Mantel;
      cout<<"Berechne die Oberflache!!"<<endl;
      //Berechnung            
      KegelOberflaeche= float(grundflaeche)+Mantel;             
      cout<<"Die Oberflaeche des Kegels betraegt:"<<endl<< KegelOberflaeche<<"    quadrat(hier MASSEINHEiT)"<<endl<<endl;               

     }               

     }                   

    system("PAUSE");
    return EXIT_SUCCESS;
}

Ein bisschen komisch ich weiß ich weiß (vllt. ne Erklaerung zu den ganzen Strings Mein Gedanke war das ich alles der Übersichtlichkeits halber in strings schreibe habe das aber nicht bis zum ende durchgehalten..

Nun mal wieder eine dumme Frage:

Wenn ich jetzt m eingebe kommt die Kugel alles gut
Dann erscheint aber sofort , dass ich mich fuer eine Pyramide entschieden habe =????

Achja wenn ihr noch Verbesserungsvorschläge habt postest sie alle^^.

Wie gesagt ich "programmiere" erst seit 1-2 Wochen insofern...

Immer diese Operatoren mir ist aufgefallen dass ich wieder = anstatt == geschrieben habe trotzdem danke ich euch allen !

cooler78

Kann mir dann noch iwer die korrekte Benutzung von float Variablen erklaeren
ich dachte , dass man diese immer in Berrechnungen verwenden muss?

Nexus

Wenn du konkrete Fragen und einen minimalen Code dazu posten kannst, gerne. Aber was float s sind, liest du vielleicht besser in deinem C++-Buch nach...

P.S.: Natürlich muss man float nicht in Berechnungen verwenden.

Blackskyliner

Ist es nicht sogar eigentlich bessern ebend _keine_ floats aufgrund deren Ungenauigkeit zu verwenden? von wegen, 1.0 + 1.0 = 1.99999999999->
Je nachdem wie genau da grade das Rechenwerk ist :>

cooler78

Ok danke aber hier

float  VolumenPyramide=float(1/3) * grundflaeche * Hoehe ;

Hab ich doch theoretisch alles richitg gemacht 1/3 = 0.333.. ergo Fließkommazahl
ergo float aber warum komm da immer 0 raus egal was ich fuer grundlaeche und Hoehe eingebe?

Hab ich doch theoretisch alles richitg gemacht 1/3 = 0.333.. ergo Fließkommazahl

Eben nicht.

float(1/3) heißt für den Kompiler:
Werte die Integerdivision (!) 1/3 aus und konvertiere das Ergebnis in ein float. D.h. float(1/3) == 0.0

Richtig wäre: float(1)/3, oder einfacher 1.f/3 oder 1.0/3

....

unskilled

weil du die beiträge über dir nicht gelesen hast...
1/3 ist nicht 0.3333333 oder 0.33 oder was auch immer.
es ist eben 0
1./3 ist 0.333333
oder 1.0/3 oder 1/3. oder 1/3.0 oder 1.0/3.0 oder float(1)/3 oder double(1)/3 usw.

bb