Approved

Scala translation

Python fork with random tests and test grouping

D translation

thank you bro, don't get description of this kata at all) Can you add this link to kata's details?

approved

It's ok in the current version.

COBOL translation (author is inactive).

In c++ they forget to put a ";" at the end of the class initialization. It must be:

#include <string>

class Negabinary{
public:
  static std::string ToNegabinary(int i);
  static int ToInt(std::string s); 
};

This is the only case that I cannot understand and tried to recognize which forgotten ";" there in class initialization. So please fix it.

Giving examples doesn't mean there is no need to define what a negabinary system is.

https://en.wikipedia.org/wiki/Negative_base

Unfortunately -2,863,311,530 is not in the Integer range, so you would truncate it by casting to 1,431,655,766. Also there was no requirement in the description that it should work for the whole possible input range. I just translated test cases from the JavaScript version.

How about adding the following tests?

  @Test
  public void negabinaryMinMaxTest() {
    // 1,431,655,765 is the maximum value that can be represented with 32 bits
    assertEquals("1010101010101010101010101010101", Kata.intToNegabinary(1431655765));
    // -2,863,311,530 is the minimum value that can be represented with 32 bits
    assertEquals("10101010101010101010101010101010", Kata.intToNegabinary((int) -2863311530L));
  }

  @Test
  public void nativeMinMaxReversibilityTest() {
    assertEquals(Integer.MAX_VALUE, Kata.negabinaryToInt(Kata.intToNegabinary(Integer.MAX_VALUE)));
    assertEquals(Integer.MIN_VALUE, Kata.negabinaryToInt(Kata.intToNegabinary(Integer.MIN_VALUE)));
  }

Thanks, enjoyed a lot. =)