pylint valideringsfel #46

Activity

1. AndreasArne

   added

   FAQInformationInformation

   on May 31, 2021
2. AndreasArne commented on May 31, 2021

   

   AndreasArne

   on May 31, 2021

   Last edited by AndreasArne

   MemberAuthor

   More actions

   Länkar till egna förklaring av fel:

   • line-to-long
3. AndreasArne

   pinned this issue on Sep 6, 2021
4. AndreasArne commented on Oct 6, 2022

   

   AndreasArne

   on Oct 6, 2022

   MemberAuthor

   More actions

   C0123: Using type() instead of isinstance() for a typecheck. (unidiomatic-typecheck)
   Svar på varför vi ska använda isinstance istället för type:

   Skillnaden är egentligen inget vi har nytta av i denna (python) kursen, men är något som är relevant i den objektorienterade kursen efter jul. Efter jul kommer vi att lära oss att skapa egna klasser/datatyper och då kan vi göra en klass som bygger på en annan. Tänk, om vi har en klass som heter Animal och sen skapar vi en klass som heter Dog och Dog bygger/ärver Animal. Om vi då använder type funktionen när vi vill jämföra data typer då kommer vi få False om vi jämför en Dog med en Animal. Men oftast vill vi att det ska blir True, för att Dog bygger på Animal.
   Numera är rekommenderat att man bara använder sig av isinstance() för att den gör samma sak och mer än vad type() gör.
5. AndreasArne commented on Oct 6, 2022

   

   AndreasArne

   on Oct 6, 2022

   MemberAuthor

   More actions

   C0116: Missing function or method docstring (missing-function-docstring)
   Alla funktioner behöver innehålle en docstring där du förklarar vad koden som funktionen innehåller gör.

   def a_func():
       """
       Detta är en docstring som förklarar vad funktionen gör
       """
       pass
   
6. AndreasArne commented on Oct 6, 2022

   

   AndreasArne

   on Oct 6, 2022

   MemberAuthor

   More actions

   R1710: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)

   Är vad den säger, om du har en funktion som innehåller en if-sats med elif och else, och du i något av de if-blocken har en return. Då borde du ha en return i alla if-blocken. Annars bör du skriva om koden så du inte har return i något if-block utan du har en return i slutet av funktionen.

   T.ex.

   def a_function(in_value):
       if isinstance(in_value, int):
           return in_value * in_value
   
       if isinstance(in_value, str): 
           return in_value + "hej"

   Här har vi ingen return om båda if-satserna blir False. En lösning som typ alltid funkar är att skriva om koden så den bara innehåller en return och den ligger sist i funktionen.

       result = ""
       if isinstance(in_value, int):
           result = in_value * in_value
   
       if isinstance(in_value, str): 
           result = in_value + "hej"
       return result
7. AndreasArne commented on Jun 16, 2023

   

   AndreasArne

   on Jun 16, 2023

   MemberAuthor

   More actions

   R1703 (simplifiable-if-statement)

   Exempel på felaktigt kod,

   is_true = False
   
   if bar > baz:
       is_true= True
   else:
       is_true = False

   Vi använder if-satsen för att kolla om något är sant eller falskt. Om det är sant sätter vi en variabel till sant, om det är falskt sätter vi variabeln till falskt. Här behövs inte if-satsen, den är onödig. Det räcker med att bara använda jämförelse operatorn och tilldela vad den returnerar till variabeln.

   Rätt lösning

   is_true = bar > baz

   Raden ovanför gör samma sak som hela if-satsen och är bättre lösning.