Oprettet søn. d. 01. juni 2008 kl. 22:28:56

30 point uddelt | 22 kommentarer
bigtime
bigtime (7.250 point. Point ude: 60)

Forkorte den her script kode

Hey
kan man ikke forkorte den her kode så den ikke er så lang ?
jeg kan forstille mig der er noget kode der ikke skal bruges
<?php


include("database.php");



$sum = 2; // Antal billeder ved siden af hinanden


$query = mysql_query("SELECT * FROM reklame ORDER BY RAND() LIMIT 3 ") or die(mysql_error());

print "<table class='top22'><tr>";

while($row = mysql_fetch_array($query)) {

    echo "<td width=250 height='60'><a href=\"$row[http]\" target=_blank> $row[alt]<img src=\"small/$row[navn]\" height=60 width=250 border=1 onMouseUp=\"className='visited'\" alt=\"$row[alt]\"></a></td>";

    if ($sumcount == $sum) {

        $sumcount = 0;

        echo "</tr><tr>";

    }

    $sumcount++;
}



echo "</td></tr></table>";

?>
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:32:54| #1

jakobdo
jakobdo (180.847 point)
www.simitu.dk
Al kode bliver brugt.
Du kan evt. ændre en linje eller 2, men ikke fjerne noget, med mindre andet tilføjes.
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:35:28| #2

w13
w13 (109.551 point)
www.2biaz.dk
$sum = 2; // Antal billeder ved siden af hinanden

kan jo spares væk, hvis du sætter:

    if ($sumcount == $sum) {
til:
    if ($sumcount == 2) {

Men det vil selvfølgelig gøre koden sværere at rette i.
Ellers bliver den vist ikke meget kortere nej. Hvorfor vil du have den kortere?
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:36:33| #3

bigtime
bigtime (7.250 point)
tænkte det nok
men hvad med den her kan den ikke fjernes for i mine øjne bliver den ikke brugt til noget  $sum = 2;

jeg har jo den her i stedet for
$query = mysql_query("SELECT * FROM reklame ORDER BY RAND() LIMIT 3 ") or die(mysql_error());
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:37:57| #4

bigtime
bigtime (7.250 point)
jo ville bare have den lidt mere overskulig og for det meste når ting bliver mindre bliver det også mere over skuligt :D
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:38:51| #5

w13
w13 (109.551 point)
www.2biaz.dk
$sum = 2;
bliver jo brugt her:
    if ($sumcount == $sum) {

Men som før nævnt, kan du jo bare skrive 2-tallet direkte ind der.
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:39:41| #6

olebole
olebole (214.028 point)
<ole>

bigtime >> Nej, meget ofte er en kompakt kode langt vanskeligere at læse  =)

/mvh
</bole>
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:40:05| #7

w13
w13 (109.551 point)
www.2biaz.dk
Skal:
  className='visited'
i øvrigt ikke være:
  this.className='visited'
?
Og evt.:
  this.className=this.class='visited'
hvis det skal virke i flere browsere.
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:40:53| #8

w13
w13 (109.551 point)
www.2biaz.dk
Ja, din kode bliver ikke meget bedre. =) Hvis dette er din mest komplekse kode, så har du en temmelig overskuelig kodestruktur på din side. ;)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:42:17| #9

olebole
olebole (214.028 point)
w13 >> Nej, der er ingen browsere, der kan bruge this.class="bla" til noget. Problemet opstår først, hvis du bruger set-/getAttribute  ;o)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:45:43| #10

olebole
olebole (214.028 point)
- og for at være mere specifik, så forstår alle browsere ELEMENT.className="bla", men IE forstår ikke ELEMENT.setAttribute("class", "bla")  =)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:46:18| #11

olebole
olebole (214.028 point)
... på trods af, at den sidste er valid
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:49:40| #12

w13
w13 (109.551 point)
www.2biaz.dk
olebole>> Sorry, ja. Den smuttede lige. :)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet søn. d. 01. juni 2008 kl. 22:55:31| #13

bigtime
bigtime (7.250 point)
jeg siger bare mange tak for det :D
det er altid godt der er nogen at spørge når man er i trivel :D
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet man. d. 02. juni 2008 kl. 00:06:13| #14

olebole
olebole (214.028 point)
Et lille eksempel på en kompakt, men ikke synderlig læselig kode:

<script type="text/JavaScript">
var isGIF = document.getElementById("menu").getElementsByTagName("img")[0].split().pop().toLowerCase().indexOf("gif")!=-1;
alert(isGIF)
</script>

Jeg ville klart foretrække 1000 linjer udfoldet kode, end 300 af den slags kompakte linjer  ;o)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet man. d. 02. juni 2008 kl. 00:07:07| #15

olebole
olebole (214.028 point)
Ups ... jeg manglede et tegn at splitte over  =)

var isGIF = document.getElementById("menu").getElementsByTagName("img")[0].split(".").pop().toLowerCase().indexOf("gif")!=-1;
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet man. d. 02. juni 2008 kl. 00:09:55| #16

olebole
olebole (214.028 point)
- og 'isGIF' er en boolean (true eller false), der tager værdi alt efter, om det første billede i elementet med id="menu" har extension'en 'gif' eller 'GIF'  =)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet man. d. 02. juni 2008 kl. 00:11:34| #17

