Ajout d'un caractère après selection des checkbox

Version imprimable

Bonjour à tous,

N'ayant pas eu de réponse sur un autre forum, je viens vous demander de l'aide.
Je vais vous mettre le script la dessus, mais avant je m'explique

Je lis les données d'un fichier texte et les affiches dans un input checkbox pour pouvoir sélectionner ce que je désire modifier en rajoutant "#" au début des lignes sélectionner.
Le soucis, c'est au moment de la modification dans le fichier txt, c'est bien l'ensemble du fichier qui est effacé et provoque une erreur.

Je pense être entièrement en défaut au moment de l'écriture du fichier txt. Pourriez-vous m'aider, svp ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
<?php
   $path = './Fichier_test/';
    $fichier = 'exemple';
    $ext = '.conf';
    $chemin = $path . $fichier . $ext ;
 
 if (file_exists($chemin)) {
 
         $line = file($chemin);
         $ligne_cpt = count($line);
 
}
 
/************** Debut section de controle *************** */
 
if(isset($_POST['env_1'])){  
 
  if(!empty($_POST['valid']))
   {
 
      $valid = $_POST['valid'];
      $op = fopen("$chemin","w");
 
      for ($i=0; $i<count($valid); $i++) 
          {
            $txt   = str_replace('','#', $valid[$i]) ;
            $result = file_put_contents($op, $text);            
            fclose($op);          
 
            echo "num de ligne :" . $valid[$i] ."Fichier modifié !"."<br>";  
          }
    } 
    else
    {
        echo "Vous n'avez rien selectionner, aucune modification a été apportée !";
    }
}
 
/* Fin section de controle  */
/*******************************************/
 
foreach( $line as $lines => $value ) 
             {                 
                 if(!preg_match("[#]", $value)) 
                  {               
  ?>
 
<form action="" method="post">
<div class="elem">
  <i class="fa-solid fa-check ivert"></i>
         <input type="checkbox" name="valid[]" id="" value="<?= $lines ?>"  >
             <label for=""><?= $value ?></label> 
</div>     
 
<?php
      }
    }
?>
<div class="form-button">
    <button type="submit" name="env_1">Modifier les selections</button>
</div>
</form>

Si tu me permets, quelle horreur, il n'y a rien qui va :mouarf:

Le script est mal indenté et dur à lire
Les variables sont mal nommées, ex. foreach( $line as $lines => $value)
file_put_contents() est mal utilisé (on l'utilise avec un nom de fichier, pas un gestionnaire)
<form> est dans le foreach(), mais pas le </form>
etc.

J'ai refait le script en gardant ta logique :
-- On affiche les lignes de f443.conf ne commençant pas par un #
-- On ajoute un # au début des lignes sélectionnées selon leur position dans le fichier

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
<?php
 
// Chargement des données
$file = __DIR__ . '/f443.conf';
$data = @file($file);
if ($data === false) {
    throw new Error('Impossible de lire ' . $file);
}
 
// Traitement des données
if ($_SERVER['REQUEST_METHOD'] === 'POST') { // Formulaire soumis ?
    if (!empty($_POST['rows']) && is_array($_POST['rows'])) {
        foreach ($_POST['rows'] as $i) { // On travaille sur $data
            $data[$i] = '#' . $data[$i];
        }
        file_put_contents($file, implode('', $data)); // Màj du fichier selon $data
    } else {
        $error = 'Vous n\'avez rien sélectionné';
    }
}
 
?>
 
<!-- Gestion de la vue -->
 
<!-- Affichage de $error si besoin -->
<?php if (isset($error)): ?>
    <p><?= $error ?></p>
<?php endif ?>
 
<!-- Affichage de $data -->
<form method="post">
    <?php foreach ($data as $i => $value): ?>
        <?php if (strpos($value, '#') !== 0): ?>
            <label>
                <input type="checkbox" name="rows[]" value="<?= $i ?>">
                <?= $value ?>
            </label><br>
        <?php endif ?>
    <?php endforeach ?>
    <button type="submit">Modifier la sélection</button>
</form>

03/01/2022, 19h50
gesualda

Bonjour SEB,

Alors tout d'abord merci pour ces remarques et surtout d'avoir pris du temps pour moi. Je pense en effet, qu'il faut que je prenne des noms de variables plus appropriées dans mes scripts. (Je vais y penser).
Quand au script, je vais le modifier, le comprendre et je reviens car j'aurai une question supplémentaire.

JLuc
03/01/2022, 20h53
Séb.

Au plaisir ;)

