Invert nil checks to improve performance

Check for `nil` before checking for more expensive conditions which
include a method call or an array scan.

Also removes redundant comments on self-explanatory code

1 files changed, 1 insertions, 1 deletions

diff --git a/lib/axlsx/workbook/worksheet/cell_serializer.rb b/lib/axlsx/workbook/worksheet/cell_serializer.rb
index 30a2b5fe..9075c9c3 100644
--- a/lib/axlsx/workbook/worksheet/cell_serializer.rb
+++ b/lib/axlsx/workbook/worksheet/cell_serializer.rb
@@ -27,7 +27,7 @@ module Axlsx
         if cell.is_text_run?
           valid = RichTextRun::INLINE_STYLES - [:value, :type]
           data = Axlsx.instance_values_for(cell).transform_keys(&:to_sym)
-          data = data.select { |key, value| valid.include?(key) && !value.nil? }
+          data = data.select { |key, value| !value.nil? && valid.include?(key) }
           RichText.new(cell.value.to_s, data).to_xml_string(str)
         elsif cell.contains_rich_text?
           cell.value.to_xml_string(str)