[PHP MYSQL] Dalando's SHORT URL script!

Door Dalando De Zuil, 17 jaar geleden, 10.377x 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

  1. shorturl.php

 

Er zijn 15 reacties op 'Php mysql dalandos short url script '

PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen
- Raoul -
- Raoul -
17 jaar geleden
 
0 +1 -0 -1
Kan je gemakkelijk doen met
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
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
}
?>
Dalando De Zuil
Dalando De Zuil
17 jaar geleden
 
0 +1 -0 -1
Quote:
Kan je gemakkelijk doen met

[...]


Hoe waar wat?
Www JdeRuijterNL
www JdeRuijterNL
17 jaar geleden
 
0 +1 -0 -1
mijn website is ineens : http://tinyurldalando.eu5.org/voorbeeld/?u=5645992495 langer geworden.

zo short is dat niet.
Dalando De Zuil
Dalando De Zuil
17 jaar geleden
 
0 +1 -0 -1
Maar dat komt omdat de url van die website ook lang is...
Pim -
Pim -
17 jaar geleden
 
0 +1 -0 -1
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']
Joren de Wit
Joren de Wit
17 jaar geleden
 
0 +1 -0 -1
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.
Joakim Broden
Joakim Broden
17 jaar geleden
 
0 +1 -0 -1
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?
Chris -
Chris -
17 jaar geleden
 
0 +1 -0 -1
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?

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
<input type='hidden' name='gen' value='<?php echo rand(0, 9999999999); ?>' />

Wat is dit voor rommel? Je gebruikt het niet eens..
Dalando De Zuil
Dalando De Zuil
17 jaar geleden
 
0 +1 -0 -1
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...
Chris -
Chris -
17 jaar geleden
 
0 +1 -0 -1
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...
Joren de Wit
Joren de Wit
17 jaar geleden
 
0 +1 -0 -1
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...
Niek s
niek s
17 jaar geleden
 
0 +1 -0 -1
Nog een toevoeging op de 2 bovenstaande replies:
Ik denk dat dat voor meer van je scripts zo is. (lees: tot nu toe alle)
Niels K
Niels K
17 jaar geleden
 
0 +1 -0 -1
Inderdaad, en hoe vaak is het niet gezegd?
- Ariën  -
- Ariën -
17 jaar geleden
 
0 +1 -0 -1
Leuk script verder. :-)
Licht het aan mij of kan je een bestaande short-URL overschrijven met een andere URL? :x
PHP hulp
PHP hulp
0 seconden vanaf nu
 

Gesponsorde koppelingen


17 jaar geleden
 
0 +1 -0 -1
mysql_select_db('$dbname',$db)
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.

Inhoudsopgave

  1. shorturl.php

Labels

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.