Forum d'entraide PHPFrance

Bonjour,

J'ai un formulaire qui écrie dans une TABLE et qui fonctionne mais je pense que je devrais pouvoir simplifier mon code

<?php
session_start();
if (!isset($_SESSION['login'])) {
header ('Location: ../index.php');
exit();
}
include "../connect/fc-phpconnect.php";

// ligne 1 du form//

if ($_POST['ref_1'] != '' && $_POST['nom_1'] != '' && $_POST['adr_1'] != '' && $_POST['rem_1'] != '' && $_POST['tech_1'] != '' && $_POST['date_1'] != '' && $_POST['tel_1']) {
$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES ('".mysql_escape_string($_POST['ref_1'])."','".mysql_escape_string($_POST['nom_1'])."','".mysql_escape_string($_POST['adr_1'])."','".mysql_escape_string($_POST['rem_1'])."','".mysql_escape_string($_POST['tech_1'])."','".mysql_escape_string($_POST['date_1'])."','".mysql_escape_string($_POST['tel_1'])."')";
mysql_query($sql)or die('Erreur SQL !'.$sql.''.mysql_error());
}

// ligne 2 fu form//

if ($_POST['ref_2'] != '' && $_POST['nom_2'] != '' && $_POST['adr_2'] != '' && $_POST['rem_2'] != '' && $_POST['tech_2'] != '' && $_POST['date_2'] != '' && $_POST['tel_2']) {
$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES ('".mysql_escape_string($_POST['ref_2'])."','".mysql_escape_string($_POST['nom_2'])."','".mysql_escape_string($_POST['adr_2'])."','".mysql_escape_string($_POST['rem_2'])."','".mysql_escape_string($_POST['tech_2'])."','".mysql_escape_string($_POST['date_2'])."','".mysql_escape_string($_POST['tel_2'])."')";
mysql_query($sql)or die('Erreur SQL !'.$sql.''.mysql_error());
}


// ligne 3 du form//

if ($_POST['ref_3'] != '' && $_POST['nom_3'] != '' && $_POST['adr_3'] != '' && $_POST['rem_3'] != '' && $_POST['tech_3'] != '' && $_POST['date_3'] != '' && $_POST['tel_3']) {
$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES ('".mysql_escape_string($_POST['ref_3'])."','".mysql_escape_string($_POST['nom_3'])."','".mysql_escape_string($_POST['adr_3'])."','".mysql_escape_string($_POST['rem_3'])."','".mysql_escape_string($_POST['tech_3'])."','".mysql_escape_string($_POST['date_3'])."','".mysql_escape_string($_POST['tel_3'])."')";
mysql_query($sql)or die('Erreur SQL !'.$sql.''.mysql_error());
 }

// ligne 4 du form//

if ($_POST['ref_4'] != '' && $_POST['nom_4'] != '' && $_POST['adr_4'] != '' && $_POST['rem_4'] != '' && $_POST['tech_4'] != '' && $_POST['date_4'] != '' && $_POST['tel_4']) {
$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES ('".mysql_escape_string($_POST['ref_4'])."','".mysql_escape_string($_POST['nom_4'])."','".mysql_escape_string($_POST['adr_4'])."','".mysql_escape_string($_POST['rem_4'])."','".mysql_escape_string($_POST['tech_4'])."','".mysql_escape_string($_POST['date_4'])."','".mysql_escape_string($_POST['tel_4'])."')";
mysql_query($sql)or die('Erreur SQL !'.$sql.''.mysql_error());

 }

// ligne 6 du form//

if ($_POST['ref_5'] != '' && $_POST['nom_5'] != '' && $_POST['adr_5'] != '' && $_POST['rem_5'] != '' && $_POST['tech_5'] != '' && $_POST['date_5'] != '' && $_POST['tel_5']) {
$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES ('".mysql_escape_string($_POST['ref_5'])."','".mysql_escape_string($_POST['nom_5'])."','".mysql_escape_string($_POST['adr_5'])."','".mysql_escape_string($_POST['rem_5'])."','".mysql_escape_string($_POST['tech_5'])."','".mysql_escape_string($_POST['date_5'])."','".mysql_escape_string($_POST['tel_5'])."')";
mysql_query($sql)or die('Erreur SQL !'.$sql.''.mysql_error());
 }


//ligne 6 du form//

if ($_POST['ref_6'] != '' && $_POST['nom_6'] != '' && $_POST['adr_6'] != '' && $_POST['rem_6'] != '' && $_POST['tech_6'] != '' && $_POST['date_6'] != '' && $_POST['tel_6']) {
$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES ('".mysql_escape_string($_POST['ref_6'])."','".mysql_escape_string($_POST['nom_6'])."','".mysql_escape_string($_POST['adr_6'])."','".mysql_escape_string($_POST['rem_6'])."','".mysql_escape_string($_POST['tech_6'])."','".mysql_escape_string($_POST['date_6'])."','".mysql_escape_string($_POST['tel_6'])."')";
mysql_query($sql)or die('Erreur SQL !'.$sql.''.mysql_error());
}
mysql_close();
?>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
</head>
<body>
<body background="../img/grd-4px.gif">
<?php
echo "Merci les informations sont enregistr&eacutees.";
?>
<a href="../deconnexion.php"><font color="#FF0000">D&eacuteconnexion</font></a>

</body>
</html>

Ci quelqu'un pouvais me donner sont avis

Merci

Garder cette ligne :

$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES

dans une variable, et l'utiliser à chaque fois, plutôt que de la réécrire.

Tu aurais utilisé des tableaux (nom[1]) au lieu de nom_1, tu aurais pû utiliser une boucle et donc diviser par 6 le nombre de lignes

Garder cette ligne :

$sql="INSERT INTO prestation_juillet_2006 (ref, nom, adr, rem, tech, date, tel) VALUES

dans une variable, et l'utiliser à chaque fois, plutôt que de la réécrire.

Tu aurais utilisé des tableaux (nom[1]) au lieu de nom_1, tu aurais pû utiliser une boucle et donc diviser par 6 le nombre de lignes

Salut
Pour modifier nom_1 par nom[1] c'est trés facile et rapide à faire.
Pourrais tu m'expliquez voir me donnée un exemple concernant la boucle et la $variable ? stp

Merci

Tu réécris six fois la même chose, donc quand le nombre d'itérations est connu, utilise une boucle for :
for ($i = 1; $i <= 6; $i++) {
$ref = $_POST['ref_'.$i];
$nom = $_POST['nom_'.$i];
// tu récupères tes autres vars selon le même principe...
// puis tu fais pour chaque itération ta requête
}

++

Tu réécris six fois la même chose, donc quand le nombre d'itérations est connu, utilise une boucle for :
for ($i = 1; $i <= 6; $i++) {
$ref = $_POST['ref_'.$i];
$nom = $_POST['nom_'.$i];
// tu récupères tes autres vars selon le même principe...
// puis tu fais pour chaque itération ta requête
}

++