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...

I see no value whatsoever in FIELDS. It's stating the obvious. I mean what else could those be, bananas? Auto-completion should react to SELECT, not FIELDS IMHO. In general seems...

There isn't a simple and perfect solution, unfortunately. We ran into this when writing the ABAP Intro book. At the time it was written, many companies still used pre-7.4 ABAP...

Came here to comment about the comments. :) Not sure where this suggestion came from, it's first time I'm hearing a recommendation to use " instead of *. It's an...

@fabianlupa The key here is to use SE80 where nothing is highlighted in red. :) Also, sometimes I even write the comments some place else and copy-paste them. Obviously I...