[GELÖST] Problem mit 2 foreach-Schleifen: Alle Werte werden angezeigt

[m-werk]

Hallo Leute, ich steh zur Zeit voll auf dem Schlauch.

Ich komm hier nicht weiter. Folgendes:

Ich hab 2 Tabellen. In der 1. Tabelle existieren nur die Grunddaten eines Wertpapieres. In der 2. Tabelle werden täglich die Kurse zu den Wertpapieren eingetragen. Soweit sogut. Das funktioniert alles.

Aber im Frontend sehe ich nun für jeden Tag den Kurs zum dazugehörigen Wertpapier. Ich weiß nicht, wie ich es anstellen kann, dass zum Wertpapier immer nur der letzte Kurs angezeigt wird. Zur Zeit wird alles angezeigt.

Kann mir jemand auf die Sprünge helfen?

Hier sind meine Models:

Dieses holt alle Informationen zum Wertpapier aus der DB:

class MarketPriceModel extends \Model
{
    /**
     * Table name
     * @var string
     */
    protected static $strTable = 'tl_market_price';
    
    public static function findAllByPublished($offset, $limit, array $arrOptions=array())
    {
        $t = static::$strTable;
        $arrColumns = array("$t.published=?");
        $arrOptions = array('order'=>"$t.sorting asc", 'offset'=>$offset, 'limit'=>$limit);
                
        return static::findBy($arrColumns, 1, $arrOptions);
    }
} 

Dieses holt die Kurse aus der Datenbank

class MarketCourseModel extends \Model
{
    /**
     * Table name
     * @var string
     */
    protected static $strTable = 'tl_market_course';
    
    public static function findAllByPublished($wid, array $arrOptions=array())
    {
        $t = static::$strTable;
        $arrColumns = array("$t.wid=?");
        $arrOptions = array('order' => "$t.date desc");
                
        return static::findBy($arrColumns, $wid, $arrOptions);
    }
} 

Und das ist dann mein Modul:

protected function compile()
    {
        \System::loadLanguageFile('tl_market_price');
        
        $objTemplate = new \FrontendTemplate(($this->market_template ? $this->market_template : 'mod_market_price'));
                    
        $limit = 4;
        $offset = 0;
        
        $result_wertpapier = \MarketPriceModel::findAllByPublished($offset, $limit);    
        
        foreach ($result_wertpapier as $wertpapier_result)
        {
            $result_course = \MarketCourseModel::findAllByPublished($wertpapier_result->id);
            
            foreach($result_course as $course_result)
            {
                $market[] = array
                (
                    'title'         => specialchars($wertpapier_result->title),
                    'title_short'     => specialchars($wertpapier_result->title_short),
                    'course'        => specialchars($course_result->share),
                    'date'            => $course_result->date,
                );
            }
        }
        $objTemplate->wertpapier = $market;
        
        $this->Template->markets = $objTemplate->parse();
    } 

Und jetzt noch das Template

    <?php foreach ($this->wertpapier as $market): ?>

    <ul class="result_min">
        <li>
            <div class="title"><?= $market['title_short']; ?><?= $market['course']; ?> <?= date($GLOBALS['TL_CONFIG']['dateFormat'], $market['date']); ?></div>
        </li>
    </ul>

    <?php endforeach; ?>

Kann mir bitte jemand helfen?

[fiedsch]

Dieses holt die Kurse aus der Datenbank

class MarketCourseModel extends \Model
{
    /**
     * Table name
     * @var string
     */
    protected static $strTable = 'tl_market_course';
    
    public static function findAMostRecentPublished($wid)
    {
        $t = static::$strTable;
        $arrColumns = array("$t.wid=?");
        $arrOptions = array('order' => "$t.date desc", 'limit'=>1); 
                
        return static::findBy($arrColumns, $wid, $arrOptions);
    } 

und dann anstelle

$result_course = \MarketCourseModel::findAllByPublished($wertpapier_result->id); 

die neue Methode

$result_course = \MarketCourseModel::findMostRecentPublished($wertpapier_result->id); 

verwenden.

Bei der Gelegenheit: die Methodennamen mit Published kommen mir etwas komisch vor, denn Du püfst nirgends, on etwas published ist. Und den optionalen Parameter $arrOptions ignorierst Du auch immer.

[m-werk]

Hallo

Danke für die Info. Dies klappt prima.

die Methodennamen mit Published kommen mir etwas komisch vor, denn Du püfst nirgends, on etwas published ist.

Doch, beim 1. Code wird es geprüft. Beim 2. hast recht, dort hab ich kein Pubished.

Und den optionalen Parameter $arrOptions ignorierst Du auch immer.

Ja, ich hab dies aus einem anderen Code herangezogen. Ist schon bereinigt.

[m-werk]

Nun komm ich jetzt schon zum finale, wo ich noch probleme habe.

Ich hab jetzt ein 2. Model gebaut, wo ich von einem Wertpapier die letzten beiden Kurse habe. Dies möchte ich jetzt berechnen.

Ich möchte den Kurs vom Array0 / Kurs vom Array1 * 100 berechnen.

So sieht nun mein Modul aus:

$result_wertpapier = \MarketPriceModel::findAllByPublished($offset, $limit);    
        foreach ($result_wertpapier as $wertpapier_result)
        {
            $result_course = \MarketCourseModel::findMostRecent($wertpapier_result->id);
            $result_percent = \MarketCourseModel::findMostRecentSeccond($wertpapier_result->id);
            
            foreach($result_percent as $percent_result)
            {
                $percent[] = array
                (
                    'share_percent'        =>specialchars($percent_result->share),
                    'date_percent'        =>$percent_result->date,        
                );
            }
            
            foreach($result_course as $course_result)
            {
                $market[] = array
                (
                    'title'         => specialchars($wertpapier_result->title),
                    'title_short'     => specialchars($wertpapier_result->title_short),
                    'course'        => specialchars($course_result->share),
                    'date'            => $course_result->date,
                    'test'            => specialchars($percent_result->wid),
                );
            }
        } 

Es geht hier um das $percent_result.

Wenn ich im Template nur das $percent mir anzeigen lasse, sieht es so aus:

Array
(
    [0] => Array
        (
            [share_percent] => 15,76
            [date_percent] => 1523224800
        )

    [1] => Array
        (
            [share_percent] => 15,75
            [date_percent] => 1522879200
        )
)

Das sind genau die beiden Daten zu einem Wertpapier. Array0 ist aktuell, Array1 ist alt. Ich stell mir jetzt nur noch die Frage, wie ich in diesem Foreach dies berechnen kann.

Ich hab jetzt zwar was probiert, aber das Ergebnis wirft mir 0,00 raus bei jedem Wertpapier

            foreach($result_percent as $percent_result)
            {
                $percent[] = array
                (
                    'share_percent'        =>specialchars($percent_result->share),
                    'date_percent'        =>$percent_result->date,        
                );
                
                $percent_ergebnis = number_format((($percent[0]['share_percent']/$percent[1]['share_percent'])*100)-100,2, ',', '.');
            }
            
            foreach($result_course as $course_result)
            {
                $market[] = array
                (
                    'title'         => specialchars($wertpapier_result->title),
                    'title_short'     => specialchars($wertpapier_result->title_short),
                    'course'        => specialchars($course_result->share),
                    'date'            => $course_result->date,
                    'test'            => $percent_ergebnis,
                );
            } 

Aber die Berechnung müsste ja stimmen, nur wie gelangt diese dann in den $market[]?

[m-werk]

So, zwar scheint mir das ganze nicht ganz Sauber zu sein, aber ich hab es gelöst.

[mvo]

Nutze Namespaces!