[PHP MYSQL] Dalando's SHORT URL script!
Door Dalando De Zuil, 17 jaar geleden, 10.454x bekeken
Al mijn kennis is er in gestoken, met dit als resultaat:
Dalando's SHORT URL script!
tadaa... +)
SQL code zit in de file!
Je moet zelf 6 variables zetten!
achtergrond info:
Ik heb het voornamelijk gemaakt om te oefenen...
Met succes! :P
Voorbeeld: http://tinyurldalando.eu5.org/
Of natuurlijk: http://tinyurldalando.eu5.org/voorbeeld/?u=voorbeeld :P
Komt nog: Kijken of short al bestaat
Gesponsorde koppelingen
PHP script bestanden
Er zijn 15 reacties op 'Php mysql dalandos short url script'
0 
-0 
-0
Kan je gemakkelijk doen met
Code (php)
1
2
3
4
5
6
7
8
2
3
4
5
6
7
8
<?php
$sql = mysql_query('SELECT * FROM codes WHERE id=''.mysql_real_escape_string($codeVar).''');
if (mysql_num_rows($sql) > 0) {
echo 'bestaat al';
}else{
//maken
}
?>
$sql = mysql_query('SELECT * FROM codes WHERE id=''.mysql_real_escape_string($codeVar).''');
if (mysql_num_rows($sql) > 0) {
echo 'bestaat al';
}else{
//maken
}
?>
0 -0
-0
mijn website is ineens : http://tinyurldalando.eu5.org/voorbeeld/?u=5645992495 langer geworden.
zo short is dat niet.
zo short is dat niet.
0 -0
-0
Ik weet niet of je er echt iets mee kan, maar je zou misschien een exploit kunnen maken door een return toe te voegen aan de lange url en extra headers toevoegen. Doe daar even een eenvoudige check bij.
Ook is het mss leuk om ipv ?u=x ?x te doen door $_GET['u'] te vervangen voor $_SERVER['QUERY_STRING']
Ook is het mss leuk om ipv ?u=x ?x te doen door $_GET['u'] te vervangen voor $_SERVER['QUERY_STRING']
0 -0
-0
Een eerste voorwaarde voor dit script om nuttig te zijn, is dat het moet draaien onder een zeer korte domeinnaam (zoals bit.ly bijvoorbeeld).
Verder kun je zoals Pim al zegt de ?u= uit de url laten, dat scheelt je alweer 3 tekens.
Tenslotte is de mogelijkheid om bezoekers zelf een korte versie te kiezen leuk, maar controleer dan wel of de korte url niet al bestaat. Verder vind ik het random nummer dat je nu genereert als geen korte url opgegeven is, veeeeel te lang. Waarom begin je niet gewoon bij 'a' en gebruik je alle alfanumerieke tekens? Dus:
a, b, c, ..., 0, 1, 2
En als je alle enkele tekens gebruikt hebt, ga je over op combinaties:
aa, ab, ..., 89, 99
Kortom, het begin is er maar er zijn nog veel dingen die verbeterd kunnen worden om het echt een volwaardig script te laten zijn.
Verder kun je zoals Pim al zegt de ?u= uit de url laten, dat scheelt je alweer 3 tekens.
Tenslotte is de mogelijkheid om bezoekers zelf een korte versie te kiezen leuk, maar controleer dan wel of de korte url niet al bestaat. Verder vind ik het random nummer dat je nu genereert als geen korte url opgegeven is, veeeeel te lang. Waarom begin je niet gewoon bij 'a' en gebruik je alle alfanumerieke tekens? Dus:
a, b, c, ..., 0, 1, 2
En als je alle enkele tekens gebruikt hebt, ga je over op combinaties:
aa, ab, ..., 89, 99
Kortom, het begin is er maar er zijn nog veel dingen die verbeterd kunnen worden om het echt een volwaardig script te laten zijn.
0 -0
-0
het is irritant dat je perse http:// er voor moet zetten... doe je toch ook niet in de browser? dan ga je toch ook gewoon naar www.google.nl of google.nl?
0 -0
-0
Slecht opgezet naar mijn idee.. Verkeerde database opzet, je had hier nog wat extra dingen in kunnen vermelden voor statistieken bijvoorbeeld. Qua code is het minimaal, maar waarom heb je geen gebruik gemaakt van PDO of MySQLi?
Wat is dit voor rommel? Je gebruikt het niet eens..
Wat is dit voor rommel? Je gebruikt het niet eens..
0 -0
-0
Bedankt. Maar zoals jullie kunnen lezen heb ik het vooral gemaakt om te leren. Ik heb nou even de basic mysql afgerond, omdat ik nu een werkend systeem heb.
Ik zal, als ik daar wat beter in PDO, Sqli, en etc, het verder uitbouwen...
Ik zal, als ik daar wat beter in PDO, Sqli, en etc, het verder uitbouwen...
0 -0
-0
Je hebt het gemaakt om van te leren, maar dat betekend niet meteen dat je het hier in de library hoeft te plaatsen. Op dit moment nemen 'leergierige' mensen dit over en denken ze dat ze een pracht van een systeem hebben. Je kan het imho beter plaatsen nadat het systeem is geoptimaliseerd, goed werkt en een goede opbouw heeft. Dat heeft het op dit moment niet...
0 -0
-0
Vraag jezelf dan af of het in deze staat ook echt meerwaarde heeft voor de scriptlib. Als je de genoemde punten niet aanpast, ben ik van mening dat dat niet het geval is...
0 -0
-0
Nog een toevoeging op de 2 bovenstaande replies:
Ik denk dat dat voor meer van je scripts zo is. (lees: tot nu toe alle)
Ik denk dat dat voor meer van je scripts zo is. (lees: tot nu toe alle)
0 -0
-0
Leuk script verder. :-)
Licht het aan mij of kan je een bestaande short-URL overschrijven met een andere URL? :x
Licht het aan mij of kan je een bestaande short-URL overschrijven met een andere URL? :x
0 -0
-0
mysql_select_db('$dbname',$db)
WTF IS DAT VOOR ONZIN!
Foutcontrole ontbreekt.
Je kunt ook gebruik maken van sql's UNIQUE.
WTF IS DAT VOOR ONZIN!
Foutcontrole ontbreekt.
Je kunt ook gebruik maken van sql's UNIQUE.
Om te reageren heb je een account nodig en je moet ingelogd zijn.
PHP hulp
0 seconden vanaf nu