Error encoding arguments: SyntaxError: Unexpected token e in JSON at position 2 while string concatenation

Question

I am just adding 2 strings into one string but getting below error:

transact to TestStrings.testCOntract errored: Error encoding arguments: SyntaxError: Unexpected token e in JSON at position 2

Below is my contract code:

pragma solidity ^0.5.1;

library Strings {

    function concat(string memory _base, string memory _value) internal returns (string memory) {

        bytes memory _baseBytes = bytes(_base);
        bytes memory _valueBytes = bytes(_value);

        string memory _tmpValue = new string(_baseBytes.length + _valueBytes.length);
        bytes memory _newValue = bytes(_tmpValue);

        uint i;
        uint j;

        for(i = 0;i<_baseBytes.length;i++) {
            _newValue[j++] = _baseBytes[i];
        }

        for(i = 0;i<_valueBytes.length;i++) {
            _newValue[j++] = _baseBytes[i];
        }
        return string(_newValue);
    }
}

contract TestStrings {

    using Strings for string;

    function testCOntract(string memory _base) public returns(string memory) {
        return _base.concat("_suffix");
    }
}

I have tried THIS accepted answer and it's working fine. But I am not sure what I am missing with my code.

how are you calling the testCOntract? with web3js? i get sometimes that error when i dont give a valid string as parameter — Majd TL, Jun 11 '19 at 09:17
can you post your code somewhere, i think the error is not in solidity code — Majd TL, Jun 11 '19 at 09:57
Sorry I am not getting you because I have already posted my code in my question. — Dharmesh Kheni, Jun 11 '19 at 09:58
your javascript code, not solidity :), or are you doing this with remix? — Majd TL, Jun 11 '19 at 11:08

score 1 · Accepted Answer · answered Jun 11 '19 at 11:23

You Error is in the second for, it should be _valueBytes[i], and use pure in your function

pragma solidity ^0.5.1;

library Strings {

    function concat(string memory _base, string memory _value) internal pure returns (string memory) {

        bytes memory _baseBytes = bytes(_base);
        bytes memory _valueBytes = bytes(_value);

        string memory _tmpValue = new string(_baseBytes.length + _valueBytes.length);
        bytes memory _newValue = bytes(_tmpValue);

        uint i;
        uint j;

        for(i = 0;i<_baseBytes.length;i++) {
            _newValue[j++] = _baseBytes[i];
        }

        for(i = 0;i<_valueBytes.length;i++) {
            _newValue[j++] = _valueBytes[i];
        }
        return string(_newValue);
    }
}

contract TestStrings {

    using Strings for string;

    function testCOntract(string memory _base) public pure returns(string memory) {
        return _base.concat("_suffix");
    }
}

Great and that was a silly mistake.. :) – Dharmesh Kheni Jun 11 '19 at 11:27 — Dharmesh Kheni, Jun 11 '19 at 11:27
welcome :P happen to everyone – Majd TL Jun 11 '19 at 11:27 — Majd TL, Jun 11 '19 at 11:27

Error encoding arguments: SyntaxError: Unexpected token e in JSON at position 2 while string concatenation

1 Answers1