Coding Challenge

[Lost User]

I bet that's the last programming challenge you try!

MVVM# - See how I did MVVM my way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

[Lost User]

Chris Maunder wrote:

Given a string of text, trim from each end of the text each all occurrences of a given set of strings

Assuming (from your sample) you only want to remove a single item from each end (hence the leading 'cat' remains), then the below would seem to provide the requisite output and I feel earn bonus points for its simplicity, elegance and the fact that it's the first opportunity I've had to do c# for nearly a year!

private string CodingChallenge(string source, string[] removals)
{
string delimiter = " ";
bool leftDone = false;
bool rightDone = false;

        foreach (string s in removals)
        {
            if (!leftDone && source.Substring(0, s.Length) == s && source.Substring(s.Length, 1) == delimiter)
            {
                source = source.Substring(s.Length);
                leftDone = true;
            }

            if (!rightDone && source.Substring(source.Length - s.Length) == s && source.Substring(source.Length - s.Length - 1, 1) == delimiter)
            {
                source = source.Substring(0, source.Length - s.Length);
                rightDone = true;
            }
        }

        return source;
    }

MVVM# - See how I did MVVM my way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

[ErnestoNet]

Need to check this with other strings, debug some more, improve comments and string handling in main(), etc. But still....here it goes:

#include "stdafx.h"

const wchar_t whitechar = ' ';
const bool IncludeWhiteSpace = true;
const wchar_t* wordstoremove[] = {L"dog", L"cat"};

const wchar_t* strtoparse = L"dog cat monkey dog horse dog"; //Len 28
int ltoparse;
const int numwords = 2;

int wordlen[numwords];

//Positions in the original string to rip
int pos_orig_from;
int pos_orig_to;

//Len of left and right copies
int left_len;
int right_len;

//Gets hoy many bytes per char
void ProcessLeft(wchar_t* strresult)
{
bool bContinue = true;

pos\_orig\_from = left\_len = 0;

while (bContinue)
{
	//Check for whitespaces. If there are, copy them
	if (strtoparse\[pos\_orig\_from\] == whitechar)
	{
		pos\_orig\_from ++;
		if (IncludeWhiteSpace)
		{
			strresult\[left\_len\] = whitechar;
			left\_len ++;
		}
	}
	else
	{
		bContinue = false;
		for (int i = 0; i < numwords; i++)
		{
			//Compare strings
			wchar\_t\* strcompare = (wchar\_t\*)strtoparse + pos\_orig\_from;
			if (wcsncmp(strcompare, wordstoremove\[i\], wordlen\[i\]) == 0)
			{
				pos\_orig\_from += wordlen\[i\];
				bContinue = true;
				break;
			}
		}
	}
}

}
void ProcessRight(wchar_t* strresult)
{
bool bContinue = true;

pos\_orig\_to = right\_len = ltoparse - 1;

while (bContinue)
{
	//Check for whitespaces. If there are, copy them
	if (strtoparse\[pos\_orig\_to\] == whitechar && IncludeWhiteSpace)
	{
		pos\_orig\_to -= 1;
		if (IncludeWhiteSpace)
		{
			strresult\[right\_len\] = whitechar;
			right\_len -= 1;
		}
	}
	else
	{
		bContinue = false;
		for (int i = 0; i < numwords; i++)
		{
			//Compare strings
			//To check right, start from end and substract string to compare len
			wchar\_t\* strcompare = (wchar\_t\*)strtoparse + pos\_orig\_to - wordlen\[i\] + 1;
			if (wcsncmp(strcompare, wordstoremove\[i\], wordlen\[i\]) == 0)
			{
				pos\_orig\_to -= wordlen\[i\];
				bContinue = true;
				break;
			}
		}
	}
}

}