olebole
olebole (214.028 point)
- og så manglede der dæleme også et lighedstegn:

var isGIF == document.getElementById("menu").getElementsByTagName("img")[0].split(".").pop().toLowerCase().indexOf("gif")!=-1;

Der kan du se. Jeg kan zq ikke engang finde ud af at skrive koden!  :D
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet man. d. 02. juni 2008 kl. 08:33:26| #18

jakobdo
jakobdo (180.847 point)
www.simitu.dk
olebole: Du drikker for meget kaffe! :o)
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet man. d. 02. juni 2008 kl. 22:17:45| #19

olebole
olebole (214.028 point)
- tværtimod ... derfor!  ;D
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet ons. d. 01. september 2010 kl. 19:47:56| #20

Accepteret svar!15 point tildelt
bigtime
bigtime (7.250 point)
kom med nogen svar for tror det er lukke tid
Anmeld misbrug
Skriv et svar | Skriv en kommentar

Skrevet tor. d. 02. september 2010 kl. 08:18:56| #21

Accepteret svar!15 point tildelt

Skrevet fre. d. 03. september 2010 kl. 10:57:54| #22

Skriv et indlæg




Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] [img]link til billede[/img]
Web- og emailadresser omdannes automatisk til links
Se preview
Opret

Log ind

   
Opret bruger | Glemt adgangskode
   

Seneste spørgsmål

Procentregning

Oprettet den 11. februar 2012 kl. 11.26
sevinding giver 60 point for svar | Giv et svar »

Ville ikke tildele at upload det til database eller komme...

Oprettet den 10. februar 2012 kl. 21.23
tobrukDk giver 15 point for svar | Giv et svar »

HTTP Post XML og modtag svar

Oprettet den 10. februar 2012 kl. 17.55
mrkaynex giver 200 point for svar | Giv et svar »

Seneste guides

Installer win 7
9. februar 2012 kl. 14.11 | Læs »
Installation af Apache2, PHP5, MySQL5, Phpmyadmin og...
8. februar 2012 kl. 20.05 | Læs »
Den hurtige og effektive metoder til at knække...
4. februar 2012 kl. 23.16 | Læs »
Guide til usb internet også kaldet mobilt bredbånd
4. februar 2012 kl. 22.33 | Læs »
Den gode bruger
30. januar 2012 kl. 04.45 | Læs »
   

Seneste nyheder

Måske snart slut med Androids helt store problem
10. februar 2012 kl. 13.40 | Læs »
Her er fem sjove danske websider du skal kende
10. februar 2012 kl. 13.31 | Læs »
NemId sprængt: Hackere bryder ind i Danske Bank
10. februar 2012 kl. 13.06 | Læs »
Danmark kigger langt efter musikbranchens guldkalv
10. februar 2012 kl. 12.20 | Læs »
Nu kan du snart hente Windows 8
10. februar 2012 kl. 11.41 | Læs »

Tips & Tricks fra PC World

Teaser billede

Her er fem sjove danske websider du skal kende

Trænger dine lattermuskler til en omgang fitness på dansk? Vi viser vej til fem websider fyldt med humor og vanvittig satire.

Læs mere »

Anmeldelser fra PC World

Teaser billede

Test: Denne super-tablet er iPads hårdeste konkurrent

Eee Pad Transformer Prime er frygtindgydende med sin quadcore processor og evne til at trylle sig om til bærbar. Apple bør kigge i bagspejlet, for Asus' tablet-pc kommer buldrende - og gør det...

Læs mere »

Seneste blogindlæg

Teaser billede

Tvangslukke spørgsmål: Hvad er den bedste løsning?

Hej Vi har mange åbne spørgsmål på Eksperten. Vi ville gerne tvangslukke dem - så et spørgsmål efter f.eks. 6 måneder lukkes. Men der er et par uklarheder som ville være gode at få lidt input til:...

Læs mere »

Nyheder fra PC World

Teaser billede

Nu kan du snart hente Windows 8

Den nye offentlige betaversion af Windows 8 er klar i denne måned.

Læs mere »

Nyheder fra Computerworld

Teaser billede

Måske snart slut med Androids helt store problem

Android-platformen har længe været plaget af et særligt problem. Men måske er problemet nu ved at være elimineret.

Læs mere »

Kurser
Lederuddannelse for it medarbejdere
Word kursus 2007-2010 VBA Programmering
Adobe Photoshop CS4-CS5 Udvidet
Microsoft Certified Technology Specialist, Sharepoint 2010, for ledige
Microsoft Certified Systems Administrator for ledige
Cisco Certified Network Associate (CCNA) for ledige
Lær at anvende WordPress
Microsoft Certified IT Professional (Windows 7) for ledige
Adobe Photoshop CS4-CS5 Grundlæggende
Internet Explorer Grundlæggende
Alle kurser »
Samarbejdspartnere
Ferienhäuser in Dänemark.
Webhotel
Bærbar
Computer
Digital TV fra Viasat
VM fodbold
Udgiver · © 2012 IDG Danmark A/S · Hørkær 18 · 2730 Herlev · Tlf.: 77 300 300 · Fax: 77 300 301 · Brug af personoplysninger