mirrors/bitcoinj
1
0
Fork 0
mirror of https://github.com/bitcoinj/bitcoinj.git synced 2025-03-10 09:20:04 +01:00
Code Issues Projects Releases Packages Wiki Activity

CoinSelection: require a List of outputs, rather than a Collection

Browse source 
Rationale:

* The comment says that it is a "list"
* Immutable Lists are the collection that maps most easily/logically to and from Stream
* Lists are easier to enforce reproducibility (mainly for unit tests)
This commit is contained in:
Sean Gilligan 2022-07-19 10:05:25 -07:00 • committed by Andreas Schildbach
parent 900af2bde9
commit e52e4d0e72
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
core/src/main/java/org/bitcoinj/wallet/CoinSelection.java
View file

@ -19,6 +19,7 @@ package org.bitcoinj.wallet;
import org.bitcoinj.base.Coin;
import org.bitcoinj.core.TransactionOutput;
import java.util.ArrayList;
import java.util.Collection;
import java.util.List;
@ -31,23 +32,23 @@ import java.util.List;
*/
public class CoinSelection {
public final Coin valueGathered;
public final Collection<TransactionOutput> gathered;
public final List<TransactionOutput> gathered;
public CoinSelection(Collection<TransactionOutput> gathered) {
public CoinSelection(List<TransactionOutput> gathered) {
this.valueGathered = sumOutputValues(gathered);
this.gathered = gathered;
}
/**
* @deprecated use {@link #CoinSelection(Collection)}
* @deprecated use {@link #CoinSelection(List)}
*/
@Deprecated
public CoinSelection(Coin valueGathered, Collection<TransactionOutput> gathered) {
// ignore valueGathered
this(gathered);
this(new ArrayList<>(gathered));
}
private static Coin sumOutputValues(Collection<TransactionOutput> outputs) {
private static Coin sumOutputValues(List<TransactionOutput> outputs) {
return outputs.stream()
.map(TransactionOutput::getValue)
.reduce(Coin.ZERO, Coin::add);