php - How do I check uploaded file is empty? -

-

i attempting validation on uploaded images. when check see if images have been selected , uploaded should return error message if no images have been uploaded. in method returns false.

heres method:

class event{          private $dbh;         private $post_data;           public function __construct($post_data, pdo $dbh){                 $this->dbh = $dbh;                 $this->post_data = array_map('trim', $post_data);          }  public function checkvalidimages(){             $errors = array();              if(empty($this->post_data['event-images'])){                 $errors[] = 'please select @ least 1 image upload.';             }              if(count($errors) > 0){                 return $errors;             }else{                 return false;             }          }

and calling here:

// check images valid         $images = new event($_files, $dbh);         var_dump($imageerrors = $images->checkvalidimages());

the var_dump() returns bool(false).

heres form:

<form name="submit-event" action="submit-event.php" method="post" enctype="multipart/form-data"> <div class="large-12 columns no-padding"> <p>select images event</p><br /> <input type="file" class="right" name="event-images[]" size="50" multiple="multiple" /> </div> </form>

so why method returning false when don't select images.

when html file input left empty, browser still submit name of form element, still entry in $_files array, error code of upload_err_no_file , filename of "".

you should check the error code anyway lots of things can go wrong. validation code becomes like:

$numokayfiles = 0; $numintendedfiles = 0; foreach ($_files['event-images']['error'] $errorcode) {     $numintendedfiles += ($errorcode != upload_err_no_file);     switch ($errorcode) {     case upload_err_ok:         $numokayfiles++;         break;     case upload_err_ini_size:     case upload_err_form_size:         $errors[] = 'your file bigger maximum allowed size.';         break;     case upload_err_no_file:         // ignore         break;     default:         $errors[] = 'a problem occured during file upload.';     } } if ($numintendedfiles == 0) {     $errors[] = 'please select @ least 1 image upload.'; }

Comments

Post a Comment

Popular posts from this blog

c# - How Configure Devart dotConnect for SQLite Code First? -

-
can configure following sample code ? when run following code, error : no entity framework provider found ado.net provider invariant name 'devart.data.sqlite' setting in machine.config <system.data> <dbproviderfactories> <add name="dotconnect sqlite" invariant="devart.data.sqlite" description="devart dotconnect sqlite" type="devart.data.sqlite.sqliteproviderfactory, devart.data.sqlite, version=4.6.287.0, culture=neutral, publickeytoken=09af7300eec23701" /> </dbproviderfactories> </system.data> after add below block code in app.config <?xml version="1.0"?> <configuration>   <configsections> <!-- more information on entity framework configuration, visit http://go.microsoft.com/fwlink/?linkid=237468 --> <section name="entityframework" type="system.data.entity.internal.configfile.entityframeworksection, entit...
Read more

java - Copying object fields -

-
what best way of copying classes when need have 2 independent variables? have simple class: public class myclass { boolean = false; string b= "not empty"; } do need make method : assign(myclass data ) { a= data.a; b= data.b; } is there automatic method copy (duplicate) objects in java? do need make method : pretty close. instead of making method, should make constructor. such constructors called copy constructor , , create them this: myclass(myclass data) { = data.a; b = data.b; } and create copy of instance, use constructor this: myclass obj1 = new myclass(); myclass obj2 = new myclass(obj1); copy constructor can tedious : using copy constructor create deep-copy can tedious, when class has mutable fields. in case, assignment create copy of reference, , not object itself. have create copy of fields (if want deep copy). can go recursive. a better way create deep copy serialize , deserialize object. why not use clone()? ...
Read more

c++ - Clear the memory after returning a vector in a function -

-
i have function processes , stores lot of data in , returns results vector of class. amount of data stored in function tremendous , want clear storage memory of function after finished job. necessary (does function automatically clear memory) or should clear memory function? update: vector<customers> process(char* const *filename, vector<int> id) { vector<customers> list_of_customers; (perform actions) return list_of_customers; } variables defined locally within function automatically released @ end of function scope. destructors objects called, should free memory allocated objects (if objects coded correctly). example of such object std::vector . anything you've allocated new must have corresponding delete release storage. try avoid doing own allocation , use raii instead, i.e. containers or smart pointers.
Read more