Method names that are vague or lying - they say one thing and do something completely different
addWaterToCup() {
// code that boils water, adds a portion of it to cup and adds a tea bag
}
Method names that doesn't really make sense
doExternalTeaBag() {
}
Method names that leak variable names or doesn't match the parameters when leaking params:
getEntityById(int id, int someotherid, int anotherfield){
// do lookup on entity using all three params, but method name only mentions first param
}
This is also horrible, overloaded methods that simply call another method using internal params, I've seen several of these today.
class Abc {
int a, b, c;
doSomething(){
doSomething(this.a, this.b, this.c)
}
doSomething(int a, int b, int c){
// do something with a, b, c even though method has direct access to this.a,this.b and this.c
}
}