Sample

pyint-e42523adfb4f-I13

Lower every eligible synchronous for loop to a while loop

Matched original source

Lines427

Scores

MethodScoreScale
Posnett00–1; higher is more readable
Dorn0.0060–1; higher is more readable
Mi0.5610–1; higher is more readable
LLOC196Logical lines; fewer is more readable
DeepSeek V4 Pro14.670–20; mean of three runs
GPT-6.1 Sol150–20; mean of three runs
BTTF0.3870–1; higher is more readable

Views

Source code

from __future__ import annotations import datetime import encodings.idna # noqa: F401 from collections.abc import Callable, Generator, Iterable, Iterator, Mapping from io import UnsupportedOperation from typing import ( TYPE_CHECKING, Any, Final, Literal, cast, overload, ) from urllib3.exceptions import ( DecodeError, LocationParseError, ProtocolError, ReadTimeoutError, SSLError, ) from urllib3.fields import RequestField from urllib3.filepost import encode_multipart_formdata from urllib3.util import parse_url from . import _types as _t from ._internal_utils import to_native_string, unicode_is_ascii from .auth import HTTPBasicAuth from .compat import ( JSONDecodeError, basestring, builtin_str, chardet, cookielib, urlencode, urlsplit, urlunparse, ) from .compat import json as complexjson from .cookies import ( _copy_cookie_jar, cookiejar_from_dict, get_cookie_header, ) from .exceptions import ( ChunkedEncodingError, ConnectionError, ContentDecodingError, HTTPError, InvalidJSONError, InvalidURL, MissingSchema, StreamConsumedError, ) from .exceptions import JSONDecodeError as RequestsJSONDecodeError from .exceptions import SSLError as RequestsSSLError from .hooks import default_hooks from .status_codes import codes from .structures import CaseInsensitiveDict from .utils import ( check_header_validity, get_auth_from_url, guess_filename, guess_json_utf, iter_slices, parse_header_links, requote_uri, stream_decode_response_unicode, super_len, to_key_val_list, ) class PreparedRequest(RequestEncodingMixin, RequestHooksMixin): """The fully mutable :class:`PreparedRequest <PreparedRequest>` object, containing the exact bytes that will be sent to the server. Instances are generated from a :class:`Request <Request>` object, and should not be instantiated manually; doing so may produce undesirable effects. Usage:: >>> import requests >>> req = requests.Request('GET', 'https://httpbin.org/get') >>> r = req.prepare() >>> r <PreparedRequest [GET]> >>> s = requests.Session() >>> s.send(r) <Response [200]> """ method: str | None url: str | None headers: CaseInsensitiveDict[str | bytes] _cookies: RequestsCookieJar | CookieJar | None body: _t.BodyType hooks: dict[str, list[_t.HookType]] _body_position: int | object | None def __init__(self) -> None: #: HTTP verb to send to the server. self.method = None #: HTTP URL to send the request to. self.url = None #: dictionary of HTTP headers. self.headers = None # type: ignore[assignment] # The `CookieJar` used to create the Cookie header will be stored here # after prepare_cookies is called self._cookies = None #: request body to send to the server. self.body = None #: dictionary of callback hooks, for internal usage. self.hooks = default_hooks() #: integer denoting starting position of a readable file-like body. self._body_position = None def prepare( self, method: str | None = None, url: _t.UriType | None = None, headers: Mapping[str, str | bytes] | None = None, files: _t.FilesType = None, data: _t.DataType = None, params: _t.ParamsType = None, auth: _t.AuthType = None, cookies: RequestsCookieJar | CookieJar | dict[str, str] | None = None, hooks: _t.HooksInputType | None = None, json: _t.JsonType = None, ) -> None: """Prepares the entire request with the given parameters.""" url = cast("_t.UriType", url) self.prepare_method(method) self.prepare_url(url, params) self.prepare_headers(headers) self.prepare_cookies(cookies) self.prepare_body(data, files, json) self.prepare_auth(auth, url) # Note that prepare_auth must be last to enable authentication schemes # such as OAuth to work on a fully prepared request. # This MUST go after prepare_auth. Authenticators could add a hook self.prepare_hooks(hooks) def __repr__(self) -> str: return f"<PreparedRequest [{self.method}]>" def copy(self) -> PreparedRequest: p = PreparedRequest() p.method = self.method p.url = self.url p.headers = self.headers.copy() if self.headers is not None else None # type: ignore[assignment] p._cookies = _copy_cookie_jar(self._cookies) p.body = self.body p.hooks = self.hooks p._body_position = self._body_position return p def prepare_method(self, method: str | None) -> None: """Prepares the given HTTP method.""" self.method = method if self.method is not None: self.method = to_native_string(self.method.upper()) @staticmethod def _get_idna_encoded_host(host: str) -> str: import idna try: host = idna.encode(host, uts46=True).decode("utf-8") except idna.IDNAError: raise UnicodeError return host def prepare_url( self, url: _t.UriType, params: _t.ParamsType, ) -> None: """Prepares the given HTTP URL.""" #: Accept objects that have string representations. #: We're unable to blindly call unicode/str functions #: as this will include the bytestring indicator (b'') #: on python 3.x. #: https://github.com/psf/requests/pull/2238 if isinstance(url, bytes): url = url.decode("utf8") else: url = str(url) # Remove leading whitespaces from url url = url.lstrip() # Don't do any URL preparation for non-HTTP schemes like `mailto`, # `data` etc to work around exceptions from `url_parse`, which # handles RFC 3986 only. if ":" in url and not url.lower().startswith("http"): self.url = url return # Support for unicode domain names and paths. try: scheme, auth, host, port, path, query, fragment = parse_url(url) except LocationParseError as e: raise InvalidURL(*e.args) if not scheme: raise MissingSchema( f"Invalid URL {url!r}: No scheme supplied. " f"Perhaps you meant https://{url}?" ) if not host: raise InvalidURL(f"Invalid URL {url!r}: No host supplied") # In general, we want to try IDNA encoding the hostname if the string contains # non-ASCII characters. This allows users to automatically get the correct IDNA # behaviour. For strings containing only ASCII characters, we need to also verify # it doesn't start with a wildcard (*), before allowing the unencoded hostname. if not unicode_is_ascii(host): try: host = self._get_idna_encoded_host(host) except UnicodeError: raise InvalidURL("URL has an invalid label.") elif host.startswith(("*", ".")): raise InvalidURL("URL has an invalid label.") # Carefully reconstruct the network location netloc = auth or "" if netloc: netloc += "@" netloc += host if port: netloc += f":{port}" # Bare domains aren't valid URLs. if not path: path = "/" if isinstance(params, (str, bytes)): params = to_native_string(params) if params is not None: enc_params = self._encode_params(params) else: enc_params = "" if enc_params: if query: query = f"{query}&{enc_params}" else: query = enc_params url = requote_uri(urlunparse((scheme, netloc, path, "", query, fragment))) self.url = url def prepare_headers(self, headers: Mapping[str, str | bytes] | None) -> None: """Prepares the given HTTP headers.""" self.headers = CaseInsensitiveDict() if headers: for header in headers.items(): # Raise exception on invalid header value. check_header_validity(header) name, value = header self.headers[to_native_string(name)] = value def prepare_body( self, data: _t.DataType, files: _t.FilesType, json: _t.JsonType = None ) -> None: """Prepares the given HTTP body data.""" # Check if file, fo, generator, iterator. # If not, run through normal process. # Nottin' on you. body = None content_type = None if not data and json is not None: # urllib3 requires a bytes-like body. Python 2's json.dumps # provides this natively, but Python 3 gives a Unicode string. content_type = "application/json" try: body = complexjson.dumps(json, allow_nan=False) except ValueError as ve: raise InvalidJSONError(ve, request=self) if not isinstance(body, bytes): body = body.encode("utf-8") # data that proxies attributes to underlying objects needs hasattr is_iterable = isinstance(data, Iterable) or hasattr(data, "__iter__") if is_iterable and not isinstance(data, (str, bytes, list, tuple, Mapping)): try: length = super_len(data) except (TypeError, AttributeError, UnsupportedOperation): length = None body = data if getattr(body, "tell", None) is not None: # Record the current file position before reading. # This will allow us to rewind a file in the event # of a redirect. try: self._body_position = body.tell() # type: ignore[union-attr] # guarded by getattr check except OSError: # This differentiates from None, allowing us to catch # a failed `tell()` later when trying to rewind the body self._body_position = object() if files: raise NotImplementedError( "Streamed bodies and files are mutually exclusive." ) if length: self.headers["Content-Length"] = builtin_str(length) else: self.headers["Transfer-Encoding"] = "chunked" else: # After is_stream filtering, remaining data is raw (not streamed) raw_data = cast("_t.RawDataType | None", data) # Multi-part file uploads. if files: (body, content_type) = self._encode_files(files, raw_data) else: if raw_data: body = self._encode_params(raw_data) if isinstance(data, basestring) or _t.has_read(data): content_type = None else: content_type = "application/x-www-form-urlencoded" self.prepare_content_length(body) # Add content-type if it wasn't explicitly provided. if content_type and ("content-type" not in self.headers): self.headers["Content-Type"] = content_type self.body = body # type: ignore[assignment] # body transforms from DataType to BodyType def prepare_content_length(self, body: _t.BodyType) -> None: """Prepare Content-Length header based on request method and body""" if body is not None: length = super_len(body) if length: # If length exists, set it. Otherwise, we fallback # to Transfer-Encoding: chunked. self.headers["Content-Length"] = builtin_str(length) elif ( self.method not in ("GET", "HEAD") and self.headers.get("Content-Length") is None ): # Set Content-Length to 0 for methods that can have a body # but don't provide one. (i.e. not GET or HEAD) self.headers["Content-Length"] = "0" def prepare_auth( self, auth: _t.AuthType, url: _t.UriType = "", ) -> None: """Prepares the given HTTP auth data.""" # If no Auth is explicitly provided, extract it from the URL first. if auth is None: url_auth = get_auth_from_url(cast(str, self.url)) auth = url_auth if any(url_auth) else None if auth: if isinstance(auth, tuple) and len(auth) == 2: # type: ignore[arg-type] # pyright widens tuple from Callable in AuthType # special-case basic HTTP auth auth_handler = HTTPBasicAuth(*auth) # type: ignore[arg-type] # pyright widens tuple from Callable in AuthType else: # TODO: can be fixed by flipping the conditionals auth_handler = cast("Callable[..., PreparedRequest]", auth) # Allow auth to make its changes. r = auth_handler(self) # Update self to reflect the auth changes. self.__dict__.update(r.__dict__) # Recompute Content-Length self.prepare_content_length(self.body) def prepare_cookies( self, cookies: RequestsCookieJar | CookieJar | dict[str, str] | None ) -> None: """Prepares the given HTTP cookie data. This function eventually generates a ``Cookie`` header from the given cookies using cookielib. Due to cookielib's design, the header will not be regenerated if it already exists, meaning this function can only be called once for the life of the :class:`PreparedRequest <PreparedRequest>` object. Any subsequent calls to ``prepare_cookies`` will have no actual effect, unless the "Cookie" header is removed beforehand. """ if isinstance(cookies, cookielib.CookieJar): self._cookies = cookies else: self._cookies = cookiejar_from_dict(cookies) cookies_jar = cast("CookieJar", self._cookies) cookie_header = get_cookie_header(cookies_jar, self) if cookie_header is not None: self.headers["Cookie"] = cookie_header def prepare_hooks(self, hooks: _t.HooksInputType | None) -> None: """Prepares the given hooks.""" # hooks can be passed as None to the prepare method and to this # method. To prevent iterating over None, simply use an empty list # if hooks is False-y hooks = hooks or {} loopIterator = __import__("builtins").iter(hooks) while True: try: event = __import__("builtins").next(loopIterator) except __import__("builtins").StopIteration: break self.register_hook(event, hooks[event])

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.13313.165−439.334
Source linesThe number of non-empty source lines.427+170.8
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.464−6.697

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

