php - Forbedre koden på siden - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php mysqli

Bruger #17136 @ 10.06.13 12:41

Hej

Jeg vil høre om jeg måske kan være med til at forbedre min hjemmeside med hvor hurtig den skal være hvis jeg tilpasser noget kode altså sæt dem i array,

PHP kode

 function alarm_112_indhold()
     {
         if ($stmt = $this->mysqli->prepare('SELECT `melding`, `tidspunkt` FROM `alarm` ORDER BY  `alarm`.`id` DESC LIMIT 10')) { 
             $stmt->execute();
 
             /* Bind resultatet */
             $stmt->bind_result($melding, $tidspunkt);
 
             /* Hent rækker og udskriv data */
             while ($stmt->fetch()) {
             ?>
             <div class="alarm_box">
             <h4>Melding: <?php echo $melding;?></h4>
             <p>Tidspunkt: <?php echo $tidspunkt;?></p>
             </div>
             <?php
             }
 
             /* Luk statement */
             $stmt->close();
 
         } else {
             /* Der er opstået en fejl */
             echo 'Der opstod en fejl i erklæringen: ' . $this->mysqli->error;
         }    
     }

Jeg læse et sted på nette det var f.eks hvis jeg skal bruge alarm flere 1 gange i min kode så er det godt at sæt det op i array, så der efter sådan hente koden ned.

f.eks sådan her

PHP kode

 $select = array("SELECT `melding`, `tidspunkt` FROM `alarm`")
     
     function alarm_112_indhold()
     {
         if ($stmt = $this->mysqli->prepare('$select[0] ORDER BY  `alarm`.`id` DESC LIMIT 10')) { 
             $stmt->execute();
 
             /* Bind resultatet */
             $stmt->bind_result($melding, $tidspunkt);
 
             /* Hent rækker og udskriv data */
             while ($stmt->fetch()) {
             ?>
             <div class="alarm_box">
             <h4>Melding: <?php echo $melding;?></h4>
             <p>Tidspunkt: <?php echo $tidspunkt;?></p>
             </div>
             <?php
             }
 
             /* Luk statement */
             $stmt->close();
 
         } else {
             /* Der er opstået en fejl */
             echo 'Der opstod en fejl i erklæringen: ' . $this->mysqli->error;
         }    
     }

(Ved godt der er fejl i koden / Kan ikke lige se hvad problemet er lige pt

), men det måske en måde at gøre det på eller hvordan vil du gøre det?

Nu er det sådan på den hjemmesider jeg arbejder på er der meget at den samme kode igen igen igen men for hver gang bliver der bare tilføj noget nyt eller ligne.

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

Sorter efter stemmer Sorter efter dato

Bruger #13363 @ 10.06.13 19:37

160

Det der er med dobbelt quotes og single quotes i PHP er, at alt inde i dobbelt quotes bliver tjekket for om det indeholder variabler.
f.eks.

PHP kode

 $var = "hej";
 echo "$var med dig";

Giver

Kode

hej med dig

Med single quotes fortæller man PHP at der ikke skal ledes efter variabler i stregen og at den skal udskrive stregen som den står.

PHP kode

 $var = "hej";
 echo '$var med dig';

Giver

Kode

$var med dig

Hvis du vil vise variabler skal de skrives uden for single quoutes'ene som en tilføjelse.

PHP kode

 $var = "hej";
 echo $var.' med dig';

Giver

Kode

hej med dig

Derfor skal din sql se nogenlunde sådan her ud

PHP kode

 $stmt = $this->mysqli->prepare($select[0].' ORDER BY alarm.id DESC LIMIT 10')

Derudover mangler der vist et ; efter $select = array("SELECT `melding`, `tidspunkt` FROM `alarm`")
Og du behøver ikke at bruge `` uden om de felter du vil hente fra databasen.
Det kan bare skrives som "SELECT melding, tidspunkt FROM alarm".

Indlæg senest redigeret d. 10.06.2013 19:39 af Bruger #13363

Bruger #14645 @ 10.06.13 13:58

806

Nu har jeg ikke været i PHP verdenen for nylig men tror det er fordi du bruger single-quoutes (') her:
if ($stmt = $this->mysqli->prepare('$select[0] ORDER BY `alarm`.`id` DESC LIMIT 10')) {

Hvis du gerne vil have at PHP forstår at hente værdien fra en variabel ind i en streng f.eks. $select skal du bruge dobbelt quotes:
if ($stmt = $this->mysqli->prepare("$select[0] ORDER BY `alarm`.`id` DESC LIMIT 10")) {

Bruger #14645 @ 10.06.13 14:29

806

Du skal stadig have [0] på for at trække strengen ud af din array.

Bruger #17136 @ 10.06.13 14:16

Nu har jeg ikke været i PHP verdenen for nylig men tror det er fordi du bruger single-quoutes (') her:
if ($stmt = $this->mysqli->prepare('$select[0] ORDER BY `alarm`.`id` DESC LIMIT 10')) {

Hvis du gerne vil have at PHP forstår at hente værdien fra en variabel ind i en streng f.eks. $select skal du bruge dobbelt quotes:
if ($stmt = $this->mysqli->prepare("$select[0] ORDER BY `alarm`.`id` DESC LIMIT 10")) {

Sådan her:

PHP kode

 if ($stmt = $this->mysqli->prepare("$select ORDER BY  `alarm`.`id` DESC LIMIT 10"))

Den viser bare ikke den rigtig farve

Bruger #17081 @ 10.06.13 14:28

1.258

det er lige meget, du skal skrive det med " (dobbelt) hvis du vil have PHP til, at forstå det som en variabel

Bruger #17136 @ 10.06.13 17:36

det er lige meget, du skal skrive det med " (dobbelt) hvis du vil have PHP til, at forstå det som en variabel

Forstår ikke lige helt hvad du mener med det

Bruger #16742 @ 10.06.13 17:40

Du kunne lave det til en try-catch exception - ville gerne give et eksempel men sidder i toget.

Bruger #17136 @ 11.06.13 14:00

Det der er med dobbelt quotes og single quotes i PHP er, at alt inde i dobbelt quotes bliver tjekket for om det indeholder variabler.
f.eks.

PHP kode
 $var = "hej";
 echo "$var med dig";
Giver
Kode
hej med dig
Med single quotes fortæller man PHP at der ikke skal ledes efter variabler i stregen og at den skal udskrive stregen som den står.
PHP kode
 $var = "hej";
 echo '$var med dig';
Giver
Kode
$var med dig
Hvis du vil vise variabler skal de skrives uden for single quoutes'ene som en tilføjelse.

PHP kode
 $var = "hej";
 echo $var.' med dig';
Giver
Kode
hej med dig
Derfor skal din sql se nogenlunde sådan her ud
PHP kode
 $stmt = $this->mysqli->prepare($select[0].' ORDER BY alarm.id DESC LIMIT 10')
Derudover mangler der vist et ; efter $select = array("SELECT `melding`, `tidspunkt` FROM `alarm`")
Og du behøver ikke at bruge `` uden om de felter du vil hente fra databasen.
Det kan bare skrives som "SELECT melding, tidspunkt FROM alarm".

Åååh lækkert, nu virker det!. Jeg skulle bare tilføj og ligne nogle ting

men tak for hjælp det virker fint nu!!

Indlæg senest redigeret d. 11.06.2013 15:16 af Bruger #17136

Forbedre koden på siden

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler