Skip to content
This repository has been archived by the owner on Feb 11, 2019. It is now read-only.

i002_z004 (Logika gry, zapisywanie itp) #7

Closed
mzglicz opened this issue Mar 31, 2013 · 8 comments
Closed

i002_z004 (Logika gry, zapisywanie itp) #7

mzglicz opened this issue Mar 31, 2013 · 8 comments
Assignees
@mzglicz
Labels
enhancement
Milestone
i002

Comments

@mzglicz
Copy link
Collaborator

mzglicz commented Mar 31, 2013

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
@ghost ghost assigned mzglicz Mar 31, 2013
@mzglicz
Copy link
Collaborator Author

mzglicz commented Apr 22, 2013

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.

@ghost ghost assigned mm4tt Apr 22, 2013
@mm4tt
Copy link
Owner

mm4tt commented Apr 22, 2013

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ę

@ghost ghost assigned mzglicz Apr 22, 2013
@mzglicz
Copy link
Collaborator Author

mzglicz commented Apr 22, 2013

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//issues/7#issuecomment-16803598
.

Pozdrawiam,
Maciej Zgliczyński

@mm4tt
Copy link
Owner

mm4tt commented Apr 22, 2013

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//issues/7#issuecomment-16805844
.

Pozdrawiam,

Mateusz Matejczyk

@mm4tt
Copy link
Owner

mm4tt commented May 2, 2013

Zamykam. Task przeszedł do kolejnej iteracji

@mm4tt mm4tt closed this as completed May 2, 2013
@mm4tt
Copy link
Owner

mm4tt commented May 4, 2013

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
Copy link
Collaborator

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//issues/7#issuecomment-17431281
.

@mzglicz
Copy link
Collaborator Author

mzglicz commented May 4, 2013

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//issues/7#issuecomment-17431293
.

Pozdrawiam,
Maciej Zgliczyński

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mzglicz mzglicz

Labels
enhancement
Projects
None yet
Milestone
i002
Development

No branches or pull requests
3 participants
@mm4tt @MaciejPuczkowski @mzglicz