Question

I often see code a function defined without visibility keywords. e.g:

class Foo() {
  function bar() {
    // ...
  }
}

Is it a shorthand of public function? Is it a good practice to omit it?

class Foo() {
  public function bar() {
    //..
  }
}

Answer 1

As written in the PHP Doc,

Methods declared without any explicit visibility keyword are defined as public.

So, yes, in

class Foo() { public function bar() { //.. } }

Foo::bar() is public, but omitting the visibility keyword is never a good practice. If it’s a fast and ugly script why not, but in other cases you should specify it.