I've been working on a 3D-vector class which should be as streamlined as possible for use in numerical simulations. It will be used to model 3D-physical vectors. I hoped to design it in a modern and fast way - but one is never perfect. So, I would be interested in some input from your side. //t...

Things you did well Declaring your parameters as void when you don't take in any arguments. Keeping your dependencies to a minimum. Things you could improve Bugs: You have a bad test conditional. if(value < 0) int first = 0; I think you mean to return a value there instead, but forgo...

First, it should be noted that an NSDictionary can use any object as it's key, as such, your setValue:forKey: methods should probably be rewritten to take id as the key argument type, rather than NSString*. Second, while the structure of your binary search tree is important for somethings, the...

Can someone please let me know if they see glaring issues with this code? I tested a handful of cases and it seems to work but in other threads. I often see much more concisely written versions of this. I just want to know if anything I have done is superfluous or just stylistically poor form. ...

Things you did well Declaring your parameters as void when you don't take in any arguments. Keeping your dependencies to a minimum. Things you could improve Bugs: You have a bad test conditional. if(value < 0) int first = 0; I think you mean to return a value there instead, but forgo...

Generally, very well written code. I just have a few suggestions. In the Start method, I understand why, but I don't like that there are two calls to check if the service is running. It feel kind of redundant, and I don't think you'll get a speed increase by short circuiting the lock. I would...

In one of my assignments I faced a problem: there are an unknown number of pairs of words. A line consists of: a word, max 10000 chars a space another word containing letters which are separators a new line sign (btw I mustn't use strings) I have to write the first word with spaces instead ...

Your implementation looks good. =)

I plan to revisit the more C++ side of this later, but I've noticed a few things in general that are wrong with this or that could be improved. Since you're not allowed to use std::string, this review biases towards C. You have two major bugs in this. 1) new[] requires a size. I'm surprised ...

Generally, very well written code. I just have a few suggestions. In the Start method, I understand why, but I don't like that there are two calls to check if the service is running. It feel kind of redundant, and I don't think you'll get a speed increase by short circuiting the lock. I would...

First, it should be noted that an NSDictionary can use any object as it's key (as long as it conforms to NSCopying protocol), as such, your setValue:forKey: methods should probably be rewritten to take id as the key argument type, rather than NSString*. Second, while the structure of your bina...

The comments in the clearmem procedure for the Linux block look a bit confusing. You could just have a summary of the procedure commented above, and have the individual comments for each line specify the meaning of the assembly instructions. Specifically, the lines that describe the C++ code d...

I know that recursion is sometimes a lot cleaner than looping, and I'm not asking anything about when I should use recursion over iteration, I know there are lots of questions about that already. What I'm asking is, is recursion ever faster than a loop? To me it seems like, you would always be a...