Follow-up to #2610146: Create API for scrambling methods

Problem/Motivation

Make different methods in order for other modules to make use of these methods.

Proposed resolution

Create methods:

  • Shuffle characters
  • Shuffle words
  • Empty values

API changes

Add scramble methods.

Comments

nico.knaepen created an issue. See original summary.

nico.knaepen’s picture

nico.knaepen CreditAttribution: nico.knaepen at Logic in Motion for Colruyt Group Services commented
Title: Create API for scrambling methods: Method Scramble characters » Create API for scrambling methods: Create methods
Issue summary: View changes

  • nico.knaepen committed 9837a92 on 7.x-1.x 
    Issue #2617530 by nico.knaepen: Added scrambler methods
nico.knaepen’s picture

nico.knaepen CreditAttribution: nico.knaepen at Logic in Motion for Colruyt Group Services commented
Status: Needs work » Needs review
nico.knaepen’s picture

nico.knaepen CreditAttribution: nico.knaepen at Logic in Motion for Colruyt Group Services commented
Assigned: nico.knaepen » Unassigned
nico.knaepen’s picture

nico.knaepen CreditAttribution: nico.knaepen at Logic in Motion for Colruyt Group Services commented
Status: Needs review » Needs work

Check the following use case:

What if the word can not be scrambled?
Example: ee, rr, ff

What if the sentence contains two words which are exactly the same?
Example: word word

Avoid having unlimited loops.

Also include unit testing for this part.

nico.knaepen’s picture

nico.knaepen CreditAttribution: nico.knaepen at Logic in Motion for Colruyt Group Services commented
Status: Needs work » Fixed
nico.knaepen’s picture

nico.knaepen CreditAttribution: nico.knaepen at Logic in Motion for Colruyt Group Services commented
Status: Fixed » Closed (fixed)