nouveau client main.php en cours

1 files changed, 13 insertions, 7 deletions

diff --git a/src/view/Zenity.php b/src/view/Zenity.php
index a04b794..04c9221 100644
--- a/src/view/Zenity.php
+++ b/src/view/Zenity.php
@@ -13,7 +13,7 @@ abstract class ZenityCmd
         protected $width = 300;
         protected $height = 200; // recalculée en fonction du contenu, vaut au minimum 150
         
-        protected function __construct($text, array $rows = []) // $rows est optionnel
+        protected function __construct(string $text, array $rows = []) // $rows est optionnel
         {
                 $this->text = $text;
                 $this->rows= $rows;
@@ -31,7 +31,7 @@ abstract class ZenityCmd
 
 class ZenityList extends ZenityCmd
 {
-        public function __construct($text, array $rows)
+        public function __construct(string $text, array $rows)
         {
                 $this->command_type = ' --list';
                 parent::__construct($text, $rows);
@@ -39,15 +39,17 @@ class ZenityList extends ZenityCmd
         $this->command .= ' --width=' . $this->width;
                 $this->command .= ' --height=' . $this->height;
                 $this->command .= ' --hide-header'; // ligne inutile, il y a déjà le --text
-                self::one_column_zenity_list($this->rows);
+                self::oneColumnZenityList($this->rows);
         }
         
-        public function set_entries($rows_set) // variable renseignée après la construction
+        // cas ou $this->rows est renseignée après
+        public function setEntries(array $rows_set)
         {
                 $this->rows = $rows_set;
         }
         
-        private function one_column_zenity_list($rows)
+        // cas où on veut une seule colonne
+        private function oneColumnZenityList(array $rows)
         {
                 $output = ' --column=""';
                 foreach($rows as $entry)
@@ -56,6 +58,8 @@ class ZenityList extends ZenityCmd
                 }
                 $this->command .= $output;
         }
+        
+        // cas où on veut plusieurs colonnes, le remplissage est horizontal, oui c'est tordu!
 }
 
 class ZenityQuestion extends ZenityCmd
@@ -70,6 +74,8 @@ class ZenityQuestion extends ZenityCmd
         }
 }
 
+// note: le formulaire renvoie une chaine avecdes pipes | entre les zones de texte (qui peuvent être vides)
+// si on clique sur 'Annuler', renvoie une chaine vide
 class ZenityForms extends ZenityCmd
 {
     public function __construct($text, array $rows)
@@ -77,10 +83,10 @@ class ZenityForms extends ZenityCmd
                 $this->command_type = ' --forms';
                 parent::__construct($text, $rows);
         //$this->height = 80 + count($this->rows) * 25; // à tester, mais devrait produire le rendu attendu
-        self::entries_zenity_forms($this->rows);
+        self::entriesZenityForms($this->rows);
         }
     
-    private function entries_zenity_forms($entries)
+    private function entriesZenityForms($entries)
     {
         $output = '';
                 foreach($entries as $one_entry)