Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > comp.lang.python > #73905

Re: OOP with MyTime

Newsgroups comp.lang.python
Date 2014-07-03 08:21 -0700
References <7020a5d8-96a9-4fa4-8e69-4c7593dedee3@googlegroups.com> <mailman.11418.1404331330.18130.python-list@python.org> <fd30a5f9-c446-435a-9f0a-74dd2d608c5c@googlegroups.com> <mailman.11458.1404393113.18130.python-list@python.org>
Message-ID <d3e593c1-4d61-4c33-b879-d16e700bc42f@googlegroups.com> (permalink)
Subject Re: OOP with MyTime
From kjakupak@gmail.com

Show all headers | View raw


On Thursday, July 3, 2014 9:11:49 AM UTC-4, MRAB wrote:
> On 2014-07-03 13:51, kjakupak@gmail.com wrote:
> 
> > On Wednesday, July 2, 2014 4:02:00 PM UTC-4, MRAB wrote:
> 
> >> >
> 
> >>
> 
> >> If you want 'between' to be an instance method of the MyTime class, it
> 
> >>
> 
> >> needs 'self' as well as the 2 arguments 't1' and 't2'.
> 
> >>
> 
> >>
> 
> >>
> 
> >> You can then compare the hours, minutes and seconds of self against
> 
> >>
> 
> >> those of t1 and t2:
> 
> >>
> 
> >>
> 
> >>
> 
> >>      def between(self, t1, t2):
> 
> >>
> 
> >>          return (t1.hours, t1.minutes, t1.seconds) <= (self.hours,
> 
> >>
> 
> >> self.minutes, self.seconds) and (self.hours, self.minutes, self.seconds)
> 
> >>
> 
> >> <= (t2.hours, t2.minutes, t2.seconds)
> 
> >>
> 
> >>
> 
> >>
> 
> >> That could be shortened further using chained comparisons.
> 
> >>
> 
> >>
> 
> >>
> 
> >> Note that the code assumes that the times t1 and t2 are ordered, i.e.
> 
> >>
> 
> >> that time t1 is not later/greater than time t2.
> 
> >
> 
> > So I've now gotten this:
> 
> > class MyTime:
> 
> >
> 
> >      def __init__(self, hrs=0, mins=0, secs=0):
> 
> >          self.hours = hrs
> 
> >          self.minutes = mins
> 
> >          self.seconds = secs
> 
> >
> 
> >          if self.seconds >= 60:
> 
> >              self.minutes += self.seconds // 60
> 
> >              self.seconds = self.seconds % 60
> 
> >
> 
> >          if self.minutes >= 60:
> 
> >              self.hours += self.minutes // 60
> 
> >              self.minutes = self.minutes % 60
> 
> >
> 
> >          if self.hours >= 24:
> 
> >              self.hours = self.hours % 24
> 
> >
> 
> >      def get_sec(self):
> 
> >          return (self.hours * 60 + self.minutes) * 60 + self.seconds
> 
> >
> 
> >      def __str__(self):
> 
> >          return "{:02d}:{:02d}:{:02d}".\
> 
> >                 format(self.hours, self.minutes, self.seconds)
> 
> >
> 
> >      def between(self, t1, t2):
> 
> >          return (t1.hours, t1.minutes, t1.seconds) <= (self.hours, self.minutes, self.seconds) and (self.hours, self.minutes, self.seconds) <= (t2.hours, t2.minutes, t2.seconds)
> 
> >
> 
> >
> 
> > t1 = MyTime(9, 59, 59)
> 
> > print("t1 =", t1)
> 
> >
> 
> > t2 = MyTime(10, 0, 1)
> 
> > print("t2 =", t2)
> 
> >
> 
> > t3 = MyTime(10, 0, 0)
> 
> > print("t3 =", t3)
> 
> >
> 
> > print("between(t2, t3, t1) =", between(t2, t3, t1))
> 
> > print("between(t1, t3, t2) =", between(t1, t3, t2))
> 
> > print("between(t3, t1, t2) =", between(t3, t1, t2))
> 
> > print("between(t1, t2, t3) =", between(t1, t2, t3))
> 
> >
> 
> > Am I on the right track or? Not sure where to go from here
> 
> >
> 
> You need to decide whether you want 'between' to be a method of the
> 
> MyTime class or a separate function.
> 
> 
> 
> In the above code it's defined as a method, so you can say:
> 
> 
> 
>      t2.between(t3, t1)
> 
> 
> 
> which means "is t2 between t3 and t1?".
> 
> 
> 
> That would return False because t3 is greater than t1, but:
> 
> 
> 
>      t2.between(t1, t3)
> 
> 
> 
> would return True.
> 
> 
> 
> (I _did_ say that it assumes that the times are ordered.)
> 
> 
> 
> BTW, gmail is messing up your messages. This will tell you how to fix
> 
> it:
> 
> 
> 
> https://wiki.python.org/moin/GoogleGroupsPython

I keep getting an invalid syntax on the t1 = (9, 59, 59) line, not sure why?

t1 = (9, 59, 59)
print("t1 =", t1)

t2 = (10, 0, 1)
print("t2 =", t2)

t3 = (10, 0, 0)
print("t3 =", t3)

print("between(t2, t3, t1) =", t2.between(t3, t1))

Back to comp.lang.python | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

OOP with MyTime kjakupak@gmail.com - 2014-07-02 12:20 -0700
  Re: OOP with MyTime Akira Li <4kir4.1i@gmail.com> - 2014-07-02 23:45 +0400
  Re: OOP with MyTime MRAB <python@mrabarnett.plus.com> - 2014-07-02 21:02 +0100
    Re: OOP with MyTime kjakupak@gmail.com - 2014-07-03 05:51 -0700
      Re: OOP with MyTime Chris Angelico <rosuav@gmail.com> - 2014-07-03 23:01 +1000
        Re: OOP with MyTime kjakupak@gmail.com - 2014-07-03 06:08 -0700
          Re: OOP with MyTime Chris Angelico <rosuav@gmail.com> - 2014-07-03 23:17 +1000
      Re: OOP with MyTime MRAB <python@mrabarnett.plus.com> - 2014-07-03 14:11 +0100
        Re: OOP with MyTime kjakupak@gmail.com - 2014-07-03 08:21 -0700
          Re: OOP with MyTime Chris Angelico <rosuav@gmail.com> - 2014-07-04 01:35 +1000
          Re: OOP with MyTime Mark Lawrence <breamoreboy@yahoo.co.uk> - 2014-07-03 17:23 +0100

csiph-web