Program, ktory wygral konkurs

pytanie zadane 18 lutego 2017 w Nasze projekty przez 10kw10 Pasjonat (22,880 p.)

Jakies 2 miesiace temu dostalem polecenie od nauczyciele zebym zrobil program na konkurs. No wiec zrobilem i udalo sie, zostalem laureatem konkursu. Teraz chce zebyscie to Wy ocenili ten projekt. Ogolnie program robilem z kolega "grafikiem".

Tresc zadania:

W dowolnym języku programowania (np.: Baltie, Scratch, Python, C++, Java, App Inventor) napisz program, który na życzenie użytkownika będzie rysował dowolny obrazek składający się z 7 elementów, a każdy element należy do osobnej kategorii i posiada 6 wariantów.

Celem programu jest narysowanie 7-elementowego rysunku, przy czym elementy kategorii powinny być wybierane losowo (można zasymulować rzuty kostką).

link do programu: Balwan

Dla tych co mysla ze sa tam wirusy ;)

https://github.com/10kw10/Balwan

Teraz widze, ze wiele rzeczy moglem napisac duzo lepiej. Najwiekszym bledem wedlug mnie bylo to ze nie dalem limitu fpsow przez co program dzialal inaczej na roznych komputerach.

Prosze o konstruktywna krytyke ;)

Pokaż 9 poprzednich komentarzy

komentarz 18 lutego 2017 przez 10kw10 Pasjonat (22,880 p.)

sprobuj uruchomic w trybie zgodnosci z win 7

komentarz 18 lutego 2017 przez robertwe Gaduła (4,620 p.)

Działa ze zgodnością z win 7:

http://pl.tinypic.com/r/2hhonef/9

komentarz 18 lutego 2017 przez 10kw10 Pasjonat (22,880 p.)

to podziwiaj ;) [joke]

komentarz 18 lutego 2017 przez Piter10p Bywalec (2,750 p.)

Wojewódzki konkurs dla szkół ponadgimnazjalnych?

Przyznam szczerze że troszeczkę mnie to dziwi.

komentarz 18 lutego 2017 przez 10kw10 Pasjonat (22,880 p.)

czemu ?

1 odpowiedź

odpowiedź 18 lutego 2017 przez Ehlert Ekspert (212,670 p.)

Język polski razi
Klasa która nazywa się BUTTON? Dziwna konwencja nazw.
Teraz sprawa tych wszystkich tekstur. Czemu trzymasz je dynamicznie? Dlaczego jest ich aż tyle? Można posklejać te obrazki. To rozwiązanie jest zupełnie niewydajne.
Niektórzy zapewne wypowiedzą się w temacie std::rand. Losowanie mogłoby być tym nowszym silnikiem.
std::to_string
Nie mam pojęcia czemu miało służyć rysuj i dlaczego nie dziedziczyłeś z sf::Drawable.
Uważam, że wskaźnik na teksturę w każdym obiekcie typu KATEGORIA jest niepotrzebny ale to moje zdanie. Skoro to było na konkurs warto było popisać się np CRTP.
Nie ma sensu przekazywać inta przez referencję. Bardziej const na argument.
Używałeś dynamicznych tablic. Szkoda że nie jakiejś ze struktur danych.
Nie obsłużony przypadek niewczytania tekstury.
enum class zamiast enum.
Czytelność kodu z piekła rodem
Cały program zapowiadał się obiektowo a main ma 162 linijki.
Szkoda że nie ma pętli stałokrokowej.
Jeszcze kilka wywołań w funkcji pozycjonowanie i wymagania przewyższyłyby Wiedźmina 3.

komentarz 18 lutego 2017 przez 10kw10 Pasjonat (22,880 p.)

2. brakowalo mi nie raz slow

3. zrobic jeden obrazek i pobierac np 50x50 pikseli z wybranej czesci ?

6. I tak nie korzystalem z Shaderow. No bo chcialem przypisac sprite z obiektu na ktory wskazuje wsk i nie mialem innego pomyslu. W sumie jak teraz patrze i w metodzie rysuj jest 1 linijka to nie wiem po co nie raz tworzylem ta metode.

9. uzywalem vectorow

11. ze niby przed mainem powinienem zdeklarowac enum ?

12. a wydawalo mi sie ze jest czytelnie

13. to duzo ?

14. nie mialem innego pomyslu

komentarz 18 lutego 2017 przez Ehlert Ekspert (212,670 p.)

Co do metody to na pewno jej odpowiedzialność należała do obiektów. sf::Drawable nie jest związane z shaderami. enum class. To jest czytelne?

        kosc.na_kosci(mysz);
        //animacja kostki
        if(czas.asMilliseconds()>=90){if(kosc.anime==true && kosc.wylosowano==false)kosc.animacja();if(ins.anime==true)ins.animacja_kostki();zegar.restart();}
        if(kosc.wylosowano==true)kosc.losowanie();

        //Pokazywanie sie wyrazow w instrukcji
        if(cz2.asMilliseconds()>=400){if(ins.wyrazy[0]<ins.ins1.size() && stan==instrukcja)ins.wyrazy[0]++;if(ins.wyrazy[1]<ins.ins2.size() && ins.wyrazy[0]>=ins.ins1.size() && stan==instrukcja)ins.wyrazy[1]++;if(ins.wyrazy[2]<ins.ins3.size() && ins.wyrazy[1]>=ins.ins2.size() && stan==instrukcja)ins.wyrazy[2]++;z2.restart();}

        //sniezki
        if(przycisk.anime==true)if((przycisk.przejechano(mysz,przycisk.najechany,przycisk.image))==true)for(int i=0;i<8;i++)s[i].pierwszy=true;
        //losowanie melodii
        if(cz.asSeconds()>=1){if(muza.sek>0)muza.sek--;if(ins.klik==true && ins.cd>0)ins.cd--;if(ins.kiedy>0 && stan==instrukcja)ins.kiedy--;if(ins.anime==true)ins.c_k--;z.restart();}
        if(muza.sek==0)muza.wczytanie=false;
        if(muza.wczytanie==false && muza.sek<=0 && muza.pierwsze==false)muza.melodie();

komentarz 18 lutego 2017 przez 10kw10 Pasjonat (22,880 p.)

tzn ze powinienem dawac wiecej komentarzy i czytelniejsze nazwy ?

dawalem takie krotkie nazwy, zazwyczaj skroty bo szybciej sie je pisze.

komentarz 18 lutego 2017 przez Ehlert Ekspert (212,670 p.)

W pracy to nie przejdzie. Jeśli będziesz pisać czytelny kod to komentarze będą całkowicie zbędne. Grunt to dobra nazwa.

2	komentarz 18 lutego 2017 przez HaKIM Szeryf (87,590 p.) 10kw10, zakup książkę: http://helion.pl/ksiazki/czysty-kod-podrecznik-dobrego-programisty-robert-c-martin,czykov.htm Nie będą to zmarowane pieniądze. :) Wersja mini: https://drive.google.com/open?id=0B_m8fC4d_bzNWDROajVnd3dlYUk