Почему цикл вносить пять раз одни и те же значения?

Question

[LP-DIMAN]

public function edit_rating($id,$dateid,$obj){
         

	       		$result = $this->get_pupils($id);
	           	$pupils = array();
	           	foreach ($result as $pupil) 
           		$pupils[] = $pupil['id_user'];

        
           	for($i=0,$j=0;$i<count($pupils),$j<count($this->input->post('rating[]'));$i++,$j++){

					
			

						$this->db->where('id_journal',$id);
						$this->db->where('id_object',$obj);
						$this->db->where('id_pupil',$pupils[$i])
						$this->db->where('date',$dateid);
						$this->db->set('rating',$this->input->post("rating[$j]"));
						$this->db->update('view_journal');
						
					
					}
           }

Код код должен обновлять оценки 5 ученикам, но это не суть. Проблема в том, что в бд он вносит 5 раз данные о последнем ученике. Допустим, у нас ученики:Вася,Петя,Маша,Федя и Ваня. У них оценки: 5,4,3,2,1. Функция должна внести в бд 5 этих учеников и их оценки. Но она внесет только 5 раз Ваню и 1 соответственно. var_dump выдают правильные данные . Не пойму, где косяк
Перед этим писал похожую функцию вставки данных. Все работает

Comments

[Дмитрий Кравченко]

покажи, что выдает var_dump

[LP-DIMAN]

string(3) "143"
string(1) "5"
string(3) "142"
string(1) "4"
string(3) "139"
string(1) "3"
string(3) "141"
string(1) "2"
string(3) "140"
string(1) "1"

id учеников и их оценки

[LP-DIMAN]

Дмитрий Кравченко: а по сути он 5 раз вставляет 140 и 1

[Алексей Уколов]

Форматирования никакого, однобуквенные или бессмысленные названия переменных, неоправданно усложненный цикл - не удивительно, что вы не можете разобраться с ошибками.

[LP-DIMAN]

Алексей Уколов: я новичок, уж простите

[Алексей Уколов]

LP-DIMAN: береги честь смолоду

public function edit_rating($id, $date, $objectID)
{
	$pupilIDs = [];

	foreach ($this->get_pupils($id) as $pupil)
	{
		$pupilIDs[] = $pupil['id_user'];
	}

	$limit = min(count($pupilIDs), count($this->input->post('rating[]')));

	for ($index = 0; $index < $limit; $index++)
	{
		$pupilID = $pupilIDs[$index];
		$rating = $this->input->post("rating[$index]");

		$this->db->where('id_journal', $id);
		$this->db->where('id_object', $objectID);
		$this->db->where('id_pupil', $pupilID);
		$this->db->where('date', $date);
		$this->db->set('rating', $rating);
		$this->db->update('view_journal');
	}
}

[LP-DIMAN]

Алексей Уколов: все равно ваш чистый код заносит в бд 5 одинаковых значений

[Алексей Уколов]

Алексей Уколов посыпает голову пеплом

Answer 1

[Walt Disney]

У вас к кода бардак. Вы не понимаете что там происходит, мы тем более.
Тем не менее:
1) Вы используете слово "внести", что скорее всего $this->db->insert, а не $this->db->update
2) Реализация $this->db не понятна, похоже что вы 5 раз переназначаете where и set для 5 запросов update, а выполнение самих запросов вызывается где то ещё. Возможно у вас должно быть, что то вроде:

$db = $this->db->getNewInstance(); // возможно  $db = $this->db->update('view_journal');
    $db->where('id_journal', $id);
    $db->where('id_object', $objectID);
    $db->where('id_pupil', $pupilID);
    $db->where('date', $date);
    $db->set('rating', $rating);
    $db->update('view_journal');
    $db->exec();

Comments

[Дмитрий]

$this->db это фреймворк CodeIgniter,

$db->where('id_journal', $id);
    $db->where('id_object', $objectID);
    $db->where('id_pupil', $pupilID);
    $db->where('date', $date);

В результате получится "... WHERE id_journal='$id' AND id_object='$objectID' AND date='$date'"

Answer 2

[Дмитрий]

Если var_dump в цикле выдает правильные результаты, то ищи ошибку в методах $this->db

Comments

[LP-DIMAN]

string(3) "112"
string(1) "1"
string(3) "143"
string(10) "2015-08-31"
string(1) "5"
string(3) "112"
string(1) "1"
string(3) "142"
string(10) "2015-08-31"
string(1) "4"
string(3) "112"
string(1) "1"
string(3) "139"
string(10) "2015-08-31"
string(1) "3"
string(3) "112"
string(1) "1"
string(3) "141"
string(10) "2015-08-31"
string(1) "2"
string(3) "112"
string(1) "1"
string(3) "140"
string(10) "2015-08-31"
string(1) "1"

Вот все var_dump. ид журнала, ид предмета, ид ученика,дата и оценка. По идее все правильно, но в бд вносится последний почему-то.

[Дмитрий]

LP-DIMAN: Выведи вар_дампом SQL запросы

[LP-DIMAN]

Дмитрий: это встроенные в CI функции. Выводится тонна всякой информации

[Дмитрий]

LP-DIMAN: Должно же быть что-то такое - "UPDATE my_table WHERE ....."

[Дмитрий]

Так, погоди. Тебе нужно внести новые данные или обновить существующие?