C++ Bennginer test, Menu AutoIT

[Ozzzj]

WITAM

Mam do pokazania program w zasadzie to malutki programik, został on stworzony na rzecz lekcji informatyki.
Program to zbór 3 testów : Łatwy, Średni, Trudny (Zakres c++) można jedynie odpowiedzieć tak, lub nie.
Każdy test ma 5 pytań, aby dojść do następnego należy przejść poprzedni.
Menu zostało napisane w AutoIT a natomiast "Środek" w c++. Uczę się tych języków, więc z góry thx za wyrozumiałość.
Co powinienem poprawić by program lepiej działał, lub ładniej wyglądał? Jeśli wiesz pisz pod komentarzem.
 
Daje kody źródłowe bo już mi się nie przydadzą, a może komuś tak.

 

AutoIT
 

#include <ButtonConstants.au3>
#include <GUIConstantsEx.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
Opt('MustDeclareVars', 1)

Example()

Func Example()
        Local $Button_1, $Button_2, $Button_3, $msg
GUICreate("C++", 181, 309, 484, 161)

 Opt("GUICoordMode", 3)
$Button_1 = GUICtrlCreateButton("Latwy", 32, 152, 115, 41)
GUICtrlSetCursor (-1, 3)
$Button_2 = GUICtrlCreateButton("Sredni", 32, 200, 115, 41)
GUICtrlSetCursor (-1, 3)
$Button_3 = GUICtrlCreateButton("Trudny", 32, 248, 115, 41)
GUICtrlSetCursor (-1, 3)
GUICtrlCreatePic("pliki\tlo.jpg", 0, 0, 180, 308)
GUICtrlSetCursor (-1, 3)
GUISetState(@SW_SHOW)


While 1
    $Msg = GUIGetMsg()
       Select
        Case $msg = $GUI_EVENT_CLOSE
            ExitLoop
            Case $msg = $Button_1
                                Run('pliki\latwy.exe')

Case $msg = $Button_2
                                Run("pliki\sredni.exe")

                                Case $msg = $Button_3
                                Run("pliki\trudny.exe")

                                EndSelect
 WEnd
EndFunc

C++ (1 TEST)

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

string jeden,dwa,trzy,cztery,piec;

int main()
{
    cout << "Witaj w latwym tescie c++! Pamietaj... odpowiadaj jedynie tak lub nie" << endl;
    cout << "1. C++ ma 4 petle?" << endl;
    cin >> jeden;

if (jeden=="nie")
{
        cout << "Gratuluje! C++ ma 3 petle: For, while oraz do...while!"<< endl;
}

else
{
   cout<<"Zle! C++ ma 3 petle!"<<endl;
   Sleep(3000);
   return 0;
}

cout << "2. C++ powstal w 1997 roku?" <<endl;
cin >> dwa;

if (dwa=="nie")
{
        cout << "Gratuluje! C++ powstal rok pozniej [] "<< endl;
}

else
{
   cout<<"Zle! C++ powstal rok pozniej!"<<endl;
   Sleep(3000);
   return 0;
}

cout << "3. Aby dac mozliwosc odpowiedzi nalezy dac >> po cin?" <<endl;
cin >> trzy;

if (trzy=="tak")
{
        cout << "Gratuluje! Prawidlowy strumien to >>!"<< endl;
}

else
{
   cout<<"Zle! Strumien wejsciowy to >>..."<<endl;
   Sleep(3000);
   return 0;
}

cout << "4. Wiele jezykow pochodzi od C++?" <<endl;
cin >> cztery;

if (cztery=="tak")
{
        cout << "Gratuluje! Od jezyka C++ pochodzi duzo innych jezykow!"<< endl;
}

else
{
   cout<<"Zle! Od C++ Pochodzi duzo jezykow!"<<endl;
   Sleep(3000);
   return 0;
}

cout << "5. C++ mozna uzyc do tworzenia gier?" <<endl;
cin >> trzy;

if (trzy=="tak")
{
        cout << "Gratuluje! C++ mozna uzywac do stworzenia gier!"<< endl;
        cout << "To koniec latwego testu, do nastepnego potrzebujesz tajnego kodu ponizej:"<<endl;
        cout << "linux"<<endl;
        Sleep(20000);
}

else
{
   cout<<"Zle! C++ mozna uzywac do stworzenia gier!"<<endl;
   Sleep(3000);
   return 0;
}
    return 0;
}

C++ (3 TEST)

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

string kod,jeden,dwa,trzy,cztery,piec;

