codeformunich/Muenchen-Transparent

Method createIfNotExistsReferentIn has 33 lines of code (exceeds 30 allowed). Consider refactoring.

Open

    public function createIfNotExistsReferentIn(ReferatData $data, Referat $referat): void
    {
        /** @var StadtraetIn $str */
        $str = StadtraetIn::model()->findByPk($data->referentInId);
        if ($str) {

Avoid using count() function in for loops.

Open

        for ($i = 0; $i < count($parts); $i++) {
            $parsed = ReferatData::parseFromHtml($parts[$i]);
            $referat = $this->createIfNotExistsReferat($parsed);
            $this->createIfNotExistsReferentIn($parsed, $referat);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {

  public function bar()
  {
    $array = array();

    for ($i = 0; count($array); $i++) {
      // ...
    }
  }
}