codeIgniter: передать параметр в запрос на выборку из предыдущего запроса

Question

Я создаю небольшой инструмент для управления браузерной игрой travian. Поэтому я выбираю все деревни из базы данных и хочу отобразить контент, уникальный для каждой из деревень. Но для того, чтобы запросить эти уникальные данные, мне нужно передать идентификатор деревни. Как мне это сделать?

это мой код (контроллер):

function members_area()
    {
        global $site_title;
        $this->load->model('membership_model');

        if($this->membership_model->get_villages())
        {
            $data['rows'] = $this->membership_model->get_villages();
            $id = 1;//this should be dynamic, but how?
            if($this->membership_model->get_tasks($id)):
                $data['tasks'] = $this->membership_model->get_tasks($id);
            endif;
        }

        $data['title'] = $site_title." | Your account";
        $data['main_content'] = 'account';
        $this->load->view('template', $data);
    }

и вот две функции, которые я использую в модели:

function get_villages()
{
    $q = $this->db->get('villages');

    if($q->num_rows() > 0) {
        foreach ($q->result() as $row) {
            $data[] = $row;
        }
    return $data;
    }
}

function get_tasks($id)
{
    $this->db->select('name');
    $this->db->from('tasks');
    $this->db->where('villageid', $id);

    $q = $this->db->get();

    if($q->num_rows() > 0) {
        foreach ($q->result() as $task) {
            $data[] = $task;
        }
    return $data;
    }
}

и, конечно, вид:

<?php foreach($rows as $r) : ?>
        <div class="village">
            <h3><?php echo $r->name; ?></h3>
            <ul>
                <?php foreach($tasks as $task): ?>
                    <li><?php echo $task->name; ?></li>
                <?php endforeach; ?>
            </ul>
            <?php echo anchor('site/add_village/'.$r->id.'', '+ add new task'); ?>
        </div>
    <?php endforeach; ?>

ps: не удаляйте комментарий в первом блоке кода!

Answer 1

Согласны с обоими выше, присоединяйтесь к ним:)

функция member_model:

function get_villages_and_tasks() {
  $query_results = $this->db
  ->select('villages.id,villages.name AS villagename,tasks.name AS taskname',false)
  ->join('tasks','tasks.villageid = villages.id','left')
  ->get('villages')->result_array();

  $return_array = array();

  foreach($query_results as $row) {
    if(!isset($return_array[$row['id']]) {
      $return_array[$row['id']] = array(
        'villagename'=>$row['villagename']
      );
    } 
    if(!empty($row['taskname'])) {
      $return_array[$row['id']]['tasks'][]=array('taskname'=>$row['taskname']);
    }
  }
  return $return_array;
}

Просмотр:

<?php foreach($villages as $villageid=>$village) : ?>
  <div class="village">
    <h3><?php echo $village['villagename']; ?></h3>
    <ul>
      <?php if(!empty($village['tasks'])): ?>
        <?php foreach($village['tasks'] as $task): ?>
          <li><?php echo $task['taskname']; ?></li>
        <?php endforeach; ?>
      <?php endif; ?>
    </ul>
    <?php echo anchor('site/add_village/'.$villageid.'', '+ add new task'); ?>
  </div>
<?php endforeach; ?>

Answer 2

Я спросил, что я думаю, является связанным вопросом, на форумах Codeigniter ранее на этой неделе.

http://ellislab.com/forums/viewthread/151624/

Надеюсь, это поможет ...

Answer 3

С вашими текущими функциями модели вы можете перебирать деревни, выбирая задачи для каждой из них.

function members_area()
{
    global $site_title;
    $this->load->model('membership_model');

    // Fetch villages
    $villages = $this->membership_model->get_villages();

    if($villages)
    { 
        $tasks = array();

        // Iterate through villages fetching tasks
        foreach ($villages as $village)
        {
            $tasks[$village->id] = $this->membership_model->get_tasks($village->village->id);
        }

        $data['villages'] = $villages;
        $data['tasks'] = $tasks;
    }

    $data['title'] = $site_title." | Your account";
    $data['main_content'] = 'account';
    $this->load->view('template', $data);
}

Но для этого требуется N + 1 SQL-запросов, где N - количество деревень. Лучшим решением было бы получить все данные в одном запросе, присоединив таблицу задач к таблице деревень.