use @length rather than magic number