Versio

Wedstrijden class

Dit is mijn eerste class die ik gemaakt heb, bedankt aan iedereen van phphulp die mij er mee geholpen heeft.

het is dus een class om de wedstrijden van je sportclub mee te tonen.
je kan er een heel lijstje mee maken, een enkele wedstrijd tonen, en de verslagen ervan tonen. Je kan de wedstrijden selecteren met 1of meerdere limiten en bijhorende 'WHERE's

het is wel met een database erbij.
hieronder de tabellen;
CREATE TABLE wedstrijden (
id int(11) NOT NULL auto_increment,
datum datetime NOT NULL default '0000-00-00 00:00:00',
pid int(11) NOT NULL default '0',
tegenstander varchar(255) NOT NULL default '',
locatie enum('Thuis','Verplaatsing') NOT NULL default 'Thuis',
score varchar(9) default NULL,
info varchar(255) default NULL,
PRIMARY KEY (id)
) TYPE=MyISAM;

CREATE TABLE ploegen (
id int(11) NOT NULL default '0',
naam varchar(50) NOT NULL default '',
klassement text NOT NULL,
foto varchar(11) NOT NULL default 'images/geen-foto.jpg',
FULLTEXT KEY naam (naam)
) TYPE=MyISAM;

CREATE TABLE verslagen (
id int(11) NOT NULL auto_increment,
mid int(11) NOT NULL default '0',
wid int(11) NOT NULL default '0',
datum datetime default NULL,
onderwerp tinytext,
verslag longtext NOT NULL,
ip varchar(15) NOT NULL default '',
PRIMARY KEY (id),
FULLTEXT KEY onderwerp (onderwerp)
) TYPE=MyISAM PACK_KEYS=0;

Voorbeeld: http://bbczedelgem.be/wedstrijden/

Gesponsorde koppelingen

BHosted Hosting al vanaf € 1,- per maand

Controleer nu gratis jouw domeinnaam:

  

PHP script bestanden

  1. wedstrijden-class

 

18 reacties op 'Wedstrijden class'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen


7 jaar geleden
 
0 +1 -0 -1
Ziet er prima uit.

Het enige wat me zo gauw opvalt is dat je een beetje overbodige syntax bij destroy gebruikt, je roept die functie toch niet aan zo lang je dat niet wilt, dus hoef je binnen de functie niet nog een extra controle te houden.
Hipska BE
Hipska BE
7 jaar geleden
 
0 +1 -0 -1
ah ja, dat komt omdat ik niet echt wist hoe een destroy te gebruiken.

ik doe het wel even weg
edit: Done!
Pim Vernooij
Pim Vernooij
7 jaar geleden
 
0 +1 -0 -1
als je je object wilt legen, kan je toch gewoon dit doen:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?php unlink( $object ); ?>
daar hoef je geen aparte funtie voor te schrijven


7 jaar geleden
 
0 +1 -0 -1
Ziet er professioneel uit. Mooi enkele quotes gebruikt.
Goede inspringing. Wel een tip voor de volgende keer met classes en html: Gebruik templates (of in Word taal 'Sjablonen').
Steff an
Steff an
7 jaar geleden
 
0 +1 -0 -1
Heb (nog) geen commentaar :)

@sebastiaan
Quote:
Gebruik templates (of in Word taal 'Sjablonen').


Heb je een betere vertaling voor 'template'? En, nee... 'malplaatje' is geen betere vertaling.
Hipska BE
Hipska BE
7 jaar geleden
 
0 +1 -0 -1
waarom dan die template? wat is het en wat doet het?
Eric Cartman
Eric Cartman
7 jaar geleden
 
0 +1 -0 -1
Een klasse hoort toch geen HTML te outputten? Zo kun je hem niet in ieder project dat je wilt gebruiken inzetten en is de functie van een klasse er zegmaar een beetje af. Alleen dit stukje:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
    function destroy($destroy){
        if($destroy){
            unset($this->result,$this->rows,$this->lijst,$this->wedstrijd,$this->verslag);
        }
    }

Waarom kijk je of $destroy bestaat terwijl het een verplichte parameter is? Bovendien zit er geen else aan vast, dus is die if in twee opzichten niet nodig...

Verder raad ik je aan om meer commentaar te gebruiken, ook in klasse.
Jelmer rrrr
Jelmer rrrr
7 jaar geleden
 