Me voila de retour, avec une question.

Seb, j'ai modifié et etudié le script pour l'adapter à mes tests, comme cela :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<form method="post">
            <fieldset>
                <legend> Cocher et Valider pour de-activer </legend>    
            <?php foreach ($data as $i => $value):?>                
                <?php  if (strpos(ltrim($value), '#') !== 0): ?>
                    <?php  if(trim($value) !== ''): ?>                  
                        <i class="fas fa-square ivert"></i>
                    <label for="">                          
                            <input type="checkbox" name="rows[]" value="<?= $i ?>">                        
                        <?= $value?> 
                    </label><br>
                    <?php endif ?>
                <?php endif ?>                
            <?php endforeach ?>
            <div class="form-button"><button type="submit" name="valid_form_2"><span><i class="far fa-paper-plane"></i>Modifier la sélection</span></button></div>
            </fieldset>
        </form>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
	// Traitement des données
if ($_SERVER['REQUEST_METHOD'] === 'POST') { // Formulaire soumis ?
 
    if(isset($_POST['valid_form_2'])){
     if (!empty($_POST['rows']) && is_array($_POST['rows'])) {
         foreach ($_POST['rows'] as $i) { // On travaille sur $data                
               $data[$i] = '#' . $data[$i];
         }
         file_put_contents($file, implode('', $data)); // Màj du fichier selon $data
        var_dump($_POST['rows']);
     } else {
        $error_vert = 'Vous n\'avez rien sélectionné';
        }
  } //unset($data[$i]);
}

Jusque la, tout à l'air de fonctionner.

Par contre lorsque je rafraichi la page avec F5 ou avec la touche enter, et sans qu'aucune coche ne soit selectionnée, l'écriture se fait à nouveau, mais en supprimant à chaque fois le 1er caractère qui peut etre # ou autre !
ex: ligne 1 => igne 1 => gne 1 (etc...)

J'ai cherché, mais pas trouvé comment empêcher cela. Peut-être une suggestion ?
Merci d'avance

Citation:

Par contre lorsque je rafraichi la page avec F5 ou avec la touche enter, et sans qu'aucune coche ne soit selectionnée, l'écriture se fait à nouveau

C'est normal, F5 rejoue la requête
L'objectif de la requête étant de préfixer les lignes sélectionnées avec un #, on peut donc se retrouver avec autant de # en début de ligne que de refreshes

Pour éviter cela il faut vérifier si la ligne commence déjà par un # :
-- Si oui on ignore la ligne
-- Si non on ajoute un #

Cela pourrait donner :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 // Traitement des données if ($_SERVER['REQUEST_METHOD'] === 'POST') { // Formulaire soumis ? if(isset($_POST['valid_form_2'])){ // 2 form sur la page, donc control de l'envoie if (!empty($_POST['rows']) && is_array($_POST['rows'])) { foreach ($_POST['rows'] as $i) { // On travaille sur $data if (substr($data[$i], 0, 1) === '#') { // Ligne débute déjà par un "#" ? // Si oui, on ne fait rien de spécial } else { // Si non, on préfixe la ligne avec "#" $data[$i] = '#' . $data[$i]; } } file_put_contents($file, implode('', $data)); // Màj du fichier selon $data } else { $error_vert = 'Vous n\'avez rien sélectionné'; } } }
Dans ton affichage il y a des l/trim() indiquant que tu boucles sur un $data qui n'est pas propre
Ça n'a rien à faire là, il faut nettoyer $data lors de son chargement ;)

08/01/2022, 11h32
gesualda

Parfois la solution est tellement simple, que notre esprit cherche la complication (enfin le mien) !8O8O
Je vais tenir compte également de ton approche pour le => ltrim.

Dans la continuité de ce sujet, j'aurai certainement d'autres questions si je trouve pas la bonne formulation.
En attendant et une fois de plus un grand MERCI !