Sample

pyint-8496a2685137-I13

Lower every eligible synchronous for loop to a while loop

Matched original source

Lines338

Scores

MethodScoreScale
Posnett00–1; higher is more readable
Dorn0.7430–1; higher is more readable
Mi0.5980–1; higher is more readable
LLOC177Logical lines; fewer is more readable
DeepSeek V4 Pro6.670–20; mean of three runs
GPT-6.1 Sol12.670–20; mean of three runs
BTTF0.4040–1; higher is more readable

Views

Source code

from __future__ import annotations import calendar import copy import time from collections.abc import Iterator, MutableMapping from http.cookiejar import Cookie, CookieJar, CookiePolicy from typing import TYPE_CHECKING, Any, TypeVar, overload from ._internal_utils import to_native_string from ._types import is_prepared as _is_prepared from .compat import Morsel, cookielib, urlparse, urlunparse import threading class RequestsCookieJar(CookieJar, MutableMapping[str, str | None]): # type: ignore[misc] """Compatibility class; is a http.cookiejar.CookieJar, but exposes a dict interface. This is the CookieJar we create by default for requests and sessions that don't specify one, since some clients may expect response.cookies and session.cookies to support dict operations. Requests does not use the dict interface internally; it's just for compatibility with external client code. All requests code should work out of the box with externally provided instances of ``CookieJar``, e.g. ``LWPCookieJar`` and ``FileCookieJar``. Unlike a regular CookieJar, this class is pickleable. .. warning:: dictionary operations that are normally O(1) may be O(n). """ _policy: CookiePolicy def get( # type: ignore[override] self, name: str, default: str | None = None, domain: str | None = None, path: str | None = None, ) -> str | None: """Dict-like get() that also supports optional domain and path args in order to resolve naming collisions from using one cookie jar over multiple domains. .. warning:: operation is O(n), not O(1). """ try: return self._find_no_duplicates(name, domain, path) except KeyError: return default def set( self, name: str, value: str | Morsel[dict[str, str]] | None, **kwargs: Any ) -> Cookie | None: """Dict-like set() that also supports optional domain and path args in order to resolve naming collisions from using one cookie jar over multiple domains. """ # support client code that unsets cookies by assignment of a None value: if value is None: remove_cookie_by_name( self, name, domain=kwargs.get("domain"), path=kwargs.get("path") ) return if isinstance(value, Morsel): c = morsel_to_cookie(value) else: c = create_cookie(name, value, **kwargs) self.set_cookie(c) return c def iterkeys(self) -> Iterator[str]: """Dict-like iterkeys() that returns an iterator of names of cookies from the jar. .. seealso:: itervalues() and iteritems(). """ loopIterator = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIterator) except __import__("builtins").StopIteration: break yield cookie.name def keys(self) -> list[str]: # type: ignore[override] """Dict-like keys() that returns a list of names of cookies from the jar. .. seealso:: values() and items(). """ return list(self.iterkeys()) def itervalues(self) -> Iterator[str | None]: """Dict-like itervalues() that returns an iterator of values of cookies from the jar. .. seealso:: iterkeys() and iteritems(). """ loopIteratorA = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIteratorA) except __import__("builtins").StopIteration: break yield cookie.value def values(self) -> list[str | None]: # type: ignore[override] """Dict-like values() that returns a list of values of cookies from the jar. .. seealso:: keys() and items(). """ return list(self.itervalues()) def iteritems(self) -> Iterator[tuple[str, str | None]]: """Dict-like iteritems() that returns an iterator of name-value tuples from the jar. .. seealso:: iterkeys() and itervalues(). """ loopIteratorB = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIteratorB) except __import__("builtins").StopIteration: break yield cookie.name, cookie.value def items(self) -> list[tuple[str, str | None]]: # type: ignore[override] """Dict-like items() that returns a list of name-value tuples from the jar. Allows client-code to call ``dict(RequestsCookieJar)`` and get a vanilla python dict of key value pairs. .. seealso:: keys() and values(). """ return list(self.iteritems()) def list_domains(self) -> list[str]: """Utility method to list all the domains in the jar.""" domains: list[str] = [] loopIteratorC = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIteratorC) except __import__("builtins").StopIteration: break if cookie.domain not in domains: domains.append(cookie.domain) return domains def list_paths(self) -> list[str]: """Utility method to list all the paths in the jar.""" paths: list[str] = [] loopIteratorD = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIteratorD) except __import__("builtins").StopIteration: break if cookie.path not in paths: paths.append(cookie.path) return paths def multiple_domains(self) -> bool: """Returns True if there are multiple domains in the jar. Returns False otherwise. :rtype: bool """ domains: list[str] = [] for cookie in iter(self): if cookie.domain is not None and cookie.domain in domains: # type: ignore[reportUnnecessaryComparison] # defensive check return True domains.append(cookie.domain) return False # there is only one domain in jar def get_dict( self, domain: str | None = None, path: str | None = None ) -> dict[str, str | None]: """Takes as an argument an optional domain and path and returns a plain old Python dict of name-value pairs of cookies that meet the requirements. :rtype: dict """ dictionary: dict[str, str | None] = {} loopIteratorE = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIteratorE) except __import__("builtins").StopIteration: break if (domain is None or cookie.domain == domain) and ( path is None or cookie.path == path ): dictionary[cookie.name] = cookie.value return dictionary def __iter__(self) -> Iterator[Cookie]: # type: ignore[override] """RequestCookieJar's __iter__ comes from CookieJar not MutableMapping.""" return super().__iter__() def __contains__(self, name: object) -> bool: try: return super().__contains__(name) except CookieConflictError: return True def __getitem__(self, name: str) -> str | None: """Dict-like __getitem__() for compatibility with client code. Throws exception if there are more than one cookie with name. In that case, use the more explicit get() method instead. .. warning:: operation is O(n), not O(1). """ return self._find_no_duplicates(name) def __setitem__( self, name: str, value: str | Morsel[dict[str, str]] | None ) -> None: """Dict-like __setitem__ for compatibility with client code. Throws exception if there is already a cookie of that name in the jar. In that case, use the more explicit set() method instead. """ self.set(name, value) def __delitem__(self, name: str) -> None: """Deletes a cookie given a name. Wraps ``http.cookiejar.CookieJar``'s ``remove_cookie_by_name()``. """ remove_cookie_by_name(self, name) def set_cookie(self, cookie: Cookie, *args: Any, **kwargs: Any) -> None: if ( (value := cookie.value) is not None and value.startswith('"') and value.endswith('"') ): cookie.value = value.replace('\\"', "") return super().set_cookie(cookie, *args, **kwargs) def update( # type: ignore[override] self, other: CookieJar | SupportsKeysAndGetItem[str, str] ) -> None: """Updates this jar with cookies from another CookieJar or dict-like""" if isinstance(other, cookielib.CookieJar): loopIteratorF = __import__("builtins").iter(other) while True: try: cookie = __import__("builtins").next(loopIteratorF) except __import__("builtins").StopIteration: break self.set_cookie(copy.copy(cookie)) else: super().update(other) def _find( self, name: str, domain: str | None = None, path: str | None = None ) -> str | None: """Requests uses this method internally to get cookie values. If there are conflicting cookies, _find arbitrarily chooses one. See _find_no_duplicates if you want an exception thrown if there are conflicting cookies. :param name: a string containing name of cookie :param domain: (optional) string containing domain of cookie :param path: (optional) string containing path of cookie :return: cookie.value """ loopIteratorG = __import__("builtins").iter(iter(self)) while True: try: cookie = __import__("builtins").next(loopIteratorG) except __import__("builtins").StopIteration: break if cookie.name == name: if domain is None or cookie.domain == domain: if path is None or cookie.path == path: return cookie.value raise KeyError(f"name={name!r}, domain={domain!r}, path={path!r}") def _find_no_duplicates( self, name: str, domain: str | None = None, path: str | None = None ) -> str: """Both ``__get_item__`` and ``get`` call this function: it's never used elsewhere in Requests. :param name: a string containing name of cookie :param domain: (optional) string containing domain of cookie :param path: (optional) string containing path of cookie :raises KeyError: if cookie is not found :raises CookieConflictError: if there are multiple cookies that match name and optionally domain and path :return: cookie.value """ toReturn = None for cookie in iter(self): if cookie.name == name: if domain is None or cookie.domain == domain: if path is None or cookie.path == path: if toReturn is not None: # if there are multiple cookies that meet passed in criteria raise CookieConflictError( f"There are multiple cookies with name, {name!r}" ) # we will eventually return this as long as no cookie conflict toReturn = cookie.value if toReturn is not None: return toReturn raise KeyError(f"name={name!r}, domain={domain!r}, path={path!r}") def __getstate__(self) -> dict[str, Any]: """Unlike a normal CookieJar, this class is pickleable.""" state = self.__dict__.copy() # remove the unpickleable RLock object state.pop("_cookies_lock") return state def __setstate__(self, state: dict[str, Any]) -> None: """Unlike a normal CookieJar, this class is pickleable.""" self.__dict__.update(state) if "_cookies_lock" not in self.__dict__: self._cookies_lock = threading.RLock() def copy(self) -> RequestsCookieJar: """Return a copy of this RequestsCookieJar.""" new_cj = RequestsCookieJar() new_cj.set_policy(self.get_policy()) new_cj.update(self) return new_cj def get_policy(self) -> CookiePolicy: """Return the CookiePolicy instance used.""" return self._policy

