René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Replace from an array of strings","articleBody":"It would be useful if the ReplaceAll method could accept an array.\r\n\r\nAs an example, I altered the ReplaceAll code to produce this:\r\n// from javascript call as:   sb.replaceFromArray()\r\n\r\nnapi_value ReplaceFromArray(napi_env env, napi_callback_info info){\r\n        size_t argsLength = 1;\r\n        napi_value args[2];   // will hold parameters from a 2d array during a for loop\r\n        napi_value me;\r\n        int n;\r\n\r\n\t\t// test with hard-coded array containing 2 pairs of strings\r\n\t\t//  \"john is some writer\"  =\u003e \"john is this joke\"\r\n        const char* arr[][2] = {{\"some\", \"this\"},\r\n\t\t\t                    {\"writer\", \"joke\"}};\r\n\r\n       \r\n        napi_get_cb_info(env, info, \u0026argsLength, args, \u0026me, 0);  // place input parameters into args[]\r\n        /* if (argsLength \u003c 2){  // this bit no longer needed\r\n             return me;\r\n       */\r\n        uint16_t* buffer;\r\n        int64_t* metadata;\r\n        getBufferAndMetaData(env, me, \u0026buffer, \u0026metadata);\r\n        uint16_t* pattern;\r\n        int64_t patternLength;\r\n        bool patternFreeAble;\r\n        for (n = 0; n \u003c 2;  n++ )    \r\n\t\t{\r\n\t\t \tnapi_create_string_utf8(env, arr[n][0],\r\n                                     NAPI_AUTO_LENGTH ,\r\n                                     \u0026args[0]);\r\n\r\n\t\t\tnapi_create_string_utf8(env, arr[n][1],\r\n                                     NAPI_AUTO_LENGTH ,\r\n                                     \u0026args[1]);\r\n                  // .....  the rest of the replaceAll code\r\n                }\r\n            return me;\r\n}\r\n\r\nI also note that the main section from replaceAll is mostly identical to that inside the ReplacePattern function.  There is a potential  therefore for refactoring, by which this section could be called from a separate function. \r\n\r\nA more sophisticated implementation of ReplaceFromArray() would pass the array as a parameter, although embedding a hard-coded array has the advantage of \"hiding\" the array contents, while still suiting many possible applications.\r\n\r\n","author":{"url":"https://github.com/xthdraft","@type":"Person","name":"xthdraft"},"datePublished":"2018-11-13T04:52:11.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/2/node-stringbuilder/issues/2"}