Правильно ли написан код?

@MOCTPECT · Регистрация: 09.03.2013

Author24 — интернет-сервис помощи студентам

Здравствуйте.

Вношу кое-какие изменения в сайт.
Решил не обращаться к программисту, сделать сам. Тем более, это, насколько я понял, простейшие вещи.
У меня сомнения по поводу конструкции if-elseif-else.

PHP

<?php 
$str = ($data['montag']+$data['montag_price']);
$com = $data['comment'];
$pp = "123";
$ppp = "456";
$bsp = "<b>$com</b>";
$all = "789";
if ($data['code'] > 10118 && $data['code'] < 10129 ){
echo $pp;
}elseif ($data['code'] >= 10129){
echo $ppp;
}else{
  if ($data['montag_price'] > 1) 
  echo $all. $com;
  if ($data['montag_price'] < 1000) 
  echo $bsp;
}
 
?>

Добавлено через 11 минут
Забыл упомянуть, что код работает, и работает как мне нужно. У меня сомнения в правильности самого кода.

KOPOJI · 11.02.2016, 18:57

Не по теме:

Kerry_Jr, насколько помню, паскаль не позволяет использовать select-case по булеву.. ну а в пыхе можно подобное реализовать при помощи in_array и range, при условии небольших диапазонов и целых чисел..

@Kerry_Jr · 11.02.2016, 20:29

Не по теме:

Сообщение от KOPOJI

паскаль не позволяет использовать select-case по булеву

Не помню, в каком именно языке я видел подобную конструкцию с case'ами, но точно помню, что видел.

KOPOJI
	11.02.2016, 18:57 Правильно ли написан код? #21
	Не по теме: Kerry_Jr, насколько помню, паскаль не позволяет использовать select-case по булеву.. ну а в пыхе можно подобное реализовать при помощи in_array и range, при условии небольших диапазонов и целых чисел.. 0

@Kerry_Jr
	11.02.2016, 20:29 Правильно ли написан код? #22
	Не по теме: Сообщение от KOPOJI паскаль не позволяет использовать select-case по булеву Не помню, в каком именно языке я видел подобную конструкцию с `case`'ами, но точно помню, что видел. 0