3

注:(最初の2つの提案以降、これを更新しました...ここで古い投稿をtxt形式で表示できます:http://bennyland.com/old-2554127.txt)。私が行った更新は、何がうまくいかなかったかをよりよく理解することでした-そして今、私は少なくとも何が起こっているのかをある程度知っていますが、それを修正する方法がわかりません。

とにかく、DjangoとPistonを使用して、BaseApiHandlerという名前の新しいBaseHandlerクラスを設定しました。これは、すべてのハンドラーで行っていた作業のほとんどを実行します。これは、結果に適用されるフィルターを制限する機能を追加するまではうまく機能しました(たとえば、「最初の結果のみを表示する」)。

例(これ以上URLを送信できないため、「:」を削除する必要がありました):- http // localhost / api / hours_detail / empid / 22は、従業員#22からのすべてのhours_detail行を提供します-http // localhost / api / hours_detail / empid / 22 / limit / firstは、従業員#22からの最初のhours_detail行を提供します

何が起こっているのかというと、/ limit / firstを連続して数回実行すると、最初の例が壊れて、そうでないときは/ limit/urlのふりをします。

今、私はそれが制限であるかどうか、そして新しいクラスの制限が何であるかを保存しています-このスタックオーバーフロー編集の前は、2つのエントリを持つリストを使用していました(初期化時のlimit = []、limit = [0、 1]設定時)。このスタックオーバーフロー編集の前に、/ limit / firstをスパムすると、最初の例に進むと、「limit」が[0,1]に事前設定され、ハンドラーはこのためにクエリを制限します。追加したデバッグデータを使用すると、リストは事前に設定されており、コードの実行中に設定されていないことは確かです。

何が起こっているかを確認できるように、応答にデバッグ情報を追加しています。現在、例1のURLを最初に要求すると、次の正しいstatusmsg応答が返されます。

"statusmsg": "2 hours_detail found with query: {'empid':'22','datestamp':'2009-03-02',}",

例2のURLを要求すると、次の正しいstatusmsg応答が返されます。

"statusmsg": "1 hours_detail found with query: {'empid':'22','datestamp':'2009-03-02','limit','first',with limit[0,1](limit,None... limit set 1 times),}",

ただし、何度も更新すると、制限設定値が増加し始めます(この値を増やすことは、この変数が何らかの形で維持されているかどうかを確認するために私の友人が提案したものでした)

"statusmsg": "1 hours_detail found with query: {'empid':'22','datestamp':'2009-03-02','limit','first',with limit[0,1](limit,None... limit set 10 times),}",

その数が「1回」を超えると、例1のURLの取得を開始できます。例1を更新するたびに、奇妙な結果が得られます。異なる更新からの3つの異なるステータスメッセージを次に示します(それぞれから、'limit':'first'がkwargのデバッグ出力から正しく欠落しているのに対し、islimitの実際の値は8から10の間でホバリングしています):

"statusmsg": "1 hours_detail found with query: {'empid':'22','datestamp':'2009-03-02',with limit[0,1](limit,None... limit set 10 times),}",
"statusmsg": "1 hours_detail found with query: {'empid':'22','datestamp':'2009-03-02',with limit[0,1](limit,None... limit set 8 times),}",
"statusmsg": "1 hours_detail found with query: {'empid':'22','datestamp':'2009-03-02',with limit[0,1](limit,None... limit set 9 times),}",

したがって、このオブジェクトはキャッシュされているように見えます。リストからクラスに「limit」を変更する前は、例2のURLに移動した後のように、「limit」のリストバージョンがキャッシュされているように見えました。制限として[0,1]が使用されることがありました。

コードの更新されたスニペットは次のとおりです(最初の投稿はここで表示できます:bennyland.com/old-2554127.txt

URLS.PY-inside'urlpatterns = Patterns('

    #hours_detail/id/{id}/empid/{empid}/projid/{projid}/datestamp/{datestamp}/daterange/{fromdate}to{todate}
    #empid is required
    url(r'^api/hours_detail/(?:' + \
        r'(?:[/]?id/(?P<id>\d+))?' + \
        r'(?:[/]?empid/(?P<empid>\d+))?' + \
        r'(?:[/]?projid/(?P<projid>\d+))?' + \
        r'(?:[/]?datestamp/(?P<datestamp>\d{4,}[-/\.]\d{2,}[-/\.]\d{2,}))?' + \
        r'(?:[/]?daterange/(?P<daterange>(?:\d{4,}[-/\.]\d{2,}[-/\.]\d{2,})(?:to|/-)(?:\d{4,}[-/\.]\d{2,}[-/\.]\d{2,})))?' + \
        r')+' + \
        r'(?:/limit/(?P<limit>(?:first|last)))?' + \
        r'(?:/(?P<exact>exact))?$', hours_detail_resource),

HANDLERS.PY

class ResponseLimit(object):
    def __init__(self):
        self._min = 0
        self._max = 0
        self._islimit = 0

    @property
    def min(self):
        if self.islimit == 0:
            raise LookupError("trying to access min when no limit has been set")
        return self._min

    @property
    def max(self):
        if self.islimit == 0:
            raise LookupError("trying to access max when no limit has been set")
        return self._max

    @property
    def islimit(self):
        return self._islimit

    def setlimit(self, min, max):
        self._min = min
        self._max = max
        # incrementing _islimit instead of using a bool so I can try and see why it's broken
        self._islimit += 1

class BaseApiHandler(BaseHandler):
    limit = ResponseLimit()
    def __init__(self):
        self._post_name = 'base'

    @property
    def post_name(self):
        return self._post_name

    @post_name.setter
    def post_name(self, value):
        self._post_name = value

    def process_kwarg_read(self, key, value, d_post, b_exact):
        """
        this should be overridden in the derived classes to process kwargs
        """
        pass

    # override 'read' so we can better handle our api's searching capabilities
    def read(self, request, *args, **kwargs):
        d_post = {'status':0,'statusmsg':'Nothing Happened'}
        try:
            # setup the named response object
            # select all employees then filter - querysets are lazy in django
            # the actual query is only done once data is needed, so this may
            # seem like some memory hog slow beast, but it's actually not.
            d_post[self.post_name] = self.queryset(request)
            s_query = ''

            b_exact = False
            if 'exact' in kwargs and kwargs['exact'] <> None:
                b_exact = True
                s_query = '\'exact\':True,'

            for key,value in kwargs.iteritems():
                # the regex url possibilities will push None into the kwargs dictionary
                # if not specified, so just continue looping through if that's the case
                if value is None or key == 'exact':
                    continue

                # write to the s_query string so we have a nice error message
                s_query = '%s\'%s\':\'%s\',' % (s_query, key, value)

                # now process this key/value kwarg
                self.process_kwarg_read(key=key, value=value, d_post=d_post, b_exact=b_exact)

            # end of the kwargs for loop
            else:
                if self.limit.islimit > 0:
                    s_query = '%swith limit[%s,%s](limit,%s... limit set %s times),' % (s_query, self.limit.min, self.limit.max, kwargs['limit'],self.limit.islimit)
                    d_post[self.post_name] = d_post[self.post_name][self.limit.min:self.limit.max]
                if d_post[self.post_name].count() == 0:
                    d_post['status'] = 0
                    d_post['statusmsg'] = '%s not found with query: {%s}' % (self.post_name, s_query)
                else:
                    d_post['status'] = 1
                    d_post['statusmsg'] = '%s %s found with query: {%s}' % (d_post[self.post_name].count(), self.post_name, s_query)
        except:
            e = sys.exc_info()[1]
            d_post['status'] = 0
            d_post['statusmsg'] = 'error: %s %s' % (e, traceback.format_exc())
            d_post[self.post_name] = []

        return d_post


class HoursDetailHandler(BaseApiHandler):
    #allowed_methods = ('GET', 'PUT', 'POST', 'DELETE',)
    model = HoursDetail
    exclude = ()

    def __init__(self):
        BaseApiHandler.__init__(self)
        self._post_name = 'hours_detail'

    def process_kwarg_read(self, key, value, d_post, b_exact):
        # each query is handled slightly differently... when keys are added
        # handle them in here.  python doesn't have switch statements, this
        # could theoretically be performed using a dictionary with lambda
        # expressions, however I was affraid it would mess with the way the
        # filters on the queryset work so I went for the less exciting
        # if/elif block instead

        # querying on a specific row
        if key == 'id':
            d_post[self.post_name] = d_post[self.post_name].filter(pk=value)

        # filter based on employee id - this is guaranteed to happen once
        # per query (see read(...))
        elif key == 'empid':
            d_post[self.post_name] = d_post[self.post_name].filter(emp__id__exact=value)

        # look for a specific project by id
        elif key == 'projid':
            d_post[self.post_name] = d_post[self.post_name].filter(proj__id__exact=value)

        elif key == 'datestamp' or key == 'daterange':
            d_from = None
            d_to = None
            # first, regex out the times in the case of range vs stamp
            if key == 'daterange':
                m = re.match('(?P<daterangefrom>\d{4,}[-/\.]\d{2,}[-/\.]\d{2,})(?:to|/-)(?P<daterangeto>\d{4,}[-/\.]\d{2,}[-/\.]\d{2,})', \
                             value)
                d_from = datetime.strptime(m.group('daterangefrom'), '%Y-%m-%d')
                d_to = datetime.strptime(m.group('daterangeto'), '%Y-%m-%d')
            else:
                d_from = datetime.strptime(value, '%Y-%m-%d')
                d_to = datetime.strptime(value, '%Y-%m-%d')

            # now min/max to get midnight on day1 through just before midnight on day2
            # note: this is a hack because as of the writing of this app,
            # __date doesn't yet exist as a queryable field thus any
            # timestamps not at midnight were incorrectly left out
            d_from = datetime.combine(d_from, time.min)
            d_to = datetime.combine(d_to, time.max)

            d_post[self.post_name] = d_post[self.post_name].filter(clock_time__gte=d_from)
            d_post[self.post_name] = d_post[self.post_name].filter(clock_time__lte=d_to)

        elif key == 'limit':
            order_by = 'clock_time'
            if value == 'last':
                order_by = '-clock_time'
            d_post[self.post_name] = d_post[self.post_name].order_by(order_by)
            self.limit.setlimit(0, 1)

        else:
            raise NameError

    def read(self, request, *args, **kwargs):
        # empid is required, so make sure it exists before running BaseApiHandler's read method
        if not('empid' in kwargs and kwargs['empid'] <> None and kwargs['empid'] >= 0):
            return {'status':0,'statusmsg':'empid cannot be empty'}
        else:
            return BaseApiHandler.read(self, request, *args, **kwargs)
4

2 に答える 2

0

get_limitプロパティアクセサーを介して、内部制限リストのエイリアスを作成している可能性があります。このアクセサー内の削除(または少なくともprintステートメントの追加)を試してください。ローカルリストをget_limitにバインドするコードが外部にある場合は、append、del、または[:]などのスライスへの割り当てを使用してコンテンツを更新できます。またはこれを試してください:

def get_limit(self): 
    return self._limit[:]

内部リストを外部名にバインドする代わりに、内部リストのコピーを作成します。

于 2010-03-31T20:42:12.520 に答える
0

limit が長さ 2 のリストである場合に True を返すことができる場合、コードに基本的な欠陥があると言えますがhas_limit()、 limit が 3 要素の長さよりも短い場合、この行は失敗します。

s_query = '%swith limit[%s,%s](limit,%s > traceback:%s),' % 
          (s_query, self.limit[0], self.limit[1], kwargs['limit'], 
           self.limit[2])

self.limit を無効な長さのリストに初期化しているのはなぜですか? このコードをもう少し防御的にすることもできます。

if self.has_limit():
    s_query += 'with limit[%s,%s]' % self.limit[0:1]
    if 'limit' in kwargs and len(self.limit) > 2:
        s_query += '(limit,%s > traceback:%s),' % 
              (kwargs['limit'], self.limit[2])
于 2010-03-31T17:05:36.750 に答える