| Conditions | 10 |
| Total Lines | 74 |
| Lines | 0 |
| Ratio | 0 % |
| Tests | 36 |
| CRAP Score | 11.5625 |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
Complex classes like _generate() often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
| 1 | 1 | import os |
|
| 67 | 1 | def _generate(top, bottom, font, background, width, height): |
|
| 68 | """Add text to an image and save it.""" |
||
| 69 | 1 | log.info("Loading background: %s", background) |
|
| 70 | 1 | image = ImageFile.open(background) |
|
| 71 | 1 | if image.mode not in ('RGB', 'RGBA'): |
|
| 72 | 1 | if image.format == 'JPEG': |
|
| 73 | 1 | image = image.convert('RGB') |
|
| 74 | 1 | image.format = 'JPEG' |
|
| 75 | else: |
||
| 76 | 1 | image = image.convert('RGBA') |
|
| 77 | 1 | image.format = 'PNG' |
|
| 78 | |||
| 79 | # Resize to a maximum height and width |
||
| 80 | 1 | ratio = image.size[0] / image.size[1] |
|
| 81 | 1 | if width and height: |
|
| 82 | if width < height * ratio: |
||
| 83 | dimensions = width, int(width / ratio) |
||
| 84 | else: |
||
| 85 | dimensions = int(height * ratio), height |
||
| 86 | 1 | elif width: |
|
| 87 | dimensions = width, int(width / ratio) |
||
| 88 | 1 | elif height: |
|
| 89 | dimensions = int(height * ratio), height |
||
| 90 | else: |
||
| 91 | 1 | dimensions = 800, int(800 / ratio) |
|
| 92 | 1 | image = image.resize(dimensions, ImageFile.LANCZOS) |
|
| 93 | 1 | image.format = 'PNG' |
|
| 94 | |||
| 95 | # Draw image |
||
| 96 | 1 | draw = ImageDraw.Draw(image) |
|
| 97 | |||
| 98 | 1 | max_font_size = int(image.size[1] / 5) |
|
| 99 | 1 | min_font_size_single_line = int(image.size[1] / 12) |
|
| 100 | 1 | max_text_len = image.size[0] - 20 |
|
| 101 | 1 | top_font_size, top = _optimize_font_size(font, top, max_font_size, |
|
| 102 | min_font_size_single_line, |
||
| 103 | max_text_len) |
||
| 104 | 1 | bottom_font_size, bottom = _optimize_font_size(font, bottom, max_font_size, |
|
| 105 | min_font_size_single_line, |
||
| 106 | max_text_len) |
||
| 107 | |||
| 108 | 1 | top_font = ImageFont.truetype(font, top_font_size) |
|
| 109 | 1 | bottom_font = ImageFont.truetype(font, bottom_font_size) |
|
| 110 | |||
| 111 | 1 | top_text_size = draw.multiline_textsize(top, top_font) |
|
| 112 | 1 | bottom_text_size = draw.multiline_textsize(bottom, bottom_font) |
|
| 113 | |||
| 114 | # Find top centered position for top text |
||
| 115 | 1 | top_text_position_x = (image.size[0] / 2) - (top_text_size[0] / 2) |
|
| 116 | 1 | top_text_position_y = 0 |
|
| 117 | 1 | top_text_position = (top_text_position_x, top_text_position_y) |
|
| 118 | |||
| 119 | # Find bottom centered position for bottom text |
||
| 120 | 1 | bottom_text_size_x = (image.size[0] / 2) - (bottom_text_size[0] / 2) |
|
| 121 | 1 | bottom_text_size_y = image.size[1] - bottom_text_size[1] * (7 / 6) |
|
| 122 | 1 | bottom_text_position = (bottom_text_size_x, bottom_text_size_y) |
|
| 123 | |||
| 124 | 1 | _draw_outlined_text(draw, top_text_position, |
|
| 125 | top, top_font, top_font_size) |
||
| 126 | 1 | _draw_outlined_text(draw, bottom_text_position, |
|
| 127 | bottom, bottom_font, bottom_font_size) |
||
| 128 | |||
| 129 | # Pad image if a specific dimension is requested |
||
| 130 | 1 | if width and height: |
|
| 131 | base_width, base_height = image.size |
||
| 132 | padding = ImageFile.new('RGBA', (width, height), (0, 0, 0)) |
||
| 133 | padding.format = 'PNG' |
||
| 134 | padding_width, padding_height = padding.size |
||
| 135 | offset = ((padding_width - base_width) // 2, |
||
| 136 | (padding_height - base_height) // 2) |
||
| 137 | padding.paste(image, offset) |
||
| 138 | image = padding |
||
| 139 | |||
| 140 | 1 | return image |
|
| 141 | |||
| 221 |
This can be caused by one of the following:
1. Missing Dependencies
This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands.
2. Missing __init__.py files
This error could also result from missing
__init__.pyfiles in your module folders. Make sure that you place one file in each sub-folder.