Beefy Boxes and Bandwidth Generously Provided by pair Networks
more useful options
 
PerlMonks  

Re^10: Faster Luhn Check Digit Calculation?

by BrowserUk (Patriarch)
on Dec 02, 2018 at 09:08 UTC ( [id://1226619]=note: print w/replies, xml ) Need Help??


in reply to Re^9: Faster Luhn Check Digit Calculation?
in thread Faster Luhn Check Digit Calculation?

At least according to godbolt.org, the three routines don't get folded into the same assembly code.

I didn't really expect that they would be; but the empirical evidence is that they take the same amount of time. The routine is now so stripped down to the essentials that the calling overheads are beginning to dominate.

For example, the OPs original test code used a numeric range for testing:401135000000000..401135000999999 which I just copy&pasted. That means that every call to the function carries the overhead of converting those integers to strings. Switching the range to strings: '401135000000000' .. '401135000999999' avoids that conversion and saves more time than tybalt's optimisations.

Other changes that would require knowledge of how that code is used are also more likely to affect the performance. Eg. If the code's purpose is to verify existing CC numbers (rather than constructing new ones) then is is likely that the 16-digits strings are being read from a file or DB. Then the 16-digits are truncated to 15, passed into the routine, the checkdigit is calculated and returned and that is then compared against the 16th digit of the input.

If the routine was rewritten to accept the 16 digits and return a boolean indicating T/F then several operations external to the routine could be eliminated and overall throughput improved even though the function itself might be a little slower.

Only the OP can decide what comes next.


With the rise and rise of 'Social' network sites: 'Computers are making people easier to use everyday'
Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
"Science is about questioning the status quo. Questioning authority". The enemy of (IT) success is complexity.
In the absence of evidence, opinion is indistinguishable from prejudice. Suck that fhit

Replies are listed 'Best First'.
Re^11: Faster Luhn Check Digit Calculation?
by kschwab (Vicar) on Dec 02, 2018 at 13:28 UTC
    "Only the OP can decide what comes next."

    Based on a huge performance difference with C, I think I'll make an XS clone of Algorithm::LUHN so this is more generally useful for others. It introduces some requirements that will slow things down (like this, but it seems to be the most popular module with LUHN10 in it.

    Appreciate all the help from everyone. If I do get get it done, I'll reference/credit all in the module.

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://1226619]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others goofing around in the Monastery: (9)
As of 2024-03-28 09:29 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found