Feature contributions

FeatureValueContribution
(log-odds)
Comment layout frequencyHow frequently the layout switches between lines containing only comments or whitespace and lines containing code.428−14.118
Indentation frequencyHow rapidly indentation changes from line to line.338+10.954
Number layout frequencyHow rapidly the number of numeric values per line changes through the code.338−7.335
Identifier layout (vertical)How rapidly the visual pattern of identifier characters changes from top to bottom.426.963+5.558
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.958−2.009
Comment layout (horizontal)How rapidly the visual pattern of comment characters changes from left to right.133.986+1.357
Operator visual areaThe share of the code’s visual area occupied by operators such as +, =, and *.0.05+0.451

Intercept 0.07 + feature contributions = -5.071 (log-odds).
Score = sigmoid(-5.071) = 0.006.

Score0.561
Score196
Run 1

The code scores 15/20. Structure is logical with clear class organization and well-documented methods, but `prepare_body` and `prepare_url` contain complex conditional blocks. Nesting reaches 3-4 levels in spots, particularly in `prepare_body`'s iterable checks and `prepare_cookies`' type checks, though most early returns help flatten flow. Intent is generally clear from comprehensive docstrings and inline comments explaining OAuth ordering constraints. Length is reasonable for a 400-line request preparation class, though type casting comments clutter several lines. Action granularity is mostly one-action-per-line but breaks down with complex conditionals like `if is_iterable and not isinstance(...)`. Reading flow follows a natural progression (method → URL → headers → body → auth → cookies → hooks), but awkward constructs like `__import__('builtins').iter()` in `prepare_hooks` break idiomatic Python readability.