int _tmain(int argc, _TCHAR* argv[])
{

//Load len of words to avoid rechecking
for (int i=0; i

[Pete OHanlon]

Your former client base is showing mate - calling a method Stripper.

Forgive your enemies - it messes with their heads

"Mind bleach! Send me mind bleach!" - Nagy Vilmos

My blog | My articles | MoXAML PowerToys | Mole 2010 - debugging made easier - my favourite utility

[AnthonyN1974]

I was fed up, so here it is in c#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace CodeChallenge
{
class Program
{
private static List<string> wordsToRemove = new List<string>() { "dog", "cat" };

static void Main(string\[\] args)
{
  string text = "dog cat monkey dog horse dog";

  string\[\] words = text.Split(' ');
  string\[\] output;
  int start = 0;
  int end = 0;

  // test the start
  if (wordsToRemove.Contains(words\[0\]) && words.Length > 0)
    start = 1;

  // test the end
  if (wordsToRemove.Contains(words\[words.Length - 1\]) && words.Length != start)
    end = (words.Length - 1) - start;
  else
    end = words.Length - start;

  //build
  output = new string\[end\];
  Array.Copy(words, start, output, 0, end);

  //output
  Console.WriteLine(string.Join(" ",output));
}

}
}

[mrchief_2000]

Probably not the most efficient, but in C#, this works:

using System.Text.RegularExpressions;

var source = "dog cat dog horses monkeys dog";
var stringsToTrim = new[] { "dog", "cat" };
var trimmedString = Regex.Replace(source, string.Format("^({0})|({0})$", string.Join("|", stringsToTrim )), "");
Console.WriteLine(trimmedString);

Live demo: http://rextester.com/GTLWO64640[^] Making above shorter:

using System.Text.RegularExpressions;

Console.WriteLine(Regex.Replace("dog cat dog horses monkeys dog", string.Format("^({0})|({0})$", string.Join("|", "dog", "cat")), ""));

[Clumpco]

Here's a solution in APL V←'dog cat monkey dog horse dog' Q←'dog' V←(((V≠Q)⍳1)-⎕IO)↓V V←(⎕IO-(Q≠⌽V)⍳1)↓V Q←'cat' V←(((V≠Q)⍳1)-⎕IO)↓V V←(⎕IO-(Q≠⌽V)⍳1)↓V No doubt a true APL programmer could do it in one line.

[Marc Clifton]

Chris Maunder wrote:

result = Regex.Replace

That's sweet. And amazingly simple. Marc

My Blog
An Agile walk on the wild side with Relationship Oriented Programming

[Marc Clifton]

Pete O'Hanlon wrote:

Your former client base is showing mate - calling a method Stripper.

I am scarred for life! ;P Marc

My Blog
An Agile walk on the wild side with Relationship Oriented Programming

[DariusLegion]

What the hell is the matter with you guys...? :sigh:

[jsc42]

5th Jan: But you said there was nothing in the spec about whitespace, so only removing the first dog is permissible. FWIW: Here is my solution in JavaScript (not the fastest, but still quite short a) Only removing the first dog (if there are multiple dogs)

alert(/^(dog|cat)*(.*?)(dog|cat)*$/.exec("dog cat monkey dog horse dog")[2])

b) Allowing removal of packs of dogs and cats optionally separated by whitespace chars:

alert(/^(\s*(dog|cat))*(.*?)((dog|cat)\s*)*$/.exec("dog cat monkey dog horse dog")[3])

6th Jan: Modified Clarifications in other responses suggest that whitespace is to be preserved and is not significant when looking for leading / trailing texts and that a general solution is required rather than looking only for dogs and cats bracketing "dog cat monkey dog horse dog". So, today's JavaScript version is ...

function strim(str, texts)
{
return str.replace(
new RegExp(
'^((\\s*)(' + // $2 = leading whitespace
texts.join('|') +
'))*(.*?)((' + // $4 = middle portion
texts.join('|') +
')(\\s*)?)*$', // $7 = trailing whitespace
'ig'
),
'$2$4$7'
);
}

alert(strim('dog cat monkey dog horse dog', [ 'dog', 'cat' ]));
alert(strim('dog dog text', [ 'dog', 'cat' ]));

To see the spaces, change the alerts to

alert(strim('dog cat monkey dog horse dog', [ 'dog', 'cat' ]).replace(/ /g, '~')); // ~~monkey~dog~horse~
alert(strim('dog dog text', [ 'dog', 'cat' ]).replace(/ /g, '~')); // ~~text

Note: This will not work if the texts for testing at the start and end include any special RegExp chars, e.g.

. * + ? {

or }.

[klasbj]

Hi, I have gotten the CodeProject newsletters for several years and used the site, but this it the first time I have posted, but this topic was so fun I just had to! :) I found the task was not clear on one topic, namely which resulting string should be the answer if several are possible? It can happen if the matches at the beginning and end overlap, if my assumption that the patterns you remove can not overlap is correct? Anyway, I solved it with the assumption that what is wanted is the shortest possible resulting string, why else would you trim a string :laugh:

#include <cstdio>
#include <vector>

#include "ahocorasick.h"

using namespace std;
using namespace ac;

// arbitraty length limits
#define MAX_TEXT 1000000
#define MAX_PATTERN 1000000

struct interval {
int begin,end;

interval(int b, int e) : begin(b), end(e) { }

// reverse the interval given the total length is n
interval reverse(int n) const {
    return interval(n - end, n - begin);
}

};

interval trim(int len, vector<interval> & intervals);

int main() {
char * patternbuf;
char ** patterns;
char text[MAX_TEXT];
int npatterns;

scanf("%d", &npatterns); getchar();
patterns = new char\*\[npatterns\];
patternbuf = new char\[npatterns \* MAX\_PATTERN\];
for (int i = 0; i < npatterns; ++i) {
    patterns\[i\] = &patternbuf\[i\*MAX\_PATTERN\];
    gets(patterns\[i\]);  // is dangerous and should not be used!
}
gets(text);

// match the patterns to find all the possible places to trim the string
aho\_corasick matcher(npatterns, patterns);
vector<int> \* matches = matcher.get\_matches(text);

// build a list of intervals of the matches
vector<interval> intervals;
for (int i = 0; i < npatterns; ++i) {
    int len = matcher.get\_pattern\_size(i);
    for (vector<int>::iterator it = matches\[i\].begin();
            it != matches\[i\].end(); ++it)
        intervals.push\_back(interval(\*it, \*it+len));
}

// trim the string as much as possible, i.e. find the shortest interval of
// the original string that can result from trimming
interval result = trim(strlen(text), intervals);

// print the result
text\[result.end\] = '\\0';
printf("\\"%s\\"\\n", &text\[result.begin\]);


// clean up
delete \[\] patternbuf;
delete \[\] patterns;

return 0;

}

