Как сделать чтоб для каждого Задания выводилось свое время завершения?

@TVPNERO · Регистрация: 29.03.2016

Есть таблица Таскс в которой хранятся название задачи и проект задачи.

Есть таблица Таймс,в ней хранятся дата завершения и дата начала задач.

В Виде выводятся список всех записей из таблицы Таскс --- название Задачи и название Проекта задачи .
Они выводятся с помощью цикла foreach

Благодаря запросу в контроллере

PHP

1
2

$model=Tasks::find()->addOrderBy(['timecreation' =>SORT_DESC])->all();  // Выводит все Таски
          $model_vivod_stop_task=Time::find()->one();  //  выводит данные о завершении таска

PHP/HTML

         <table class="table_price">                            
        <caption>Все задачи</caption>                      
        <tbody>                                            
          <tr id="headnone" >                                
        <th class="headnone">Задачи</th>               
        <th class="headnone" colspan="2">Проект</th>  
    </tr>                                              
                                                       
          <?php foreach ($model  as $current): ?>
 
            <table class="table_price">                            
    <caption>Все задачи</caption>                      
    <tbody>                                            
    <tr id="headnone" >                                
        <th class="headnone">Задачи (Таск)</th>               
        <th class="headnone" colspan="2">Проект</th>  
    </tr>                                              
                                                       
           <?php foreach ($model  as $current): ?>
 
              
 
       <tr>                                                                                                        
             <td height="40" class="tskdetails"> Начало выполнения : <?php echo $current['timecreation']; ?> </td>   
      </tr>                                                                                                       
                                                                                                             
       <tr>                                                                                                              
            <td height="40" class="tskdetails"> Завершение выполнения :  <?php echo $model_vivod_stop_task->stop; ?> </td>
        </tr>                                                                                                             
                                                                                                                          
 
                                                                                                    
          
 
 
         <?php endforeach; ?>
 
 
      </tbody>
</table>

Эта строка обращается как к свойству обьекта а не массива и выводит только первую запись из таблицы Таймс

PHP

1	<td height="40" class="tskdetails"> Завершение выполнения : <?php echo $model_vivod_stop_task->stop; ?> </td>

И проблема как раз заключается в том что мне нужна не первая запись (скриншот прилагаю) из таблицы Таймс мне нужно для каждого Задания свое время завершения ,для этого я пробовал переписать запрос чтоб возвращало массив вот так

PHP

1	$model_vivod_stop_task=Time::find()->all();

А Вывод соотвествено написал так,но в результате после таких изменений вывод Завершения выполнения для каждого Задания имел все значения из таблицы а не для конкретного задания ( скрин 2 прилагаю )

PHP/HTML

        <?php if(is_array($model_vivod_stop_task)) :?>
 
    <?php foreach ($model_vivod_stop_task  as $current2): ?>
 
 
 
        <tr>
            <td height="40" class="tskdetails"> Завершение выполнения :  <?php echo $current2['stop']; ?> </td>
        </tr>
            <?php endforeach; ?>
        
    <?php endif; ?>
 
 
    <?php endforeach;  ?>
 
    </tbody>
</table>

Проблема в том как сделать чтоб для каждого Задания выводилось свое время завершения ?
сейчас для каждого Задания выводятся все даты завершения

@FloppyDisc · 29.06.2018, 10:29

Сообщение от TVPNERO

Есть таблица Таймс,в ней хранятся дата завершения и дата начала задач.

Зачем данное разделение? Разве нельзя в таблице Task реализовать что-то эдакое:

PHP

$this->createTable('{{%tasks}}', [
   'id' => $this->primaryKey(),
   'title' => $this->string(255),
   'description' => $this->text(),
   'status' => $this->smallInteger()->notNull(),
   'priority' => $this->smallInteger()->notNull(),
   'started_at' => $this->dateTime(),
   'deadline' => $this->dateTime(),
   'finished_at' => $this->dateTime(),
], '');

Сообщение от TVPNERO

Эта строка обращается как к свойству обьекта а не массива и выводит только первую запись из таблицы Таймс

Оно и понятно, вы же делаете непонятно зачем запрос Time::findOne();

Сообщение от TVPNERO

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

Тоже оно и понятно, опять же только findAll();

Вообще логики нет в данных запросах.