Run 2

The code is generally readable with good structure, but has some issues. It's well-organized with clear method separation and good docstrings, but the massive import block at the top is verbose. Nesting is mostly flat, though prepare_body() has multiple levels. Intent is clear with descriptive method names, but some comments are overly verbose or unnecessary. Code length is reasonable per method but could be more concise in places. Action granularity is mostly one-action-per-line, though some lines are overly long (e.g., the type ignores). Reading flow is natural in most methods, but prepare_hooks() uses an awkward __import__ pattern for iteration that breaks flow, and prepare_url() is quite dense. The code is professional but not exceptional.

Run 3

The code shows good structure with clear class organization and type hints, but has readability issues. The imports section is very long, though logically grouped. Nesting is moderate with some deep conditionals in `prepare_body`. Most methods have clear intent with descriptive names and docstrings. Code length is reasonable for functionality, but `prepare_body` is lengthy. Action granularity is mostly good, though some lines combine multiple operations. The main concern is `prepare_hooks` which uses unconventional `__import__('builtins')` calls instead of standard Python iteration, breaking reading flow. Comments are helpful when present. Type annotations are extensive but sometimes require casting which adds noise. Overall, the code is maintainable for experienced Python developers but has sections that could be refactored for clarity.

Run 1

Code Structure: Well separated into focused preparation methods, with a clear orchestration method. Nesting: Mostly shallow, but prepare_body contains several nested branches and exception handlers that increase cognitive load. Clarity of intent: Descriptive method names, annotations, and docstrings explain most behavior; vague comments and extensive type-ignore annotations add noise. Code length: Generally justified by HTTP edge cases, though verbose imports, comments, and the manual iterator loop could be simplified. Action granularity: Statements generally perform one action per line, making changes easy to follow. Reading flow: The preparation sequence is natural and ordering constraints are documented. However, prepare_hooks disrupts readability with repeated dynamic builtins imports and a while/try loop instead of a straightforward for loop. Overall, readable and organized, with localized complexity and unnecessary verbosity.

