Structure conditionnelle

Résolu
LaChaux78 Messages postés 585 Date d'inscription   Statut Membre Dernière intervention   -  
LaChaux78 Messages postés 585 Date d'inscription   Statut Membre Dernière intervention   -
Bonjour,
J'ai souhaité faire une condition avec 3 variables, c'est 3 cases à cocher. Je voudrais vous demander votre aide pour que je puisse simplifier cela. Pour l' INSERT dans ma BDD et dans mon dossier où sont stocker les fichiers cela fonctionne parfaitement, mais je trouve cela très lourd et peu compréhensible. Bien entendu je vous donne uniquement la partie concernée, toutes les variables sont définis en amont.
Je vous remercie.
			$dossier = 'compte_rendu/';
			$extension_valide = array('pdf', 'doc', 'docx');
			$extension_upload = strtolower(  substr(  strrchr($fileName, '.')  ,1)  );
			
			if ($crag AND !$crbur AND !$crfd) {
				$file_new = '' .'AG_'.$datecr. '.' . $extension_upload;
				$file_dest = $dossier.$file_new;
				$file_upload = move_uploaded_file($fileTemp, $file_dest);
			} else if (!$crag AND $crbur AND !$crfd) {
				$file_new = '' .'BUR_'.$datecr. '.' . $extension_upload;
				$file_dest = $dossier.$file_new;
				$file_upload = move_uploaded_file($fileTemp, $file_dest);
			} else if (!$crag AND !$crbur AND $crfd) {
				$file_new = '' .'FD_'.$datecr. '.' . $extension_upload;
				$file_dest = $dossier.$file_new;
				$file_upload = move_uploaded_file($fileTemp, $file_dest);
			} else {
				$erreur = "<span class='erreur'>Sélectionne uniquement une des trois options !</span>";
			}

