it indeed does not work for string length