初心者です。こちらのコードを効率良く描きたいのですがアドバイス頂けないでしょうか。

Question

プログラミング初心者です。少しだけ組んでみたのですが、もう少し短くまたは効率のよい方法はありますでしょうか。どうぞ宜しくお願いします。

<?php
			// 全国
			$pref = array(
				'北海道', //pref_01
				'青森県', //pref_02
				'岩手県', //pref_03
				'宮城県', //pref_04
				'秋田県', //pref_05
				'山形県', //pref_06
				'福島県', //pref_07
				'茨城県', //pref_08
				'栃木県', //pref_09
				'群馬県', //pref_10
				'埼玉県', //pref_11
				'千葉県', //pref_12
				'東京都', //pref_13
				'神奈川県', //pref_14
				'新潟県', //pref_15
				'富山県', //pref_16
				'石川県', //pref_17
				'福井県', //pref_18
				'山梨県', //pref_19
				'長野県', //pref_20
				'岐阜県', //pref_21
				'静岡県', //pref_22
				'愛知県', //pref_23
				'三重県', //pref_24
				'滋賀県', //pref_25
				'京都府', //pref_26
				'大阪府', //pref_27
				'兵庫県', //pref_28
				'奈良県', //pref_29
				'和歌山県', //pref_30
				'鳥取県', //pref_31
				'島根県', //pref_32
				'岡山県', //pref_33
				'広島県', //pref_34
				'山口県', //pref_35
				'徳島県', //pref_36
				'香川県', //pref_37
				'愛媛県', //pref_38
				'高知県', //pref_39
				'福岡県', //pref_40
				'佐賀県', //pref_41
				'長崎県', //pref_42
				'熊本県', //pref_43
				'大分県', //pref_44
				'宮崎県', //pref_45
				'鹿児島県', //pref_46
				'沖縄県', //pref_47
			);
			$pref_count = count($pref);
			$pref_num = array(
				'pref_01',
				'pref_02',
				'pref_03',
				'pref_04',
				'pref_05',
				'pref_06',
				'pref_07',
				'pref_08',
				'pref_09',
				'pref_10',
				'pref_11',
				'pref_12',
				'pref_13',
				'pref_14',
				'pref_15',
				'pref_16',
				'pref_17',
				'pref_18',
				'pref_19',
				'pref_20',
				'pref_21',
				'pref_22',
				'pref_23',
				'pref_24',
				'pref_25',
				'pref_26',
				'pref_27',
				'pref_28',
				'pref_29',
				'pref_30',
				'pref_31',
				'pref_32',
				'pref_33',
				'pref_34',
				'pref_35',
				'pref_36',
				'pref_37',
				'pref_38',
				'pref_39',
				'pref_40',
				'pref_41',
				'pref_42',
				'pref_43',
				'pref_44',
				'pref_45',
				'pref_46',
				'pref_47',
			);
			for ($i=0; $i<$pref_count; $i++) {
					if(isset($_GET['area']) ? $_GET["area"] : null){
						if($_GET['area'] === $pref_num[$i]){
							echo "<h1>表示結果：".$pref[$i]."</h1>";
							
						}
					}
			}
			
				
		?>

Answer 1

Tak1wa さんの回答にプラスして、if の条件式が謎なことになっています。

次のように三項演算子は不要で、代わりに $pref にあるかどうかをチェックすればいいでしょう。

<?php
// 全国
$pref = array(
    'pref_01' => '北海道',
    'pref_02' => '青森県',
     …省略…
    'pref_46' => '鹿児島県',
    'pref_47' => '沖縄県',
);
if (isset($_GET['area']) && isset($pref[$_GET['area']])) {
    echo "<h1>表示結果：".$pref[$_GET['area']]."</h1>";
}

※未検証！

Comments

[the]

ありががとうございました！なるほど・・・ただ中身をチェックするだけでよかったんですね！　

Answer 2

こんにちは。

http://php.net/manual/ja/language.types.array.php
キーバリューペアで配列を作成し、都道府県コードで都道府県名称をひっぱってきては如何でしょう。

// 全国
$pref = array(
    'pref_01' => '北海道',
    'pref_02' => '青森県',
     …省略…
    'pref_46' => '鹿児島県',
    'pref_47' => '沖縄県',
);
if(isset($_GET['area']) ? $_GET["area"] : null){
    echo "<h1>表示結果：".$pref[$_GET['area']]."</h1>";
}

※未検証！

Comments

[the]

ありがとうございました！！！
短くなり、スッキリしました。とても非効率なやり方だったようですね・・・。