[Lacwik]

#csharp #unity3d #unity #stream

Pomocy bo zaraz wyrzucę laptopa przez okno, otóż problem jest wręcz trywialny - zapisać string po stringu do pliku tekstowego.

Zapis odbywa się po osiągnięciu zwycięstwa. Po pierwszej iteracji gry wszystko bangla fajnie, jednak przy następnych wyrzuca exception. Już nie mam pojęcia gdzie może leżeć błąd i zdecydowanie zajmuje mi to więcej czasu niż powinno, błagam o pomoc...

Kod:
https://pastebin.com/E2b6GMG4

Głównie chodzi mi o konstrukcję Update() oraz saveIteration() gdzie odbywa się zapis. Z góry dziękuję za jakąkolwiek pomoc

[budyn]

@Lacwik: Na 90% jest problem z tym, że Streama masz cały czas otwartego. Chodzi o ten static.
Bez zagłębiania się musisz za każdym razem nowego tworzyć albo go do początku dawać.
Albo może ten StreamWriter.Close() zamyka też ten static stream i musisz go na nowo otworzyć.

Jak często zapisujesz do tego to trzymaj StreamWritera otwartego po prostu cały czas.

[nielegalny_imigrant]

@Lacwik: na drugiej iteracji masz zamknięty stream, musisz nowy utworzyć, fileWriter.Close(); <- to Ci zamyka streama, nie tylko writera

najlepiej w 94 lini zamień na
StreamWriter fileWriter = new StreamWriter("answers.txt", true);

[mcr90]

@Lacwik nie trzymaj otwartych streamów i writerow. Używaj ich w usingach - implementuja interfejs IDisposable i będą dzięki temu zwalniane po zakończeniu scope-a nawet jak poleci exception. Żadnych staticow w pracy z zasobami zewnetrznymi.

[Lacwik]

@nielegalny_imigrant @mcr90
Po usunięciu słowa kluczowego static i/lub zmiany parametru na StremaWriter("answers.txt", true) wywala następujący błąd:

IOException: Sharing violation on path

[nielegalny_imigrant]

@Lacwik: StreamWriter fileWriter = new StreamWriter(Application.persistentDataPath + "answers.txt", true);

[mcr90]

@Lacwik nie możesz otworzyć do zapisu tego samego zasobu.

W metodzie SaveIteration "otwórz" plik. Zapisz i zamknij stream.

[jaggi]

@Lacwik: czemu nie zrobisz po prostu System.IO.File.AppendLine(costam)? albo File.Write(asdasd, FileMode.Append)?
czy tam File.Create(asdasd, FileMode.Append) czy jakoś tak to było

[jaggi]

@Lacwik: nie musisz się bawić ze streamami, serio

[Lacwik]

@mcr90 a nie robie tego właśnie? StreamWriter otwiera mi plik, zapisuje do niego wartość po czym ręcznie zamykam, chyba że czegoś nie kumam, późna godzina jest...

@nielegalny_imigrant pierwsza iteracja znowu idzie ok, przy drugiej dzieją się dzie rzeczy:
- IOException: Sharing violation on path
- ArgumentNullException: Argument cannot be null.
Parameter name: stream

[jaggi]

@nielegalny_imigrant: to na 100% zadziała jeżeli tylko za każdym razem będzie tworzył ten stream writer i go zamykał. Tylko nadal nie rozumiem - czy ręczne używanie streamów przy Unity jest naprawdę konieczne? nie bawiłem się tym nigdy, ale wydaje mi się to zbędne, strasznie toporne i bardzo na około, zwłaszcza do tak trywialnych zadań

[nielegalny_imigrant]

@jaggi: gość pisze sieci neuronowe więc pewnie jest wyjadaczem z chwilowym zaćmieniem i ma pewnie swoje powody :P

[jaggi]

@nielegalny_imigrant: dopiero teraz spojrzałem na kod z pastebina dłużej niż 2 sek. Wygląda jakby przyszedł z c++ i nie akceptował innego sposobu obsługi plików ( ͡° ͜ʖ ͡°)

[Lacwik]

@jaggi otusz tag, przyznaje sie bez bicia :)

[jaggi]

@Lacwik: Skoro tak, to się trochę poczepiam:
1) Jeżeli nie zapisujesz czegoś kilka razy na sekundę albo miliona bajtów to spróbuj tak jak mówiłem - File.AppendAllText() albo AppendAllBytes czy co tam bardziej Ci pasuje.
2) Jeżeli bardzo chcesz streama to pewnie ostatnia linijka kodu od @nielegalny_imigrant powinna być ok StreamWriter fileWriter = new StreamWriter(new FileStream("answers.txt", FileMode.Append, FileAccess.ReadWrite, FileShare.None)); fileWriter.Write(asdasdasd); fileWriter.Close()
3) Zlituj się z tą pętlą for w 97

[nielegalny_imigrant]

@jaggi: Ad4 ten PascalCase mnie strasznie wkurza w C# i szczerze mówiąc sam mam go w dupie i robię camelem

[Lacwik]

@jaggi
2) FileMode.Append nie chce łączyć się z FileAccess.ReadWrite
4) Oczywiście że powinno sie unikać public ale chciałem to po prostu napisać na szybkości bo potrzebuje jednorazowo tych danych, by użyć ich jako spodziewanego outa perceptronu żeby miał sie jak uczyć. Potem wywalam tek kod
PS. ja również ;)

[jaggi]

@nielegalny_imigrant: mi w sumie nie robi róznicy czy PascalCase czy camelCase, bardziej chodzi mi o porządek. Np. jak widzę wywołanie metody DoSth(); to wiem, że to jest publiczna metoda, a jak widzę doSth() to wiem że prywatna itd. Ale to jest w sumie kwiestia umowna. Byle by się trzymać jednej konwencji w projekcie, to jakoś bardzo mnie nie boli. Zresztą - Visual Studio 2017 pilnuje tego za mnie, bez żadnych