Вам нужно почитать про реляции, я думаю у вас в таблице Task есть поле-внешний ключ к таблице Time, так вот по этому ключу и делайте связи, например при создании CRUD это делается автоматически, если в миграциях прописано:

PHP

$this->addForeignKey(
   'fk_task_time',
   '{{%task}}',
   'time_id',
   '{{%time}}',
   'id',
   'CASCADE',
   'CASCADE'
);

или наоборот, я не знаю как вы там связывали(а может и вообще не связывали) записи.

Из идеи:

Сообщение от TVPNERO

Проблема в том как сделать чтоб для каждого Задания выводилось свое время завершения ?

Я понял, что у каждого Task всего по одной связанной записи в Times, тогда вопрос, который задал выше: "Зачем данное разделение?" очень актуальна.

Но хозяин барин, так что ваша реляция должна выглядеть примерно так:

1. В Tasks model: думаю у вас этот вариант

PHP

public function getTime()
{
     return $this->hasOne(Time::className(), ['id' => 'time_id']);
}
 
// example usage:
$tasks = Task::findAll();
foreach($tasks as $task) {
   echo $task->title, PHP_EOL, $task->time->start, ' - ', $task->time->stop;
}

2. Time model

PHP

public function getTask()
{
     return $this->hasMany(Task::className(), ['time_id' => 'id']);
}
 
// example usage: в этом случае все как-то коряво, получается у одного Time может быть несколько Task
$times = Time::findAll();
foreach($times as $time) {
   echo $time->task->title, PHP_EOL, $time->start, ' - ', $time->stop;
}

@TVPNERO · 29.06.2018, 22:11 **[ТС]**

FloppyDisc,

Я разделил данные на две таблицы в учебных целях ведь в реальных проектах нельзя сложить все в одну таблицу))

У меня таблицы не связаны внешним ключом,я просто в контроллере записываю сразу айди в таблицу Тайм

PHP

               public function actionStarttask($id){
 
                   $model=Tasks::find()->addOrderBy(['timecreation' =>SORT_DESC])->all();
 
                   $model2=new Time();
 
                $model2->taskid=$id;
      $model2->save();
 
\\ итд

И я так и не понял как связывание таблиц позволит мне выводить свое время для каждого таска.
и не понял где использовать функции getTime() и getTask().у вас в коде они нигде не используются

А модели у меня простые, просто дают достп к таблицам,там нету методов.

К примеру модель Таскс

PHP

<?php
 
namespace frontend\models;
 
 
 
 
class Tasks extends \yii\db\ActiveRecord
{
 
    public static function tableName()
    {
        return 'tasks';
    }
 
 
}
 
 
?>

и модель Таймс

PHP

<?php
 
namespace frontend\models;
 
use yii\db\ActiveRecord;
 
 
class Time extends \yii\db\ActiveRecord
{
 
 
 
 
    public static function tableName()
    {
        return 'time';
    }
 
 
 
 
 
 
 
 
 
 
 
}
 
 
?>

@FloppyDisc · 29.06.2018, 23:04

Сообщение от TVPNERO

Я разделил данные на две таблицы в учебных целях ведь в реальных проектах нельзя сложить все в одну таблицу))

Почему же, можно, если сущность удовлетворяет правилам нормализации и целостности, но ваш пример не очень удачен и совсем не по правилам.

Сообщение от TVPNERO

У меня таблицы не связаны внешним ключом,я просто в контроллере записываю сразу айди в таблицу Тайм

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

Сообщение от TVPNERO

и не понял где использовать функции getTime() и getTask().у вас в коде они нигде не используются

Используются, просто в виде геттеров, вот более понятный вариант:

PHP

// example usage:
$tasks = Task::findAll();
foreach($tasks as $task) {
   echo $task->title, PHP_EOL, $task->getTime()->start, ' - ', $task->getTime()->stop;
}

Сообщение от TVPNERO

А модели у меня простые

Я бы сказал минималистические, прям как новый вид искусства.

Сообщение от TVPNERO

И я так и не понял как связывание таблиц позволит мне выводить свое время для каждого таска.

Советую вам почитать данный раздел документации: Работа со связными данными

@TVPNERO · 08.07.2018, 18:25 **[ТС]**

FloppyDisc, матод findAll

PHP

1	$tasks = Task::findAll();

не работает

