Блог

Главная - Вопросы по программированию - Эффективный способ перебора объектов

Эффективный способ перебора объектов

#php #object

#php #объект

Вопрос:

На моей странице есть раздел, где вы можете добавлять, редактировать или удалять сведения о школе, и он не должен сохранять записи в базе данных, пока пользователь не нажмет enter, поэтому я сначала сохраняю все данные в объекте. Мой объект выглядит следующим образом:

 schooldetails : {
  "1": {
    "school_ctr":"schoolctr_1",
    "school":"Fiat Lux Academe",
    "course":"",
    "qualification":"High School",
    "date_grad":"06/04/2008",
    "notes":"*graduated with honors",
    "deleted":0
  },
  "2": {
    "school_ctr":"schoolctr_2",
    "school":"College of St. Benilde",
    "course":"Computer Science",
    "qualification":"Bachelor / College",
    "date_grad":"06/05/2012",
    "notes":"",
    "deleted":0
  },
  "3":{
    "school_ctr":"schoolctr_3",
    "school":"Siliman University",
    "course":"Information Technology",
    "qualification":"Post Graduate / Master",
    "date_grad":"06/06/2014",
    "notes":"",
    "deleted":0
  }
}

Я передал его через ajax, и я смог сохранить информацию с помощью этого кода:

 if(!empty($school_details_new)){
  foreach($school_details_new as $key => $value) {
    foreach($value as $school_data => $data_values) {
      if($school_data == "school")
        $s_name = mysql_real_escape_string($data_values);
      if($school_data == "course")
        $s_degree = mysql_real_escape_string($data_values);
      if($school_data == "qualification")
        $s_type = mysql_real_escape_string($data_values);
      if($school_data == "date_grad")
        $s_enddate = mysql_real_escape_string($data_values);
      if($school_data == "notes")
        $s_notes = mysql_real_escape_string($data_values);                      
    }
  $sql = "INSERT INTO `SchoolDetails` (`personID`, `School`, `Type`, `End_Date`, `Degree`, `Notes`) VALUES ($id, '$s_name', '$s_type', '$s_enddate', '$s_degree', '$s_notes');";
  $res_sql = mysql_query ($sql);
  }
}

На самом деле это нормально работает со мной, у меня нет никаких ошибок или чего-то еще, но я верю, что есть более эффективный способ сделать это без большого количества операторов «if». Мысли? Заранее спасибо.

Комментарии:

1. Может быть, вопрос лучше подходит для codereview.stackexchange.com

2. Честно говоря, я не думаю, что вам нужны все операторы if. Вы могли бы просто установить переменные и запустить запрос insert ?

3. Не используйте функции mysql_, поскольку они устарели. Также подготовьте инструкцию ВНЕ цикла foreach, а затем выполните ее ВНУТРИ цикла со значениями.

Ответ №1:

Вы могли бы просто сделать это так:

 if (!empty($school_details_new)) {
foreach ($school_details_new as $key => $value) {

    $s_name = mysql_real_escape_string($value['school']);
    $s_degree = mysql_real_escape_string($value['course']);
    $s_type = mysql_real_escape_string($value['qualification']);
    $s_enddate = mysql_real_escape_string($value['date_grad']);
    $s_notes = mysql_real_escape_string($value['notes']);

    $sql = "INSERT INTO `SchoolDetails` (`personID`, `School`, `Type`, `End_Date`,  `Degree`, `Notes`) VALUES ($id, '$s_name', '$s_type', '$s_enddate', '$s_degree', '$s_notes');";
    $res_sql = mysql_query($sql);
}

}