das kann man nicht pauschal beantworten, und einen aspekt habe ich ja schon genannt.
wenn du eine grosse menge von prozessen hast, die alle eine bestimmte zeit lang versuchen sollen, eine datei zu locken, dann kann es bei deiner methode theoretisch dazu kommen, dass der, der als erster versucht hat, trotzdem nie drankommt, weil immer mit LOCK_NB abgebrochen und erneut versucht wird und in der zwischenzeit der nächste gelockt hat.
es gibt aber auch anwendungfsfälle, in denen das vollkommen ok ist.