"PHP" / Говнокод #7813 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
  23. 23
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
  33. 33
  34. 34
  35. 35
function resetIsetSorting()
	{
		$db = 'i_set';
		$co = -1;
		$q = $this->db->get($db);
		$r = $q->result();
		foreach($r as $row)
		{
			$co++;
			$upd['sort'] = $co;
			$this->db->where('id_set', $row->id_set); 
			$this->db->update($db, $upd);	
		}
	}
	function isetSortExchange($sort, $canBeSort)
	{
		$db = 'i_set';
		$bigNumber = 1470000;
		$this->db->where('sort', $canBeSort);
		$q = $this->db->get($db);
		if ($q->num_rows() == 1)
		{
			$upd['sort'] = $bigNumber;
			$this->db->where('sort', $canBeSort); 
			$this->db->update($db, $upd);	
			//
			$upd['sort'] = $canBeSort;
			$this->db->where('sort', $sort); 
			$this->db->update($db, $upd);
			//
			$upd['sort'] = $sort;
			$this->db->where('sort', $bigNumber); 
			$this->db->update($db, $upd);
		}
	}

Первая функция призвана обновить поле sort на уникальные последовательные значение. Сначала хотел провернуть такое дело на SQL в PHPmyADMIN, но мозгов моих оказалось мало, и я сделал это на CodeIgniter. Говнокод ли это?

Вторая функция призвана для обмена значениями полей sort. Это нужно для сортировки чего либо в баз. Чутьё мне подсказывает что это можно проделать каким-то белее правильным путём, без использования $bigNumber = 1470000; (а вдруг количество строк достигнет этого числа??!!). Говнокод ли это?

Всё рабочее, только что проверил. Свежий код, только что написан!

Запостил: increazon increazon, (Updated )

Комментарии (4) RSS

Добавить комментарий

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?


    8