int main()
{
    cout << "Podaj tajny kod ktory otrzymales wczesniej: ";
    cin >> kod;

if (kod=="hard")
{
        cout << "Witaj w trudnym tescie c++! Pamietaj... odpowiadaj jedynie tak lub nie"<< endl;
}

else
{
   cout<<"Aby zdobyc kod musisz przejsc wczesniejszy test!"<<endl;
Sleep(3000);
   return 0;
}

    cout << "1. Jezyk C++ jest jezykiem wieloparadygmatowym?"<<endl;
    cin >> jeden;

if (jeden=="tak")
{
        cout << "Gratuluje!"<< endl;
}

else
{
   cout<<"Zle!"<<endl;
Sleep(4000);
   return 0;
}

    cout << "2. Jezyk C++ zaklada statyczna kontrole typow, oraz posiada tez elementy dynamicznej kontroli typow?"<<endl;
    cin >> dwa;

if (dwa=="tak")
{
        cout << "Gratuluje!"<< endl;
}

else
{
   cout<<"Zle!"<<endl;
Sleep(4000);
   return 0;
}

    cout << "3. Jezyk C++ umozliwia bezposrednie zarzadzanie wolna pamiecia?"<<endl;
    cin >> trzy;

if (trzy=="tak")
{
        cout << "Gratuluje!"<< endl;
}

else
{
   cout<<"Zle!"<<endl;
Sleep(4000);
   return 0;
}

    cout << "4. Projekt jezyka zaklada, ze zadna nowa (wzgledem jezyka C) cecha jezyka C++ nie moze miec negatywnego wplywu na szybkosc dzialania programu?"<<endl;
    cin >> cztery;

if (cztery=="nie")
{
        cout << "Gratuluje!"<< endl;
}

else
{
   cout<<"Zle!"<<endl;
Sleep(4000);
   return 0;
}

    cout << "5. Do dzis (2015) zaden popularny kompilator nie jest w pelni zgodny z obowiazujacym standardem jezyka?"<<endl;
    cin >> piec;

if (piec=="tak")
{
        cout << "Gratuluje! Przebyles przez wszystkie testy!"<< endl;
        cout << "Ten program zostal zrobiony przez Oskara Smolinskiego"<< endl;
        cout << "Na rzecz pracy z informatyki, data wykonania: 2015-02-21 , godz 01:30"<< endl;
        Sleep(20000);
}

else
{
   cout<<"Zle!"<<endl;
Sleep(4000);
   return 0;
}

    return 0;
}

LINK DO POBRANIA: http://runfornight.y0.pl/pucharseiscomplete.php (Klikamy te na końcu, c++)
LINK DO SKANU: https://www.virustotal.com/pl/url/a8a2d38dfa2a4bd3b39ba52ee58b4e6b0339181c940098e15769079f2e14a8f0/analysis/1424487223/
ZDJĘCIE  |

                              |

                 V

 

[NopeDotAvi]

słabe formatowanie, brak funkcji dla rzeczy, które często powtarzasz w kodzie, zmienne globalne. 5 pytań i 5 zmiennych dla wczytywania, dla miliona pytań, zrobiłbyś milion zmiennych?

Nie musisz mieć osobnych zmiennych do wczytywania dla każdego pytania.

 

 

 

5. Do dzis (2015) zaden popularny kompilator nie jest w pelni zgodny z obowiazujacym standardem jezyka?

gcc/clang? (to nie ironia, tylko pytanie).

[Gość]

• C++ został po raz pierwszy wydany w 1979 roku, w 1998 tylko zaczął obowiązywać standard ISO
• Pytanie 3Ł jest dość dziwnie sformułowane, duża część innych zresztą też
• >> to nie strumień

Co do kodu AutoIT:

• Może lepiej byłoby przechowywać uchwyty przycisków w tablicy?
• Nie używaj Select, tylko Switch
• Zamiast tła mógłbyś po prostu dać tekst, GUICtrlCreateLabel
• Po co jest to Opt("GUICoordMode", 3)?

Co do kodu C++:

• Niepotrzebnie tyle zmiennych, wystarczy jedna(bo przecież z poprzednich nie korzystasz)
• Brzydki kod z ogromną ilością powtórzeń, lepiej było zrobić jedną funkcję przyjmującą treść pytania, odpowiedź i reakcje. Natomiast w mainie byłyby tablice z tymi danymi i w pętli by przekazywały kolejne wartości

[Ozzzj]

 

słabe formatowanie, brak funkcji dla rzeczy, które często powtarzasz w kodzie, zmienne globalne. 5 pytań i 5 zmiennych dla wczytywania, dla miliona pytań, zrobiłbyś milion zmiennych?

Nie musisz mieć osobnych zmiennych do wczytywania dla każdego pytania.

 

 

 

gcc/clang? (to nie ironia, tylko pytanie).

 

http://pl.wikipedia.org/wiki/C%2B%2B

Wikipedia pisała że w 2012 nie było ;P

Cytuje : "Do dziś (2012) żaden popularny kompilator nie jest w pełni zgodny z obowiązującym standardem języka, aczkolwiek ewentualne niezgodności dotyczą już tylko drugorzędnych cech języka (np. implementacji słowa kluczowego export)"

 

Ale co do programu, to jak go oceniacie? Chodzi mi teraz o wygląd, kod może brzydki ale działa jak powinien. :P