Dette stykke PHP til evaluering

tarj1974
Antal: 62
Tilmeldt:
30-03-2014
User is offline
Dette stykke PHP til evaluering

Hej Linux Hjerner :)

Jeg har lavet følgende stykke kode i PHP for at generere 20 tomme databaser til MySQL server, kig selv. :)
Det er for at kunne installere Joomla! vha. disse databaser

Det skal senere tilføjes et CSV spreadsheet :)

Og det skal senere laves med udfyldning af form :)

Jeg vil meget gerne have forslag til ændringer :)

Jeg modtager meget gerne krav til uddybninger hvis, nødvendigt :)

Mvh,
Thomas :)

< title>Database creation in PHP

<?php
$servername
= "localhost";
$username = "username";
$password = "password";
$startDB= 0;
$endDB= 20;
$TJDB="prefixDB";
//$spreadsheetCSV="databases.csv";

for($DBnr = $startDB; $DBnr <= $endDB; $DBnr++){

// Create connection
$conn = new mysqli($servername, $username, $password);
// Check connection
if ($conn->connect_error) {
    die(
"Connection failed: " . $conn->connect_error);
}

//$myDB=system("mkpasswd 'date +%N' ");

// Create DATABASE

$sql= "CREATE DATABASE ${TJDB}${DBnr};";

if ($conn->query($sql) === TRUE) {
    echo
"DATABASE created successfully";
} else {
    echo
"Error creating DATABASE: " . $conn->error;
}

// Create USER

$myUsername = system("mkpasswd 'date +%N' ");

$sql="CREATE USER ${myUsername}@localhost;";

if ($conn->query($sql) === TRUE) {
     echo
"USER created successfully";
} else {
    echo
"Error creating USER: " . $conn->error;
}
// Create PASSWORD

$myPassword = system("mkpasswd 'date +%N' ");

$sql="IDENTIFIED BY PASSWORD ${myPassword};";

if ($conn->query($sql) === TRUE) {
    echo
"PASSWORD created successfully";
} else {
    echo
"Error creating PASSWORD: " . $conn->error;
}
// GRANT ALL ON myDB.* TO USERNAME

$sql = "GRANT ALL ON ${TJDB}${myDB}.* TO ${myUsername}@localhost;";

if ($conn->query($sql) === TRUE) {
    echo
"Granting created successfully";
} else {
    echo
"Error creating GRANTING: " . $conn->error;
}
$conn->close();

};
?>


julemand101
julemand101's picture
Antal: 1640
Tilmeldt:
17-01-2004
User is offline
Spørg dig selv: "Virker det

Spørg dig selv: "Virker det for mig?"

[JA] - Fint for dig.
[NEJ] - Hvad er problemet?

Jeg forstår ikke hvad det er vi kan give dig af input til et script hvor ingen af os ved i detaljer hvad for nogle krav du har. Vil du have os til at køre dit script og så se efter om det gør hvad du skriver det skal gøre?

Dit script løser tydeligvis et meget snævert problem som er opstået pga. et projekt du er ved at lave. Hvad har du af interesse i at modtage forslag til ændringer af scriptet som ikke har noget at gøre med det problem du er selv ved at løse? Jeg kunne jo foreslå at scriptet skulle tage imod argumenter for antal databaser der skal oprettes men det kan du ikke bruge til noget når din opgave er kun at oprette 20 databaser. Og en sådan ændring kan jeg hurtigt selv lave til scriptet og bruge det i min egen kontekst.


zity
zity's picture
Antal: 87
Tilmeldt:
11-01-2004
User is offline
Det eneste der lige springer

Det eneste der lige springer mig i øjnene er at du laver en ny sql forbindelse hver gang løkken kører. Det ville måske være smartere at oprette forbindelsen før du kører løkken og lukke den igen efter løkken er færdig (eftersom du forbinder til den samme server hver gang).