Безусловный вызов FinamTradeSource.CommitLoad()


Безусловный вызов FinamTradeSource.CommitLoad()
Atom
11/8/2011


Доброго времени суток.
Метод FinamTradeSource.CommitLoad() вызывается даже если не каких данных загружено не было (например по причине того дата близка к текущей). В CommitLoad производится инкремент:
security.ExtensionInfo["FinamLastTradeTime"] = (date + TimeSpan.FromDays(1)).To<long>();

В результате если получение данных стартует более одного раза в сутки, то FinamLastTradeTime "убегает" в будущее.

Tags:


Thanks:


Mikhail Sukhov

Avatar
Date: 11/9/2011
Reply


Залил на КодеПлекс фикс.
Thanks:

Marcopolo

Avatar
Date: 11/9/2011
Reply


Да, сейчас в будущее не "убегает", но как и в топике про РТС через выходные не переходит. Возможно необходимо инкрементировать FinamLastTradeTime в случае "пустого" дня, если мы дальше от текущего дня чем FinamOffset. То есть видимо в любом случае передавать весь список:

source.CommitLoad(data.Keys);

но в CommitLoad инкрементировать только в случае если мы дальше от текущего дня чем FinamOffset:

public void CommitLoad(IEnumerable<Security> securities)
{
if (securities == null)
throw new ArgumentNullException("securities");
foreach (var security in securities)
{
var date = security.ExtensionInfo["FinamLastTradeTime"].To<DateTime>();
if (date + TimeSpan.FromDays(_settings.FinamOffset) < DateTime.Today)
{
security.ExtensionInfo["FinamLastTradeTime"] = (date + TimeSpan.FromDays(1)).To<long>();
_storage.Securities.Save(security);
}
}
}
Thanks:

Mikhail Sukhov

Avatar
Date: 11/9/2011
Reply


Пофиксил.
Thanks: Marcopolo

Marcopolo

Avatar
Date: 11/10/2011
Reply


Да, судя по всему заработало.
Спасибо!
Thanks:


Attach files by dragging & dropping, , or pasting from the clipboard.

loading
clippy