Run 2

Code Structure: Request preparation is logically divided into focused methods, although prepare_body combines several responsibilities. Nesting: Most methods are relatively flat, but body handling contains multiple nested conditionals and exception handlers. Clarity of intent: Descriptive method names, annotations, and docstrings explain the purpose well; cryptic variables, an irrelevant comment, and numerous type-ignore annotations add noise. Code length: Much of the length supports legitimate HTTP edge cases, but verbose comments and the expanded hook iteration are unnecessarily long. Action granularity: Statements generally perform one action per line, making individual operations easy to follow. Reading flow: The main prepare method presents a clear sequence, with helpful explanations of ordering constraints. However, prepare_hooks abruptly replaces straightforward iteration with dynamic builtins imports and manual StopIteration handling, disrupting otherwise conventional Python flow.

Run 3

Code Structure: Preparation responsibilities are logically separated into focused methods, although prepare_body combines several complex paths. Nesting: Most methods are shallow, but body handling contains multiple nested conditionals and exception blocks. Clarity of intent: Descriptive method names, annotations, and documentation explain the request lifecycle; obscure comments, terse variables, and type-ignore clutter reduce clarity. Code length: Generally justified by protocol edge cases, but verbose comments and the manually expanded iterator loop add unnecessary length. Action granularity: Statements mostly perform one action per line, making state changes easy to track. Reading flow: prepare provides a clear sequence, and individual methods mostly follow natural validation-to-transformation order. However, prepare_hooks disrupts this flow with repeated dynamic builtins imports and a while/try/except loop instead of a straightforward for loop. Overall, readable and well organized, with localized complexity and avoidable verbosity.