php - Hjælp til funktion - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

<< < 1 2 > >>

Bruger #15993 @ 30.07.10 10:43

Hej alle,

Jeg prøver at lave en funktion, som henter information fra min db og udskriver det.

Man skal indsætte 2 variabler, for at finde det rette felt i db'en, som den skal hente det fra.
Jeg ved ikke om det er muligt eller om jeg bare har lavet en fejl

Er ikke så stærk i det endnu.

Det som jeg har lavet ser sådan ud og jeg kalder funktionen med f.eks. getSetting("msg", "1");

Kode

function getSetting($name, $field){
	$res_setting = mysql_query("SELECT * FROM settings WHERE name = '$name'") or die(mysql_error());
	$row_setting = mysql_fetch_array($res_setting);
	if ($field == "1") {
		echo $row_setting['field1'];
	}
	elseif ($field == "2") {
		echo $row_setting['field2'];
	}
	elseif ($field == "3") {
		echo $row_setting['field3'];
	}
}

På forhånd tak,
Morten

14 svar postet i denne tråd vises herunder
3 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #6565 @ 30.07.10 11:56

Har du i din db.php en linje ala:

PHP kode

 $conn = mysql_connect('xx','xx','xx');

Hvis ja, så prøv og ændre funktionen

PHP kode

 function getSettings($name, $field = 1) {
   global $conn;
   switch ($field) {
     case 1:
     case 2:
     case 3:
       $sql = "SELECT field".(int)$field." FROM settings WHERE name=".mysql_real_escape_string($name)." LIMIT 1";
     break;
     default:
       return "Error";
     break;
   }
   $res = mysql_query($sql, $conn);
   $row = mysql_fetch_array($res, MYSQL_NUM);
   return $row[0];
 }

Og hvis det heller ikke virker, så prøv et tilfældigt sted at udskrive $conn.

PHP kode

 var_dump($conn);

Det skulle gerne srkive noget i stil med resource og et ID.

Hvis den ikke gør det, så har du ikke forbindelse til databasen.

Bruger #12245 @ 30.07.10 10:55

181

Er det blot en træningsøvelse eller hvad? Ellers er det ikke en ret god måde at lave det på.

Men du kan prøve at udskrive din SQL udtræk med f.eks. var_dump($row_setting);

du kan ligeledes skrive din if-else lidt mere effektivt, med mindre det kun må være bestemte felter som må udskrives..

Kode

function getSetting($name, $field){
	$res_setting = mysql_query("SELECT * FROM settings WHERE name = '$name'") or die(mysql_error());
	$row_setting = mysql_fetch_array($res_setting);

	if($field) {
		echo $row_setting[$field];
	} elseif(count($row_setting) == 0) {
		echo "SQL result returned empty.";
	}
}

Indlæg senest redigeret d. 30.07.2010 10:56 af Bruger #12245

Bruger #5620 @ 30.07.10 11:22

1.500

Dette er mere eller mindre Anders kode skrevet anderledes, brug af funktionen kræver at der er en mysql_connection.

Kode

function getSettings($name, $field = 1) {
  switch ($field) {
    case 1:
    case 2:
    case 3:
      $sql = "SELECT field".$field." FROM settings WHERE name=".mysql_real_escape_string($name)." LIMIT 1";
    break;
    default:
      return "Error";
    break;
  }
  $res = mysql_query($sql);
  $row = mysql_fetch_array($res, MYSQL_NUM);
  return $row[0];
}

Indlæg senest redigeret d. 30.07.2010 11:23 af Bruger #5620

Bruger #12245 @ 30.07.10 10:58

181

BTW, hvis de heder field1 osv felterne skal echo sætningen hede:

Kode

echo $row_setting['field'.$field];

Bruger #6565 @ 30.07.10 10:59

Hej Morten

Er der kun 1 række i din database for hver ting?

Hvis ja så ville jeg nok lave det på følgende måde.

PHP kode

 function getSettings($name, $field = 1) {
     switch ($field) {
         case 1:
             $sql = "SELECT field1 FROM settings WHERE name='%s' LIMIT 1";
             break;
         
         case 2:
             $sql = "SELECT field2 FROM settings WHERE name='%s' LIMIT 1";
             break;
 
         case 3:
             $sql = "SELECT field3 FROM settings WHERE name='%s' LIMIT 1";
             break;
 
         default:
             $sql = "SELECT default FROM settings LIMIT 1";
             break;
     }
 
     $str_query = sprintf($sql, mysql_real_escape_string($name));
     $res = mysql_query($str_query);
     $row = mysql_fetch_array($res, MYSQL_NUM);
 
     return($row[0]);
 }

Bruger #15993 @ 30.07.10 11:01

Hej Frank,

Træningsøvelse? Både og.. Jeg vil gerne blive bedre, men dette er noget jeg skal bruge, så det skal helst fungere optimalt.

Jeg har forsøgt mig med den kode du skrev og kaldt den med
getSetting("msg", "field1");

Så får jeg en blank side frem

Mvh Morten

Bruger #15993 @ 30.07.10 11:10

Hej Anders,

Nu har jeg også forsøgt mig med din kode.
Og så kaldt den med
<? echo getSetting("msg", "1"); ?>

Der bliver ikke udskrevet noget.

Bruger #6565 @ 30.07.10 11:16

Hvordan ser resten af din kode ud?
Har du lavet en forbindelse til databasen?

Bruger #15993 @ 30.07.10 11:33

Jeg har functions.php:

Kode

<?
include("../db.php");

function getSettings($name, $field = 1) {
switch ($field) {
case 1:
$sql = "SELECT field1 FROM settings WHERE name='%s' LIMIT 1";
break;

case 2:
$sql = "SELECT field2 FROM settings WHERE name='%s' LIMIT 1";
break;

case 3:
$sql = "SELECT field3 FROM settings WHERE name='%s' LIMIT 1";
break;

default:
$sql = "SELECT default FROM settings LIMIT 1";
break;
 }
$str_query = sprintf($sql, mysql_real_escape_string($name));
$res = mysql_query($str_query);
$row = mysql_fetch_array($res, MYSQL_NUM);

return($row[0]);
}
?>

Den includer jeg i min index.php:

include("inc/functions.php");
og kalder den med:
<? echo getSetting("msg", "1"); ?>

Bruger #15993 @ 30.07.10 11:59

Nu virker din løsning Anders

funktionens navn var skiftet til getSettings() istedet for getSetting().

Kan jeg godt lave en mysql_close($res); i slutningen af funktionen, og stadig kalde den flere gange på index.php?

Mvh
Morten

<< < 1 2 > >>

Hjælp til funktion

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler