excelexporters.offlinemeterbatch.generate_excel() - Code Metrics - Inspection of "Merge branch 'develop'" - MyEMS/myems - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( e39e9b...029c30 )

by Guangyu

created 2022-06-10 06:27 UTC

excelexporters.offlinemeterbatch.generate_excel() B

↳ Parent: excelexporters.offlinemeterbatch

Complexity

Conditions

Size

Total Lines	64
Code Lines	50

Duplication

Lines	64
Ratio	100 %

Importance

Changes

Metric	Value
eloc	50
dl	64
loc	64
rs	8.1696
c	0
b	0
f	0
cc	5
nop	4

How to fix Long Method

import base64
import uuid
import os
from openpyxl.styles import PatternFill, Border, Side, Alignment, Font
from openpyxl.drawing.image import Image
from openpyxl import Workbook


########################################################################################################################
# PROCEDURES
# Step 1: Validate the report data
# Step 2: Generate excelexporters file
# Step 3: Encode the excelexporters file to Base64
########################################################################################################################

def export(result, space_name, reporting_start_datetime_local, reporting_end_datetime_local):

    ####################################################################################################################
    # Step 1: Validate the report data
    ####################################################################################################################
    if result is None:
        return None

    ####################################################################################################################
    # Step 2: Generate excel file from the report data
    ####################################################################################################################
    filename = generate_excel(result,
                              space_name,
                              reporting_start_datetime_local,
                              reporting_end_datetime_local)
    ####################################################################################################################
    # Step 3: Encode the excel file to Base64
    ####################################################################################################################
    binary_file_data = b''
    try:
        with open(filename, 'rb') as binary_file:
            binary_file_data = binary_file.read()
    except IOError as ex:
        pass

    # Base64 encode the bytes
    base64_encoded_data = base64.b64encode(binary_file_data)
    # get the Base64 encoded data using human-readable characters.
    base64_message = base64_encoded_data.decode('utf-8')
    # delete the file from server
    try:
        os.remove(filename)
    except NotImplementedError as ex:
        pass
    return base64_message


def generate_excel(report, space_name, reporting_start_datetime_local, reporting_end_datetime_local):


    wb = Workbook()
    ws = wb.active
    ws.title = "OfflineMeter"

    # Col width
    for i in range(ord('A'), ord('L')):
        ws.column_dimensions[chr(i)].width = 20.0

    # Head image
    ws.row_dimensions[1].height = 105
    img = Image("excelexporters/myems.png")
    ws.add_image(img, 'A1')

    # Query Parameters
    b_r_alignment = Alignment(vertical='bottom',
                              horizontal='right',
                              text_rotation=0,
                              wrap_text=True,
                              shrink_to_fit=False,
                              indent=0)
    ws['A3'].alignment = b_r_alignment
    ws['A3'] = 'Space:'
    ws['B3'] = space_name
    ws['A4'].alignment = b_r_alignment
    ws['A4'] = 'Start Datetime:'
    ws['B4'] = reporting_start_datetime_local
    ws['A5'].alignment = b_r_alignment
    ws['A5'] = 'End Datetime:'
    ws['B5'] = reporting_end_datetime_local

    # Title
    title_font = Font(size=12, bold=True)
    ws['A6'].font = title_font
    ws['A6'] = 'ID'
    ws['B6'].font = title_font
    ws['B6'] = 'Name'
    ws['C6'].font = title_font
    ws['C6'] = 'Space'

    ca_len = len(report['energycategories'])
    for i in range(0, ca_len):
        col = chr(ord('D') + i)
        ws[col + '6'].font = title_font
        ws[col + '6'] = report['energycategories'][i]['name'] + " (" + \
            report['energycategories'][i]['unit_of_measure'] + ")"

    current_row_number = 7
    for i in range(0, len(report['offline_meters'])):
        ws['A' + str(current_row_number)] = str(report['offline_meters'][i]['id'])

        ws['B' + str(current_row_number)] = report['offline_meters'][i]['offline_meter_name']
        ws['C' + str(current_row_number)] = report['offline_meters'][i]['space_name']
        ca_len = len(report['offline_meters'][i]['values'])
        for j in range(0, ca_len):
            col = chr(ord('D') + j)
            ws[col + str(current_row_number)] = report['offline_meters'][i]['values'][j]

        current_row_number += 1

    filename = str(uuid.uuid4()) + '.xlsx'
    wb.save(filename)

    return filename


