I like the "is not None" part, but I think "== 0" on the length communicates better that we are checking explicitly for zero-length strings instead of "not length" which is confusing.