• Custom User Avatar
    • Test framework and solution should be imported explicitly
    • The assertions in the random tests are set up in a completely nonsensical way
    • Practically a duplciate of the "Multiply" kata
  • Custom User Avatar

    The description needs to clear and concise, right now it is unnecessarily verbose, and despite of its verbosity it is not clear at all.

  • Custom User Avatar
    • C state is not explained anywhere
    • Floodfill is not a novel idea
  • Custom User Avatar
  • Custom User Avatar

    Bin to hex (and other similar conversions) were done already many times, in better quality (e.g this one); furthermore, they can be done with simple builtin functions; so there is nothing new here.

  • Custom User Avatar

    Input validation should be removed.

  • Custom User Avatar

    Could someone help me to fix test cases?

  • Custom User Avatar

    Sample tests:

    main.cpp:11:9: error: C++ requires a type specifier for all declarations
    Assert::That(alien_logo(2, 3), Equals(
            ^
    main.cpp:11:32: error: no matching function for call to 'Equals'
    Assert::That(alien_logo(2, 3), Equals(
                                   ^~~~~~
    
  • Custom User Avatar
    main.cpp:15:45: error: use of undeclared identifier 'alien'
        Assert::That(alien_logo(7, 7),   Equals(alien(7, 7)));
                                                ^
    ...
    main.cpp:25:29: error: use of undeclared identifier 'par'
        Assert::That(alien_logo(par.gen_N(), par.gen_skip()),  Equals(alien(par.get_N(), par.get_skip())));
                                ^