Transaction splitting *seems* to be working!!!

This commit is contained in:
tom 2014-06-16 21:52:06 -04:00 committed by Thomas Winget
parent 9bfe0b9b6c
commit 62109840d6
2 changed files with 24 additions and 6 deletions

View file

@ -812,15 +812,21 @@ std::vector<std::vector<cryptonote::tx_destination_entry>> simple_wallet::split_
cryptonote::tx_destination_entry de; cryptonote::tx_destination_entry de;
uint64_t amount; uint64_t amount;
amount = dsts[j].amount / num_splits; amount = dsts[j].amount;
std::cout << "Amount before split: " << amount << "; num_splits: " << num_splits;
amount = amount / num_splits;
std::cout << "; amount after split: " << amount;
// if last split, add remainder // if last split, add remainder
if (i + 1 == num_splits) if (i + 1 == num_splits)
{ {
amount += dsts[j].amount % num_splits; amount += dsts[j].amount % num_splits;
std::cout << "; amount after remainder: " << amount;
} }
std::cout << std::endl;
de.addr = dsts[j].addr; de.addr = dsts[j].addr;
de.amount = amount;
new_dsts.push_back(de); new_dsts.push_back(de);
} }
@ -839,13 +845,19 @@ std::vector<std::vector<cryptonote::tx_destination_entry>> simple_wallet::split_
void simple_wallet::create_transactions(std::vector<cryptonote::tx_destination_entry> dsts, const size_t fake_outs_count, const uint64_t unlock_time, const uint64_t fee, const std::vector<uint8_t> extra) void simple_wallet::create_transactions(std::vector<cryptonote::tx_destination_entry> dsts, const size_t fake_outs_count, const uint64_t unlock_time, const uint64_t fee, const std::vector<uint8_t> extra)
{ {
// for now, limit to 5 attempts. TODO: discuss a good number to limit to. // for now, limit to 5 attempts. TODO: discuss a good number to limit to.
const size_t MAX_ATTEMPTS = 30; const size_t MAX_ATTEMPTS = 5;
// failsafe split attempt counter // failsafe split attempt counter
size_t attempt_count = 0; size_t attempt_count = 0;
for(attempt_count = 1; attempt_count <= 5 ;attempt_count++) for(attempt_count = 1; ;attempt_count++)
{ {
if (attempt_count > 1)
{
std::string prompt = "Attempt #";
prompt.append(std::to_string(attempt_count));
command_line::input_line(prompt);
}
auto split_values = split_amounts(dsts, attempt_count); auto split_values = split_amounts(dsts, attempt_count);
// Throw if split_amounts comes back with a vector of size different than it should // Throw if split_amounts comes back with a vector of size different than it should
@ -857,11 +869,12 @@ void simple_wallet::create_transactions(std::vector<cryptonote::tx_destination_e
std::vector<tools::wallet2::pending_tx> ptx_vector; std::vector<tools::wallet2::pending_tx> ptx_vector;
try try
{ {
for (size_t i=0; i < attempt_count; i++) // for each new destination vector (i.e. for each new tx)
for (auto & dst_vector : split_values)
{ {
cryptonote::transaction tx; cryptonote::transaction tx;
tools::wallet2::pending_tx ptx; tools::wallet2::pending_tx ptx;
m_wallet->transfer(dsts, fake_outs_count, unlock_time, fee, extra, tx, ptx); m_wallet->transfer(dst_vector, fake_outs_count, unlock_time, fee, extra, tx, ptx);
ptx_vector.push_back(ptx); ptx_vector.push_back(ptx);
// mark transfers to be used as "spent" // mark transfers to be used as "spent"
@ -884,7 +897,7 @@ void simple_wallet::create_transactions(std::vector<cryptonote::tx_destination_e
if (attempt_count > 1) if (attempt_count > 1)
{ {
std::string prompt_str = "Your transaction needs to be split into "; std::string prompt_str = "Your transaction needs to be split into ";
prompt_str += attempt_count; prompt_str += std::to_string(attempt_count);
prompt_str += " transactions. This will result in a fee of "; prompt_str += " transactions. This will result in a fee of ";
prompt_str += print_money(attempt_count * DEFAULT_FEE); prompt_str += print_money(attempt_count * DEFAULT_FEE);
prompt_str += ". Is this okay? (Y/Yes/N/No)"; prompt_str += ". Is this okay? (Y/Yes/N/No)";
@ -906,6 +919,8 @@ void simple_wallet::create_transactions(std::vector<cryptonote::tx_destination_e
ptx_vector.pop_back(); ptx_vector.pop_back();
} }
return;
} }
// only catch this here, other exceptions need to pass through to the calling function // only catch this here, other exceptions need to pass through to the calling function
catch (const tools::error::tx_too_big& e) catch (const tools::error::tx_too_big& e)

View file

@ -23,6 +23,7 @@
#include "wallet_errors.h" #include "wallet_errors.h"
#include <iostream>
#define DEFAULT_TX_SPENDABLE_AGE 10 #define DEFAULT_TX_SPENDABLE_AGE 10
#define WALLET_RCP_CONNECTION_TIMEOUT 200000 #define WALLET_RCP_CONNECTION_TIMEOUT 200000
@ -329,6 +330,8 @@ namespace tools
THROW_WALLET_EXCEPTION_IF(needed_money < dt.amount, error::tx_sum_overflow, dsts, fee); THROW_WALLET_EXCEPTION_IF(needed_money < dt.amount, error::tx_sum_overflow, dsts, fee);
} }
std::cout << "Attempting to create transaction, needed money = " << needed_money << std::endl;
// randomly select inputs for transaction // randomly select inputs for transaction
// throw if requested send amount is greater than amount available to send // throw if requested send amount is greater than amount available to send
std::list<transfer_container::iterator> selected_transfers; std::list<transfer_container::iterator> selected_transfers;