выдает ошибку
Missing argument 1 for yii\db\BaseActiveRecord::findAll(), called in /var/www/html/timetracker/frontend/controllers/TrackController.php on line 43 and defined

@FloppyDisc · 09.07.2018, 17:59

Сообщение от TVPNERO

не работает

::find()->all();

@TVPNERO · 11.07.2018, 21:28 **[ТС]**

Сообщение от FloppyDisc

Используются, просто в виде геттеров, вот более понятный вариант:

только такой вариант не сработал,ошибка Getting unknown property: yii\db\ActiveQuery::start

PHP/HTML

1	echo $task->title, PHP_EOL, $task->getTime()->start, ' - ', $task->getTime()->stop;

а в виде геттеров сработал

@FloppyDisc · 12.07.2018, 12:03

Я не понимаю логики разделения, у вас у каждого Task (задачи) один или несколько Time (времен завершения)? В таком случае придется переделать реляцию в модели:

PHP

public function getTimes()
{
     return $this->hasMany(Time::className(), ['id' => 'taskid']);
}

он вернет массив значений, которые так же надо перебирать через цикл:

PHP

$tasks = Task::find()->all();
foreach($tasks as $task) {
  /**
    *@var $task Task
    */
   echo 'Задача: ', $task->title, PHP_EOL; 
   /**
    *@var $time Time
    */
   foreach($task->times as $time) {
      echo  'Время: ', $time->start, ' - ', $time->stop, PHP_EOL;
   }
}

@TVPNERO · 15.07.2018, 17:16 **[ТС]**

FloppyDisc, нет для каждого Таска одно время в таблице Тайм .То есть вреся завершения одно для каждого таска.

Просто вариант этот вариант цыкла с геттерами работает

PHP

foreach($tasks as $task) {
      echo $task->name, PHP_EOL, $task->time->start, ' - ', $task->time->stop,' - ',$task->time->wasted;
 
   
 
 
  }

А этот не работает.И непонятно почему.

PHP

1	$task->name, PHP_EOL, $task->getTime()->start, ' - ', $task->getTime()->stop;

@FloppyDisc · 16.07.2018, 09:04

Сообщение от TVPNERO

А этот не работает.И непонятно почему.

потому что там массив, но если вы переделали реляцию, то они оба не должны работать...

@TVPNERO · 16.07.2018, 22:18 **[ТС]**

Сообщение от FloppyDisc

но если вы переделали реляцию, то они оба не должны работать..

FloppyDisc, я поробовал переделать так,но все равно работает только один вариант

PHP

public function getTimes()
{
     return $this->hasMany(Time::className(), ['id' => 'taskid']);
}

Сообщение от FloppyDisc

потому что там массив

Так а разве в первом варианте не массив ?
foreach же не будет работать с обьектами.Или в этом случае работает потому что тут массив обьектов ??

PHP

public function getTime()
{
     return $this->hasOne(Time::className(), ['id' => 'time_id']);
}

@FloppyDisc · 16.07.2018, 22:55

Сообщение от TVPNERO

foreach же не будет работать с обьектами

кто это вам сказал

Сообщение от TVPNERO

Так а разве в первом варианте не массив ?

Я уже запутался, что тут первое, а что второе...

Сообщение от TVPNERO

я поробовал переделать так,но все равно работает только один вариант

Окей, в модели Task создаете "связь" 1:1 между таблицами, это так, образно

PHP

task model
 
public function getTime()
{
     return $this->hasOne(Time::className(), ['id' => 'time_id']);
}

в принципе усе

PHP

/* SELECT * FROM `task` WHERE `id` = 1
$task = Task::findOne(1);
 
/* SELECT * FROM `time` WHERE `task_id` = 1
echo $task->time->start; //

@FloppyDisc Особый статус 615 / 214 / 163 Регистрация: 18.11.2015 Сообщений: 1,073
	09.07.2018, 17:59	6
	Сообщение от TVPNERO не работает ::find()->all(); 0

@FloppyDisc Особый статус 615 / 214 / 163 Регистрация: 18.11.2015 Сообщений: 1,073
	16.07.2018, 09:04	10
	Сообщение от TVPNERO А этот не работает.И непонятно почему. потому что там массив, но если вы переделали реляцию, то они оба не должны работать... 0

Как сделать чтоб для каждого Задания выводилось свое время завершения?

Решение