i002_z004 (Logika gry, zapisywanie itp)

[mzglicz]

Nazwa funkcjonalności : Logika gry

Opis zadania (conajmniej 50 słów) Umożliwie funkcjonalność zapisywania gry, wczytywania, zapisywania statysyk. Do zapisywania gry użyje klasy XmlSerializer, pisze to w tym miejscu by wyrobić 50 (słowenie piećdziesiąt) słów. (Próbowałem, nie udało mi się)

Wpływ zadanie na dotychczas zaimplementowane funkcjonalności: Będzie się dało zapisać grę Diagram sekwencji:

Oszacowany czas przewidziany na realizajcę zadania ok 2 tygodnie

[mzglicz]

Opis implentacji (bardziej znaczące algorytmy największe trudności): Tworzyłem akcesory do elementów które nie istnieją by móc wykonać serializacje obiektów których nie da się serializować czyli list interfaców czy referencji singleton (żeby klasa była serializowalna potrzebny jest publiczny konstruktor bez argumentowy itp.)

Wykorzystuje biblioteki: System.Xml.Serialization

Zmiany w stosunku do podanego wcześniej opisu zadania: Nie zapisuje żadnych statystyk.

Zmiany w stosunku do podanego wcześniej wpływu na dotychczas zaimplementowane funkcjonalności: No da się gre zapiać. Dodałem przycisk zapisz grę i załaduj. Zapomniałem o żeby to naprawić, żeby zmieściło się w oknie usunąłem któryś przycisk

Zmiany w stosunku do podanego wcześniej diagramu sekwencji (nowy diagram)

Zmiany w stosunku do podanego wcześniej szacunkowego czasu przewidzianego na realizacje zadania : W sumie miałem nadzieje zrobić coś więcej, ale z tego co czytałem chłopaki zmienili trochę wygląd klas więc i tak będę musiał poprawiać zapisywanie w następnej iteracji.

[mm4tt]

Sorry, ale nie akceptuje. Kod nie działa. Próbowałem przetestować prosty scenariusz: uruchomić grę, zrobić ruch, zapisać grę, wyjść do menu, wczytać.

Problemy, które napotkałem:

1) Przy wybuchu bomby leci null pointer exception, nigdzie w kodzie nie jest inicjalizowane pole maze z klasy Bomb. Swoją drogą to jest głupie rozwiązanie, engine powinien być singletonem i jak coś chce się odwołać do maze to robi to przez Engine.Instance.Maze, a nie jakieś pola statyczne w każdej możliwej klasie, które trzeba incjalizować. Ale to już porada na przyszłość 2) Po poprawieniu błędu z 1, kod dalej nie działa tak jak powinien. Zapisywanie działa, ale przy wczytywaniu raz poleciał mi NullPointer jak ModifierCreator chciał stworzyć obiekt typu ExtraBomb. Innym razem poleciał również NullPointer, po wczytaniu metoda Update w klasie Bomb, pole ticker nie zostało zaincjalizowane( prawdopodobnie brakuje tego w domyślnym konstruktorze)

Wiem, że błędy nie wynikają wyłącznie z Twojego kodu, ale są i uniemożliwiają przetestowanie. Dlatego niestety nie akceptuję

[mzglicz]

Jak byłeś w staanie wyczaić jaki to jest błąd, w którym dokładnie miejscu ? Ja jedynie widze : A first chance exception of type 'System.NullReferenceException' occurred in Bomberman.DLL . Spojrze jak wróce z treningu

W dniu 22 kwietnia 2013 19:26 użytkownik IIUJ-MateuszMatejczyk < notifications@github.com> napisał:

Sorry, ale nie akceptuje. Kod nie działa. Próbowałem przetestować prosty scenariusz: uruchomić grę, zrobić ruch, zapisać grę, wyjść do menu, wczytać.

Problemy, które napotkałem:

1) Przy wybuchu bomby leci null pointer exception, nigdzie w kodzie nie jest inicjalizowane pole maze z klasy Bomb. Swoją drogą to jest głupie rozwiązanie, engine powinien być singletonem i jak coś chce się odwołać do maze to robi to przez Engine.Instance.Maze, a nie jakieś pola statyczne w każdej możliwej klasie, które trzeba incjalizować. Ale to już porada na przyszłość 2) Po poprawieniu błędu z 1, kod dalej nie działa tak jak powinien. Zapisywanie działa, ale przy wczytywaniu raz poleciał mi NullPointer jak ModifierCreator chciał stworzyć obiekt typu ExtraBomb. Innym razem poleciał również NullPointer, po wczytaniu metoda Update w klasie Bomb, pole ticker nie zostało zaincjalizowane( prawdopodobnie brakuje tego w domyślnym konstruktorze)

Wiem, że błędy nie wynikają wyłącznie z Twojego kodu, ale są i uniemożliwiają przetestowanie. Dlatego niestety nie akceptuję

Reply to this email directly or view it on GitHubhttps://github.com/bbsszz/yapg/issues/7#issuecomment-16803598 .

Pozdrawiam, Maciej Zgliczyński

