Jak byscie to skrócili. Kod

pytanie zadane 11 maja 2015 w C i C++ przez WojciechJot Gaduła (3,410 p.)

Witajcie mam taki maly programik, pewna osoba powiedziała mi, że pasowałoby to zapisać w funkcjach bo program jest za długi. Miałem kiedyś pomysł, wymyśliłem go jeszcze tego samego dnia lecz to był już wieczór i na drugi dzień mi sie to ulotniło :D Macie jakieś pomysły? (wiem, że macie :D)

http://wklej.to/Oa8CN

3 odpowiedzi

odpowiedź 11 maja 2015 przez Dorion300 Szeryf (90,250 p.)

Kod w ifie i else'ie się niepotrzebnie powtarza powtarza jak i wystarczy 1 float do kilka zmiennych na raz.


#include <iostream>
#include <math.h>
#include <iostream>
#include <cstdio>

using namespace std;
float wodka,ludzie,kielon = 0.05,W,K = 0.7,KO = 0.5,A;
int plec;
int main()
{
    cout<<"Wybierz plec: "<<endl;
    cout<<"1.Mezczyzna "<<endl;
    cout<<"2.Kobieta "<<endl;
    cin>>plec;

    switch(plec)
    {
    case 1:
        cout<<"Wybrales mezczyzna."<<endl;
        break;

    case 2:
        cout<<"Wybrales kobieta."<<endl;
        break;
    }

    cout.precision(3);
    cout << " Ile masz wodki? Wpisz liczbe w litrach. \n\n\n " << endl;
    cin>> wodka;

    A = (wodka/kielon)*17;

    cout << " \n \n Ile wazysz? \n \n "<<endl;
    cin >>W;

    cout <<" \n \n Ile ludzi bedzie spozywalo alkohol?\n\n "<<endl;
    cin>> ludzie;

    cout <<"\n \n Pijecie w 50ml kielonach wiec wystarczy na: "<<wodka/kielon<<" kielonow.\n\n "<<endl;

    cout <<" Na osobe przypada "<<wodka/ludzie<<" litra wodki. \n\n "<<endl;

    if (plec==1)
    cout <<" Po spozyciu ilosc promili we krwii bedzie wynosic : "<<(A/(K*W))/ludzie<<"promili \n\n "<<endl;
    else
        cout <<" Po spozyciu ilosc promili we krwii bedzie wynosic : "<<(A/(KO*W))/ludzie<<"promili \n\n "<<endl;
    cin.get();
    cin.get();
    return 0;
}

komentarz 11 maja 2015 przez WojciechJot Gaduła (3,410 p.)

Dzieki! no to jade dalej

komentarz 11 maja 2015 przez Czeslow Obywatel (1,220 p.)

if/else zamiast switcha tez powinien odrobine skrocic kod.

odpowiedź 11 maja 2015 przez toaspzoo Dyskutant (8,300 p.)

odpowiedź 11 maja 2015 przez Czeslow Obywatel (1,220 p.)

Co do funkcji to w sumie widzę jedną która by mogła coś skrócić. Mianowicie Zadajesz tam kilka pytań i mógłbyś zrobić funkcję zadania pytania z wczytaniem danych.Z wzkorzystaniem wskaników wyglądało by to mniej więcej tak:

void pyt(string a, int *b){
     cout<<a<<endl;
     cin>>*b;
}

A wywołanie danej funkcji:

pyt("Wybierz plec:\n1.Mezczyzna\n2.Kobieta", &plec);

Przy okazji widzisz, że zamiast 'endl' można wykorzystać '\n'.

Powodzenia

komentarz 12 maja 2015 przez Dorion300 Szeryf (90,250 p.)

\n nie działa na innych systemach niż Windows. Przez co program przestaje być wieloplatformowy i trzeba zmieniać kawałek kodu aby przekompilowac na inny system.