Which method is a better way to code?

[gandolf989]

let the logging package decide when to log.

BEGIN
   my_log_package.start_logging (
      'test', 'test', my_log_package.g_no_logging, 
      TRUE, FALSE, 80 );

   FOR i IN 1..10000000
   LOOP
      my_log_package.record_event('TEST','TRUE',
          my_log_package.g_detail_logging);
   END LOOP;
END;
/

or putting an if statement in the code to decide
whether or not to call the logging package.

BEGIN
   my_log_package.start_logging (
      'test', 'test', my_log_package.g_no_logging, 
       TRUE, FALSE, 80 );

   FOR i IN 1..10000000
   LOOP
      IF my_log_package.g_detail_logging <= 
         my_log_package.g_no_logging
      THEN
         my_log_package.record_event('TEST','TRUE',  
            my_log_package.g_detail_logging);
      END IF;
   END LOOP;
END;
/