0 +1 -0 -1
$destroy moet true zijn, als een soort extra beveiliging. destroy() alleen aanroepen is niet goed genoeg, het moet dus destroy(true) zijn. Denk ik.

En ik denk dat er niets mis is met een tabel genereren in een klasse, het is best handig. Maar zorg er dan wel voor dat die tabel geen enkele opmaak bevat, misschien alleen wat class-attributen, zodat je hem kan opmaken met een stylesheet. Kan je hem overal weer gebruiken. Je zet dat soort info immers altijd in een tabel, nooit ergens anders in... En dan is er hopelijk wel een equivalente functie die gewoon de waarden in een array terug geeft...
Hipska BE
Hipska BE
7 jaar geleden
 
0 +1 -0 -1
Ja jelmer heeft volkomen gelijk;

met de functie toon_wedstrijd kan je je een array krijgen van de waarden apart. Je kan hem ook in een while lus steken net als bij toon_verslag bij voorbeeld1

Edit: even een voorbeeldje gemaakt zonder html uitvoer voor een lijst wedstrijden. Zie voorbeeld3
PHP erik
PHP erik
7 jaar geleden
 
0 +1 -0 -1
Quote:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<?php unlink( $object ); ?>
Dit moet unset zijn, niet unlink.
Hipska BE
Hipska BE
7 jaar geleden
 
0 +1 -0 -1
zijn er nog mensen die een site maken voor een sportclub/vereniging?

ik zoek namelijk inspiratie voor een vernieuwend script ;-) welke scripts hebben jullie zoal?
Niek Kasius
Niek Kasius
7 jaar geleden
 
0 +1 -0 -1
wat moet dit voor stellen in Create Table: wedstrijden: locatie enum('Thuis','Verplaatsing') NOT NULL default 'Thuis',
krijg dit als melding:
Fout
SQL-query:

CREATE TABLE `wedstrijden` (

`id` INT( 11 ) NOT NULL AUTO_INCREMENT PRIMARY KEY ,
`datum` DATETIME NOT NULL DEFAULT '0000-00-00 00:00:00',
`pid` INT( 11 ) NOT NULL DEFAULT '0',
`tegenstander` VARCHAR( 255 ) NOT NULL DEFAULT '"',
`locatie` ENUM( Thuis, Verplaatsing ) NOT NULL DEFAULT 'Thuis',
`score` VARCHAR( 9 ) NULL ,
`info` VARCHAR( 255 ) NULL
) ENGINE = MYISAM

MySQL retourneerde:

#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Thuis, Verplaatsing) NOT NULL DEFAULT 'Thuis', `score` VARCHAR(9) NULL, `info` V' at line 1

of is het soms de bedoeling dat het zo moet zijn:
locatie enum('Thuis','Verplaatsing') NOT NULL default 'Thuis'
Thuis enum NOT NULL default 'Thuis'
Verplaatsing enum NOT NULL default 'Thuis'
Stijn Defieuw
Stijn Defieuw
7 jaar geleden
 
0 +1 -0 -1
Waar zijn de andere pagina's naar toe (zoals ploegen etc?)?
Hipska BE
Hipska BE
7 jaar geleden
 
0 +1 -0 -1
@Stijn: dat zijn pagina's die op mijn voorbeeld server staan. maar als je de query wilt zien, dan lijkt die heel goed op die van voorbeeld3
Stijn Defieuw
Stijn Defieuw
7 jaar geleden
 
0 +1 -0 -1
require('connect.php');

Ik zie nergens de pagina connect...
Niek Kasius
Niek Kasius
6 jaar geleden
 
0 +1 -0 -1
Leuk geprobeerd dat script zal ook wel werken!
maar een onvolledig wat heb je daar aan, want ik zal toch ergens clubs en standen moeten in voegen, maar ik vraag me af waar?
Jan geen
Jan geen
6 jaar geleden
 
0 +1 -0 -1
jammer dat je dan weer geen gebruik maakt van een database class...
Mike van Dongen
Mike van Dongen
6 jaar geleden
 
0 +1 -0 -1
@Eric:
Dat betekend verkeerde DB gegevens ingevoerd.
Die kun je veranderen in connect.php

Om te reageren heb je een account nodig en je moet ingelogd zijn.

  • Details
  • Hipska BE
    Door:
    Hipska BE
  • 7 jaar geleden
  • 584 x bekeken
  • Labels
  • Geen tags toegevoegd.