[darkDercane]

where i can send my solution?? :P

[Rizean]

I'm sorry but this is not a challenge. If you want a real challenge go to ProjectEuler.com. There you will find over 350 easy to difficult challenges. I have learned a ton from the site and have only completed 45 problems so far. Wish I had more time to work the problems. Regards

[Chris Maunder]

I'm sorry but I think you missed the point of this entire thread. Alternatively you could post your own challenge. I'd love to see everyone else throwing their challenges into the ring.

cheers, Chris Maunder The Code Project | Co-founder Microsoft C++ MVP

[User 7888059]

Hmm, i read here since more than one, year but never posted something, but this sounds not to difficult. euphoria 4.0.x include std/text.e include std/console.e puts(1,trim("dog cat monkey dog horse dog","dogcat",0)&"\n") any_key() this should produce your final output Andreas

[PIEBALDconsult]

OK, this is simpler than what I was concocting yesterday and it's not fully tested, but it works with your sample strings. I started by writing a simple Spell Check Tree class (in C# it's much simpler than it was with C back in college):

namespace PIEBALD.Types
{
public sealed class SpellCheckTree
{
public enum SearchState
{
NotFound = -1
,
Inconclusive = 0
,
Found = 1
}

private sealed class TreeNode : System.Collections.Generic.Dictionary<char,TreeNode> {}

private readonly TreeNode tree ;

private TreeNode current ;

public SpellCheckTree
(
)
{
  this.current = this.tree = new TreeNode() ;

  return ;
}

public void
Reset
(
)
{
  this.current = this.tree ;

  return ;
}

public void
Add
(
  System.Collections.Generic.IEnumerable<char> String
)
{
  TreeNode temp = this.tree ;

  foreach ( char c in String )
  {
    if ( !temp.ContainsKey ( c ) )
    {
      temp \[ c \] = new TreeNode() ;
    }                                        
                                             
    temp = temp \[ c \] ;    
  }

  temp \[ System.Char.MaxValue \] = new TreeNode() ;

  return ;
}

public SearchState
Find
(
  char C
)
{
  SearchState result = SearchState.Inconclusive ;

  if ( !this.current.ContainsKey ( C ) )
  {
    result = SearchState.NotFound ;
  }
  else
  {
    this.current = this.current \[ C \] ;

    if ( this.current.ContainsKey ( System.Char.MaxValue ) )
    {
      result = SearchState.Found ;
    }
  }

  return ( result ) ;
}

}
}

Then I wrote a Trim method to use it:

namespace PIEBALD.Lib
{
using System.Linq ; /* Laziness/expedience */

public enum TrimFrom
{
Left = 1
,
Right = 2
,
Both = 3
}

namespace LibExt.Trim
{
public static partial class LibExt
{
public static string
Trim
(
this string Subject
,
TrimFrom TrimFrom
,
params string[] Strings
)
{
System.Text.StringBuilder result = new System.Text.StringBuilder ( Subject ) ;
in

[PIEBALDconsult]

I read it as, "if you want to trim whitespace, then supply them as a 'string to be removed'".

[IAbstract]

You would be correct. Until you look in the box - the cat is neither dead nor alive. Quantum theory can be crudely demonstrated with this example. Anyway ...back to the topic...

[brian oslick]

Chris Maunder... without having to think too much (and thus stress my feeble brain)

include 'win32a.inc'
format PE console 4.0
entry start
;*********************************************
section '.data' data readable writeable

_s db '%s',13,0

i_string db 'dog cat monkey dog horse dog'
o_string db 21 dup(0)

;*********************************************
section '.text' code readable executable
;*********************************************
start: ;fasm_dogcat.fasm

mov ebx,3
xor ecx,ecx
above:
mov eax,dword[i_string+ebx+ecx*4]
mov dword[o_string+ecx*4],eax
inc ecx
cmp ecx,5
jnz above
mov ax,word[i_string+ebx+ecx*4]
mov word[o_string+ecx*4],ax

cinvoke printf,_s,o_string

; >"C:\_sys\temp\shell.exe"
;$20,'cat monkey dog horse',$20,0
; >Exit code: 0
;*********************************************
invoke ExitProcess,0
;***********************************************
section '.idata' data import readable writeable
library kernel32,'kernel32.dll',\
user32,'user32.dll',\
msvcrt,'msvcrt.dll'
include 'api\kernel32.inc'
include 'api\user32.inc'
import msvcrt,\
printf,'printf'
;***********************************************

brianO