1		import base64
2		import uuid
3		import os
4		from openpyxl.styles import PatternFill, Border, Side, Alignment, Font
5		from openpyxl.drawing.image import Image
6		from openpyxl import Workbook
7
8
9		########################################################################################################################
10		# PROCEDURES
11		# Step 1: Validate the report data
12		# Step 2: Generate excelexporters file
13		# Step 3: Encode the excelexporters file to Base64
14		########################################################################################################################
15
16	View Code Duplication	def export(result, space_name, reporting_start_datetime_local, reporting_end_datetime_local):
		0 ignored issues – show Duplication introduced 2022-06-10 06:35 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
17		####################################################################################################################
18		# Step 1: Validate the report data
19		####################################################################################################################
20		if result is None:
21		return None
22
23		####################################################################################################################
24		# Step 2: Generate excel file from the report data
25		####################################################################################################################
26		filename = generate_excel(result,
27		space_name,
28		reporting_start_datetime_local,
29		reporting_end_datetime_local)
30		####################################################################################################################
31		# Step 3: Encode the excel file to Base64
32		####################################################################################################################
33		binary_file_data = b''
34		try:
35		with open(filename, 'rb') as binary_file:
36		binary_file_data = binary_file.read()
37		except IOError as ex:
38		pass
39
40		# Base64 encode the bytes
41		base64_encoded_data = base64.b64encode(binary_file_data)
42		# get the Base64 encoded data using human-readable characters.
43		base64_message = base64_encoded_data.decode('utf-8')
44		# delete the file from server
45		try:
46		os.remove(filename)
47		except NotImplementedError as ex:
48		pass
49		return base64_message
50
51
52	View Code Duplication	def generate_excel(report, space_name, reporting_start_datetime_local, reporting_end_datetime_local):
		0 ignored issues – show Duplication introduced 2021-05-19 08:30 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
53
54		wb = Workbook()
55		ws = wb.active
56		ws.title = "OfflineMeter"
57
58		# Col width
59		for i in range(ord('A'), ord('L')):
60		ws.column_dimensions[chr(i)].width = 20.0
61
62		# Head image
63		ws.row_dimensions[1].height = 105
64		img = Image("excelexporters/myems.png")
65		ws.add_image(img, 'A1')
66
67		# Query Parameters
68		b_r_alignment = Alignment(vertical='bottom',
69		horizontal='right',
70		text_rotation=0,
71		wrap_text=True,
72		shrink_to_fit=False,
73		indent=0)
74		ws['A3'].alignment = b_r_alignment
75		ws['A3'] = 'Space:'
76		ws['B3'] = space_name
77		ws['A4'].alignment = b_r_alignment
78		ws['A4'] = 'Start Datetime:'
79		ws['B4'] = reporting_start_datetime_local
80		ws['A5'].alignment = b_r_alignment
81		ws['A5'] = 'End Datetime:'
82		ws['B5'] = reporting_end_datetime_local
83
84		# Title
85		title_font = Font(size=12, bold=True)
86		ws['A6'].font = title_font
87		ws['A6'] = 'ID'
88		ws['B6'].font = title_font
89		ws['B6'] = 'Name'
90		ws['C6'].font = title_font
91		ws['C6'] = 'Space'
92
93		ca_len = len(report['energycategories'])
94		for i in range(0, ca_len):
95		col = chr(ord('D') + i)
96		ws[col + '6'].font = title_font
97		ws[col + '6'] = report['energycategories'][i]['name'] + " (" + \
98		report['energycategories'][i]['unit_of_measure'] + ")"
99
100		current_row_number = 7
101		for i in range(0, len(report['offline_meters'])):
102		ws['A' + str(current_row_number)] = str(report['offline_meters'][i]['id'])
		0 ignored issues – show Comprehensibility Best Practice introduced 2022-06-10 06:35 UTC by Report Bug Copy Issue Report The variable `str` does not seem to be defined. Loading history...
103		ws['B' + str(current_row_number)] = report['offline_meters'][i]['offline_meter_name']
104		ws['C' + str(current_row_number)] = report['offline_meters'][i]['space_name']
105		ca_len = len(report['offline_meters'][i]['values'])
106		for j in range(0, ca_len):
107		col = chr(ord('D') + j)
108		ws[col + str(current_row_number)] = report['offline_meters'][i]['values'][j]
109
110		current_row_number += 1
111
112		filename = str(uuid.uuid4()) + '.xlsx'
113		wb.save(filename)
114
115		return filename
116

MyEMS / myems

Push — master ( e39e9b...029c30 )

excelexporters.offlinemeterbatch.generate_excel() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like