Resource id #11

Du prøver at udskrive noget, som ikke er en normal variable, men bare er et resource id. Det kan for eksempel være resultatet fra en mysql_query() eller en fopen() eller andre lignende ting, der ikke indeholder information i sig selv, men bare er et "håndtag", som kan bruges til at få fat i information.

nok fordi du har noget i retning af

  $r=mysql_query("...");
  print $r;

<?
require "database.php";
//titlen
$title = "Brugere";
//querien til at finde medlemmer
$brugq = mysql_query("SELECT * FROM medlemmer") or die(mysql_error());
//antallet af medlemmer
$antal = mysql_num_rows($brugq);
//wile løkken
while($raew = mysql_fetch_array($brugq))
    {

    $brug .=
    "
    <tr>
      <td height=\"25\">$raew[nick]</td>
      <td>$raew[mail]</td>
      <td>$raew[oprettet]</td>
    </td>
    ";
   
    }
er her noget?

Nej, det er ikke i den del, der er noget galt.

Kan det være, at du kommer til at udskrive $brugq et sted?

ok her kommer der hele:

<?
require "database.php";

//titlen
$title = "Brugere";

//querien til at finde medlemmer
$brugq = mysql_query("SELECT * FROM medlemmer ORDER BY opret ASC") or die(mysql_error());

//antallet af medlemmer
$antal = mysql_num_rows($brugq);

//wile løkken
while($raew = mysql_fetch_array($brugq))
    {
$opret = date("j / n - Y", $raew[oprettet]);
    $brug .=
    "
    <tr>
      <td height=\"25\">$raew[nick]</td>
      <td>$raew[mail]</td>
      <td>$opret</td>
    </td>
    ";
   
    }

//selve content
$content =
"
<table width=\"100%\">
<tr>
  <td colspan=\"3\" valign=\"top\" align=\"center\"><u><b>Brugere oprettet på $hjemmeside ($antal)</b></u></td>
</tr>
<tr>
  <td colspan=\"3\" height=\"30\">&nbsp;</td>
</tr>
$brug
</table>
";


replace();
?>

det er ikke replace() der er noget galt med

Fejlen er nødt til enten at være et sted, hvor du fylder i variabler til udskrift, eller hvor du udskriver. Hvis den kode, du har vist her er hele din side, så er fejlen i database.php et sted.

hvis jeg fjerner:

//querien til at finde medlemmer
$kh = mysql_query("SELECT * FROM medlemmer ORDER BY oprettet ASC") or die(mysql_error());

//wile løkken
while($yf = mysql_fetch_array($kh))
    {
   
    $opret = date("j / n - Y | H:i", $yf[oprettet]);
   
    $brug .=
    "
    <tr>
      <td height=\"25\">$yf[nick]</td>
      <td>$yf[mail]</td>
      <td>$opret</td>
    </td>
    ";
   
    }

så kommer "fejlen" ikke

Hjalp det ikke bare at kalde den $kh i stedet for $brugq?

Kan jeg få et link, og gerne også replace(), så jeg kan se nøjagtigt hvor fejlen optræder? Det er letter end at skulle skyde sig frem i blinde.

link = http://balmer1.frac.dk/brugere.php


function replace()
    {
    global $design, $buffer, $alert, $content, $title, $status, $hoejre;
   
    $design = fopen("template.php", "r");   
   
    while(!feof($design))
        {
   
          $buffer .= fgets($design, 4096);
   
        }   

    fclose($design);
   
   
    $buffer = ereg_replace('<--hoejre-->', $hoejre, $buffer);
    $buffer = ereg_replace('<--content-->', $content, $buffer);
    $buffer = ereg_replace('<--alert-->', $alert, $buffer);
    $buffer = ereg_replace('<--title-->', $title, $buffer);
    $buffer = ereg_replace('<--status-->', $status, $buffer);
   
    echo $buffer;
    }
?>

Hvis du trykker "vis kilde" så kommer:

<table width="100%">
<tr>
  <td colspan="3" height="40" valign="top"  align="center"><u><b>Brugere oprettet på Wicez.com</b></u></td>
</tr>
<tr>
  <td colspan="3" height="22" valign="top"><b>fed</b> = Administrator</td>
</tr>
<tr>
  <td colspan="3" height="22" valign="top"><u>understreget</u> = Redaktør</td>
</tr>
<tr>
  <td colspan="3" height="22" valign="top"><em>kursiv</em> = inaktiv</td>
</tr>
<tr>
  <td colspan="3" height="30">&nbsp;</td>
