Что насчёт QuantStamp думаете?
Добрый день, проект пока свежий, не слышали, чтобы с ними кто-то работал.
function () payable whenNotPaused beforeDeadline afterStartTime saleNotClosed nonReentrant {
вот эта фукнция перегружена модификаторами. Так, конечно можно, но читабельность снижается.
// Transfer the tokens from the crowdsale supply to the sender
if (tokenReward.transferFrom(tokenReward.owner(), msg.sender, numTokens)) {
FundTransfer(msg.sender, amount, true);
/ Check if the funding goal or cap have been reached
/ TODO check impact on gas cost
checkFundingGoal();
checkFundingCap();
}
А здесь проверки достижения надо делать до отправки, тк может быть ситуация что (funding + newEther) > Goal и таким образом, нужно делать отмену транзакции. На наш взгляд, это надо ставить до If.