{"id":556,"date":"2011-04-06T08:51:08","date_gmt":"2011-04-06T06:51:08","guid":{"rendered":"https:\/\/d-mueller.de\/blog\/?p=556"},"modified":"2011-04-06T08:56:17","modified_gmt":"2011-04-06T06:56:17","slug":"aufteilen-von-funktionen-doppelte-validierung","status":"publish","type":"post","link":"https:\/\/d-mueller.de\/blog\/aufteilen-von-funktionen-doppelte-validierung\/","title":{"rendered":"Aufteilen von Funktionen = doppelte Validierung"},"content":{"rendered":"

F\u00fcr ein kleines Nebenprojekt bin ich am Werkeln mit codeigniter<\/a>. Soweit so gut. Ich brauchte eine Funktion, die den Bildupload und anschlie\u00dfendes resizing erledigte. Kein Problem, dank Upload-Klasse<\/a> und Image Manipulation-Klasse<\/a>. Okay, das hier soll aber keine Lobhymne auf ein Framework werden, sondern ein allgemeines Thema ansprechen.<\/p>\n

Ich hab also meine Funktion, die upload und<\/b> resize erledigt. Wens interessiert:<\/p>\n

\r\npublic function process_image($cat_id)\r\n{\r\n\t\/\/----------------------------------\r\n\t\/\/Upload\r\n\t\r\n\tif (intval($cat_id) <= 0)\r\n\t\tthrow new Exception("Invalid Video ID");\r\n\t\r\n\t$upload_config = array\r\n\t(\r\n\t\t'upload_path' => IMAGES_PATH,\r\n\t\t'allowed_types' => 'gif|jpg|png|jpeg',\r\n\t\t'overwrite' => true\r\n\t);\r\n\t\r\n\tif (!is_dir(IMAGES_PATH))\r\n\t\tthrow new Exception("Could not find Upload directory");\r\n\r\n\t$this->load->library('upload', $upload_config);\r\n\r\n\tif (!$this->upload->do_upload())\r\n\t\tthrow new Exception("Upload Error: " . $this->upload->display_errors()); \r\n\r\n\t$upload_info = $this->upload->data();\r\n\r\n\t\/\/----------------------------------\r\n\t\/\/Resize\r\n\t$thumbname = IMAGES_PATH . 'thumb_' . $cat_id . '_' . uniqid() . "." . $upload_info['image_type'];\r\n\t\t\t\t\r\n\t$resize_config = array\r\n\t(\r\n\t\t'source_image' => $upload_info['full_path'],\r\n\t\t'maintain_ratio' => false,\r\n\t\t'width' => 65,\r\n\t\t'height' => 65,\r\n\t\t'new_image' => $thumbname\r\n\t);\r\n\t\r\n\t$this->load->library('image_lib', $resize_config); \r\n\t\r\n\tif (!$this->image_lib->resize())\r\n\t{\r\n\t\t@unlink($upload_info['full_path']);\r\n\t\t\r\n\t\tthrow new Exception("Resize Error: " . $this->image_lib->display_errors());\r\n\t}\r\n\t\r\n\t@unlink($upload_info['full_path']);\r\n\t\r\n\treturn true;\r\n}\r\n<\/pre>\n
Nunja, als Mensch der die OOP Grundprinzipien<\/a> hochhalten m\u00f6chte, schmerzt es nat\u00fcrlich, dass die Funktion sich um Upload und<\/b> Resize k\u00fcmmert. Ich trenne also die Funktionen auf, Wiederverwendbarkeit und so… <\/p>\n
Der Haken an der Sache<\/h2>\n
Dabei passiert nun das, was immer passiert: Ich f\u00fchle mich getrieben, den ganzen Kram neu zu validieren<\/b>. <\/p>\n
\nIst die $cat_id<\/i> noch g\u00fcltig? Gibt es das Verzeichnis IMAGE_PATH noch? Existiert mein zu resizendes Bild und ist es wirklich ein Bild?\n<\/p><\/blockquote>\n
Toll. H\u00e4tte ich die Funktionen nicht aufgetrennt, k\u00f6nnte mich auf all das verlassen. Weil ich jetzt genau wei\u00df, dass ich die Resize-Methode alleine nie in diesem Projekt brauchen werde<\/b>, ist der Refactoring-Aufwand also komplett f\u00fcr die Katz, weil er mir keinen Vorteil bringt. Man kann argumentieren, dass man sp\u00e4ter ja irgendwann nochmal einen isolierten Resize brauchen werde, ja…<\/p>\n
Naja, ich werde wohl zuk\u00fcnftig diesbez\u00fcglich meine Prinzipien ab und an \u00fcber den Haufen werfen, wenn ich in so einem kleinen Projekt gewiss sein kann, dass dadurch kein Nachteil entsteht. Wie l\u00f6st ihr solche Fragen?<\/b><\/p>\n","protected":false},"excerpt":{"rendered":"
F\u00fcr ein kleines Nebenprojekt bin ich am Werkeln mit codeigniter. Soweit so gut. Ich brauchte eine Funktion, die den Bildupload und anschlie\u00dfendes resizing erledigte. Kein Problem, dank Upload-Klasse und Image Manipulation-Klasse. Okay, das hier soll aber keine Lobhymne auf ein … Weiterlesen →<\/span><\/a><\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[4,5,3],"tags":[],"class_list":["post-556","post","type-post","status-publish","format-standard","hentry","category-php","category-software-engineering","category-webdev"],"_links":{"self":[{"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/posts\/556","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/comments?post=556"}],"version-history":[{"count":0,"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/posts\/556\/revisions"}],"wp:attachment":[{"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/media?parent=556"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/categories?post=556"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/d-mueller.de\/blog\/wp-json\/wp\/v2\/tags?post=556"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}