</tr>
Resource id #11
    <tr>
      <td height="25">Wicez</td>
      <td>andersbalmer@hotmail.com</td>
      <td>10 / 7 - 2004 | 17:07</td>
    </tr>

så det står uden for tabellen?

Du skal nu også have rettet
    <tr>
      <td height=\"25\">$yf[nick]</td>
      <td>$yf[mail]</td>
      <td>$opret</td>
    </td>
til
    <tr>
      <td height=\"25\">$yf[nick]</td>
      <td>$yf[mail]</td>
      <td>$opret</td>
    </tr>

Så står fejlbeskeden måske det rigtige sted. Husk altid at lave en HTML-varlidering af dine sider.

jeg bruger også database.php på alle mine andre sider, og der kommer det ikke det der "resource id"

ja det har jeg også haft gjort ^^, men det virkede også med </td> :?

Sæt $brug="";
inden din whileløkke. Den kommer nok fra din include fil

"virkede også med </td>"  ... browsere er underlige, men det er altså forkert.

ja i know.. men stadig mystisk o_O

det med $brug = ""; virkede

hvordan kan du gøre det erik?

det er gang på gang på gang på gang på gang på gang du har det rigtige svar :D

men du vil ikke have point?

Jeg har psykokinetiske evner, og kan se alle bits på din computer. Derfor.

Nej tak, ingen points.

;-) Hvorfor kom du så ikke bare med løsningen med det samme?

ok :) jeg tror godt på det der psykokinetiske evner, er du freelance programmør?

du får point så sand ^^

Tak.

Så vil jeg tillade mig at lege synsk også... Mon ikke $brug har været brugt til din brugervalidering i database.php?

$brug = mysql_query("SELECT * FROM online WHERE brugerid>0");

i database.php

mens vi er i gang, en der kan sige mig om man kan lave det her lidt nemmere?

require "counter.php";
$brugerq = mysql_query("SELECT * FROM medlemmer");
$brugere = mysql_num_rows($brugerq);
$onlineq = mysql_query("SELECT * FROM online");
$online = mysql_num_rows($onlineq);
$count = mysql_query("SELECT * FROM counter");
$besogende = mysql_num_rows($count);   
$be = mysql_query("select distinct ip from counter") or die(mysql_error());
$raaa = mysql_num_rows($be);
$brug = mysql_query("SELECT * FROM online WHERE brugerid>0");
$bru = mysql_num_rows($brug);

    $nu = time();
    $month = date("m", $nu);
    $year = date("y", $nu);
    $dag = date("d", $nu);
    $start = mktime(0,0,0,$month,$dag,$year);
   
$daag = mysql_query("SELECT * FROM medlemmer WHERE oprettet>'$start'");
$toda = mysql_num_rows($daag);
$koe = mysql_query("SELECT count(distinct ip) AS antal, tid FROM counter GROUP BY tid ORDER BY antal DESC LIMIT 0,1") or die(mysql_error());
$hoejst = mysql_fetch_array($koe);
$hojst = $hoejst['antal'];
$dat = explode("-", $hoejst['tid']);
$dathj = mktime(0, 0, 0, $dat[1], $dat[2], $dat[0]);
$datj = date("d/m-y", $dathj);

Du kan lave en funktion.

function hentAntal($query){
  $result=mysql_query($query);
  if($result){
    $antal=mysql_num_rows($result);
    mysql_free_result($result);
  } else {
    $antal=false;
  }
  return $antal;
}

Du bruger den således:

