Skip to content

Commit 92499bc

Browse files
committed
Refactoring.
1 parent 9d65d2b commit 92499bc

File tree

4 files changed

+14
-2
lines changed

4 files changed

+14
-2
lines changed

src/FirebirdSql.Data.FirebirdClient/Common/PageSizeHelper.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@
1515

1616
//$Authors = Jiri Cincura ([email protected])
1717

18+
using System;
19+
1820
namespace FirebirdSql.Data.Common;
1921

2022
internal static class PageSizeHelper
@@ -28,4 +30,9 @@ public static bool IsValidPageSize(int value)
2830
|| value == 16384
2931
|| value == 32768;
3032
}
33+
34+
public static Exception InvalidPageSizeException()
35+
{
36+
return new InvalidOperationException("Invalid page size.");
37+
}
3138
}

src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbConnectionInternal.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,9 @@ public void CreateDatabase(int pageSize, bool forcedWrites, bool overwrite)
124124
dpb.Append(IscCodes.isc_dpb_overwrite, (overwrite ? 1 : 0));
125125
if (pageSize > 0)
126126
{
127+
if (!PageSizeHelper.IsValidPageSize(pageSize))
128+
throw PageSizeHelper.InvalidPageSizeException();
129+
127130
dpb.Append(IscCodes.isc_dpb_page_size, pageSize);
128131
}
129132

@@ -169,6 +172,8 @@ public async Task CreateDatabaseAsync(int pageSize, bool forcedWrites, bool over
169172
dpb.Append(IscCodes.isc_dpb_overwrite, (overwrite ? 1 : 0));
170173
if (pageSize > 0)
171174
{
175+
if (!PageSizeHelper.IsValidPageSize(pageSize))
176+
throw PageSizeHelper.InvalidPageSizeException();
172177
dpb.Append(IscCodes.isc_dpb_page_size, pageSize);
173178
}
174179

src/FirebirdSql.Data.FirebirdClient/Services/FbRestore.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public int? PageSize
3333
set
3434
{
3535
if (value is int v && !PageSizeHelper.IsValidPageSize(v))
36-
throw new InvalidOperationException("Invalid page size.");
36+
throw PageSizeHelper.InvalidPageSizeException();
3737

3838
_pageSize = value;
3939
}

src/FirebirdSql.Data.FirebirdClient/Services/FbStreamingRestore.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public int? PageSize
3737
set
3838
{
3939
if (value is int v && !PageSizeHelper.IsValidPageSize(v))
40-
throw new InvalidOperationException("Invalid page size.");
40+
throw PageSizeHelper.InvalidPageSizeException();
4141

4242
_pageSize = value;
4343
}

0 commit comments

Comments
 (0)