php switch文を関数に入れる

Question

現在phpにて、switch文で出た値を複数個所で反映する為,
switch文を関数に入れようとしているのですが反映されません。

functhon(){　をつける前は正常に反映しておりました。

関数の構文の間違えでしょうか？

又、switchで取り出した値を他で使う方法が他にあればご教授ください。

 switch($result['date']){
  case preg_match('/-10$/', $result['date'])===1 && preg_match('/100/', $result['CDN'])===1:
    echo $result['price']*0;
 return $result['price']*0;
    break;
  case preg_match('/-18$/', $result['date'])===1 && preg_match('/100/', $result['CDN'])===1:
    echo  $result['price']*0.01;
    return $result['price']*0.01;
    break;
  case preg_match('/-10$/', $result['date'])===1 && preg_match('/49/', $result['CDN'])===1:
  case preg_match('/-27$/', $result['date'])===1 && preg_match('/49/', $result['CDN'])===1:
  case preg_match('/-29$/', $result['date'])===1 && preg_match('/49/', $result['CDN'])===1:
    echo $result['price']*0.02;
    return  $result['price']*0.02;
    break;
   
  case preg_match('/-28$/', $result['date'])===1 && preg_match('/194/', $result['CDN'])===1:
  case preg_match('/-30$/', $result['date'])===1 && preg_match('/194/', $result['CDN'])===1:
    echo  $result['price']*0.02;
    return  $result['price']*0.02;
    break;

  case  preg_match('/145/', $result['CDN'])===1:
  case  preg_match('/^9$/', $result['CDN'])===1:
  case  preg_match('/81/', $result['CDN'])===1:
      echo 0;
    return 0;
            break;
        default:
        echo $result['price']*0.03;  
       return $result['price']*0.03;   
      }   
    }

?>

Comments

[maisumakun]

> switch文を関数に入れようとしているのですが反映されません。

その、失敗した状態のコードをご提示いただけませんでしょうか？

[tecnopolicy]

申し訳ありません。
コード再送いたします。

<?php

function tes(){
switch($result['date']){
case preg_match('/-10$/', $result['date'])===1 && preg_match('/100/', $result['CDN'])===1:
echo $result['price']*0;
break;
case preg_match('/-18$/', $result['date'])===1 && preg_match('/100/', $result['CDN'])===1:
echo $result['price']*0.01;
break;
case preg_match('/-10$/', $result['date'])===1 && preg_match('/49/', $result['CDN'])===1:
case preg_match('/-27$/', $result['date'])===1 && preg_match('/49/', $result['CDN'])===1:
case preg_match('/-29$/', $result['date'])===1 && preg_match('/49/', $result['CDN'])===1:
echo $result['price']*0.02;
break;
case preg_match('/-28$/', $result['date'])===1 && preg_match('/194/', $result['CDN'])===1:
case preg_match('/-30$/', $result['date'])===1 && preg_match('/194/', $result['CDN'])===1:
echo $result['price']*0.02;
break;
case preg_match('/145/', $result['CDN'])===1:
case preg_match('/^9$/', $result['CDN'])===1:
case preg_match('/81/', $result['CDN'])===1:
echo 0;
return 0;
break;
default:
echo $result['price']*0.03;

}
}
?>

[m.ts10806]

質問は編集できますので適宜調整してください

Answer 1

function tes(){
switch($result['date']){

PHPの関数では、関数の外側にある変数を、原則として参照できません。引数で渡すなど、何かしらの対応を行う必要があります。

Comments

[tecnopolicy]

ありがとうございます！

初歩的な質問にも関わらず、ご返答頂きありがとうございました。

一番早く返答頂きましたので、ベストアンサーとさせていただきます。

Answer 2

すでに指摘があるようにユーザー関数への値引き継ぎはできない問題もそうですが
そもそもswitchの使い方も間違えています
returnの仕方もおかしいです

Answer 3

switch にこだわる理由がよくわかりません。

function tes($result) {
    //$result['date'] のチェックはとりあえずスルーで

    if (preg_match('/100/', $result['CDN'])===1) {
        if (preg_match('/-10$/', $result['date'])===1) return $result['price']*0;
        if (preg_match('/-18$/', $result['date'])===1) return $result['price']*0.01;
    }
    if (preg_match('/49/', $result['CDN'])===1) {
        foreach (['/-10$/','/-27$/,'/-29$/'] as $x) {
            if (preg_match($x, $result['date'])===1) return $result['price']*0.02;
        }
    }
    if (preg_match('/194/', $result['CDN'])===1) {
        foreach (['/-28$/','/-30$/'] as $y) {
            if (preg_match($y, $result['date'])===1) return $result['price']*0.02;
        }
    }
    foreach (['/145/','/^9$/','/81/'] as $z) {
        if (preg_match($z, $result['CDN'])===1) return 0;
    }
    return $result['price']*0.03;
}

$p = tes($result);
echo $p;
//$pを使った処理