4 réponses

  1. jee pee Messages postés 9439 Date d'inscription   Statut Modérateur Dernière intervention   9 977
     
    Bonjour,

    Si la seule différence au niveau du test c'est le préfixe, AG_, BUR_ F8 tu peux au niveau de ces tests, créer une nouvelle variable qui contient ce préfixe, et constituer les 3 variables après le test.

    Et plutôt que 3 cases à cocher, si la réponse doit être unique, tu ferais mieux dans ton formulaire de mettre un radio bouton : http://www.lehtml.com/htmlplus/fmradio.htm

    0
    1. LaChaux78 Messages postés 585 Date d'inscription   Statut Membre Dernière intervention   32
       
      Bonjour,
      Merci pour ton retour. Pour les cases à cocher type radio je n'y avait pas pensé. je vais revoir ma structure.
      0
  2. LaChaux78 Messages postés 585 Date d'inscription   Statut Membre Dernière intervention   32
     
    Bonjour,
    Je reviens sur mon problème de bouton radio. En fait dans mon cas je ne peux pas utiliser les boutons radio je crois. Voici la structure de ma table si vous pouvez me dire ce que vous en pensez. Merci
    mysql> show full columns from cdc_compte_rendu;
    +--------------+--------------+-----------------+------+-----+---------+----------------+---------------------------------+------------------------------------+
    | Field        | Type         | Collation       | Null | Key | Default | Extra          | Privileges                      | Comment                            |
    +--------------+--------------+-----------------+------+-----+---------+----------------+---------------------------------+------------------------------------+
    | idcr         | int(11)      | NULL            | NO   | PRI | NULL    | auto_increment | select,insert,update,references | ID Compte rendu                    |
    | compte_rendu | varchar(255) | utf8_general_ci | NO   |     | NULL    |                | select,insert,update,references | Nom du Compte rendu                |
    | datecr       | date         | NULL            | NO   |     | NULL    |                | select,insert,update,references | Date du Compte rendu               |
    | crag         | tinyint(1)   | NULL            | NO   |     | NULL    |                | select,insert,update,references | 1 Compte rendu Assemblée Générale  |
    | crbur        | tinyint(1)   | NULL            | NO   |     | NULL    |                | select,insert,update,references | 1 Compte rendu bureau              |
    | crfd         | tinyint(1)   | NULL            | NO   |     | NULL    |                | select,insert,update,references | 1 Compte rendu Fédération musicale |
    +--------------+--------------+-----------------+------+-----+---------+----------------+---------------------------------+------------------------------------+
    6 rows in set (0.03 sec)
    
    0
  3. jee pee Messages postés 9439 Date d'inscription   Statut Modérateur Dernière intervention   9 977
     
    Les radios boutons remplacent les cases à cocher.

    Tu as un radio bouton et en plus cela initialise le préfixe

    <form method="POST">
        <input type="radio" name="btprefix" value="AG_"/>Option 1 - 
        <input type="radio" name="btprefix" value="BUR_"/>Option 2 - 
        <input type="radio" name="btprefix" value="FD_"/>Option 3 - 
        <input type="submit" value="OK"/>
    </form>
    
    <?php
    print $_POST['btprefix'];
    ?>


    Donc tu peux simplifier tout ton code par (mes lignes sont des exemples car je ne fais pas de php)

    $dossier = 'compte_rendu/';
    $extension_valide = array('pdf', 'doc', 'docx');
    $extension_upload = strtolower(  substr(  strrchr($fileName, '.')  ,1)  );
    $btprix = .... ($_POST['btprefix']; ???)
    
    $file_new = '' .'$btprix.$datecr. '.' . $extension_upload;
    $file_dest = $dossier.$file_new;
    $file_upload = move_uploaded_file($fileTemp, $file_dest); 


    0
    1. jee pee Messages postés 9439 Date d'inscription   Statut Modérateur Dernière intervention   9 977
       
      mais si tu stockes la réponse des cases à cocher ...

      avoir trois champs, ce n'est peut être pas la meilleure solution pour connaitre le type de l'objet
      0
  4. LaChaux78 Messages postés 585 Date d'inscription   Statut Membre Dernière intervention   32
     
    Bonjour
    J'ai trouvé la solution à mon problème en ayant les boutons radio. J'ai souhaité également avoir un prefixe different pour mes 3 variables.
    Voilà ce que ça donne.
    Merci.
    $cr 		= !empty($_POST['cr']) ? $_POST['cr'] : '';
    $crag 		= !empty($_POST['crag']) ? $_POST['crag'] : '';
    $crbur 		= !empty($_POST['crbur']) ? $_POST['crbur'] : '';
    $crfd 		= !empty($_POST['crfd']) ? $_POST['crfd'] : '';
    // On initialise les variables à 0
    $crag 	= 0; 
    $crbur 	= 0; 
    $crfd 	= 0;
    
    // On test le choix du visiteur
    if ($cr == 'crag') 
    $crag = 1; 
    
    if ($cr == 'crbur')
    $crbur = 1;
    
    if ($cr == 'crfd')
    $crfd = 1;
    
    			$dossier = 'compte_rendu/';
    			$extension_valide = array('pdf', 'doc', 'docx');
    			$extension_upload = strtolower(  substr(  strrchr($fileName, '.')  ,1)  );
    		
    			if ($crag)
    				$file_new = '' .'AG_'.$datecr. '.' . $extension_upload;
    
    			if ($crbur)
    				$file_new = '' .'BUR_'.$datecr. '.' . $extension_upload;
    
    			if($crfd)
    				$file_new = '' .'FD_'.$datecr. '.' . $extension_upload;
    
    			$file_dest = $dossier.$file_new;
    			$file_upload = move_uploaded_file($fileTemp, $file_dest);
    
    	echo '<p>';
    		echo '<input type="radio" id="crag" class="css-crag" name="cr" value="crag"/>';
    		echo '<label>Compte rendu Assemblée Générale !</label>';
    	echo '</p>';
    	echo '</tr><tr>';
    	echo '<p>';
    		echo '<input type="radio" id="crbur" class="css-crbur" name="cr" value="crbur"/>';
    		echo '<label>Compte rendu Bureau</label>';
    	echo '</p>';
    	echo '</tr><tr>';
    	echo '<p>';
    		echo '<input type="radio" id="crfd" class="css-crfd" name="cr" value="crfd"/>';
    		echo '<label>Compte rendu Fédération musicale</label>';
    	echo '</p>';
    
    0