Build a Medical Data Validator - Step 39

Tell us what’s happening:

please help me, how to finish this, i don’t understand the instructions

Your code so far

import re

medical_records = [
    {
        'patient_id': 'P1001',
        'age': 34,
        'gender': 'Female',
        'diagnosis': 'Hypertension',
        'medications': ['Lisinopril'],
        'last_visit_id': 'V2301',
    },
    {
        'patient_id': 'p1002',
        'age': 47,
        'gender': 'male',
        'diagnosis': 'Type 2 Diabetes',
        'medications': ['Metformin', 'Insulin'],
        'last_visit_id': 'v2302',
    },
    {
        'patient_id': 'P1003',
        'age': 29,
        'gender': 'female',
        'diagnosis': 'Asthma',
        'medications': ['Albuterol'],
        'last_visit_id': 'v2303',
    },
    {
        'patient_id': 'p1004',
        'age': 56,
        'gender': 'Male',
        'diagnosis': 'Chronic Back Pain',
        'medications': ['Ibuprofen', 'Physical Therapy'],
        'last_visit_id': 'V2304',
    }
]

def find_invalid_records(
    patient_id, age, gender, diagnosis, medications, last_visit_id
):
    constraints = {
        'patient_id': isinstance(patient_id, str)
        and re.fullmatch('p\d+', patient_id, re.IGNORECASE),
        'age': isinstance(age, int) and age >= 18,
        'gender': isinstance(gender, str) and gender.lower() in ('male', 'female'),
        'diagnosis': isinstance(diagnosis, str) or diagnosis is None,
        'medications': isinstance(medications, list)
        and all([isinstance(i, str) for i in medications]),
        'last_visit_id': isinstance(last_visit_id, str)
        and re.fullmatch('v\d+', last_visit_id, re.IGNORECASE)
    }

# User Editable Region

    return constraints.items()

# User Editable Region


def validate(data):
    is_sequence = isinstance(data, (list, tuple))

    if not is_sequence:
        print('Invalid format: expected a list or tuple.')
        return False
        
    is_invalid = False
    key_set = set(
        ['patient_id', 'age', 'gender', 'diagnosis', 'medications', 'last_visit_id']
    )

    for index, dictionary in enumerate(data):
        if not isinstance(dictionary, dict):
            print(f'Invalid format: expected a dictionary at position {index}.')
            is_invalid = True

        if set(dictionary.keys()) != key_set:
            print(
                f'Invalid format: {dictionary} at position {index} has missing and/or invalid keys.'
            )
            is_invalid = True

    if is_invalid:
        return False
    print('Valid format.')
    return True

validate(medical_records)
print(find_invalid_records(**medical_records[0]))

Your browser information:

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/147.0.0.0 Safari/537.36

Challenge Information:

Build a Medical Data Validator - Step 39

you’ll need to create a for loop that evaluates each key for each key: value pair in constraints.items() and return the key. In the next step you will add an if statement to return the key only if it is invalid. for now you only need to return each key for key, value in the dictionaries items. Good Luck, Happy Coding!

If you need a refresher on list comprehensions, you can find that here:

Working with Loops and Sequences - What Are List Comprehensions and What Are Some Useful Functions to Work With Lists? | Learn | freeCodeCamp.org

i’m still don’t understand how to make it right, my result already same with the instruction, but still not working :sweat_smile:

return [(key, value) if value == True else (key, ) for key, value in constraints.items()]

Valid format.
[(‘patient_id’,), (‘age’, True), (‘gender’, True), (‘diagnosis’, True), (‘medications’, True), (‘last_visit_id’,)]

At this point, there should be no condition. You just need to return the key. And if there was a condition, it would be at the end of the list comprehension. Maybe look for more examples of list comprehensions to get comfortable with the syntax.