How to search in an id tree in the python program - python-3.x
I'm a student and working on a small assignment where I need to collect inputs from the student on factors like kind of books they like to issue from the library. I've been provided id_tree class which I need to search using. As you can see I'm getting inputs from the console and I like to use that as the search criteria and get the recommendation from the id tree.
Just for testing purpose, I'm using out.py, but that needs to be replaced with id_tree search logic for which I'm struggling.
# k-Nearest Neighbors and Identification Trees
#api.py
import os
from copy import deepcopy
from functools import reduce
################################################################################
############################# IDENTIFICATION TREES #############################
################################################################################
class Classifier :
def __init__(self, name, classify_fn) :
self.name = str(name)
self._classify_fn = classify_fn
def classify(self, point):
try:
return self._classify_fn(point)
except KeyError as key:
raise ClassifierError("point has no attribute " + str(key) + ": " + str(point))
def copy(self):
return deepcopy(self)
def __eq__(self, other):
try:
return (self.name == other.name
and self._classify_fn.__code__.co_code == other._classify_fn.__code__.co_code)
except:
return False
def __str__(self):
return "Classifier<" + str(self.name) + ">"
__repr__ = __str__
## HELPER FUNCTIONS FOR CREATING CLASSIFIERS
def maybe_number(x) :
try :
return float(x)
except (ValueError, TypeError) :
return x
def feature_test(key) :
return Classifier(key, lambda pt : maybe_number(pt[key]))
def threshold_test(feature, threshold) :
return Classifier(feature + " > " + str(threshold),
lambda pt: "Yes" if (maybe_number(pt.get(feature)) > threshold) else "No")
## CUSTOM ERROR CLASSES
class NoGoodClassifiersError(ValueError):
def __init__(self, value=""):
self.value = value
def __str__(self):
return repr(self.value)
class ClassifierError(RuntimeError):
def __init__(self, value=""):
self.value = value
def __str__(self):
return repr(self.value)
class IdentificationTreeNode:
def __init__(self, target_classifier, parent_branch_name=None):
self.target_classifier = target_classifier
self._parent_branch_name = parent_branch_name
self._classification = None #value, if leaf node
self._classifier = None #Classifier, if tree continues
self._children = {} #dict mapping feature to node, if tree continues
self._data = [] #only used temporarily for printing with data
def get_parent_branch_name(self):
return self._parent_branch_name if self._parent_branch_name else "(Root node: no parent branch)"
def is_leaf(self):
return not self._classifier
def set_node_classification(self, classification):
self._classification = classification
if self._classifier:
print("Warning: Setting the classification", classification, "converts this node from a subtree to a leaf, overwriting its previous classifier:", self._classifier)
self._classifier = None
self._children = {}
return self
def get_node_classification(self):
return self._classification
def set_classifier_and_expand(self, classifier, features):
if classifier is None:
raise TypeError("Cannot set classifier to None")
if not isinstance_Classifier(classifier):
raise TypeError("classifier must be Classifier-type object: " + str(classifier))
self._classifier = classifier
try:
self._children = {feature:IdentificationTreeNode(self.target_classifier, parent_branch_name=str(feature))
for feature in features}
except TypeError:
raise TypeError("Expected list of feature names, got: " + str(features))
if len(self._children) == 1:
print("Warning: The classifier", classifier.name, "has only one relevant feature, which means it's not a useful test!")
if self._classification:
print("Warning: Setting the classifier", classifier.name, "converts this node from a leaf to a subtree, overwriting its previous classification:", self._classification)
self._classification = None
return self
def get_classifier(self):
return self._classifier
def apply_classifier(self, point):
if self._classifier is None:
raise ClassifierError("Cannot apply classifier at leaf node")
return self._children[self._classifier.classify(point)]
def get_branches(self):
return self._children
def copy(self):
return deepcopy(self)
def print_with_data(self, data):
tree = self.copy()
tree._assign_data(data)
print(tree.__str__(with_data=True))
def _assign_data(self, data):
if not self._classifier:
self._data = deepcopy(data)
return self
try:
pairs = list(self._soc(data, self._classifier).items())
except KeyError: #one of the points is missing a feature
raise ClassifierError("One or more points cannot be classified by " + str(self._classifier))
for (feature, branch_data) in pairs:
if feature in self._children:
self._children[feature]._assign_data(branch_data)
else: #feature branch doesn't exist
self._data.extend(branch_data)
return self
_ssc=lambda self,c,d:self.set_classifier_and_expand(c,self._soc(d,c))
_soc=lambda self,d,c:reduce(lambda b,p:b.__setitem__(c.classify(p),b.get(c.classify(p),[])+[p]) or b,d,{})
def __eq__(self, other):
try:
return (self.target_classifier == other.target_classifier
and self._parent_branch_name == other._parent_branch_name
and self._classification == other._classification
and self._classifier == other._classifier
and self._children == other._children
and self._data == other._data)
except:
return False
def __str__(self, indent=0, with_data=False):
newline = os.linesep
ret = ''
if indent == 0:
ret += (newline + "IdentificationTreeNode classifying by "
+ self.target_classifier.name + ":" + newline)
ret += " "*indent + (self._parent_branch_name + ": " if self._parent_branch_name else '')
if self._classifier:
ret += self._classifier.name
if with_data and self._data:
ret += self._render_points()
for (feature, node) in sorted(self._children.items()):
ret += newline + node.__str__(indent+1, with_data)
else: #leaf
ret += str(self._classification)
if with_data and self._data:
ret += self._render_points()
return ret
def _render_points(self):
ret = ' ('
first_point = True
for point in self._data:
if first_point:
first_point = False
else:
ret += ', '
ret += str(point.get("name","datapoint")) + ": "
try:
ret += str(self.target_classifier.classify(point))
except ClassifierError:
ret += '(unknown)'
ret += ')'
return ret
################################################################################
############################# k-NEAREST NEIGHBORS ##############################
################################################################################
class Point(object):
"""A Point has a name and a list or tuple of coordinates, and optionally a
classification, and/or alpha value."""
def __init__(self, coords, classification=None, name=None):
self.name = name
self.coords = coords
self.classification = classification
def copy(self):
return deepcopy(self)
def __getitem__(self, i): # make Point iterable
return self.coords[i]
def __eq__(self, other):
try:
return (self.coords == other.coords
and self.classification == other.classification)
except:
return False
def __str__(self):
ret = "Point(" + str(self.coords)
if self.classification:
ret += ", " + str(self.classification)
if self.name:
ret += ", name=" + str(self.name)
ret += ")"
return ret
__repr__ = __str__
################################################################################
############################### OTHER FUNCTIONS ################################
################################################################################
def is_class_instance(obj, class_name):
return hasattr(obj, '__class__') and obj.__class__.__name__ == class_name
def isinstance_Classifier(obj):
return is_class_instance(obj, 'Classifier')
def isinstance_IdentificationTreeNode(obj):
return is_class_instance(obj, 'IdentificationTreeNode')
def isinstance_Point(obj):
return is_class_instance(obj, 'Point')
#id_tree
from api import *
import math
log2 = lambda x: math.log(x, 2)
INF = float('inf')
import pandas as pd
def id_tree_classify_point(point, id_tree):
if id_tree.is_leaf():
return id_tree.get_node_classification()
else:
new_tree = id_tree.apply_classifier(point)
get_point = id_tree_classify_point(point, new_tree)
return get_point
def split_on_classifier(data, classifier):
"""Given a set of data (as a list of points) and a Classifier object, uses
the classifier to partition the data. Returns a dict mapping each feature
values to a list of points that have that value."""
#Dictionary which will contain the data after classification.
class_dict = {}
#Iterating through all the points in data
for i in range(len(data)):
get_value = classifier.classify(data[i])
if get_value not in class_dict:
class_dict[get_value] = [data[i]]
else:
class_dict[get_value].append(data[i])
return class_dict
def branch_disorder(data, target_classifier):
"""Given a list of points representing a single branch and a Classifier
for determining the true classification of each point, computes and returns
the disorder of the branch."""
#Getting data after classification based on the target_classifier
class_dict = split_on_classifier(data, target_classifier)
if (len(class_dict) == 1):
#Homogenous condition
return 0
else:
disorder = 0
for i in class_dict:
get_len = len(class_dict[i])
p_term = get_len/ float(len(data))
disorder += (-1) * p_term * log2(p_term)
return disorder
def average_test_disorder(data, test_classifier, target_classifier):
"""Given a list of points, a feature-test Classifier, and a Classifier
for determining the true classification of each point, computes and returns
the disorder of the feature-test stump."""
average_disorder = 0.0
#Getting all the branches after applying test_classifer
get_branches = split_on_classifier(data, test_classifier)
#Iterating through the branches
for i in get_branches:
disorder = branch_disorder(get_branches[i], target_classifier)
average_disorder += disorder * (len(get_branches[i])/ float(len(data)))
return average_disorder
#### CONSTRUCTING AN ID TREE
def find_best_classifier(data, possible_classifiers, target_classifier):
"""Given a list of points, a list of possible Classifiers to use as tests,
and a Classifier for determining the true classification of each point,
finds and returns the classifier with the lowest disorder. Breaks ties by
preferring classifiers that appear earlier in the list. If the best
classifier has only one branch, raises NoGoodClassifiersError."""
#Base values to start with
best_classifier = average_test_disorder(data, possible_classifiers[0], target_classifier)
store_classifier = possible_classifiers[0]
#Iterating over the list of possible classifiers
for i in range(len(possible_classifiers)):
avg_disorder = average_test_disorder(data, possible_classifiers[i], target_classifier)
if avg_disorder < best_classifier:
best_classifier = avg_disorder
store_classifier = possible_classifiers[i]
get_branches = split_on_classifier(data, store_classifier)
if len(get_branches)==1:
#Only 1 branch present
raise NoGoodClassifiersError
else:
return store_classifier
def construct_greedy_id_tree(data, possible_classifiers, target_classifier, id_tree_node=None):
"""Given a list of points, a list of possible Classifiers to use as tests,
a Classifier for determining the true classification of each point, and
optionally a partially completed ID tree, returns a completed ID tree by
adding classifiers and classifications until either perfect classification
has been achieved, or there are no good classifiers left."""
#print data
#print "possible", possible_classifiers
#print "target", target_classifier
if id_tree_node == None:
#Creating a new tree
id_tree_node = IdentificationTreeNode(target_classifier)
if branch_disorder(data, target_classifier) == 0:
id_tree_node.set_node_classification(target_classifier.classify(data[0]))
else:
try:
#Getting the best classifier from the options available
best_classifier = find_best_classifier(data, possible_classifiers, target_classifier)
get_branches = split_on_classifier(data, best_classifier)
id_tree_node = id_tree_node.set_classifier_and_expand(best_classifier, get_branches)
#possible_classifiers.remove(best_classifier)
branches = id_tree_node.get_branches()
for i in branches:
construct_greedy_id_tree(get_branches[i], possible_classifiers, target_classifier, branches[i])
except NoGoodClassifiersError:
pass
return id_tree_node
possible_classifiers = [feature_test('age'),
feature_test('gender'),
feature_test('duration'),
feature_test('Mood')
]
df1 = pd.read_csv("data_form.csv")
#df1 = df1.drop("age", axis=1)
print(df1)
a = []
with open("data_form.csv") as myfile:
firstline = True
for line in myfile:
if firstline:
mykeys = "".join(line.split()).split(',')
firstline = False
else:
values = "".join(line.split()).split(',')
a.append({mykeys[n]:values[n] for n in range(0,len(mykeys))})
keys = a[0].keys()
print(keys)
with open('data_clean.csv', 'w') as output_file:
dict_writer = csv.DictWriter(output_file, keys)
dict_writer.writeheader()
dict_writer.writerows(a)
print(a)
tar = feature_test('genre')
print(construct_greedy_id_tree(a, possible_classifiers, tar))
#book_suggestion
import random
#from out import *
def genre(Mood, age, gender, duration):
print("Hi")
res_0= input("What's your name?")
res_1 = input("How are you, "+str(res_0)+"?")
if res_1 in ("good","fine","ok","nice"):
print ("Oh nice")
else:
print("Oh! It's alright")
Mood = input("What is your current mood?")
age = input("What is your age range : 10-12, 12-15,13-14,15-18,18+?")
gender = input("What is your gender?")
duration = input("How long do you want to read : 1week, 2weeks, 3weeks, 3+weeks, 2hours")
def get_book(genre):
suggestions = []
genre_to_book = {"Fantasy":["Just me and my babysitter - Mercer Mayer","Just Grandpa and me - Mercer Mayer","Just me and my babysitter - Mercer Mayer",
"The new Potty - Mercer Mayer","I was so mad - Mercer Mayer","Just me and my puppy" ,"Just a mess" ,"Me too"
,"The new Baby","Just shopping with mom"],
"Encyclopedias":["Brain Power - Paul Mcevoy", "My best books of snakes Gunzi Chrisitian","MY best books of MOON Grahame,Ian",
"The book of Planets Twist,Clint", "Do stars have points? Melvin", "Young discover series:cells Discovery Channel"]
,
"Action" : ["The Kane Chronicle:The Throne of Fire s Book 2 Riordan,Rick",
"Zane : ninja of ice Farshtey, Greg",
"Escape from Sentai Mountain Farshtey, Greg",
"Percy jackson Rick Riordan",
"The Kane Chronicle:The Throne of Fire s Book 2 Rick Riordan"],
"Comic" : ["Double Dork Diaries Russell Rachel Renée",
"Dork Dairies Russell Rachel Renee",
"Dork Dairies Russell Rachel Renée"],
"Mystery" : ["Sparkling Cyanide Christie Agatha",
"Poirot's Early Cases: Agatha Christie",
"The Name of this Book is Secret Bosch,Pseudonyuous"],
"Biographies" :["All by myself Mercer Mayer", "D Days prett bryan",
"Snake Bite Lane Andrew"] }
if (genre == "Fantasy"):
suggestions = [random.sample(genre_to_book["Fantasy"], 3)]
elif (genre == "Action"):
suggestions = [random.sample(genre_to_book["Action"], 3)]
elif (genre == "Comic"):
suggestions = [random.sample(genre_to_book["Comic"], 3)]
elif (genre == "Mystery"):
suggestions = [random.sample(genre_to_book["Mystery"], 3)]
elif (genre == "Encyclopedias"):
suggestions = random.sample(genre_to_book["Encyclopedias"], 3)
elif (genre == "Biographies"):
suggestions = random.sample(genre_to_book["Biographies"], 3)
return suggestions
print(get_book(genre(Mood, age, gender, duration)))
I want the program to not depend on out.py and and run on the information of id tree
The current implementation of the suggestions works by asking the user for a genre, then looking up a list of book titles in a dictionary using that genre as the key, then randomly selecting one of the titles and printing it. The current implementation also (presumably) constructs a IdentificationTreeNode containing recommendations, but then does nothing with it except printing it to the standard output.
The next step would be to not discard the tree, but save it in a variable and use in the recommendation process. Since the class structure is not given, it is not clear how this could be done, but it seems a reasonable assumption that it is possible to provide a keyword (the genre) and receive some collection of objects where each one contains data on a recommendation.
If constructing the IdentificationTreeNode is too costly to run on each recommendation request, it is possible to split the construction into its own script file and using python's pickle package to save the object in a file that can then be unpickled more quickly in the script performing the recommendations.
Related
Switching between classes and passing results
I am trying to add different PlugLead's to the PlugBoard and extract the combination. In the first class I am extracting the one letter from the list should they match or return the input. e.g. lead = PlugLead("AG") assert(lead.encode("A") == "G") class PlugLead: def __init__(self, c): self.c = c def encode(self, l): c0 = self.c[0] c1 = self.c[1] if len(l) == 1 and c0 == l: return c1 elif len(l) == 1 and c1 == l: return c0 else: return l class Plugboard: def __init__(self): self.__head = 0 self.leads = [] self.c = [] def add(self, item): if self.__head >= 10: print("leads already got 10 items") elif item in self.leads: print(f"leads already have this item: {item}") else: self.leads.append(item) self.__head += 1 return self.leads def encode(self) lines = plugboard.leads for word in lines: word = word.split(",") PlugLead.encode(word) In the second class I am trying to add multiple combinations and then at the end pass the one letter to see what its match is in the Class PlugLead however am not able to switch between the two. In the class PlugLead I have a add function that allows the different combinations to be added up to 10 and then I would like to encode from this list the combination of the pairs. e.g. plugboard = Plugboard() plugboard.add(PlugLead("SZ")) plugboard.add(PlugLead("GT")) plugboard.add(PlugLead("DV")) plugboard.add(PlugLead("KU")) assert(plugboard.encode("K") == "U") assert(plugboard.encode("A") == "A")
if you want to use PlugLead("{balabala}"),you need use __new__ to return a dict when you create an instance, not __init__. you want a key-value pair in Plugboard, it should be a dict not list fix some other typo and bugs. code: class PlugLead: def __new__(self, c): return {c[0]:c[1]} class Plugboard: def __init__(self): self.__head = 0 self.leads = {} def add(self, item): if self.__head >= 10: print("leads already got 10 items") elif list(item.keys())[0] in self.leads.keys(): print(f"leads already have this item: {item}") else: self.leads.update(item) self.__head += 1 return self.leads def encode(self,key): if key in self.leads: return self.leads[key] elif key in self.leads.values(): return list(self.leads.keys())[list(self.leads.values()).index(key)] return key plugboard = Plugboard() plugboard.add(PlugLead("SZ")) plugboard.add(PlugLead("GT")) plugboard.add(PlugLead("DV")) plugboard.add(PlugLead("KU")) plugboard.add(PlugLead("KU")) assert(plugboard.encode("K") == "U") assert(plugboard.encode("U") == "K") assert(plugboard.encode("A") == "A") print(plugboard.encode("K")) print(plugboard.encode("U")) print(plugboard.encode("A")) result: leads already have this item: {'K': 'U'} U K A
Object() takes no arguments
I get, "TypeError: Question() takes no arguments" (where Question is an object of class Question) when trying to execute the code below. I'm using jupyter notebook and have checked most of my indentation, Tthe class Question has 2 attributes, I have an init method... Issue is related to the function find_best_split(rows) below. I have added an image of the error console output here class Question: def __init__(self, column, value): self.column = column self.value = value def match(self, example): val = example[self.column] if is_numeric(val): return val >= self.value else: return val == self.value def __repr__(self): condition = "==" if is_numeric(self.value): condition = ">=" return "Is %s %s %s?" % (header[self.column], condition, str(self.value)) def partition(rows, question): true_rows, false_rows = [], [] for row in rows: if question.match(row): true_rows.append(row) else: false_rows.append(row) return true_rows, false_rows ` Error points to this function, specifically "question = Question(col, val)" def find_best_split(rows): best_gain = 0 best_question = None current_uncertainty = gini(rows) n_features = len(rows[0]) -1 # number of columns for col in range(n_features): values = set([row[col] for row in rows]) # unique values in the column for val in values: #now for each value question = Question(col, val) # trying to split the data set true_rows, false_rows = partition(rows, question) # skips this split if it doesn't divide the data set. if len(true_rows) == 0 or len(false_rows) == 0: continue # calculate the information gain from this split gain = info_gain(true_rows, false_rows, current_uncertainty) # you can use > instead of >= below but I wanted the data set to look a certain way for this example. if gain >= best_gain: best_gain, best_question = gain, question return best_gain, best_quesiton ` class Decision_Node: def __init__(self, question, true_branch, false_branch): self.question = question self.true_branch = true_branch self.false_branch = false_branch def build_tree(rows): gain, question = find_best_split(rows) if gain == 0: return Leaf(rows) true_rows, false_rows = partition(rows, question) true_branch = build_tree(true_rows) false_branch = build_tree(false_rows) return Decision_Node(question, true_branch, false_branch) if __name__ == '__main__': my_tree = build_tree(training_data) print_tree(my_tree) # Evaluate testing_data = [ ["Green", 3, "Mango"], ["Yellow", 4, "Mango"], ["Red", 2, "Grape"], ["Red", 1, "Grape"], ["Yellow", 3, "Lemon"], ] for row in testing_data: print ("Actual: %s. Predicted: %s" % (row[-1], print_leaf(classify(row, my_tree))))
Error with Yahtzee Code: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all()
This is my code for finding the probability of different die cases in Yahtzee. I've figured out the yahtzee, and full house methods. But when I do the is_it_large_straight method it gives me an error of this: The truth value of an array with more than one element is ambiguous. Use a.any() or a.all(). What do I need to change in my code to make it produce what I want it to? import numpy as np import random class Die: def __init__(self, sides): """A constructor method to create a die""" self.sides = sides def roll(self): """A general method to roll the die""" return random.randint(1, self.sides) ------------------------------------------------- class Yahtzee: def __init__(self): """A constructor method that can record 5 dice rolls""" self.rolls = np.zeros(5, dtype=np.int16) def roll_dice(self): """A general method that rolls 5 dice""" for i in range(len(self.rolls)): self.rolls[i] = Die(6).roll() def count_outcomes(self): """A helper method that determines how many 1s, 2s, etc. were rolled""" counts = np.zeros(7, dtype=np.int16) for roll in self.rolls: counts[roll] += 1 return counts def is_it_yahtzee(self): yahtzeefun = self.count_outcomes() if 5 in yahtzeefun: return True else: return False def is_it_full_house(self): fullhouse = self.count_outcomes() if 2 in fullhouse and 3 in fullhouse: return True else: return False def is_it_large_straight(self): largestraight = self.count_outcomes() if largestraight == [0,0,1,1,1,1,1] or largestraight == [0,1,1,1,1,1,0]: print("recognize straight") return True else: return False ------------------------------------------------- def main(how_many): yahtzees = 0 full_houses = 0 large_straights = 0 game = Yahtzee() for i in range(how_many): game.roll_dice() if game.is_it_yahtzee(): yahtzees += 1 elif game.is_it_full_house(): full_houses += 1 elif game.is_it_large_straight(): large_straights += 1 print("Number of Rolls:", how_many) print("---------------------") print("Number of Yahtzees:", yahtzees) print("Yahtzee Percent:", "{:.2f}%\n".format(yahtzees * 100 / how_many)) print("Number of Full Houses:", full_houses) print("Full House Percent:", "{:.2f}%\n".format(full_houses * 100 / how_many)) print("Number of Large Straights:", large_straights) print("Large Straight Percent:", "{:.2f}%".format(large_straights * 100 / how_many)) # ------------------------------------------------- main(5000)
In is_it_large_straight replace largestraight = self.count_outcomes() with largestraight = self.count_outcomes().tolist() currently largestraight is a numpy array and it doesn't have a definite boolean value, and thus can't be compared to your list in largestraight == [0,0,1,1,1,1,1]
Sort a unordered list class in python? solution?
I have a Unordered list class like this : from node import Node class UnorderedList: """ Unordered list """ def __init__(self): self.head = None def is_empty(self): return self.head == None def __len__(self): """ returns the length of the list O(1) """ return self.size() def add(self, item): """ Add item to list """ temp = Node(item) temp.set_next(self.head) self.head = temp def size(self): """ Return size of list """ current = self.head count = 0 while current != None: count = count + 1 current = current.get_next() return count def set(self, index, newdata): """ Set node-data in list at specific index """ current = self.head previous = None for i in range(index): previous = current current = current.get_next() if current != None: temp = Node(newdata) temp.set_next(current) if previous is None: self.head = temp else: previous.set_next(temp) else: raise("index out of range") def getIndex(self, item): """get the index of an item, assume the first one (head pointing to) is 0""" index = 0 current = self.head found = False while current != None: if current.get_data() == item: found = True break else: current = current.get_next() index += 1 if not found: index = None return index def get(self, index): """ Returns node data based on index """ current = self.head for i in range(index): current = current.get_next() if current != None: return current.get_data() else: raise("index out of range") def search(self, item): """ Returns True if item found, else return False """ # Här ska du returnera en bool (True/False) # beroende på om 'item' finns i listan current = self.head found = False while current != None and not found: if current.get_data() == item: found = True else: current = current.get_next() return found def print_list(self): """ Prints each item in list """ # Traversera listan och gör en print() på varje element result = "[" node = self.head if node != None: result += str(node.data) node = node.next while node: result += ", " + str(node.data) node = node.next result += "]" return result def remove(self, item): """ Removes item from list """ current = self.head previous = None found = False while not found: if current.get_data() == item: found = True else: previous = current current = current.get_next() if previous == None: self.head = current.get_next() else: previous.set_next(current.get_next()) I want to create a class list and then i can sort the list with my "bubblesort" function. My "bubblesort" function look like this : def bubble_sort(items): """ Bubble sort """ Size = items.size() for i in range(Size): for j in range(Size-1-i): if items.get(j) > items.get(j+1): tmps = items.get(j+1) items.set(j, items.get(j+1)) items.set(j+1, tmps) return items Now let us create a list : // create the list myListTwo = UnorderedList() // Add the elements to the list myListTwo.add(4) myListTwo.add(50) myListTwo.add(6) myListTwo.add(10) myListTwo.add(60) //print the list : print(myListTwo.print_list()) [60, 10, 6, 50, 4] In this stage all work very well but the problem is when I want to sort the list with my bubble_sort function I got this result : // bubble_sort my list sorte = bubble_sort(myListTwo) //print the list print(sorte.print_list()) [10, 10, 10, 10, 60, 10, 6, 50, 4] Any idea? Thanx/Georges
So, you have two issues to your implementation. The first one is the inner code of bubble sort. Change this tmps = items.get(j+1) to tmps = items.get(j) See here for more about swapping Python Simple Swap Function The second issue is the set method. You should remove this temp = Node(newdata) temp.set_next(current) if previous is None: self.head = temp else: previous.set_next(temp) and you should write something like this current.set_data(newData) (I don't know how exactly you implemented the Node class)
union of two default dictionaries
am trying to create a union of two default dictionaries. Here is the code for the method in my class: def __add__(self,right): mergedbag = copy.copy(self.bag_value) for item in right: if item not in mergedbag:mergedbag[item] = 0 mergedbag[item] += right[item] return mergedbag I create two default dictionaries: b = Bag(['d','a','b','d','c','b','d']) c = Bag(['d','a','b','d','c','b','d']) the result of print(b+c) should be the total count of elements after the union.. Bag(a[a],b[4],c[3],d[6]) This is the error I keep getting: Traceback (most recent call last): File "D:\workspace33\courselib\driver.py", line 229, in driver Command[print(b+c)]: exec(old,local,globl) File "<string>", line 1, in <module> File "D:\workspace33\Project2\src\bag.py", line 58, in __add__ mergedbag[item] += right[item] TypeError: 'Bag' object is not subscriptable Here is the new code: class Bag: def __init__(self, items = []): self.bag_value = defaultdict(int) for item in items: self.bag_value[item] += 1 def __repr__(self): bag_list = [] for item, count in self.bag_value.items(): bag_list.extend(list(item*count)) return 'Bag(' + str(bag_list) + ')' def __str__(self): return 'Bag(' + ','.join(str(item) + '[' + str(count) + ']' for item, count in self.bag_value.items()) + ')' def __len__(self): bag_len = 0 for value in self.bag_value: bag_len += self.bag_value[value] return bag_len def unique(self): return len(self.bag_value) def __contains__(self, item): return item in self.bag_value def count(self, item): return(self.bag_items.count(item)) def add(self, new): self.bag_value[new] += 1 def __add__(self,right): mergedbag = copy.copy(self.bag_value) for item in right: if item not in mergedbag:mergedbag[item] = 0 mergedbag[item] += right[item] return mergedbag def remove(self, item): if item in self.bag_items: del(item) else: raise ValueError(type_as_str(item) + ' not in bag.') def __eq__(self, right): if type(right) is not Bag: raise TypeError('Cannot compare Bag with' + type_as_str(right) + '. Can only compare Bag with Bag') else: return (len(self) == len(right)) and (self.unique() == right.unique()) def __ne__(self, right): return not self.__eq__(right) def _bag_gen(self, bag_value): for item in self.bag_value: for count in range(self.bag_value[item]): yield item def __iter__(self): return self._bag_gen(self.bag_value) if __name__ == '__main__': # bag = Bag(['d','a','b','d','c','b','d']) # bag2 = Bag(['d','a','b','d','c','b','d']) # bag3 = Bag(['d','a','b','d','c','b']) # print(bag == bag2) # print(bag == bag3) # print(bag != bag2) # print(bag != bag3) import driver driver.driver()
First of all, it is important to note that your Bag class is basically the same as a Counter. If you need any specific extension, just inherit from it and you are done. I would say that the following code just address all the functionality you are implementing yourself in your Bag class: from collections import Counter class Bag(Counter): pass b = Bag(['d','a','b','d','c','b','d']) c = Bag(['d','a','b','d','c','b','d']) print (b) print (c) print (b+c) If you are doing an exercise to learn, the problem is that you are not implementing setitem and getitem methods in your Bag class to allow the [] notation. You could implement them or just access the attribute bag_value: def __add__(self, right): mergedbag = Bag() mergedbag.bag_value = copy.copy(self.bag_value) for item in right.bag_value.keys(): mergedbag.bag_value[item] += right.bag_value[item] return mergedbag Be sure to create and return a Bag object
The definition is def __add__(self,other): So you ought to merge the contents of self.items with other.items and then return the result. Also, not sure if bag1+bag2 will work, but my pyfu is weak, you will explicitly have to iterate through them and add the counts together. import copy c = {'a':2,'b':1} d = {'b':1,'c':1} # mergedBag = copy.copy(c) #might wanna use deepcopy? for k in d: if k not in mergedBag: mergedBag[k] = 0 mergedBag[k] += d[k] print mergedBag