Posts by NielsB

Jopitan, dit stukje code vind ik beetje overbodig:

isset($_FILES['Bestand']) ? $bestand = $_FILES['Bestand'] : $bestand = '';

Schrijf het dan teminste om het manier dat je maar 1 keer variabel aanmaak en dan vult met waarde1 of waarde2

Dus:

$bestand = isset($_FILES['Bestand']) ? $_FILES['Bestand'] : '';

En zo geldt dat ook voor die andere waardes.

Voor de rest je kijkt of die gevuld is en zo ja laat die hem staan en anders vul je hem met niks. Kan je beter daar gelijk een if en else constructie om heen zetten, aangezien je nu dubbele check doet.

if(empty($naam) || !isset($naam) {

In je database zet je een tmp_name in en niet een name en een pad van het bestand, hierdoor krijg je de verkeerde waardes te zien.

Voorbeeld voor het uploaden:

$uploads_dir = '/uploads';
foreach ($_FILES["Bestand"]["error"] as $key => $error) {
    if ($error == UPLOAD_ERR_OK) {
        $tmp_name = $_FILES["Bestand"]["tmp_name"][$key];
        $name = $_FILES["Bestand"]["name"][$key];
        move_uploaded_file($tmp_name, "$uploads_dir/$name");
    }
}

Eerst wordt heel het bestand geopend en doorgelopen, hij zoekt hier naar een error, wanneer deze ok staat zal hij hem uploaden naar de map met het bestand. Je kan dit ook testen zonder je database te gebruiken, zodat je weet of het werkt.

Jopitan geeft ook slim in zijn code de mysql pas vrij als er geupload is, dit is dan ook zeker een tip om te gebruiken .

Sorry Koen maar ik vind het slecht dat je het volgende zegt:

Citaat

Hierdoor kunnen wij jullie garanderen dat als men ooit onze database zou stelen (je weet nooit), dat men jullie wachtwoorden NIET kan ontsleutelen.

je gaat er versteld van staan dat als je heeeeeel lang het gaat doen hij gewoon kan worden gekraakt..

voor de rest waarom geen gebruik van hashes i.p.v sha1 ? En die beveilig je gewoon zo zeer met meer dingen maar dat is logisch.

Maar ik vind het als gebruiker en webdeveloper eigenlijk niet van toepassing om dit soort nieuwtjes te melden. Aangezien je hierdoor dus weer van die mensen gaan krijgen die denken even ddossen of andere dingen..

Wanneer je een wedstrijd zal houden, bijvoorbeeld een script welk opensource is dus voor iedereen, dan kun je kijken wie het nou als beste doet. Maar letop er zijn genoeg mensen met de ervaring en kwaliteiten wie dan niet mee zullen doen.

Overigens heb ik heel vaak hulp gehad van Darsstar en schat ik de kansen in dat hij samen met Killingdevil toch de beste 2 zijn. Dit is gebaseerd in de loop van de jaren, dat ik hier actief was en hulp van ze heb gehad.

Maak gebruik van een onclick op het plaatje en zorg dat je aan de hand van kleine ajax call de php aanroept.

Anders maak je gebruik van paremeter aan een url die kijkt of je een taal mee stuurt en die dan verwerkt.

Jullie lijstjes kloppen voor geen meter..

Is al bijna opgelost.

Het gebruik van zn seo klasse is een handige manier, maar dit hoeft niet persee via een hele seo klasse he Je kan dit gewoon simpel via controller op halen en bijv via een template systeem ( twig ) makkelijk verwerken in je view. Ik raad je ook aan om een template systeem zoals twig te gebruiken, is wel even uitvogelen hoe het werkt maar kan veel voordelen opleveren.

Voor de rest over dat definen, ik gebruik ook defines! Alleen de manier hoe je het hier gebruik vind ik beetje nutteloos. Het voordeel van define is dat je de waarde niet kan aanpassen, bij een variabel kan dit natuurlijk wel. Dus weet niet wat je nu gaat verwerken via een define?

De titel van de website is handig voor een define, aangezien die niet aangepast kan worden. Maar als je de titel van elke pagina in een define gaat verwerken, kan je beter een variabel gebruiken.

Als je verstand van SEO hebt, dan weet je toch echt wel beter dat dit niet werkt met een simpele define!! Jullie zitten namelijk helemaal fout, je hebt namelijk veel meer dingen waar je op moet letten.

@ts, Ik zal je even een goeie tip geven je url die je mee geeft in de url zal je moeten omzetten naar mod rewrite. De titel die in de url komt te staan kan rare tekens bevatten.. Deze moet je namelijk omzetten via een speciale filter zodat hij als gewone url eruit komt en hem dan verwerkt op de pagina met de titel met rare chars.

Seo heeft btw 1 h1 en voor de rest opbouwend met h2 h3 h4 h5. Je gebruikt titles aan een link mee, alts aan een img. Zo heb je dan ook weer het gebruik van divjes en css html netjes scheiden. Ook heb je manieren om een allow te geven via een meta tag, zodat google heel je site gaat bekijken.
Keywords en description in de meta tag zijn bepalend, maar laat dit ook varieren per pagina.

Voor de rest kan de titel gewoon je code echoën als je dit gewoon goed mee stuurt! Maar je kan het beste je pagina functie bovenaan zetten. En dan in de html een simpele echo of include.

zolang jij geen bewijs hebt dat iemand hem officeel heeft vrijgeven, zal je sowieso geen recht hebben om het te verkopen of aan tepassen. Wanneer jij een licentie hebt en daarin staat dat je het mag doorverkopen kan dit.

Wanneer jij een GEHEEL NIEUWE CODE maakt, zullen de rechten van jou zijn! Dit betreft alleen om de scripting en eventuele vormgeving.

wanneer je een primary key + auto_increment doe is het niet eens verstandig om dit te gaan doen, aangezien je niks voor niks een id mee geef!

Denk is even aan je koppeltabellen waar je die id gebruikt, die moet je dan ook veranderen, te veel querys te veel load, website sloom.

gebruik json convert alles naar een array en geef deze via een ajax call door naar javascript, daar krijg je als input dus een json array en die kan je heel simpel uitlezen op zelfde wijze als arrays in php..

Kijk hoeveel resultaten je hebt ( num_rows of count ) daarna een teller mee laten lopen en wanneer hij dan bij het laatste count is de komma weg laten

Voorbeeld:

$test = mysql_query("SELECT * FROM users ORDER BY username ASC");
$count = mysql_num_rows($test);


$i = 0;
while($fetch = mysql_fetch_array($test)) {
echo ($i >= $count) ? $i .'.' : $i .',';
$i++;
}

Nog een tip haal alle h1 tags weg en zet er een onzichtbaar achter de header maffiaminds. dat is namelijk seo vriendelijk en zet als tekst gewoon maffiaminds maar dan wel zo dat je hem niet ziet op de achtegrond, hier kun je text-indent voor gebruiken.

Als je seo vriendelijk wil werken mag je maar 1 heading ( h1 ) tag gebruiken op een website als bijv tekst van de website. H2, h3, h4 mag je zoveel mogelijk gebruiken..

Voor de rest gebruik overal divjes en niet div en tabellen bij elkaar. Het is netter als je overal divjes maakt.

Ik heb het gevoel dat ik niet op een submit veld kan klikken, zet daar een cursor: pointer overheen zodat je weet dat je kan klikken. De input velden zo maken met een onclick dat de tekst weg gaat?

Geef wat meer ruimte tussen de content en de header, voorbeeld hiervan is het kopje criminaliteit en de balkjes erboven, die zit te dicht op elkaar.

Gebruik is een algemene class om de content bok login heen en om errors heen en om recruit etc. zodat je geen <br /> eronder hoeft te doen wat lelijk staat in je code.

Dit is niet echt netjes gesliced, <font color="#54483c"><strong>Helaas je bent opgepakt!</strong> Je moet nu 3 minuten vast zitten in de gevangenis!</font>
Probeer die font gewoon aan een class mee te geven in een span om de tekst.. Gebruik een class om tekst heen waar <strong> staat is netter, en makkelijker aanpasbaar.

Het verloop van inhoud-footer naar footer klopt niet helemaal met het plaatje die opeens stopt.

Volgens mij heb ik nu al best veel puntjes voor je.
Hoop dat je er wat aan hebt, zijn op zich wel puntjes waardoor je je slice netjes maakt.

Toevallig zie ik deze topic en dacht ik even wat puntjes geven, zit namelijk voor de rest nooit meer op deze site door het niveau.

Overigens is je top menu met je naam Frenzo zeer lelijk gemaakt, leer is meer divjes te maken i.p.v fonts mee geven in de code + bij de image staat margin-right etc om heen zet daar een div om heen en geef daar de juiste attributen, dat heet netjes slicen namelijk.

Nadeel van manier van werken wat je nu doet met die if als die gelijk is aan 18:00 dat hij alleen werkt als je dan op dat moment op de site bent.

Beste is dan een cron maken en in directadmin mocht je dit hebben een cronjob laten uitvoeren om 6 uur naar een bepaald bestand dan zal het wel kunnen!

het hele menu gaat niet vanwege het feit dat je van donker naar wit naar grijs gaat. zoek is naar goede kleuren combinaties !

voor de rest een tip maak een wedstrijd voor een mooie template voor cp.

overigens zet de tijd bij de topic is erbij ( menu )

console.log even regel er boven doen, dat kan je als het goed is niet in de innerHTML zetten? en je moet gewoon in firebug kijken wat hij aangeeft dan.

Waarom zou je categorie naam veranderen als je dat via php of html doe?

dan zal je te werk moeten gaan met een get in de url.

profiel.php?id=1

dit kan je opvragen door $_GET['id'] te doen, dan zal hij 1 weergeven.
natuurlijk is dit allemaal nog niet veilig daarom zou je moeten kijken met een intval of het wel een integer is. En daarna in de database kijken of deze bestaat. Zo niet error gebruiker bestaat niet?

Wanneer je dus het id hebt kan je een select maken met als voorbeeld:

"SELECT * FROM gebruiker WHERE id = '". (int) $_GET['id'] ."'

Hier mee haal je gegevens op van een gebruiker.
Je kan dan mysql_fetch_assoc of object of array gebruiken om het te weergeven maar je kan ook nog even een count doen om te kijken of hij wel eerst bestaat.

de volgorde waar je hem in sleept misschien zorgen dat hij hem daar neer zet?
nu wordt hij steeds ergens ingesleept en dan onderin.
zo heb je ook als je hem wil slepen dat je hem links uitgelijnt krijg en niet op de plek waar je hem beet pakt.

Verder haal bij je demo bij de li even de list-item-image weg door het op none te zetten.