CLN: Added doc and fixed variable names.

Author: asdf8601

pandas/core/generic.py

@@ -8763,21 +8763,35 @@ def transform(self, func, *args, **kwargs):
         scalar : type of index
         """
 
-    def _find_first_valid(self, direction=1):
+    def _find_first_valid(self, reversed=False):
+        """Retrieves the index of the first valid value.
+
+        Parameters
+        ----------
+        reversed : bool
+            If you want invert the original data order (axis=0). Useful to find
+            the last valid value.
+
+        Returns
+        -------
+        idx_first_valid : type of index
+        """
         if len(self) == 0:  # early stop
             return None
         is_valid = ~self.isna()
 
         if self.ndim == 2:
             is_valid = is_valid.any(1)  # reduce axis 1
 
-        if direction == 1:
+        if not reversed:
+            # first valid value case
             i = is_valid.idxmax()
             if not is_valid[i]:
                 return None
             else:
                 return i
-        elif direction == -1:
+        else:
+            # last valid value case
             i = is_valid.values[::-1].argmax()
             if not is_valid.iat[len(self) - i - 1]:
                 return None
@@ -8787,12 +8801,12 @@ def _find_first_valid(self, direction=1):
     @Appender(_shared_docs['valid_index'] % {'position': 'first',
                                              'klass': 'NDFrame'})
     def first_valid_index(self):
-        return self._find_first_valid(1)
+        return self._find_first_valid()
 
     @Appender(_shared_docs['valid_index'] % {'position': 'last',
                                              'klass': 'NDFrame'})
     def last_valid_index(self):
-        return self._find_first_valid(-1)
+        return self._find_first_valid(True)
 
 
 def _doc_parms(cls):