$brugere = hentAntal("SELECT * FROM medlemmer");
$online = hentAntal(SELECT * FROM online");

osv.

så sparer jeg mysql_num_rows oder was?

Ja, plus at dine queries bliver afsluttet med det samme, der ikke er brug for dem mere, så serveren ikke spilder hukommelse på at huske dem til dit script afslutter.

ok, thx ^^

Jeg ville nu nok sige
$brugerq = mysql_query("SELECT count(*) as antal FROM medlemmer");
...eller tilsvarende... og så aflæse feltet antal. Min intuition siger at det
er mere effektivt.

ok, hvis jeg har den er $brugerq[antal] så lig med antallet af rækker i medlemmer?

Det er nok rigtigt. Så skal alle queries til det bare skrives om.

Nej, den skal hentes med en af mysql_fetch_*-funktionerne først.

hvad er så fordelen ved at count(*) og fetch, i forhold til en alm ' og num_rows?

Med count(*) skal Mysql ikke til at gøre sig parat til at sende dig alle de rækker, du alligevel ikke gider have, for så ved den, at du bare vil tælle.

ok

Fordelen er at sql-serveren ved at den kun skal give dig een række med eet felt,
og ikke alt, som en "SELECT * FROM ..." lægger op til. Derfor vil det være
muligt for sql-serveren at gøre det hurtigere. Om den reelt gør det mærkbart
hurtigere ved jeg så ikke, men det er god skik ikke at bede om mere end man skal bruge.

ok, hvis man altid kun beder om det man skal bruge, kan man måske mærke det i sidste ende ?

Det vil jeg tro. Uden at have undersøgt nærmere, hvordan MySQL fungerer internt, vil jeg tro, at når man sender en query, vil den klargøre hele resultatet, og derefter sende rækkerne en af gangen, når man beder om dem. Og alt andet lige, vil det nok kunne mærkes på antallet af brugere, som systemet kan holde til, om man beder om hele tabeller eller enkelte tal, når man gør det så tit.

"måske" er et godt ord at vælge, ja.

Men snakker vi store databaser, rigtigt store, så vil både struktur af databasen,
og de tilhørende forespørgsler have afgørende betydning for effektiviteten.

ok så har jeg lige en lille ting mere

jeg vil godt have at hvis man ikke har været logget på i 100 dage, skal "inaktiv" i databasen være '1'.

men jeg tror at det her er en dum idé:

//finder ud af om nogle er inaktive
$mange = 60*60*24*100;
$sidst_set = $nu-$mange
mysql_query("UPDATE medlemmer SET inaktiv='1' WHERE sidst<'sidst_set'");

//finder ud af om nogle ikke er inaktive

mysql_query("UPDATE medlemmer SET inaktiv='0' WHERE sidst>'$sidst_set'");

da det jo løber alle brugere igennem, kan det passe?

"sidst" i tabellen 'medlemmer' er den dato i time() man var online sidst

Ja, de to til sammen, vil update alle medlemmer (Men muligvis har MySQL en optimering, der får den til at ignorere updates, der ikke ændrer noget...)

Du kan rette dem til

mysql_query("UPDATE medlemmer SET inaktiv='1' WHERE sidst<'$sidst_set' and inaktiv=0");

og

mysql_query("UPDATE medlemmer SET inaktiv='0' WHERE sidst>'$sidst_set' and inaktiv=1");

Derudover kan den anden gøres overflødig vedsimpelthen at sætte inaktiv=0, når du alligevel sætter sidst-feltet.

nåh ja, tak for det sand ^^

Hvorfor have et felt, inaktiv, når du kan se det på 'sidst'-feltet.
Det kræver bare at du bruger din SELECT sætning lidt anderledes, eller
laver en lille beregning når du viser status.

Derved vil du også nemmere kunne ændre de 100 dage til noget andet, eller have
en mellemting: lidt inaktiv.

Groft sagt vil du få redundans, dvs. ens og overflødig, information i tabellen,
når 'inaktivt' og 'sidst' har den kobling.

nåh ja... thx erik..!

Her er en ny funktion til dig. Den virker lidt anderledes end den første.

function hentAntal($tabel, $where=''){
  if(!empty($where)){$where="WHERE " . $where;}
  $result=mysql_query("SELECT COUNT(*) FROM $tabel $WHERE");
  if($result){
    $row=mysql_fetch_assoc($result)
    $antal=$row['antal'];
    mysql_free_result($result);
  } else {
    $antal=false;
  }
  return $antal;
}

Den bruges således:

$brugere = hentAntal('medlemmer');

eller

$bru = hentAntal('online','brugerid>0');

Det vil sige, den skal have navnet på tabellen, den skal tælle, og en eventuel betingelse.

Og en rettelse:

function hentAntal($tabel, $where=''){
  if(!empty($where)){$where="WHERE " . $where;}
  $result=mysql_query("SELECT COUNT(*) as antal FROM $tabel $WHERE");
  if($result){
    $row=mysql_fetch_assoc($result)
    $antal=$row['antal'];
    mysql_free_result($result);
  } else {
    $antal=false;
  }
  return $antal;
}

thx

Og en rettelse mere:

function hentAntal($tabel, $where=''){
  if(!empty($where)){$where="WHERE " . $where;}
  $result=mysql_query("SELECT COUNT(*) as antal FROM $tabel $where");
  if($result){
    $row=mysql_fetch_assoc($result)
    $antal=$row['antal'];
    mysql_free_result($result);
  } else {
    $antal=false;
  }
  return $antal;
}

jeg går i seng nu.. men tak for det sand og erik ^^