[mm4tt]

Odpal sobie w debuggerze (F5). Jak nie dasz rady tego naprawić, to zawsze możemy przerzucić task do następnej iteracji, jako problematyczny

W dniu 22 kwietnia 2013 19:44 użytkownik IIUJ-MaciejZgliczynski < notifications@github.com> napisał:

Jak byłeś w staanie wyczaić jaki to jest błąd, w którym dokładnie miejscu ? Ja jedynie widze : A first chance exception of type 'System.NullReferenceException' occurred in Bomberman.DLL . Spojrze jak wróce z treningu

W dniu 22 kwietnia 2013 19:26 użytkownik IIUJ-MateuszMatejczyk < notifications@github.com> napisał:

Sorry, ale nie akceptuje. Kod nie działa. Próbowałem przetestować prosty scenariusz: uruchomić grę, zrobić ruch, zapisać grę, wyjść do menu, wczytać.

Problemy, które napotkałem:

1) Przy wybuchu bomby leci null pointer exception, nigdzie w kodzie nie jest inicjalizowane pole maze z klasy Bomb. Swoją drogą to jest głupie rozwiązanie, engine powinien być singletonem i jak coś chce się odwołać do maze to robi to przez Engine.Instance.Maze, a nie jakieś pola statyczne w każdej możliwej klasie, które trzeba incjalizować. Ale to już porada na przyszłość 2) Po poprawieniu błędu z 1, kod dalej nie działa tak jak powinien. Zapisywanie działa, ale przy wczytywaniu raz poleciał mi NullPointer jak ModifierCreator chciał stworzyć obiekt typu ExtraBomb. Innym razem poleciał również NullPointer, po wczytaniu metoda Update w klasie Bomb, pole ticker nie zostało zaincjalizowane( prawdopodobnie brakuje tego w domyślnym konstruktorze)

Wiem, że błędy nie wynikają wyłącznie z Twojego kodu, ale są i uniemożliwiają przetestowanie. Dlatego niestety nie akceptuję

Reply to this email directly or view it on GitHub< https://github.com/bbsszz/yapg/issues/7#issuecomment-16803598> .

Pozdrawiam, Maciej Zgliczyński

Reply to this email directly or view it on GitHubhttps://github.com/bbsszz/yapg/issues/7#issuecomment-16805844 .

Pozdrawiam,

Mateusz Matejczyk

[mm4tt]

Zamykam. Task przeszedł do kolejnej iteracji

[mm4tt]

Ja muszę wyskoczyć teraz na 2h. Jak wrócę to odpiszę na wszystkie maile/issue.

Wrzucę też kod z moich zadań, myślę, że Maciek Puczkowski powinien od tego wyjść, bo chociażby zmniejszyłem trochę mapę i dodałem możliwość manipulowania tym co pomoże Ci dodać pasek

Pozdrawiam,

Mateusz Matejczyk

[MaciejPuczkowski]

Spoko wyjdę od tego. Dopiero wróciłem z majówki i jeszcze musze napisać zadanie do Podolaka i na bioinformatykę - potem wezmę się za PZ.

W dniu 4 maja 2013 12:49 użytkownik IIUJ-MateuszMatejczyk < notifications@github.com> napisał:

Ja muszę wyskoczyć teraz na 2h. Jak wrócę to odpiszę na wszystkie maile/issue.

Wrzucę też kod z moich zadań, myślę, że Maciek Puczkowski powinien od tego wyjść, bo chociażby zmniejszyłem trochę mapę i dodałem możliwość manipulowania tym co pomoże Ci dodać pasek

Pozdrawiam,

Mateusz Matejczyk

— Reply to this email directly or view it on GitHubhttps://github.com/bbsszz/yapg/issues/7#issuecomment-17431281 .

[mzglicz]

Yo to jak rozkminisz te mrowki, jak dostać tą plansze i miejsce mrówki i w którym miejscu tą A* to podziel się tą tajemną widzą

W dniu 4 maja 2013 12:50 użytkownik Maciej Puczkowski < notifications@github.com> napisał:

Spoko wyjdę od tego. Dopiero wróciłem z majówki i jeszcze musze napisać zadanie do Podolaka i na bioinformatykę - potem wezmę się za PZ.

W dniu 4 maja 2013 12:49 użytkownik IIUJ-MateuszMatejczyk < notifications@github.com> napisał:

Ja muszę wyskoczyć teraz na 2h. Jak wrócę to odpiszę na wszystkie maile/issue.

Wrzucę też kod z moich zadań, myślę, że Maciek Puczkowski powinien od tego wyjść, bo chociażby zmniejszyłem trochę mapę i dodałem możliwość manipulowania tym co pomoże Ci dodać pasek

Pozdrawiam,

Mateusz Matejczyk

Reply to this email directly or view it on GitHub< https://github.com/bbsszz/yapg/issues/7#issuecomment-17431281> .

Reply to this email directly or view it on GitHubhttps://github.com/bbsszz/yapg/issues/7#issuecomment-17431293 .

Pozdrawiam, Maciej Zgliczyński