Skip to content

Commit

Permalink
Merge pull request #5 from xledger/toimmdict-select-keys-vals
Browse files Browse the repository at this point in the history
Add `ToImmDict(Func<T, K> selectKey, Func<T, V> selectVal)`.
  • Loading branch information
isaksky authored Jan 31, 2025
2 parents 88fe70e + dd735cc commit 53cac77
Show file tree
Hide file tree
Showing 5 changed files with 49 additions and 6 deletions.
6 changes: 2 additions & 4 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,15 +5,13 @@ on:
branches: [ "master" ]
pull_request:
branches: [ "master" ]
workflow_dispatch:

jobs:
build:
runs-on: ubuntu-latest
runs-on: windows-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-dotnet@v4
with:
dotnet-version: 8.0.x
- run: dotnet restore
- run: dotnet build --no-restore
- run: dotnet test --no-build --verbosity normal
28 changes: 28 additions & 0 deletions Xledger.Collections.Test/TestImmDict.cs
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,34 @@ public void TestNoOps() {
Assert.ThrowsAny<NotSupportedException>(() => idict.Remove(4));
Assert.ThrowsAny<NotSupportedException>(() => ((System.Collections.ICollection)idict).CopyTo((Array)null, 0));
}

[Fact]
public void TestSelectKey() {
var dct = Enumerable.Range(-100, 1_000).ToDictionary(i => i);
var imm = Enumerable.Range(-100, 1_000).ToImmDict(i => i);

for (int i = -1000; i < 2000; ++i) {
Assert.Equal(dct.ContainsKey(i), imm.ContainsKey(i));
if (dct.TryGetValue(i, out var dctValue)) {
imm.TryGetValue(i, out var immValue);
Assert.Equal(dctValue, immValue);
}
}
}

[Fact]
public void TestSelectKeySelectValue() {
var dct = Enumerable.Range(-100, 1_000).ToDictionary(i => i, i => (i * i).ToString());
var imm = Enumerable.Range(-100, 1_000).ToImmDict(i => i, i => (i * i).ToString());

for (int i = -1000; i < 2000; ++i) {
Assert.Equal(dct.ContainsKey(i), imm.ContainsKey(i));
if (dct.TryGetValue(i, out var dctValue)) {
imm.TryGetValue(i, out var immValue);
Assert.Equal(dctValue, immValue);
}
}
}
}


2 changes: 1 addition & 1 deletion Xledger.Collections.Test/Xledger.Collections.Test.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<PropertyGroup>
<PackageId>Xledger.Collections.Test</PackageId>

<TargetFrameworks>net48;net6.0;net8.0</TargetFrameworks>
<TargetFrameworks>net48;net8.0</TargetFrameworks>
<LangVersion>12.0</LangVersion>
<ImplicitUsings>disable</ImplicitUsings>
<Nullable>disable</Nullable>
Expand Down
17 changes: 17 additions & 0 deletions Xledger.Collections/Extensions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,23 @@ public static ImmDict<K, V> ToImmDict<K, V>(this IEnumerable<(K, V)> xs) {
};
}

public static ImmDict<K, T> ToImmDict<T, K>(this IEnumerable<T> xs, Func<T, K> selectKey) {
return xs switch {
null => ImmDict<K, T>.Empty,
_ => new ImmDict<K, T>(xs.ToDictionary(selectKey)),
};
}

public static ImmDict<K, V> ToImmDict<T, K, V>(
this IEnumerable<T> xs,
Func<T, K> selectKey,
Func<T, V> selectVal
) {
return xs switch {
null => ImmDict<K, V>.Empty,
_ => new ImmDict<K, V>(xs.ToDictionary(selectKey, selectVal)),
};
}
internal static T[] ArrayOf<T>(T[] arr) {
return (T[])arr.Clone();
}
Expand Down
2 changes: 1 addition & 1 deletion Xledger.Collections/Xledger.Collections.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<RootNamespace>Xledger.Collections</RootNamespace>
<AssemblyName>Xledger.Collections</AssemblyName>

<TargetFrameworks>net48;net6.0;net8.0</TargetFrameworks>
<TargetFrameworks>net48;net8.0</TargetFrameworks>
<LangVersion>12.0</LangVersion>
<ImplicitUsings>disable</ImplicitUsings>
<Nullable>disable</Nullable>
Expand Down

0 comments on commit 53cac77

Please sign in to comment.