Forum C++ - ItHelpDesk

[Soeb]

Przeczytaj co to jest BBCode i co to jest ortografia.

[Armon]

nie przesadzaj z tą ortografią...

chodzi oto, że powinieneś zamieszczać kod pomiędzy znacznikami:

[ code ] [ /code]

aby był prawidłowo sformatowany i innym się łatwo go czytało, wyjaśnij też trochę lepiej swój problem, bo pytasz się czemu nie działa... jak dla mnie wszystko działa, powiedz raczej CO chcesz osiągnąć.

Pozdrawiam.

[Soeb]

Dobra Armon może trochę przesadziłem, ale nad ortografią można byłoby popracować.

[f@c3m4n]

No wiec witam!!

To mój pierwszy post tutaj i od razu pytanie

Otóż poniżej podałem kod mojego pierwszego samodzielnie stworzonego programu i chciałbym żebyście mi powiedzieli, jak można maksymalnie skrócić kod tego programu?

Bo chciałbym już od początku umieć pisać dobrze ale używając jak najmniej kodu (żeby sie przy większej jego ilości się nie pogubić)

A to kod:

Kod: Zaznacz cały

#include <iostream>

using namespace std;

int main()
{
long double a;
long double b;
short int opcje;
char wybor='n';   //zmienna, która będzie decydować czy powtórzyć program czy nie

   cout<<"                             OBLICZATOR  v1.0 :-)                            \n";
   cout<<"                          ------------------------ \n";
   cout<< "      Obliczator jest programem, ktory umozliwia obliczanie wybranych\n";
   cout<< "         rownan takich jak np. odejmowanie, obwod prostokata, itptd.";
   {
   cout<<"\n\nENTER aby rozpoczac...";
   getchar();
   for (wybor;wybor=='n' || wybor=='N';)
   {
   cout<<"\n\nDobra to zaczynamy!!!\n";
   cout <<"Teraz wpisz dwie liczby (dodatnie).\n\n\n";
   cout <<"liczba pierwsza (a): ";
         cin>>a;               //wprowadzanie liczby a
         cin.ignore();
         for (a;a<=0;)
         {
           cout<<"\nWpisales liczbe rowna 0 lub ujemna!!!\n";
           cout<<"Wpisz ja jeszcze raz: ";
           cin >>a;
           cin.ignore();
         }
         cout <<"liczba druga (b): ";
         cin >>b;              //wprowadzanie liczby b
         cin.ignore();
         for (b;b<=0;)
         {
           cout<<"Wpisales liczbe rowna 0 lub ujemna!!!\n";
           cout<<"Wpisz ja jeszcze raz: ";
           cin >>b;
           cin.ignore();
         }
   cout<<"\n\n-------------------------------\n";
   cout<<"1.Dodawanie dwoch liczb.       |\n";
   cout<<"2.Odejmowanie dwoch liczb.     |\n";
   cout<<"3.Dzielenie dwoch liczb.       |\n";
   cout<<"4.Mnozenie dwoch liczb.        |       <---------tu sa podane rownania,\n";
   cout<<"                               |          by wybrac rownanie, wpisz jego numer\n";
   cout<<"5.Obliczanie pola kwadratu.    |\n";
   cout<<"6.Obliczanie pola prostokata.  |\n";
   cout<<"7.Obliczanie obwodu kwadratu.  |\n";
   cout<<"8.Obliczanie obwodu prostokata.|\n";
   cout<<"9.WYJSCIE.                     |\n";
   cout<<"-------------------------------\n";
   cout<<"\n\n\n\n\n\n\n";
   cout<<"Twoj wybor to: ";
   cin>>opcje;
   cin.ignore();

switch (opcje)
   {
         case 1: cout <<"\n\nDzialanie: "<<a<<" + "<<b<<" = "<<a+b;
          cout <<"\n\nWynik to: "<<a+b; break;
         case 2: cout <<"\n\nDzialanie: "<<a<<" - "<<b<<" = "<<a-b;
         cout <<"\n\nWynik to: "<<a-b; break;
         case 3: cout <<"\n\nDzialanie: "<<a<<" : "<<b<<" = "<<a/b;
         cout <<"\n\nWynik to: "<<a/b; break;
         case 4: cout <<"\n\nDzialanie: "<<a<<" * "<<b<<" = "<<a*b;
         cout <<"\n\nWynik to: "<<a*b; break;
         case 5: cout <<"\n\nPole kwadratu a * a  wiec:    "<<a<<" * "<<a<<" = "<<a*a;
         cout <<"\n\nWynik to: "<<a*a; break;
         case 6: cout <<"\n\nPole prostokata a * b  wiec:    "<<a<<" * "<<b<<" = "<<a*b;
         cout <<"\n\nWynik to: "<<a*b; break;
         case 7: cout <<"\n\nObwod prostokata 2 * a + 2 * b  wiec:    "<<a<<"*"<<a<<" + "<<b<<"*"<<b<<"="<<(a*a)+(b*b);
         cout <<"\n\nWynik to: "<<(a*a)+(b*b); break;
         case 8: cout <<"\n\nObwod kwadratu 4 * a  wiec:    "<<4<<" * "<<a<<" = "<<4*a;
         cout <<"\n\nWynik to: "<<4*a; break;
         case 9: cout <<"\n\n    Dziekuje za uzycie mojego programu...;-)";
   }

    cout<<"\n\nJesli chcesz zakonczyc program wpisz 'T',\n";
    cout<<"a jesli chcesz ponownie dokonac obliczenia wpisz 'N'.\n";
    cout<<"\n\n\nChcesz zakonczyc program?: ";
    cin>>wybor;

if (wybor=='T' && wybor=='t')
return 0;

else if (wybor!='n' && wybor!='N')
{
cout<<"Nieznana opcja!!!\n";
cout<<"ENTER aby zakonczyc...";
getchar();
return 0;
}
    }
   }
}



[M(i)odek]

Witam serdecznie! Postaram się Panu pomóc w realizacji tego zadania, ale zanim to nastąpi muszę mieć odpowiedzi na pewne pytania. Przede wszystkim na moim komputerze w momencie próby liczenia program przypisuje swoje własne liczby. Jest to błędem oczywiście. Czy Pan testował w ogóle u siebie tę aplikację? Czy działała, jak należy? Myślę, że błędem jest nieprzypisanie wartości początkowych zmiennym w parametrach pętli. Program pomija zupełnie podane przez użytkownika liczby i dalej przypisuje swoje własne. Proszę mi powiedzieć, czy u Pana występuje podobne zjawisko i czy mogę przebudować nieco aplikację? Pozdrawiam, M(i)odek

[f@c3m4n]

Hmmm to dziwne u mnie wszystko jest w porządku, najpierw wpisuję te dwie liczby potem wybieram działanie i wszystko jest OK.

Używam C++ Buildera ale i w Dev c++ tez działa jak należy. Przebuduj ten kod jak to coś da, przynajmniej zobacze co byc może zrobiłem źle.

PS. Aha i nie mów do mnie "Pan" bo mam dopiero 14 lat

[Armon]

Jeszcze nad estetyką wypada trochę popracować żeby się milej czytało Warto poznać funkcję, które mogą skrócić program...