C++ translation (author gone)

approved

gotcha

done

fixes merge conflict

you should really start using a macro to compute array lengths, when they are determined by an initializer list (e.g. int array[] = {1, 2, 3};).

There is a mistake that demonstrates the dangers of hardcoding: you have two arrays of tests in the fixed and sample tests, one is of length 4 and the other of length 3; but you copied over the sample tests' loop code to the fixed tests:

const int tests[4][2] = {
//      input, expected
    {   1,     1 },
    {   8,    36 },
    { 100,  5050 },
    { 213, 22791 }
};
for(size_t index = 0; index < /* !!! */3/* !!! */; index++) {

#define ARRAY_LENGTH(array) ( sizeof(array) / sizeof *(array) )

const int tests[][2] = {
//      input, expected
    {   1,     1 },
    {   8,    36 },
    { 100,  5050 },
    { 213, 22791 }
};
for(size_t index = 0; index < ARRAY_LENGTH(tests); index++) {

approved

edit: cross-post

upgrade to PHP 8.0

code and in should be const

do not use const char *str in C++, use const std::string &str

the input should be const std::string &str, not const char *

• please keep things simple for the user: void *weights instead should be float[n][n]
• it's not explained how the memory is laid out in the input
• it's not explained how the return should be allocated
• the initial code crashes

please keep things simple for the end user:

• explain how the memory is supposed to be allocated
• your own solution returns a string literal on some code paths and a heap-allocated string on some others -> it should always be heap-allocated, const removed, and freed by the tests
• do not write uncomprehensible, golfed code in the test suite
• do not obfuscate the function names
• do not put the solution in preloaded