Hacker News Reader

Comment by downer | Hacker News Reader

It does work, you do the 3-step XOR with a[0] and a[last], then a[1] and a[last-1], etc. The 3-step XOR from the OP swaps the data.

Here it is with a 4-element array, a[0] through a[3]:

 a=[100,200,300,400]

 a[0] ^= a[3]
 a[3] ^= a[0]
 a[0] ^= a[3]

 a[1] ^= a[2]
 a[2] ^= a[1]
 a[1] ^= a[2]

 a
 [400, 300, 200, 100]

(Pretend this list is an array, it's just for illustrative purposes)

View on HN

Well, obviously the XOR swap works. But how do you use that to reverse an array of variable size? (i.e. where "last" isn't known at compile-time, so the number of swaps isn't known in advance).

You don't; it's basically, "here's an array, now reverse it" (keep in mind a pointer is not an array).

http://www.lysator.liu.se/faq/c-faq/c-2.html

The general point is that interviews of this type are counterproductive.

An array can be variably-sized in C (C99, that is). And thank you, I know the difference between an array and a pointer. You just seem to have arbitrarily interpreted the question to limit it to C89 and a fixed-size array in order to make your solution work.

This has nothing to do with it being statically sized or variably sized. You have to know what size it is when you are going to reverse it. The solution presented works as long as the programmer knows what the size is.

If you want to posit any kind of sane question, then you can have a sane solution.

Well, suppose I have an array `A' of size `len'. How would you write an algorithm that reverses that array in place, without using any temporary variables, and that works for arbitrary values of A and len? I think the answer is "you can't". You can't claim to have a solution for the problem if it only works for a fixed input, and if the solution itself would be different for different inputs.

> I think the answer is "you can't".

That's what I said a few posts ago.

> You can't claim to have a solution for the problem if it only works for a

> fixed input, and if the solution itself would be different for different

> inputs.

"Reverse this array". Done. "Reverse an unspecified array of arbitrary length" is a different problem :)

Reply on news.ycombinator.com