How does that work when you want to change more than one value? Gary
On Aug 15, 2017 00:17, "Benedikt Ritter" <[email protected]> wrote: > Hi, > > I very much like that CSVRecord is unmodifiable. So I’d suggest an API, > that creates a new record instead of mutating the existing one: > > CSVRecord newRecord = myRecord.put(1, „value") > > I’m not sure about „put“ as a method name since it clashes with > java.util.Map#put, which is mutation based... > > Regards, > Benedikt > > > Am 15.08.2017 um 02:54 schrieb Gary Gregory <[email protected]>: > > > > Feel free to provide a PR on GitHub :-) > > > > Gary > > > > On Aug 14, 2017 15:29, "Gary Gregory" <[email protected]> wrote: > > > >> I think we've kept the design as YAGNI as possible... :-) > >> > >> Gary > >> > >> On Mon, Aug 14, 2017 at 3:25 PM, nitin mahendru < > >> [email protected]> wrote: > >> > >>> Yeah that also is OK. I though there is a reason to keep the CSVRecord > >>> without setters. But maybe not! > >>> > >>> Nitin > >>> > >>> > >>> > >>> > >>> On Mon, Aug 14, 2017 at 2:22 PM Gary Gregory <[email protected]> > >>> wrote: > >>> > >>>> Hi All: > >>>> > >>>> Should we consider adding put(int,Object) and put(String, Object) to > the > >>>> current CSVRecord class? > >>>> > >>>> Gary > >>>> > >>>> On Mon, Aug 14, 2017 at 2:54 PM, nitin mahendru < > >>>> [email protected]> > >>>> wrote: > >>>> > >>>>> Hi Everyone, > >>>>> > >>>>> I recently pushed a change(pull request 20) to get the line ending > >>> from > >>>> the > >>>>> parser. > >>>>> > >>>>> Now I want to push another change which I feel will also be useful > for > >>>> the > >>>>> community. I want to add a CSVRecordMutable class which had a > >>> constructor > >>>>> which accepts a CSVRecord object. So when we have a CSVRecordMutable > >>>> object > >>>>> from it then we can edit individual columns using it. > >>>>> > >>>>> I would be using this to write back my edited CSV file. My use case > >>> is to > >>>>> read a csv, mangle some columns, write back a new csv. > >>>>> > >>>>> I could have directly raised a pull request but I just wanted to > float > >>>> the > >>>>> idea before and see the reaction. > >>>>> > >>>>> Thanks > >>>>> > >>>>> Nitin > >>>>> > >>>> > >>> > >> > >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
