Wedstrijden class

Door Hipska BE, 22 jaar geleden, 3.910x bekeken

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

PHP script bestanden

  1. wedstrijden-class

 

Er zijn 18 reacties op 'Wedstrijden class'

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen


22 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
22 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
22 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


22 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
22 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
22 jaar geleden
 
0 +1 -0 -1
waarom dan die template? wat is het en wat doet het?
Eric Cartman
Eric Cartman
22 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 -
Jelmer -
22 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
22 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
22 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
21 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
21 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
21 jaar geleden
 
0 +1 -0 -1
Waar zijn de andere pagina's naar toe (zoals ploegen etc?)?
Hipska BE
Hipska BE
21 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
21 jaar geleden
 
0 +1 -0 -1
require('connect.php');

Ik zie nergens de pagina connect...
Niek Kasius
Niek Kasius
20 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
20 jaar geleden
 
0 +1 -0 -1
jammer dat je dan weer geen gebruik maakt van een database class...
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
Mike van Dongen
Mike van Dongen
20 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.

Inhoudsopgave

  1. wedstrijden-class

Labels

  • Geen tags toegevoegd.

Navigatie

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.