2

無料のオンライン Python チュートリアルについては、次のことを行う必要があります。

指定されたクレジット カード番号が有効かどうかをチェックする関数を作成します。この関数check(S)は、文字列Sを入力として受け取る必要があります。"#### #### #### ####"まず、文字列がそれぞれ#が数字である形式に従っていない場合は、を返す必要がありFalseます。10次に、数字の合計が(「チェックサム」メソッド) で割り切れる場合、プロシージャは を返しTrue、そうでない場合は を返しFalseます。たとえば、Sが文字列の"9384 3495 3297 0123"場合、形式は正しいですが、桁の合計は である72ため、 を返す必要がありますFalse。

以下は、私が思いついたことを示しています。私の論理は正しいと思いますが、なぜ間違った値を与えているのかよくわかりません。私のコードに構造的な問題がありますか、それともメソッドの使い方が間違っていますか?

def check(S): 
    if len(S) != 19 and S[4] != '' and S[9] != '' and S[14] != '':
        return False                # checking if the format is correct

    S = S.replace(" ",'')         # Taking away spaces in the string
    if not S.isdigit():
        return False             # checking that the string has only numbers

    L = []            
    for i in S:
        i = int(i)                # Making a list out of the string and converting each character to an integer so that it the list can be summed 
        L.append(i)
    if sum(L)//10 != 0:        # checking to see if the sum of the list is divisible by 10
        return False
4

3 に答える 3

5

スペースをテストするのではなく、空の文字列のみをテストします。これは、python 文字列でストレート インデックスを使用する場合には決して見つかりません。

Falseさらに、これらの 4 つの条件のいずれかが真である場合に返す必要があります。それらがすべて同時に真である場合ではありません。

if len(S) != 19 or S[4] != ' ' or S[9] != ' ' or S[14] != ' ':
    return False

次に、スペースを置き換えますが、長さは再度チェックしません。19 のスペースを与えたとしたらどうなるでしょうか。

S = S.replace(" ", '')
if len(S) != 16 or not S.isdigit():
    return False

最後に、最初にすべての数字を収集し、残りがあるかどうかを確認します。

L = map(int, S)  # makes S into a sequence of integers in one step
if sum(L) % 10 != 0:  # % calculates the remainder, which should be 0
    return False

これらすべてのテストに合格した場合は、True を返すことを忘れないでください。

return True

最後に。

それをすべてまとめると、次のようになります。

>>> def check(S):
...     if len(S) != 19 or S[4] != ' ' or S[9] != ' ' or S[14] != ' ':
...         return False
...     S = S.replace(" ", '')
...     if len(S) != 16 or not S.isdigit():
...         return False
...     L = map(int, S)  # makes S into a sequence of integers in one step
...     if sum(L) % 10 != 0:  # % calculates the remainder, which should be 0
...         return False
...     return True
... 
>>> check('9384 3495 3297 0123')
False
>>> check('9384 3495 3297 0121')
True
于 2013-09-13T18:20:38.347 に答える
3

正規表現に基づく方法は次のとおりです。

import re
def cc(pat):
    # check for the pattern #### #### #### #### with each '#' being a digit
    m=re.match(r'(\d{4})\s(\d{4})\s(\d{4})\s(\d{4})$', pat.strip())
    if not m:
        return False
    # join all the digits in the 4 groups matched, 
    # turn into a list of ints, 
    # sum and 
    # return True/False if divisible by 10: 
    return sum(int(c) for c in ''.join(m.groups()))%10==0

>>> cc('9384 3495 3297 0123')
False
>>> cc('9384 3495 3297 0121')
True
于 2013-09-13T19:25:29.147 に答える
1

これが私のアプローチです。少し長いコードですが、定義関数を使用するのが好きです。Computer Science Circles の Web サイトでは何らかの理由でコードが機能しませんが、PyCharm プログラムでは機能します。 

    def CardNumber():
        global card     #  Making variable global for function SumCardNumDigits(); see below
        card = input()  # Credit card number is entered
        return card


    def check(S):
        CardNumber = S
        SplitCardNum = CardNumber.split()      # Split credit card number into a list, so we get [####, ####, ####, ####]
        for i in range(0, len(SplitCardNum)):  # Checking if each item in list has length of four strings and each item is
                                               # an actual a number

            if len(SplitCardNum[i]) == 4 and SplitCardNum[i].isdigit():
            SplitCardNum.insert(i, True)   # We add to a list a True value at position i
                del SplitCardNum[i + 1]    # We delete items at position i + 1
            return SplitCardNum


    def checkFormat(SplitCardNum):
        if SplitCardNum == [True] * 4:  # Checking if all above conditions are met in function check(S)
                                        # So the returned value from previous function is a list [True, True, True, True]
            return True
        else:
            return False

    def SumCardNumDigits():
        Ncard = card                    # Using global variable 'card' from function CardNumber()
        SumN = 0
        for i in Ncard:            # Summing up all digits in string 'Ncard', if i position in a string is empty space " "
                                   # we skip a step.
            if i == " ":
                continue
            else:
                SumN += int(i)
            return SumN


    def DivideByTen(SplitCardNum):
        if SplitCardNum == True:        # If conditions from function check(S) are met, we divide the sum of digits
                                        # of credit card number by 10
            SumNumber = SumCardNumDigits() % 10  # We do this by using the returned value of function SumCardNumDigits()
            global SumNumber       # <--- Fixed code
            return SumNumber
        else:
            return False


    def IsDivideByTen(SumNumber):
        check = checkFormat(SplitCardNum)      # Finally we check if the sum of digits of credit card number is divisible by 10
        if SumNumber == 0 and check == True:   # <--- Fixed code
            return True
        else:
            return False


    print(IsDivideByTen(DivideByTen(checkFormat(check(CardNumber())))))  # Output final result

    # You can test this code at: http://cscircles.cemc.uwaterloo.ca/visualize/#mode=edit  and see how it works.
    # Try for '9384 3495 3297 4523' and '9384 3495 3297 4526'
于 2016-09-30T13:42:17.510 に答える