TDataPoint<T> review
This commit is contained in:
@@ -117,8 +117,8 @@ type
|
|||||||
FDescriptionText: string; // User-friendly description
|
FDescriptionText: string; // User-friendly description
|
||||||
public
|
public
|
||||||
constructor Create(ATickCount: Integer);
|
constructor Create(ATickCount: Integer);
|
||||||
function Init(const ADataSet: TArray<TDataRecord<TAskBidItem>>): Boolean;
|
function Init(const ADataSet: TArray<TDataPoint<TAskBidItem>>): Boolean;
|
||||||
function IsNewBar(AIndex: Integer; const ATick: TDataRecord<TAskBidItem>): Boolean;
|
function IsNewBar(AIndex: Integer; const ATick: TDataPoint<TAskBidItem>): Boolean;
|
||||||
function GetCaption: string; // Returns the Description for UI
|
function GetCaption: string; // Returns the Description for UI
|
||||||
property Caption: string read GetCaption; // Interface property uses GetCaption
|
property Caption: string read GetCaption; // Interface property uses GetCaption
|
||||||
end;
|
end;
|
||||||
@@ -130,8 +130,8 @@ type
|
|||||||
FDescriptionText: string; // User-friendly description
|
FDescriptionText: string; // User-friendly description
|
||||||
public
|
public
|
||||||
constructor Create(ATargetTickVolume: Integer);
|
constructor Create(ATargetTickVolume: Integer);
|
||||||
function Init(const ADataSet: TArray<TDataRecord<TAskBidItem>>): Boolean;
|
function Init(const ADataSet: TArray<TDataPoint<TAskBidItem>>): Boolean;
|
||||||
function IsNewBar(AIndex: Integer; const ATick: TDataRecord<TAskBidItem>): Boolean;
|
function IsNewBar(AIndex: Integer; const ATick: TDataPoint<TAskBidItem>): Boolean;
|
||||||
function GetCaption: string; // Returns the Description for UI
|
function GetCaption: string; // Returns the Description for UI
|
||||||
property Caption: string read GetCaption; // Interface property uses GetCaption
|
property Caption: string read GetCaption; // Interface property uses GetCaption
|
||||||
end;
|
end;
|
||||||
@@ -147,12 +147,12 @@ begin
|
|||||||
FDescriptionText := Format('Every %d Ticks', [FTickCountForNewBar]);
|
FDescriptionText := Format('Every %d Ticks', [FTickCountForNewBar]);
|
||||||
end;
|
end;
|
||||||
|
|
||||||
function TEveryNTicksBuilder.Init(const ADataSet: TArray<TDataRecord<TAskBidItem>>): Boolean;
|
function TEveryNTicksBuilder.Init(const ADataSet: TArray<TDataPoint<TAskBidItem>>): Boolean;
|
||||||
begin
|
begin
|
||||||
Result := True;
|
Result := True;
|
||||||
end;
|
end;
|
||||||
|
|
||||||
function TEveryNTicksBuilder.IsNewBar(AIndex: Integer; const ATick: TDataRecord<TAskBidItem>): Boolean;
|
function TEveryNTicksBuilder.IsNewBar(AIndex: Integer; const ATick: TDataPoint<TAskBidItem>): Boolean;
|
||||||
begin
|
begin
|
||||||
Result := (AIndex mod FTickCountForNewBar = 0);
|
Result := (AIndex mod FTickCountForNewBar = 0);
|
||||||
end;
|
end;
|
||||||
@@ -173,13 +173,13 @@ begin
|
|||||||
FCurrentTickInBarCount := 0;
|
FCurrentTickInBarCount := 0;
|
||||||
end;
|
end;
|
||||||
|
|
||||||
function TVolumeAccumulationBuilder.Init(const ADataSet: TArray<TDataRecord<TAskBidItem>>): Boolean;
|
function TVolumeAccumulationBuilder.Init(const ADataSet: TArray<TDataPoint<TAskBidItem>>): Boolean;
|
||||||
begin
|
begin
|
||||||
FCurrentTickInBarCount := 0;
|
FCurrentTickInBarCount := 0;
|
||||||
Result := True;
|
Result := True;
|
||||||
end;
|
end;
|
||||||
|
|
||||||
function TVolumeAccumulationBuilder.IsNewBar(AIndex: Integer; const ATick: TDataRecord<TAskBidItem>): Boolean;
|
function TVolumeAccumulationBuilder.IsNewBar(AIndex: Integer; const ATick: TDataPoint<TAskBidItem>): Boolean;
|
||||||
begin
|
begin
|
||||||
Inc(FCurrentTickInBarCount);
|
Inc(FCurrentTickInBarCount);
|
||||||
if FCurrentTickInBarCount >= FTargetTickVolumePerBar then
|
if FCurrentTickInBarCount >= FTargetTickVolumePerBar then
|
||||||
@@ -755,8 +755,8 @@ begin
|
|||||||
and (HoveredCandle.OriginalDataIndexEnd >= 0)
|
and (HoveredCandle.OriginalDataIndexEnd >= 0)
|
||||||
and (HoveredCandle.OriginalDataIndexEnd < FDataController.GetTickDataLength) then // Use GetTickDataLength
|
and (HoveredCandle.OriginalDataIndexEnd < FDataController.GetTickDataLength) then // Use GetTickDataLength
|
||||||
begin
|
begin
|
||||||
UTCTimeStart := FDataController.GetTickDataItem(HoveredCandle.OriginalDataIndexStart).TimeStamp;
|
UTCTimeStart := FDataController.GetTickDataItem(HoveredCandle.OriginalDataIndexStart).Time;
|
||||||
UTCTimeEnd := FDataController.GetTickDataItem(HoveredCandle.OriginalDataIndexEnd).TimeStamp;
|
UTCTimeEnd := FDataController.GetTickDataItem(HoveredCandle.OriginalDataIndexEnd).Time;
|
||||||
LocalTimeStart := TTimeZone.Local.ToLocalTime(UTCTimeStart);
|
LocalTimeStart := TTimeZone.Local.ToLocalTime(UTCTimeStart);
|
||||||
LocalTimeEnd := TTimeZone.Local.ToLocalTime(UTCTimeEnd);
|
LocalTimeEnd := TTimeZone.Local.ToLocalTime(UTCTimeEnd);
|
||||||
TimeZoneStr := TTimeZone.Local.GetAbbreviation(LocalTimeStart); // Get abbreviation for the local time zone
|
TimeZoneStr := TTimeZone.Local.GetAbbreviation(LocalTimeStart); // Get abbreviation for the local time zone
|
||||||
@@ -1299,7 +1299,7 @@ begin
|
|||||||
if (CrosshairCandle.OriginalDataIndexStart >= 0)
|
if (CrosshairCandle.OriginalDataIndexStart >= 0)
|
||||||
and (CrosshairCandle.OriginalDataIndexStart < FDataController.GetTickDataLength) then // Use GetTickDataLength
|
and (CrosshairCandle.OriginalDataIndexStart < FDataController.GetTickDataLength) then // Use GetTickDataLength
|
||||||
begin
|
begin
|
||||||
UTCTimeStart := FDataController.GetTickDataItem(CrosshairCandle.OriginalDataIndexStart).TimeStamp;
|
UTCTimeStart := FDataController.GetTickDataItem(CrosshairCandle.OriginalDataIndexStart).Time;
|
||||||
LocalTimeStart := TTimeZone.Local.ToLocalTime(UTCTimeStart);
|
LocalTimeStart := TTimeZone.Local.ToLocalTime(UTCTimeStart);
|
||||||
TimeStr := FormatDateTime('hh:nn:ss', LocalTimeStart); // Display time part
|
TimeStr := FormatDateTime('hh:nn:ss', LocalTimeStart); // Display time part
|
||||||
|
|
||||||
|
|||||||
@@ -21,7 +21,7 @@ type
|
|||||||
|
|
||||||
TChartDataController = class
|
TChartDataController = class
|
||||||
private
|
private
|
||||||
FTickDataArray: TArray<TDataRecord<TAskBidItem>>;
|
FTickDataArray: TArray<TDataPoint<TAskBidItem>>;
|
||||||
FOHLCCacheList: TList<TOHLC>;
|
FOHLCCacheList: TList<TOHLC>;
|
||||||
FTimeframeSettingsArray: TArray<TTimeframeSetting>; // Now includes registered generic builders
|
FTimeframeSettingsArray: TArray<TTimeframeSetting>; // Now includes registered generic builders
|
||||||
FMemoLog: TStrings;
|
FMemoLog: TStrings;
|
||||||
@@ -48,7 +48,7 @@ type
|
|||||||
procedure ClearAllDataAndCaches;
|
procedure ClearAllDataAndCaches;
|
||||||
function GetTickDataCount: Integer;
|
function GetTickDataCount: Integer;
|
||||||
function GetTickDataLength: Integer;
|
function GetTickDataLength: Integer;
|
||||||
function GetTickDataItem(AIndex: Integer): TDataRecord<TAskBidItem>;
|
function GetTickDataItem(AIndex: Integer): TDataPoint<TAskBidItem>;
|
||||||
function HasData: Boolean;
|
function HasData: Boolean;
|
||||||
|
|
||||||
// ABuilderCaption is the unique key used for FTimeframeSettingsArray.Caption and FGenericBuilders key.
|
// ABuilderCaption is the unique key used for FTimeframeSettingsArray.Caption and FGenericBuilders key.
|
||||||
@@ -512,7 +512,7 @@ begin
|
|||||||
Result := Length(FTickDataArray);
|
Result := Length(FTickDataArray);
|
||||||
end;
|
end;
|
||||||
|
|
||||||
function TChartDataController.GetTickDataItem(AIndex: Integer): TDataRecord<TAskBidItem>;
|
function TChartDataController.GetTickDataItem(AIndex: Integer): TDataPoint<TAskBidItem>;
|
||||||
begin
|
begin
|
||||||
if (AIndex >= 0) and (AIndex < Length(FTickDataArray)) then
|
if (AIndex >= 0) and (AIndex < Length(FTickDataArray)) then
|
||||||
Result := FTickDataArray[AIndex]
|
Result := FTickDataArray[AIndex]
|
||||||
@@ -522,7 +522,7 @@ begin
|
|||||||
FMemoLog.Add(
|
FMemoLog.Add(
|
||||||
Format('DataController: Attempted to access invalid tick data index %d. Count is %d.', [AIndex, Length(FTickDataArray)])
|
Format('DataController: Attempted to access invalid tick data index %d. Count is %d.', [AIndex, Length(FTickDataArray)])
|
||||||
);
|
);
|
||||||
Result := Default(TDataRecord<TAskBidItem>);
|
Result := Default(TDataPoint<TAskBidItem>);
|
||||||
end;
|
end;
|
||||||
end;
|
end;
|
||||||
|
|
||||||
|
|||||||
+12
-12
@@ -56,8 +56,8 @@ type
|
|||||||
|
|
||||||
// Interface for generic candle building logic
|
// Interface for generic candle building logic
|
||||||
IGenericCandleBuilder = interface
|
IGenericCandleBuilder = interface
|
||||||
function Init(const ADataSet: TArray<TDataRecord<TAskBidItem>>): Boolean; // Changed DataSet to ADataSet for clarity
|
function Init(const ADataSet: TArray<TDataPoint<TAskBidItem>>): Boolean; // Changed DataSet to ADataSet for clarity
|
||||||
function IsNewBar(AIndex: Integer; const ATick: TDataRecord<TAskBidItem>): Boolean; // Changed Idx to AIndex, Tick to ATick
|
function IsNewBar(AIndex: Integer; const ATick: TDataPoint<TAskBidItem>): Boolean; // Changed Idx to AIndex, Tick to ATick
|
||||||
function GetCaption: string; // Method for the Caption property
|
function GetCaption: string; // Method for the Caption property
|
||||||
property Caption: string read GetCaption;
|
property Caption: string read GetCaption;
|
||||||
end;
|
end;
|
||||||
@@ -80,7 +80,7 @@ type
|
|||||||
public
|
public
|
||||||
constructor Create;
|
constructor Create;
|
||||||
procedure ClearCache;
|
procedure ClearCache;
|
||||||
procedure Build(const ATickData: TArray<TDataRecord<TAskBidItem>>;
|
procedure Build(const ATickData: TArray<TDataPoint<TAskBidItem>>;
|
||||||
ASelectedTimeframeSeconds: Int64;
|
ASelectedTimeframeSeconds: Int64;
|
||||||
AAutoAggCandleWidth: Integer;
|
AAutoAggCandleWidth: Integer;
|
||||||
AAutoAggCandleSpacing: Integer;
|
AAutoAggCandleSpacing: Integer;
|
||||||
@@ -188,7 +188,7 @@ begin
|
|||||||
Result := Length(FCachedCandles);
|
Result := Length(FCachedCandles);
|
||||||
end;
|
end;
|
||||||
|
|
||||||
procedure TOHLC.Build(const ATickData: TArray<TDataRecord<TAskBidItem>>;
|
procedure TOHLC.Build(const ATickData: TArray<TDataPoint<TAskBidItem>>;
|
||||||
ASelectedTimeframeSeconds: Int64;
|
ASelectedTimeframeSeconds: Int64;
|
||||||
AAutoAggCandleWidth: Integer; AAutoAggCandleSpacing: Integer; AAutoAggPaintBoxClientWidth: Integer;
|
AAutoAggCandleWidth: Integer; AAutoAggCandleSpacing: Integer; AAutoAggPaintBoxClientWidth: Integer;
|
||||||
AMemoForLogging: TStrings; AGenericCandleBuilder: IGenericCandleBuilder = nil); // Parameter changed
|
AMemoForLogging: TStrings; AGenericCandleBuilder: IGenericCandleBuilder = nil); // Parameter changed
|
||||||
@@ -200,7 +200,7 @@ var
|
|||||||
CandleList: TList<TCachedCandle>;
|
CandleList: TList<TCachedCandle>;
|
||||||
TempCandle: TCachedCandle;
|
TempCandle: TCachedCandle;
|
||||||
LogTimePerCandleStr: string;
|
LogTimePerCandleStr: string;
|
||||||
CurrentTick: TDataRecord<TAskBidItem>;
|
CurrentTick: TDataPoint<TAskBidItem>;
|
||||||
begin
|
begin
|
||||||
if AMemoForLogging <> nil then
|
if AMemoForLogging <> nil then
|
||||||
AMemoForLogging.Add(FormatDateTime('yyyy-mm-dd hh:nn:ss', Now) + Format(' - TOHLC.Build: TF Secs: %d...',
|
AMemoForLogging.Add(FormatDateTime('yyyy-mm-dd hh:nn:ss', Now) + Format(' - TOHLC.Build: TF Secs: %d...',
|
||||||
@@ -353,8 +353,8 @@ begin
|
|||||||
SetLength(FCachedCandles, NumCandlesToCacheAuto);
|
SetLength(FCachedCandles, NumCandlesToCacheAuto);
|
||||||
if (NumDataPoints > 1) and (NumCandlesToCacheAuto > 0) then
|
if (NumDataPoints > 1) and (NumCandlesToCacheAuto > 0) then
|
||||||
begin
|
begin
|
||||||
var FirstTickTimeVal_A := ATickData[0].TimeStamp;
|
var FirstTickTimeVal_A := ATickData[0].Time;
|
||||||
var LastTickTimeVal_A := ATickData[High(ATickData)].TimeStamp;
|
var LastTickTimeVal_A := ATickData[High(ATickData)].Time;
|
||||||
var TotalDataDurationDaysVal_A := LastTickTimeVal_A - FirstTickTimeVal_A;
|
var TotalDataDurationDaysVal_A := LastTickTimeVal_A - FirstTickTimeVal_A;
|
||||||
if NumCandlesToCacheAuto > 0 then // Avoid division by zero if NumCandlesToCacheAuto is somehow 0 here
|
if NumCandlesToCacheAuto > 0 then // Avoid division by zero if NumCandlesToCacheAuto is somehow 0 here
|
||||||
begin
|
begin
|
||||||
@@ -432,7 +432,7 @@ begin
|
|||||||
CurrentTickIndex := 0;
|
CurrentTickIndex := 0;
|
||||||
while CurrentTickIndex <= High(ATickData) do
|
while CurrentTickIndex <= High(ATickData) do
|
||||||
begin
|
begin
|
||||||
BucketStartTimeOADate := Floor(ATickData[CurrentTickIndex].TimeStamp / TimeframeIntervalOADays) * TimeframeIntervalOADays;
|
BucketStartTimeOADate := Floor(ATickData[CurrentTickIndex].Time / TimeframeIntervalOADays) * TimeframeIntervalOADays;
|
||||||
BucketEndTimeOADate := BucketStartTimeOADate + TimeframeIntervalOADays;
|
BucketEndTimeOADate := BucketStartTimeOADate + TimeframeIntervalOADays;
|
||||||
|
|
||||||
DataIndexStart := CurrentTickIndex;
|
DataIndexStart := CurrentTickIndex;
|
||||||
@@ -444,8 +444,8 @@ begin
|
|||||||
|
|
||||||
// Iterate through ticks that fall into the current bucket
|
// Iterate through ticks that fall into the current bucket
|
||||||
while (CurrentTickIndex <= High(ATickData)) and
|
while (CurrentTickIndex <= High(ATickData)) and
|
||||||
(ATickData[CurrentTickIndex].TimeStamp >= BucketStartTimeOADate) and
|
(ATickData[CurrentTickIndex].Time >= BucketStartTimeOADate) and
|
||||||
(ATickData[CurrentTickIndex].TimeStamp < BucketEndTimeOADate) do
|
(ATickData[CurrentTickIndex].Time < BucketEndTimeOADate) do
|
||||||
begin
|
begin
|
||||||
if ATickData[CurrentTickIndex].Data.Ask > HighPrice then HighPrice := ATickData[CurrentTickIndex].Data.Ask; // Changed
|
if ATickData[CurrentTickIndex].Data.Ask > HighPrice then HighPrice := ATickData[CurrentTickIndex].Data.Ask; // Changed
|
||||||
if ATickData[CurrentTickIndex].Data.Ask < LowPrice then LowPrice := ATickData[CurrentTickIndex].Data.Ask; // Changed
|
if ATickData[CurrentTickIndex].Data.Ask < LowPrice then LowPrice := ATickData[CurrentTickIndex].Data.Ask; // Changed
|
||||||
@@ -492,8 +492,8 @@ begin
|
|||||||
|
|
||||||
if Length(ATickData) > 0 then
|
if Length(ATickData) > 0 then
|
||||||
begin
|
begin
|
||||||
var FirstTickTimeVal := ATickData[0].TimeStamp;
|
var FirstTickTimeVal := ATickData[0].Time;
|
||||||
var LastTickTimeVal := ATickData[High(ATickData)].TimeStamp;
|
var LastTickTimeVal := ATickData[High(ATickData)].Time;
|
||||||
FTotalDataTimeSpanText := FormatTimeSpanFromSeconds((LastTickTimeVal - FirstTickTimeVal) * SecsPerDay_Double);
|
FTotalDataTimeSpanText := FormatTimeSpanFromSeconds((LastTickTimeVal - FirstTickTimeVal) * SecsPerDay_Double);
|
||||||
if AMemoForLogging <> nil then
|
if AMemoForLogging <> nil then
|
||||||
AMemoForLogging.Add(Format('TOHLC.Build: Total data time span: %s.', [FTotalDataTimeSpanText]));
|
AMemoForLogging.Add(Format('TOHLC.Build: Total data time span: %s.', [FTotalDataTimeSpanText]));
|
||||||
|
|||||||
Reference in New Issue
Block a user