Loading feature diagnosis…

Source regions

Feature contributions

Feature contributions

FeatureValueContribution
(log-odds)
Halstead volumeHow much symbolic information the code contains. It combines the total number of operators and values with the number of distinct ones.10739.276−354.396
Source linesThe number of non-empty source lines.338+135.2
Byte entropyHow varied the bytes in the source text are. Repeated bytes give a lower value; a more even mixture gives a higher value.4.378−6.566

Intercept 8.87 + feature contributions = -216.892 (log-odds).
Score = sigmoid(-216.892) = 0.

Feature contributions

FeatureValueContribution
(log-odds)
Comment layout frequencyHow frequently the layout switches between lines containing only comments or whitespace and lines containing code.339−10.967
Indentation frequencyHow rapidly indentation changes from line to line.238+7.495
Identifier layout (vertical)How rapidly the visual pattern of identifier characters changes from top to bottom.337.815+4.312
Identifier layout (horizontal)How rapidly the visual pattern of identifier characters changes from left to right. It measures spatial frequency, not the meaning of the names.133.973−2.009
Comment layout (horizontal)How rapidly the visual pattern of comment characters changes from left to right.133.971+1.357
Operator visual areaThe share of the code’s visual area occupied by operators such as +, =, and *.0.055+0.405
Number layout frequencyHow rapidly the number of numeric values per line changes through the code.1+0.398

