Что не так с этой функцией, которая использует php если еще, если?

Question

У меня есть этот код, но он не работает, и я не знаю почему ...

function link_next($local_id, $link_type) {
  if ( !is_numeric($local_id) )      die(); 
  elseif ($link_type = 'a-href')   { $something = 'a-href'; }
  elseif ($link_type = 'link-rel') { $something = 'link-rel'; }
  else                             { $something = 'blablabla'; } 

  return $local_id.'-'.$something;
} 

$id ='14678';
echo link_next($id, 'link-rel');

// редактировать

Функция всегда отображает 14678-a-href

Answer 1

вы используете = в проверке состояния. Вам нужно использовать == или === (строгое сравнение)

Answer 2

Ваш код неверен. Вы используете оператор '=' в условиях if-else вместо оператора '=='.Это назначит значение типу ссылки.Вам нужно изменить код:

function link_next($local_id, $link_type) {

    if (!is_numeric($local_id)) 
           return false;

    if ($link_type == 'a-href') {
        $something = 'a-href';
    } elseif ($link_type == 'link-rel') {
        $something == 'link-rel';
    } else {
        $something = 'blablabla';
    }
    return $local_id.'-'.$something;
} 

Надеюсь, это поможет

Answer 3

У вас есть три проблемы:

1. Ваш код не читается - структурируйте его так, чтобы его можно было прочитать
2. Вы используете назначение (=) вместо равенство (==) для сравнения строк - вот что ломает код

-

function link_next($local_id, $link_type) {

    // early exit
    if (!is_numeric($local_id)) die();

    if ($link_type == 'a-href') {
        $something = 'a-href';
    } elseif ($link_type == 'link-rel') {
        $something == 'link-rel';
    } else {
        $something = 'blablabla';
    }
    return $local_id.'-'.$something;
} 

Answer 4

Вы используете оператор присваивания =, а не оператор сравнения == или === (строгий).

Кроме того, правильная установка отступа для кода помогает легко определить его.ошибки.

function link_next($local_id, $link_type) {
 if ( !is_numeric($local_id) ){ 
       die(); 
    }
 elseif ($link_type == 'a-href') {
      $something = 'a-href'; 
    }
 elseif ($link_type == 'link-rel') { 
    $something = 'link-rel';
    }
 else { 
    $something == 'blablabla'; 
    } 

 return $local_id.'-'.$something;
} 

 $id ='14678';
 echo link_next($id, 'link-rel'); 
 echo link_next($id, 'link-rel');

Answer 5

Вы должны использовать два знака равенства:

elseif ($link_type == 'a-href')   { $something = 'a-href'; }
elseif ($link_type == 'link-rel') { $something = 'link-rel'; }

В противном случае вы устанавливаете $link_type для указанной строки!

Answer 6

Вы используете оператор присваивания вместо оператора эквивалентности. Ака вы используете = вместо ==.

Вы можете использовать присваивание в if, но тогда вы говорите: «Присвойте переменной значение, и тогда условие будет истинным, если переменная не пустая». (где пусто 0, false, '', null и т. д.)

РЕДАКТИРОВАТЬ: получил ниндзя. Наверное, потому, что я более многословен.

Answer 7

Попробуйте это:

function link_next($local_id, $link_type) { 

if ( !is_numeric($local_id) ){ 
    die(); 
} elseif ($link_type == 'a-href') { 
    $something = 'a-href'; 
} elseif ($link_type == 'link-rel') { 
    $something = 'link-rel'; 
} else { 
    $something = 'blablabla';
}

return $local_id.'-'.$something;

}

Я только исправил тот факт, что при первой проверке {} не было, не уверен, что это будет иметь значение, но, эй, стоит попробовать.