Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. The Lounge
  3. Friday Programming Quiz [modified]

Friday Programming Quiz [modified]

Scheduled Pinned Locked Moved The Lounge
debugging
39 Posts 23 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • A Andy Brummer

    :-D Please tell me you just made that up. That isn't an actual example of PE programming, it can't be that would just be absurd.

    Using the GridView is like trying to explain to someone else how to move a third person's hands in order to tie your shoelaces for you. -Chris Maunder

    A Offline
    A Offline
    amclint
    wrote on last edited by
    #17

    Andy Brummer wrote:

    Using the GridView is like trying to explain to someone else how to move a third person's hands in order to tie your shoelaces for you. -Chris Maunder

    ROFL, good sig

    1 Reply Last reply
    0
    • R Rama Krishna Vavilala

      In a language of your choice (no PE), implement the following:

      string RemoveDuplicates(string csvString) {

      }

      The function should remove all duplicate values form a string containing comma separated values. RemoveDuplicates("a,b,c,a") => "a,b,c" RemoveDuplicates("a,b,c,a,c,b,c") => "a,b,c" RemoveDuplicates("a,b,c") => "a,b,c" RemoveDuplicates("cat,dog,dog") => "cat,dog" The ideal implementation should have just 1 line of code.


      Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

      S Offline
      S Offline
      Stuart Dootson
      wrote on last edited by
      #18

      In Haskell,

      import Data.List
      removeDuplicates csvStr = nub (map (delete ',') (groupBy (\a b -> b == ',') csvStr))
      

      I had to write the 'split on ,' functionality, which takes most of the declaration (it's this bit map (delete ',') (groupBy (\a b -> b == ',') csvStr)), but Haskell handily has a 'remove duplicates from a list' function, nub. [Edit]Whoops - forgot to reconstruct the string (also, didn't cope with multi-char strings)!

      import Data.List
      removeDuplicates csvStr = concat $ intersperse "," $ nub $ map (delete ',') (groupBy (\a b -> b /= ',') csvStr)
      

      [/Edit] [Edit 2] And on further investigation of Haskell's libraries, there's a splitRegex function:

      import Data.List -- for intersperse, nub
      import Text.Regex -- for splitRegex, mkRegex
      removeDuplicates csvStr = concat $ intersperse "," $ nub $ splitRegex (mkRegex ",") csvStr
      

      [/Edit 2]

      1 Reply Last reply
      0
      • J Jon Sagara

        Plain English Function Called "Remove Duplicates" with Argument Consisting of Comma Separated Values in a Character String Remove Duplicate Values From The Plain English Function Argument Consisting of Comma Separated Values in a Character String Return The Plain English Function Argument Consisting of Comma Separated Values in a Character String, But With Duplicate Values Removed End Of Plain English Function Called "Remove Duplicates" with Argument Consisting of Comma Separated Values in a Character String Excuse me while I go hurl X|

        Jon Sagara When I grow up, I'm changing my name to Joe Kickass! My Site | My Blog | My Articles

        C Offline
        C Offline
        Chris S Kaiser
        wrote on last edited by
        #19

        This just won't ever get old... :laugh::laugh::laugh:

        What's in a sig? This statement is false. Build a bridge and get over it. ~ Chris Maunder

        1 Reply Last reply
        0
        • T ToddHileHoffer

          In C# string RemoveDuplicates(string csvString) { string[] x = csvString.Split(char.Parse(",")); System.Collections.Specialized.StringCollection c = new System.Collections.Specialized.StringCollection(); foreach (string y in x) { if (!c.Contains(y)) c.Add(y); } string result = ""; foreach (string z in c) { result += z + ","; } return result.Substring(0, result.Length - 1); }


          how vital enterprise application are for proactive organizations leveraging collective synergy to think outside the box and formulate their key objectives into a win-win game plan with a quality-driven approach that focuses on empowering key players to drive-up their core competencies and increase expectations with an all-around initiative to drive up the bottom-line. But of course, that's all a "high level" overview of things --thedailywtf 3/21/06

          P Offline
          P Offline
          PIEBALDconsult
          wrote on last edited by
          #20

          public static string RemoveDuplicates ( string Subject ) { System.Text.StringBuilder result = new System.Text.StringBuilder ( Subject.Length ) ; System.Collections.Specialized.StringCollection dic = new System.Collections.Specialized.StringCollection() ; foreach ( string temp in Subject.Split ( new char[] { ',' } , System.StringSplitOptions.None ) ) { if ( !dic.Contains ( temp ) ) { dic.Add ( temp ) ; result.Append ( temp ) ; result.Append ( "," ) ; } } return ( result.Remove ( result.Length-1 , 1 ).ToString() ) ; }

          1 Reply Last reply
          0
          • R Rama Krishna Vavilala

            In a language of your choice (no PE), implement the following:

            string RemoveDuplicates(string csvString) {

            }

            The function should remove all duplicate values form a string containing comma separated values. RemoveDuplicates("a,b,c,a") => "a,b,c" RemoveDuplicates("a,b,c,a,c,b,c") => "a,b,c" RemoveDuplicates("a,b,c") => "a,b,c" RemoveDuplicates("cat,dog,dog") => "cat,dog" The ideal implementation should have just 1 line of code.


            Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

            P Offline
            P Offline
            PIEBALDconsult
            wrote on last edited by
            #21

            Does it count if we write a class to implement a distinct StringCollection with an appropriate ToString() to do most of the work? Resultant function could be something like: string RemoveDuplicates(string csvString) { return ( (new DistinctStringCollection ( csvString.Split ( new char[] { ',' } ) )).ToString() ) ; }

            M 1 Reply Last reply
            0
            • R Rama Krishna Vavilala

              Depends on the language (probably it is better to call 1 statement instead of 1 line): Something like this[^]


              Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

              P Offline
              P Offline
              PIEBALDconsult
              wrote on last edited by
              #22

              I just wanted to point out that "lines of code" is not a very worthwhile concept in relation to "modern programming languages".

              1 Reply Last reply
              0
              • R Rama Krishna Vavilala

                In a language of your choice (no PE), implement the following:

                string RemoveDuplicates(string csvString) {

                }

                The function should remove all duplicate values form a string containing comma separated values. RemoveDuplicates("a,b,c,a") => "a,b,c" RemoveDuplicates("a,b,c,a,c,b,c") => "a,b,c" RemoveDuplicates("a,b,c") => "a,b,c" RemoveDuplicates("cat,dog,dog") => "cat,dog" The ideal implementation should have just 1 line of code.


                Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

                S Offline
                S Offline
                Shog9 0
                wrote on last edited by
                #23

                function Reduce(str)
                {
                var ret = new Array();
                var a = str.split(',');
                for (var i in a)
                {
                if ( !ret[a[i]] ) ret.push(a[i]);
                ret[a[i]] = true;
                }
                return ret.join(',');
                }

                Or, if you can use 1.7:

                function Reduce(str)
                {
                function Unique(a)
                {
                var o = {};
                for each (var i in a)
                {
                if (!o[i]) yield i;
                o[i] = true;
                }
                }

                return [i for (i in Unique(str.split(',')))].join(',');
                }

                1 Reply Last reply
                0
                • D Daniel Grunwald

                  This is what LINQ is for: return string.Join(",", csvString.Split(',').Distinct()); Edit: Note that it is also the most efficient solution - it's O(N) because Distinct() internally uses a hash table. The C++ set<> solutions are O(N log N), though probably faster in the real world. And everything running Contains() repeatedly will be O(N²). Not that anyone would store large amounts of data in CSV strings.... Second modification: Sadly, it won't work like that. Distinct() returns IEnumerable, but for some strange reason, Join only works with arrays. So if we don't get a new Join() overload in .NET 3.5, add a .ToArray() extension method call behind the Distinct().

                  Last modified: 24mins after originally posted --

                  D Offline
                  D Offline
                  David Stone
                  wrote on last edited by
                  #24

                  Daniel Grunwald wrote:

                  Not that anyone would store large amounts of data in CSV strings....

                  :laugh: How optimistic. :rolleyes:

                  Once you wanted revolution
                  Now you're the institution
                  How's it feel to be the man?

                  1 Reply Last reply
                  0
                  • D Daniel Grunwald

                    This is what LINQ is for: return string.Join(",", csvString.Split(',').Distinct()); Edit: Note that it is also the most efficient solution - it's O(N) because Distinct() internally uses a hash table. The C++ set<> solutions are O(N log N), though probably faster in the real world. And everything running Contains() repeatedly will be O(N²). Not that anyone would store large amounts of data in CSV strings.... Second modification: Sadly, it won't work like that. Distinct() returns IEnumerable, but for some strange reason, Join only works with arrays. So if we don't get a new Join() overload in .NET 3.5, add a .ToArray() extension method call behind the Distinct().

                    Last modified: 24mins after originally posted --

                    M Offline
                    M Offline
                    Matt Gerrans
                    wrote on last edited by
                    #25

                    Similarly in Python: def RemoveDuplicates(text): return {}.fromkeys( [elem.strip() for elem in text.split(',')] ).keys() (But I also stripped the spaces after splitting on commas, to allow "a, b, b,c,c,d" kind of stuff).

                    Matt Gerrans

                    1 Reply Last reply
                    0
                    • C Chris Losinger

                      template < typename _Cont > void split(const std::string& str, _Cont& _container, const std::string& delim=",")
                      {
                      std::string::size_type lpos = 0;
                      std::string::size_type pos = str.find_first_of(delim, lpos);
                      while(lpos != std::string::npos)
                      {
                      _container.insert(_container.end(), str.substr(lpos,pos - lpos));

                          lpos = ( pos == std::string::npos ) ?  std::string::npos : pos + 1;
                          pos = str.find\_first\_of(delim, lpos);
                      }
                      

                      }

                      std::string fn(std::string in)
                      {
                      std::string out;
                      std::set foo;
                      split(in, foo);

                      for (std::set::iterator it=foo.begin();it!=foo.end();it++)
                      {
                      	if ((\*it).size() > 0)
                      	{
                      		out+=(\*it);
                      		if (std::distance(it, foo.end()) > 1) out+=",";
                      	}
                      }
                      return out;
                      

                      }

                      and you can count this as my code from CP entry for the day. why is IE (or CP?) putting that sentence inside the PRE ? it's not. it looked fine in FF2.0. -- modified at 19:14 Friday 3rd November, 2006

                      image processing | blogging

                      J Offline
                      J Offline
                      Jorgen Sigvardsson
                      wrote on last edited by
                      #26

                      Boostified:

                      std::string fn(std::string in)
                      {
                      std::string out;

                      typedef boost::tokenizer<boost::char\_separator<char> > tokenizer;
                      tokenizer foo(in, boost::char\_separator<char>(",");
                      
                      tokenizer::iterator it = foo.begin(), end = foo.end();
                      while(it != end)
                      {
                          out += \*it++;
                          if(it != end)
                              out += ",";
                      }
                      return out; 
                      

                      }

                      There's probably a boost function somewhere which allows one to join strings as well, but I didn't bother to look. :)

                      -- Not based on the Novel by James Fenimore Cooper

                      C 2 Replies Last reply
                      0
                      • R Rama Krishna Vavilala

                        In a language of your choice (no PE), implement the following:

                        string RemoveDuplicates(string csvString) {

                        }

                        The function should remove all duplicate values form a string containing comma separated values. RemoveDuplicates("a,b,c,a") => "a,b,c" RemoveDuplicates("a,b,c,a,c,b,c") => "a,b,c" RemoveDuplicates("a,b,c") => "a,b,c" RemoveDuplicates("cat,dog,dog") => "cat,dog" The ideal implementation should have just 1 line of code.


                        Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

                        A Offline
                        A Offline
                        Andre xxxxxxx
                        wrote on last edited by
                        #27

                        print RemoveDuplicates("a,b,b,c,b,c");

                        sub RemoveDuplicates
                        {
                        foreach (split (/,/,$_[0])) { $_{$_} = $_; }
                        join (",",keys %_);
                        }

                        J M 2 Replies Last reply
                        0
                        • J Jorgen Sigvardsson

                          Boostified:

                          std::string fn(std::string in)
                          {
                          std::string out;

                          typedef boost::tokenizer<boost::char\_separator<char> > tokenizer;
                          tokenizer foo(in, boost::char\_separator<char>(",");
                          
                          tokenizer::iterator it = foo.begin(), end = foo.end();
                          while(it != end)
                          {
                              out += \*it++;
                              if(it != end)
                                  out += ",";
                          }
                          return out; 
                          

                          }

                          There's probably a boost function somewhere which allows one to join strings as well, but I didn't bother to look. :)

                          -- Not based on the Novel by James Fenimore Cooper

                          C Offline
                          C Offline
                          Chris Losinger
                          wrote on last edited by
                          #28

                          someday i'll try to figure out how to use boost again. the last time i tried, it was a total installation, dependency, compiler configuration nightmare. definitely not the kind of thing i wanted to get into, just to use their regexp classes.

                          image processing | blogging

                          J 1 Reply Last reply
                          0
                          • C Chris Losinger

                            someday i'll try to figure out how to use boost again. the last time i tried, it was a total installation, dependency, compiler configuration nightmare. definitely not the kind of thing i wanted to get into, just to use their regexp classes.

                            image processing | blogging

                            J Offline
                            J Offline
                            Jorgen Sigvardsson
                            wrote on last edited by
                            #29

                            I've had no problems with the releases since 1.30. This has been with VS 2k3 - don't know what'll happen with VC6 or 2k5.

                            -- -= Proudly Made on Earth =-

                            1 Reply Last reply
                            0
                            • J Jorgen Sigvardsson

                              Boostified:

                              std::string fn(std::string in)
                              {
                              std::string out;

                              typedef boost::tokenizer<boost::char\_separator<char> > tokenizer;
                              tokenizer foo(in, boost::char\_separator<char>(",");
                              
                              tokenizer::iterator it = foo.begin(), end = foo.end();
                              while(it != end)
                              {
                                  out += \*it++;
                                  if(it != end)
                                      out += ",";
                              }
                              return out; 
                              

                              }

                              There's probably a boost function somewhere which allows one to join strings as well, but I didn't bother to look. :)

                              -- Not based on the Novel by James Fenimore Cooper

                              C Offline
                              C Offline
                              Chris Losinger
                              wrote on last edited by
                              #30

                              Joergen Sigvardsson wrote:

                              *it++; if(it != end)

                              ah. nice touch.

                              image processing | blogging

                              1 Reply Last reply
                              0
                              • P PIEBALDconsult

                                Does it count if we write a class to implement a distinct StringCollection with an appropriate ToString() to do most of the work? Resultant function could be something like: string RemoveDuplicates(string csvString) { return ( (new DistinctStringCollection ( csvString.Split ( new char[] { ',' } ) )).ToString() ) ; }

                                M Offline
                                M Offline
                                Marc 0
                                wrote on last edited by
                                #31

                                Yeah piece of cake: return (new [UniqueStringList](http://www.codeproject.com/csharp/uniquestringlist.asp)[[^](http://www.codeproject.com/csharp/uniquestringlist.asp)](csvString.Split(new char[] {','})).ToString("", ",", ""); It is kind of cheating though ;P


                                "..Commit yourself to quality from day one..it's better to do nothing at all than to do something badly.." -- Mark McCormick


                                || Fold With Us! || Pensieve || VG.Net ||

                                P 1 Reply Last reply
                                0
                                • A Andre xxxxxxx

                                  print RemoveDuplicates("a,b,b,c,b,c");

                                  sub RemoveDuplicates
                                  {
                                  foreach (split (/,/,$_[0])) { $_{$_} = $_; }
                                  join (",",keys %_);
                                  }

                                  J Offline
                                  J Offline
                                  Jorgen Sigvardsson
                                  wrote on last edited by
                                  #32

                                  Ah.. Perl syntax. Gives me the shiver every time. ;)

                                  -- Now with chucklelin

                                  1 Reply Last reply
                                  0
                                  • M Marc 0

                                    Yeah piece of cake: return (new [UniqueStringList](http://www.codeproject.com/csharp/uniquestringlist.asp)[[^](http://www.codeproject.com/csharp/uniquestringlist.asp)](csvString.Split(new char[] {','})).ToString("", ",", ""); It is kind of cheating though ;P


                                    "..Commit yourself to quality from day one..it's better to do nothing at all than to do something badly.." -- Mark McCormick


                                    || Fold With Us! || Pensieve || VG.Net ||

                                    P Offline
                                    P Offline
                                    PIEBALDconsult
                                    wrote on last edited by
                                    #33

                                    Ah, very good, I'll have to take a deeper look at UniqueStringList. Here's what I just whipped up: public partial class DistinctStringCollection : System.Collections.Specialized.StringCollection { public static DistinctStringCollection FromCSV ( string CSV ) { DistinctStringCollection result = new DistinctStringCollection() ; foreach ( string temp in CSV.Trim ( new char[] { ',' } ).Split ( new char[] { ',' } ) ) { if ( !result.Contains ( temp ) ) { result.Add ( temp ) ; } } return ( result ) ; } public string ToCSV ( ) { System.Text.StringBuilder result = new System.Text.StringBuilder() ; foreach ( string temp in this ) { result.Append ( temp ) ; result.Append ( "," ) ; } return ( result.Remove ( result.Length-1 , 1 ).ToString() ) ; } } And then... public static string RemoveDuplicates ( string Subject ) { return ( DistinctStringCollection.FromCSV ( Subject ).ToCSV() ) ; }

                                    P 1 Reply Last reply
                                    0
                                    • R Rama Krishna Vavilala

                                      In a language of your choice (no PE), implement the following:

                                      string RemoveDuplicates(string csvString) {

                                      }

                                      The function should remove all duplicate values form a string containing comma separated values. RemoveDuplicates("a,b,c,a") => "a,b,c" RemoveDuplicates("a,b,c,a,c,b,c") => "a,b,c" RemoveDuplicates("a,b,c") => "a,b,c" RemoveDuplicates("cat,dog,dog") => "cat,dog" The ideal implementation should have just 1 line of code.


                                      Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

                                      M Offline
                                      M Offline
                                      Michael Dunn
                                      wrote on last edited by
                                      #34

                                      Perl (not tested, may or may not work) ;) print join ( grep { ++$tokens{$_} == 1 } split(/,/), ',' )

                                      --Mike-- Visual C++ MVP :cool: LINKS~! Ericahist | PimpFish | CP SearchBar v3.0 | C++ Forum FAQ

                                      1 Reply Last reply
                                      0
                                      • A Andre xxxxxxx

                                        print RemoveDuplicates("a,b,b,c,b,c");

                                        sub RemoveDuplicates
                                        {
                                        foreach (split (/,/,$_[0])) { $_{$_} = $_; }
                                        join (",",keys %_);
                                        }

                                        M Offline
                                        M Offline
                                        Michael Dunn
                                        wrote on last edited by
                                        #35

                                        Does keys return the keys in the same order as they were inserted? When I did my solution below, I consciously kept the order of the words the same in the output.

                                        --Mike-- Visual C++ MVP :cool: LINKS~! Ericahist | PimpFish | CP SearchBar v3.0 | C++ Forum FAQ

                                        A 1 Reply Last reply
                                        0
                                        • R Rama Krishna Vavilala

                                          In a language of your choice (no PE), implement the following:

                                          string RemoveDuplicates(string csvString) {

                                          }

                                          The function should remove all duplicate values form a string containing comma separated values. RemoveDuplicates("a,b,c,a") => "a,b,c" RemoveDuplicates("a,b,c,a,c,b,c") => "a,b,c" RemoveDuplicates("a,b,c") => "a,b,c" RemoveDuplicates("cat,dog,dog") => "cat,dog" The ideal implementation should have just 1 line of code.


                                          Debugging is twice as hard as writing the code in the first place. Therefore, if you write the code as cleverly as possible, you are, by definition, not smart enough to debug it. -Brian Kernighan

                                          E Offline
                                          E Offline
                                          Eytukan
                                          wrote on last edited by
                                          #36

                                          It's a becoming a bit of addiction to wait for friday programming post... :) hmm but I'm simply watching it now. I'll start posting my version soon :). Very nice Rama :).. in particular everytime I look for Nish to responding with his code :-D...that;s cool. Also at the end you can post your own answer right?


                                          :Gong: 歡迎光臨 吐 西批 :Gong:

                                          1 Reply Last reply
                                          0
                                          Reply
                                          • Reply as topic
                                          Log in to reply
                                          • Oldest to Newest
                                          • Newest to Oldest
                                          • Most Votes


                                          • Login

                                          • Don't have an account? Register

                                          • Login or register to search.
                                          • First post
                                            Last post
                                          0
                                          • Categories
                                          • Recent
                                          • Tags
                                          • Popular
                                          • World
                                          • Users
                                          • Groups