Intercept 0.07 + feature contributions = 1.062 (log-odds).
Score = sigmoid(1.062) = 0.743.

Score0.598
Score177
Run 1

The code has some readable aspects, such as clear docstrings and type hints, but it is severely undermined by bizarre implementation choices. The repeated use of `__import__('builtins').iter()` and `__import__('builtins').next()` instead of Python's built-in `for` loop is obfuscated and unnecessarily verbose, breaking natural reading flow. This pattern creates deeply nested while/try/except blocks, increasing cognitive load. Several methods (iterkeys, itervalues, iteritems, list_domains, list_paths, get_dict, _find, update) duplicate this convoluted iteration logic instead of using a simple `for cookie in self`. The code also mixes multiple actions per line in some conditions and suffers from disorganized structure due to repetitive boilerplate. While class-level documentation and some method intentions are clear, the execution details make the code much harder to read than necessary.

Run 2

The code suffers from severe readability issues despite having clear docstrings and type hints. The most glaring problem is the repeated use of `__import__('builtins').iter(iter(self))` and `__import__('builtins').next(loopIterator)` instead of simply using `for cookie in self:` loops. This creates massive verbosity and completely destroys the natural reading flow. The code is unnecessarily long—each iteration method repeats the same awkward boilerplate pattern (loopIteratorA through G), when a simple for loop would suffice in 2-3 lines. While the class structure is logical and methods have clear names, the action granularity is poor: multiple unnecessary statements per simple iteration. The nesting level is acceptable, but the intent of the iteration pattern is obfuscated by bizarre import calls. The inconsistent naming (loopIteratorA, loopIteratorB, etc.) suggests copy-paste coding. The use of walrus operator in set_cookie is clear, but it's overshadowed by the surrounding mess. Docstrings are excellent, but they can't compensate for the core iteration pattern being fundamentally unreadable.

