co mam źle w tym kodzie?

pytanie zadane 29 października 2017 w C i C++ przez maksymilian467 Nowicjusz (160 p.)

#include <cfloat>
#include <iostream>
#include <cstdlib>
#include <string>
#include <windows.h>
#include <cstdarg>
#include <bits/c++config.h>
using namespace std;
int wybor;

;int main()
{

    cout<<"MENU GLOWNE"<<endl;
    cout<<"-----------"<<endl;
    cout<<"1.kalkulator"<<endl;
    cout<<"2.rakieta"<<endl;
    cout<<"3.wyrazy wspak"<<endl;
    cout<<"4.srednie"<<endl;
    cout<<"5.przeliczanie na cale"<<endl;
    cout<<"6.petle"<<endl;
    cout<<"7.miesiace"<<endl;
    cout<<"          "<<endl;
    cout<<"Wybierz: "<<endl;
    cin>>wybor;

   system("cls");
   switch(wybor)
  {
      case 1:

#include <iostream>

using namespace std;
float x,y;
int wybor;
int main()


    ;cout << "Podaj 1 liczbe:" << endl;
    cin>>x;
    cout<<"Podaj 2 liczbe:"<<endl;
    cin>>y;

    cout<<endl;
    cout<<"MENU GLOWNE"<<endl;
    cout<<"-----------------"<<endl;
    cout<<"1.Dodawanie"<<endl;
    cout<<"2.Odejmowanie"<<endl;
    cout<<"3.Mnozenie"<<endl;
    cout<<"4.Dzielenie"<<endl;

    cout<<"Wybierz: ";
    cin>>wybor;

    switch (wybor)
    {
         case 1:

        cout<<"Suma= "<<x+y;

    break;
    case 2:
    cout<<"Roznica= "<<x-y;
    break;
    case 3:
    cout<<"Iloczyn= "<<x*y;
    break;
    case 4:

  cout<<"Iloraz= "<<x/y;
    break;
    default: cout<<"Takiej opcji nie ma w menu!";
    }
    {
        case 2:
        {
#include <iostream>
#include <windows.h>
#include <cstdlib>
#include <cstdarg>
using namespace std;
int i;
int main()
   ;{
    for(int i=15;i>=0; i--)

    Sleep(1000);
   system ("cls");
   cout<<i<<endl;

    cout<<"START";


    return 0;
}
        }
    }

   case 3:

#include <iostream>

using namespace std;

int main()
;{
    string wyraz;
    cout << "Podaj wyraz do odwrocenia: ";
    cin>>wyraz;

    int dlugosc = wyraz.length();

    for (int i=dlugosc-1; i>=0; i--)
    {
        cout<<wyraz[i];
    }


    return 0;

}
{
    case 4:
{
   #include <iostream>

using namespace std;
float oceny[5]; float suma=0, srednia;
int main()
;{
    for(int i=0; i<5; i++)
    {
        cout<<endl<<"Podaj "<<i+1<<" ocene: ";
       cin>>oceny[i];
      suma = suma + oceny[i];
   srednia = suma / 5;
   cout<<endl<<"Srednia = "<<srednia;

    }
    return 0;
}
}
}
{
case 5:
{
#include <iostream>

using namespace std;

float metry;

float ile_cali(float m)
;{
return metry*39.37;
}


int main()
;{
    cout << "Podaj ile metrow: ";
    cin>>metry;
    cout<<"na cale :"<<ile_cali(metry)<<endl;


    return 0;
}
}
}
{
case 6:
    {
#include <iostream>

using namespace std;
int populacja=1; int godzin=0;

int main()
;{
  while(populacja<=1000000000)
{
    populacja = populacja *2;
    cout<<"minelo godzin:"<<godzin;
    cout<<" liczba bakteri: "<<populacja<<endl;
    godzin++;
}

   return 0;
}
    }
}
{
case 7:
    {
#include <iostream>

using namespace std;
int populacja=1; int godzin=0;

int main()
;{
  while(populacja<=1000000000)
{
    populacja = populacja *2;
    cout<<"minelo godzin:"<<godzin;
    cout<<" liczba bakteri: "<<populacja<<endl;
    godzin++;
}

   return 0;
}
    }
}
  }
}
return 0;

2	komentarz 29 października 2017 przez 10kw10 Pasjonat (22,880 p.) Wtf? Takie kodu jeszcze nie widziałem :d ogladnij najpierw jakis poradnik.

1	komentarz 29 października 2017 przez Jakub 0 Pasjonat (23,120 p.) Myślałem że nikt nie pisze kodu bardziej nie czytelnie niż ja, ale się myliłem :) Z jakiego poradnika/książki/kursu korzystasz?

2 odpowiedzi

odpowiedź 29 października 2017 przez Kodeman Gaduła (4,480 p.)

odpowiedź 29 października 2017 przez DragonCoder Nałogowiec (36,500 p.)
edycja 29 października 2017 przez DragonCoder

1. Nie widzialem, takiego kodu nigdy, nie wiem skad sie uczyles, ale zmien zrodlo jak najszybciej, bo latwiej by bylo zapytac, co jest dobrze, ale do rzeczy:

2. Wywolanie funkcji main x7

3. Using namespace x 8

4. #include , mozesz mi powiedziec, po co Ci ta biblioteka?

5. Linijka 11 srednik przed main, pomimo braku deklaracjo czego kolwiek

6. Return 0, nie chcialo mi sie liczyc, ale za duzo

7. Inplementacja takze 8 razy

8. Za kazdym razem po main () pojawia sie srednik w nastepnej lini lub brak {

9. O ta jedna zmiema globalna bym sie nie przyczepil, ale jest zbedna tam. Wszystkie zmienne zadeklaruj zaraz po main ()

10. Patrz na 4, i wyjasnij mi po co cala reszta, skoro wystarczy cstdlib, iostream?

11. Nazwy zmiennych piszemy w jezyku angielskim i nie "x,y", tylko jakies ladne imiona, zeby wyrazaly to co ma roboc zmienna.

12. 2 razy ta sama petla while z bakteriami

13. Za duzo nawiasow domykajacych

14. Zamiast endl, uzywaj /n, ktore wstawiasz w cudzyslow razem z tekstem

15. Popracuj nad czytelnoscia kodu