你混淆了基于0和基于1的索引。x范围从0到org.size[0]-1。但是a的范围从org.size[0]到1,因为您从a=org.size[0]开始使用它,而不需要首先减去。
最有可能的错误来自这一行的某个地方:new.putpixel((a,y),pixel)
…在第一次调用时,当您试图在仅从(0-279,0-279)运行的图像中写入像素(280,0)。
所以,与其这样:for x in range(org.size[0]):
a=org.size[0]
for y in range(org.size[1]):
pixel=org.getpixel((x,y))
new.putpixel((a,y),pixel)
a-=1
执行以下操作:for x in range(org.size[0]):
a=org.size[0]-1
for y in range(org.size[1]):
pixel=org.getpixel((x,y))
new.putpixel((a,y),pixel)
a-=1
但一旦你解决了这个问题,你就会遇到另一个问题。每次通过外环时,您都将a设置为org.size[0],而不是第一次。然后每次通过内环而不是外环减少a。所以,你最终会把原始图像中的每一行复制到从(279,0)到(0279)的对角线上。
所以,你需要这样做:a=org.size[0]-1
for x in range(org.size[0]):
for y in range(org.size[1]):
pixel=org.getpixel((x,y))
new.putpixel((a,y),pixel)
a-=1
这正是您应该避免像这样手动修改索引的原因。你第一次尝试就不会成功。在您的第三次尝试中,您得到了一些看起来正确的内容,但是当您尝试第一个不在您的测试套件中的图像时就会崩溃。最好是计算这些值,而不是倒数。例如:for x in range(org.size[0]):
flipped_x = org.size[0] - x - 1
for y in range(org.size[1]):
pixel=org.getpixel((x,y))
new.putpixel((flipped_x,y),pixel)
在这里,如果您需要/依赖PIL 1.1.6或更高版本,那么使用Image.load()返回的数组会简单得多,效率也更高,而且通常更易于调试:orgpixels, newpixels = org.load(), new.load()
for x in range(org.size[0]):
flipped_x = org.size[0] - x - 1
for y in range(org.size[1]):
pixel=orgpixels[x, y]
newpixels[flipped_x, y] = pixel
如果你不能依赖1.1.6,你可以使用getdata获得一个像素的iterable序列,使用它生成一个新的list或其他序列(这意味着你可以使用列表理解,map,甚至通过np.array(org.getdata()).reshape(org.size)将它们输入^{}),然后使用putdata从结果创建新图像。
当然,getdata和putdata处理一维序列,而您希望将其作为二维序列处理。幸运的是,^{}文档中的grouper函数通常正是您想要的:orgrows = more_itertools.grouper(org.size[0], org.getdata())
newrows = [list(reversed(row)) for row in orgrows]
new.putdata(newrows)
需要注意的一点是Image.open(im)可能不一定在RGB模式下返回图像。如果只将像素从XYZ或P图像复制到RGB,则最终会分别出现变色的垃圾或红色通道。您可能需要print org.mode,也可能需要print org.pixel((0, 0)),以确保它实际上有3个通道(它们看起来像RGB)。
最简单的方法是在执行其他操作之前转换org:org=Image.open(im).convert('RGB')
当然,有些格式要么没有直接转换,要么需要显式矩阵或调色板。如果您得到一个ValueError,那么您必须阅读转换输入类型的内容。