graph-di5 / warspot

Automatically exported from code.google.com/p/warspot
0 stars 0 forks source link

использовать game.repay в DownloadReplay #36

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
subj
+
вероятно стоит поменять имя на что-то более 
толковое, например использовть метку 
времени 

Original issue reported on code.google.com by graph....@gmail.com on 25 Feb 2013 at 7:51

GoogleCodeExporter commented 9 years ago
Раскройте поподробней сабж. 

Original comment by gregory....@gmail.com on 27 Feb 2013 at 8:58

GoogleCodeExporter commented 9 years ago
вот тут
http://code.google.com/p/warspot/source/browse/trunk/WarSpot.Cloud.Storage/Wareh
ouse.cs#145

видно(да и в других местах тоже), что реплей 
хранится в виде без расширения
replayC5A1D2C1-E154-47C7-AF66-AA4357B5037D
// что не страшно, но немножко неудобно при 
ручной заливке файлов

соль задачи в том, что _опять_ используется 
механизм генерации имени (скопированный), 
а не уже существующее поле replay

правильно я понимаю, что поле replay должно 
быть именем реплея? если да, то лучше его и 
ипоользовать в этом месте

мутно написал, но думаю понятно

Original comment by graph....@gmail.com on 27 Feb 2013 at 12:51

GoogleCodeExporter commented 9 years ago
Да-да, содержимое поля Replay и есть имя блоба, 
к которому мы должны обратиться.

Original comment by Stanisla...@gmail.com on 27 Feb 2013 at 4:01

GoogleCodeExporter commented 9 years ago
Вероятно, закрыто. Результат в r509. 
Конкретно - реализация GetReplay  и UploadReplay в 
Warehouse.cs.  
В UploadReplay теперь генерится ID повтора, и 
заполняется поле Game.ReplayID , а сам повтор 
загружается в блоб Game.ReplayID.ToString() . GetReplay 
использует поле Game.ReplayID, чтобы достать 
повтор.
Опционально можно добавить и метку 
времени, если это нужно, только укажите 
тогда формат  имени блоба повтора.

Original comment by gregory....@gmail.com on 27 Feb 2013 at 5:04

GoogleCodeExporter commented 9 years ago
отлично, закрыто. метку времени пока не 
делаем

Original comment by graph....@gmail.com on 27 Feb 2013 at 5:32