Jelena

IMHO in the example given either the method name (drive) is not descriptive enough or it violates "the method should do one thing" concept. Either there should be 2 methods,...

Both examples below are valid, which one to use depends on the scenario and specific requirements. This part is not the matter of cleanliness. ``` METHOD read_customizing. IF keys IS...

That's what I usually do too - put the shortest part of IF / exception first. Much easier to read IMHO.

I also vote for symbols instead of characters. In addition to what's mentioned already, characters are based on English language, so especially for non-native speakers it takes an extra millisecond...

I'm with @fabianlupa on this: if it's used in multiple branches, the declare explicitly. It's OK to declare inline if it's only used in the same branch and not anywhere...

I'm relieved to see so many ABAP luminaries voting for SELECT SINGLE. I also tend to use it, whether I have full key or not. If I don't have full...