Ну есть несколько вещей, которые вы можете сделать, прежде всего, вы хотите, чтобы убедиться, что обе переменные существуют, которые вы можете сделать с isset()
Для кода, как это я, как правило, в пользу раннего возвращения над вложенным сослагательного наклонения, так что вы могли бы сделать что-то вроде:
if (!isset($_POST['postid']) || !isset($_POST['comment'])) {
$error = 'Values for postid and comment were not set.';
return $error;
}
После того, как вы проверили, что обе переменные установлены, вы могли бы начать, чтобы проверить переменные являются ли правильный тип и размер.
Вы могли потенциально заменить isset()
с empty()
так:
if (empty($_POST['postid'] || empty($_POST['comment']) {
$error = 'Values are not set.';
return $error;
}
Для примера я полагаю postid
является числовым так ...
if (!is_numeric($_POST['postid'])) {
$error = 'Value for postid must be numeric.';
return $error;
}
Вы также можете сделать чек, что это выше 0 : if (!$_POST['postid'] > 0) {}
Для комментариев вы можете проверить, что длина больше 0.
if (!strlen($_POST['comment']) > 0) {
$error = 'Comment was left blank, ensure it is filled in.';
return $error;
}
Итак, это всего лишь несколько примеров, которые помогут вам начать работу, надеюсь, что это поможет. :)