Run 3

The code has some readability strengths, including clear docstrings, type hints, and logical class organization. However, it is severely undermined by bizarre and inconsistent iterator implementations. The repeated use of `__import__('builtins').iter/next/StopIteration` inside manual while-loops is a major red flag: it is obscure, non-idiomatic, and seems designed to avoid normal language constructs. This pattern appears in nine methods, creating massive duplication and obscuring intent. The code also mixes naming styles (camelCase loop variables vs snake_case), uses unnecessary type-ignore comments, and has inconsistent formatting. The `multiple_domains` logic appears inverted. While individual method docstrings are clear, the overall reading flow is jarring due to the unnatural iteration pattern and scattered imports. Readability would improve dramatically by replacing these loops with simple `for cookie in self` constructs and standard exception handling. As written, the code reads like intentionally obfuscated Python despite otherwise clean documentation and structure.

Run 1

1. Code Structure: Methods are logically grouped within a cohesive class, with clear separation of mapping operations, lookup helpers, and serialization. Import ordering is inconsistent. 2. Nesting: Most methods are shallow, but lookup helpers contain deeply nested conditions, and manual iteration adds unnecessary try/except nesting. 3. Clarity of intent: Descriptive method names, type hints, and docstrings clarify purpose. Generic iterator names and a mismatch between multiple_domains() documentation and its duplicate-domain check undermine clarity. 4. Code length: Repeated manual iterator machinery and dynamic builtins imports make simple loops unnecessarily verbose; some documentation is repetitive. 5. Action granularity: Statements generally perform one action per line, although dynamic imports embedded in iterator operations add distracting complexity. 6. Reading flow: Method boundaries and consistent formatting support navigation, but repeated while/try/except patterns interrupt natural reading compared with ordinary for loops.

Run 2

Code Structure: Well-organized class with focused methods and recognizable mapping operations, though imports are inconsistently grouped. Nesting: Most methods are shallow, but lookup methods contain several nested conditions that could be flattened. Clarity of intent: Descriptive method names, type annotations, and extensive docstrings explain the API well; generic camelCase variables and repeated dynamic builtins imports obscure otherwise simple iteration. Code length: Unnecessarily verbose manual iterator handling repeats try/except blocks where ordinary for loops would suffice; some documentation is repetitive. Action granularity: Statements generally perform one action per line, with few readability concerns. Reading flow: Method grouping and delegation provide a natural progression, but iteration boilerplate, long inline suppression comments, and inconsistent naming interrupt scanning. Overall, the interface is understandable, but implementation verbosity and nesting materially reduce readability.

Run 3

Code Structure: Methods are logically separated within a cohesive class, with clear grouping of mapping operations and cookie utilities. Nesting: Most methods are shallow, but lookup methods contain unnecessarily deep conditional chains. Clarity of intent: Descriptive method names, type annotations, and extensive docstrings help, although generic iterator names and some documentation/implementation mismatches introduce ambiguity. Code length: Repeated manual iterator setup, while loops, and StopIteration handling substantially inflate code that simple for loops could express. Action granularity: Statements generally perform one action per line, though dynamic imports and the assignment expression add complexity within expressions. Reading flow: Method organization is predictable, but repeated __import__("builtins") calls, inconsistent naming conventions, long inline comments, and verbose iteration machinery interrupt otherwise natural Python reading flow.