Invalid argument - "foreach" problem HIlfe!

**XGremliN** · 28.02.2008, 07:33

$motwList2 ist hier kein Array, deswegen die Fehlermeldung. evtl um $n-count() noch Klammern setzen

**H2O** · 28.02.2008, 08:34

Die Klammern haben in diesem Fall absolut keinen Einfluss. Die Frage ist: Was gibt die Funktion selectNewMOTW() zurück? Offensichtlich ist es kein Array.

**Antonia** · 28.02.2008, 08:53

Hmm, hab da wegen dem selectNewMOTW schon experimentiert.
Ohne Erfolg.

O:K., hier mal der komplette text:
der foreach befindet sich im unteren Drittel.

PHP-Code:


<?php require_once('Connections/memberverbindung.php');

mysql_select_db($database_memberverbindung, $memberverbindung);





// Holt die memberids der aktuellen Mitglieder der Woche

  // oder NULL falls es keine gibt.

  function getCurrentMOTW($memberverbindung)

  {

    $query='SELECT membernr AS motw FROM membersOfTheWeek WHERE kw=WEEK(CURDATE(),3)

     AND year=YEAR(CURDATE())';

     if(false===($result=mysql_query($query, $memberverbindung)))

      die(mysql_error().' ON '.$query);

           $motwList=array();

    while(false!==($row=mysql_fetch_assoc($result)))

      $motwList[]=$row['motw'];

    return $motwList;

  }



 // Speichert ein neues Mitglied der Woche

   function storeNewMOTW($memberverbindung, $motw)

  {

    $query=sprintf('INSERT INTO membersOfTheWeek (ProductID,kw,year)

     VALUES(%d,WEEK(CURDATE(),3),YEAR(CURDATE()))', $motw);

    if(!mysql_query($query, $memberverbindung))

      die(mysql_error().' ON '.$query); 

  }



  // ermittelt $n neue Mitglieder der Woche

  function selectNewMOTW($memberverbindung, $n)

  {

    $query=sprintf('SELECT 

                      a.membernr AS motw

                    FROM

                      members a 

                       LEFT JOIN membersOfTheWeek b 

                        ON 

                          a.membernr=b.membernr 

                    ORDER BY

                      b.year ASC,

                      b.kw ASC,

                      RAND()

                    LIMIT 1,%d',$n);

    $result=mysql_query($query, $memberverbindung);

     if(!mysql_query($query,$memberverbindung))

      die(mysql_error().' ON '.$query);

    $motwList=array(); 

    while(false!==($row=mysql_fetch_assoc($result)))

      $motwList[]=$row['motw']; 

}



// ZUSAMMENBAU: gibt die aktuellen $n "Mitglieder der Woche" zurück:

  function getMotw($memberverbindung, $n)

  {

    $motwList=getCurrentMOTW($memberverbindung);

    if(count($motwList)<$n)

    {

      $motwList2=selectNewMOTW($memberverbindung,$n-count($motwList));

      foreach($motwList2 as $mid)

        storeNewMOTW($memberverbindung, $mid);

      $motwList=array_merge($motwList, $motwList2);

    }

    return $motwList;

  }  

  

      $aktmem=getMotw($memberverbindung,1);

  ?>



<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">

<LINK REL="SHORTCUT ICON" HREF="/favicon.ico">

<meta name="robots" content="INDEX,FOLLOW">



<BODY>



 <?php echo $aktmem ['memberinfo']; ?>



</BODY>

</HTML>

**jahlives** · 28.02.2008, 08:59

Und was gibt selectNewMOTW() zurück? Ich behaupte ganz mutig: Rein gar nichts!

Oder siehst du return in dieser Funktion? Allenfalls nach der while-Schleife das Array zurückgeben...

**Antonia** · 28.02.2008, 14:07

Original geschrieben von jahlives
Und was gibt selectNewMOTW() zurück? Ich behaupte ganz mutig: Rein gar nichts!

Das könnte hinkommen.

Ziel ist:

Aus der tabelle "members" einen oder zwei texte aus der Spalte "memberinfo" (pro Woche einen anderen) auslesen und anzeigen.
Dazu gibt es noch die Tabelle "membersOfTheWeek" mit Spalte "kw und "year". Beide haben die Spalte "membernr", sozusagen als ID Spalte.

Da müsst Ihr mir mal unter die Arme greifen, ich meine den betreffenden Block korrigiert einstellen.

Ich habe hiermit meine Schwierigkeiten, teils sind mir Befehle nicht bekannt, dann ist das Script auch nicht von mir erstellt, nur teils verbessert, wo ich konnte.

Mann, wenn das endlich laufen würde...uhhh...

**Kropff** · 28.02.2008, 14:12

Da müsst Ihr mir mal unter die Arme greifen, ich meine den betreffenden Block korrigiert einstellen.

hier gibt es keinen fertigen code. du musst doch nur am ende der funktion selectNewMOTW das entsprechende return einbauen. siehe auch hier: funktionen

gruß
peter

**jahlives** · 28.02.2008, 14:13

Schon probiert?

Allenfalls nach der while-Schleife das Array zurückgeben...

PHP-Code:


function selectNewMOTW($memberverbindung, $n)

  {

    $query=sprintf('SELECT 

                      a.membernr AS motw

                    FROM

                      members a 

                       LEFT JOIN membersOfTheWeek b 

                        ON 

                          a.membernr=b.membernr 

                    ORDER BY

                      b.year ASC,

                      b.kw ASC,

                      RAND()

                    LIMIT 1,%d',$n);

    $result=mysql_query($query, $memberverbindung);

     if(!mysql_query($query,$memberverbindung))

      die(mysql_error().' ON '.$query);

    $motwList=array(); 

    while(false!==($row=mysql_fetch_assoc($result))){

      $motwList[]=$row['motw']; 

    }

    //Jetzt geben wir das Array zurück

    return $motwList;

}

Gruss

tobi

**Antonia** · 29.02.2008, 08:47

Klappt jetzt, danke!

Das Script arbeitet jetzt, nur der Ablauf ist noch etwas gestört.
Ich beschreibs heute später mal. Da muss ich etwas ausholen.

Invalid argument - "foreach" problem HIlfe!